Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+154
$code_date = strtotime($sms_action_date);
$date = date(
"Y-m-d H:i:s",
mktime(
date('H', $code_date),
date('i', $code_date) + 30,
date('s', $code_date),
date("m", $code_date),
date("d", $code_date),
date("Y", $code_date)
)
);Определяем +30 минут грамотно.
miraage,
05 Марта 2013
+74
private boolean userInOneRegistrationNode() throws DfException {
String uname = OrganizationStaffStructureHelper.getCurrentUserName();
int i = 0;
IDfCollection NodesCol = DQLHelper.getCollection(DQL_GET_REGISTRATOR_DIV, new String[]{uname});
while (NodesCol.next()) {
if (!(NodesCol == null)) {
String group_name = NodesCol.getString(GROUP_NAME);
i = i + 1;
}
}
if (i == 1) {
return true;
}
return false;
}Заменяется 2мя строками один - select count(*), вторая - полученный результат Integer.ValueOf(...).
Landing,
04 Марта 2013
+140
[WebMethod]
public void runCompareService(string mAuthToken, int documentId, int origVerNum, int revisedVerNum)
{
//string result = null;
DirectoryInfo tmpDirInfo = getTempDirectoryPath(documentId);
try
{
//authToken = authClient.AuthenticateUser(username, password);
authToken = mAuthToken;
otAuth.AuthenticationToken = authToken;
try
{
string origFilePath = String.Format(tmpFilePath, tmpDirInfo.ToString(), origVerNum);
string revisedFilePath = String.Format(tmpFilePath, tmpDirInfo.ToString(), revisedVerNum);
string resultFilePath = String.Format(tmpResFilePath, tmpDirInfo.ToString(), origVerNum, revisedVerNum);
string origContextId = getDocContextId(documentId, origVerNum);
string revisedContextId = getDocContextId(documentId, revisedVerNum);
try
{
downlodFileByContextId(origContextId, origFilePath);
downlodFileByContextId(revisedContextId, revisedFilePath);
try
{
doCompare(origFilePath, revisedFilePath, resultFilePath);
try
{
DownloadToBrowser(resultFilePath);
//uploadResultToCS(targetId, resultFilePath);
}
catch (Exception e)
{
throw new Exception(String.Format("Failed to Download To Browser. Error: {0}", e.ToString()));
}
}
catch (Exception e)
{
throw new Exception(String.Format("Failed to do compare method . Error: {0}", e.ToString()));
}
}
catch (Exception e)
{
throw new Exception(String.Format("Failed to create and download file. Error: {0}", e.ToString()));
}
}
catch (Exception e)
{
throw new Exception("Failed to get Context ID for version Exeption details: " + e.ToString());
}
}
catch (Exception e)
{
throw new Exception("Failed to auth user. Exeption details: " + e.ToString());
}
finally
{
docManClient.Close();
contentServiceClient.Close();
if (Directory.Exists(tmpDirInfo.ToString()))
{
tmpDirInfo.Delete(true);
}
}
}
Landing,
04 Марта 2013
+106
using System;
using System.Windows.Forms;
using System.Collections.Generic;
using System.Text;
using System.Xml.Linq;
namespace SplitSpace {
class Program {
static void Main(string[] args) {
test();
}
static void test() {
if(testfun(0, 0, 3, 1020, 0.05*180/Utils.PI, 1, 40, 5.80020838542985, 5.80020838542985, 0.0125032560359095)) return;
if(testfun(0, 0, 3, 1020, 0.05*180/Utils.PI, 2, 5, 1.75, 9.85005209635745, 0.00156290700448869)) return;
if(testfun(0, 0, 3, 1020, 0.05*180/Utils.PI, 3, 1.5, 0.0375078144536192, 0.0375078144536192, 0.000468872101346607)) return;
if(testfun(0, 0, 3, 219, 0.00, 1, 15.000, 4.900000000, 4.900000000, 0.000000000)) return;
if(testfun(0, 0, 3, 219, 0.50, 1, 15.000, 4.900000415, 4.900000415, 0.000142791)) return;
if(testfun(0, 0, 3, 219, 5.00, 1, 15.000, 4.900415674, 4.900415674, 0.014290277)) return;
if(testfun(0, 0, 3, 219, 5.10, 1, 15.000, 4.900441130, 4.900441130, 0.014868082)) return;
if(testfun(0, 0, 3, 219, 5.90, 1, 15.000, 4.900683170, 4.900683170, 0.019903983)) return;
// Спустя 8369 строчек
// ...
// ...
// ...
if(testfun(2, 1, 4, 1420, 55.00, 2, 7.000, 5.512975201, 5.512975201, 1.316853521)) return;
if(testfun(2, 1, 4, 1420, 60.00, 2, 7.000, 5.817691454, 5.817691454, 1.558845727)) return;
if(testfun(2, 1, 4, 1420, 66.00, 2, 7.000, 6.589343466, 6.589343466, 1.708953962)) return;
if(testfun(2, 1, 4, 1420, 75.00, 2, 7.000, 7.385125168, 7.385125168, 2.367207113)) return;
if(testfun(2, 1, 4, 1420, 85.10, 2, 7.000, 8.438873737, 8.438873737, 3.248627191)) return;
if(testfun(2, 1, 4, 1420, 89.00, 2, 7.000, 8.848026526, 8.848026526, 3.610990132)) return;
if(testfun(2, 1, 4, 1420, 89.99, 2, 7.000, 8.932590260, 8.932590260, 3.699354242)) return;
if(testfun(2, 1, 4, 1420, 90.00, 2, 7.000, 8.932590181, 8.932590181, 3.700000000)) return;
MessageBox.Show("OK!");
}
static bool testfun(int dir, int coolType, int hotType, double dia, double angG, int angType, double Rad, double Tan1, double Tan2, double Bis) {
// тут всякие тесты
// ...
// ...
// ...
return false;
}
}
}
diimdeep ,
04 Марта 2013
+106
using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Text;
namespace data_base
{ public class DatA
{ public struct DATA
{ public struct Yslovie { public string yslovie; public bool ActivE;}
public Yslovie yslovie1;
public Yslovie yslovie2;
public Yslovie yslovie3;
public string deistvie;};
public DATA.Yslovie yslovie1 = new DATA.Yslovie();
public DATA.Yslovie yslovie2 = new DATA.Yslovie();
public DATA.Yslovie yslovie3 = new DATA.Yslovie();
public DATA[] BD = new DATA[5];
public void инициализация_БД()
{ BD[0].yslovie1.yslovie = "поставщик проверенный"; BD[0].yslovie1.ActivE = false;
BD[0].yslovie2.yslovie = "товар качественный"; BD[0].yslovie2.ActivE = false;
BD[0].yslovie3.yslovie = "цена приемлемая"; BD[0].yslovie3.ActivE = false;
BD[0].deistvie = "Поставщик подходит";
BD[1].yslovie1.yslovie = "поставщик давно на рынке"; BD[1].yslovie1.ActivE = false;
BD[1].yslovie2.yslovie = "жалоб на поставщика не было"; BD[1].yslovie2.ActivE = false;
BD[1].yslovie3.yslovie = null; BD[1].yslovie3.ActivE = false;
BD[1].deistvie = "поставщик проверенный";
BD[2].yslovie1.yslovie = "есть рекомендации от других заказчиков"; BD[2].yslovie1.ActivE = false;
BD[2].yslovie2.yslovie = "поставка производится в срок"; BD[2].yslovie2.ActivE = false;
BD[2].yslovie3.yslovie = "товар не получает дефектов в ходе транспатрировки"; BD[2].yslovie3.ActivE = false;
BD[2].deistvie = "жалоб на поставщика не было";
BD[3].yslovie1.yslovie = "товар сертифицирован"; BD[3].yslovie1.ActivE = false;
BD[3].yslovie2.yslovie = "товар прощел государственные экспертизы"; BD[3].yslovie2.ActivE = false;
BD[3].yslovie3.yslovie = null; BD[3].yslovie3.ActivE = false;
BD[3].deistvie = "товар качественный";
BD[4].yslovie1.yslovie = "цена чуть ниже чем у других поставщиков"; BD[4].yslovie1.ActivE = false;
BD[4].yslovie2.yslovie = null; BD[4].yslovie2.ActivE = false;
BD[4].yslovie3.yslovie = null; BD[4].yslovie3.ActivE = false;
BD[4].deistvie = "цена приемлемая";
}
public void Констотация_факта(int i)
{ switch (i)
{ case 1: if (BD[0].yslovie1.ActivE == true) BD[0].yslovie1.ActivE = false;
else BD[0].yslovie1.ActivE = true; break;
case 2: if (BD[0].yslovie2.ActivE == true) BD[0].yslovie2.ActivE = false;
else BD[0].yslovie2.ActivE = true; break;
case 3: if (BD[0].yslovie3.ActivE == true) BD[0].yslovie3.ActivE = false;
else BD[0].yslovie3.ActivE = true; break;
case 4: if (BD[1].yslovie1.ActivE == true) BD[1].yslovie1.ActivE = false;
else BD[1].yslovie1.ActivE = true; break;
case 5: if (BD[1].yslovie2.ActivE == true) BD[1].yslovie2.ActivE = false;
else BD[1].yslovie2.ActivE = true; break;
case 6: if (BD[1].yslovie3.ActivE == true) BD[1].yslovie3.ActivE = false;
else BD[1].yslovie3.ActivE = true; break;
case 7: if (BD[2].yslovie1.ActivE == true) BD[2].yslovie1.ActivE = false;
else BD[2].yslovie1.ActivE = true; break;
case 8: if (BD[2].yslovie2.ActivE == true) BD[2].yslovie2.ActivE = false;
else BD[2].yslovie2.ActivE = true; break;
case 9: if (BD[2].yslovie3.ActivE == true) BD[2].yslovie3.ActivE = false;
else BD[2].yslovie3.ActivE = true; break;
case 10: if (BD[3].yslovie1.ActivE == true) BD[3].yslovie1.ActivE = false;
else BD[3].yslovie1.ActivE = true; break;
case 11: if (BD[3].yslovie2.ActivE == true) BD[3].yslovie2.ActivE = false;
else BD[3].yslovie2.ActivE = true; break;
case 12: if (BD[3].yslovie3.ActivE == true) BD[3].yslovie3.ActivE = false;
else BD[3].yslovie3.ActivE = true; break;
case 13: if (BD[4].yslovie1.ActivE == true) BD[4].yslovie1.ActivE = false;
else BD[4].yslovie1.ActivE = true; break;
case 14: if (BD[4].yslovie2.ActivE == true) BD[4].yslovie2.ActivE = false;
else BD[4].yslovie2.ActivE = true; break;
case 15: if (BD[4].yslovie3.ActivE == true) BD[4].yslovie3.ActivE = false;
else BD[4].yslovie3.ActivE = true; break;}
}}}оставлю тут
Psilon,
04 Марта 2013
−92
def register_valid_login
login = params[:login]
return register_error("Слишком длинный ник!") if (login.length > 16)
return register_error("Слишком короткий ник!") if (login.length < 3)
return register_error("Недопустимые символы в нике!") if !(/^[a-zA-Z\-_]+$/=~login)
return register_error("Можно использовать либо '-', либо '_' в нике!") if (login.count("-") > 0 && login.count("_") > 0)
return register_error("Ник должен состоять из трёх частей максимум!") if (login.count("-") > 2 || login.count("_") > 2)
login = [login] if (login.count("-") == 0 && login.count("_") == 0)
login = login.split("-") if login.count("-") > 0
login = login.split("_") if login.count("_") > 0
return register_error("Начало ника не может быть меньше трёх символов!") if (login[0].length < 1)
case login.count
when 2
return register_error("Если у вас ник вида Xxx_Yyy, то Yyy может состоять минимум из 3х букв") if (login[1].length < 3)
when 3
return register_error("Если у вас ник вида AAA_B_CCC, то B должна быть больше одной буквы") if (login[1].length < 1)
end
for i in 0..login.count-1
return register_error("Заглавная буква в середине " + (i+1).to_s + "-ой части ника") if !(/^[a-zA-Z]?[a-z]+$/=~login[i])
end
return true
endПроцедура проверки ника в ситеме регистрации одного игрового проекта.
DropWorld,
03 Марта 2013
+134
public void Children_OnAdd(object sender, EventArgs e)
{
((MyEventArgs<Document>)e).D.Parent = this;
}
taburetka,
03 Марта 2013
+131
public class UserData : EventArgs {
public int UserCod { get; set; }
public string Name { get; set; }
public UserData(DataRow dataRow)
{
UserCod = (int)(decimal)dataRow["user_cod"];
Name = (string)dataRow["name"];
}
}
Во время рефакторинга появилось сообщение от VS UserData нельзя привести к типу UserData...
Заглянули в один из классов UserData...
теперь понятно откуда запах)))
dukhovnyi,
02 Марта 2013
+125
//сервер сайд (из colors.php)
$colors = [
'banana' => 'yellow',
'orange' => 'orange',
'cucumber' => 'green'
];
echo $colors[$_GET['fruit']];
//клиент сайд (js)
var fruits = ['banana', 'orange', 'cucumber']
for (var i in fruits) {
var fruit = fruits[i]
$.get('colors.php?fruit='+fruit, function(color){
document.write(fruit +' is '+ color +'<br/>')
})
}
Этот код показывают и задают вопрос почти на любом собеседовании на должность web-программита:
Что здесь неверно и как это исправить?
LispGovno,
02 Марта 2013
+128
/// <summary>
/// Проверка нажатия 2 клавиш (напрмер: Ctrl+W)
/// </summary>
/// <param name="firstKey">Первая клавиша</param>
/// <param name="hotKey">Сочетание из БД</param>
/// <param name="e">Аргументы события нажатия клавиши</param>
/// <param name="Scope">Область видимости</param>
/// <returns>Действие</returns>
private static string CheckMultipleKeyPress(string firstKey, DBHotKeys_new hotKey, KeyEventArgs e, HotKeyScope Scope)
{
System.Text.StringBuilder concat = new System.Text.StringBuilder();
if (!string.IsNullOrEmpty(PreviousKey))
{
concat.Append('+');
}
if (!string.IsNullOrEmpty(firstKey))
{
concat.Append(firstKey).Append('+');
}
if (!string.IsNullOrEmpty(PreviousKey))
{
concat.Append(PreviousKey).Append('+');
}
PreviousKey = ReplaceKeyCode(e.KeyCode);
if (e.KeyCode != Keys.ControlKey)
{
concat.Append(ReplaceKeyCode(e.KeyCode));
}
//if (concat[0] == '+')
//{
// concat = new System.Text.StringBuilder(concat.ToString().Substring(1));
//}
if ((hotKey.Scope == Scope) && (concat.ToString().Equals(hotKey.HotKeyString)))
{
PreviousKey = string.Empty;
return hotKey.Action;
}
PreviousKey = string.Empty;
return string.Empty;
}
HLW,
02 Марта 2013
Follow us!