По колено в коде — Говнокод.ру

PHP / Говнокод #1770

+158.9

1
2
3
4
5

static $badwords = array(
...
'(?<=[_\d]) [гg]_?[оo]_?[вvb]_?[нnh]_?[оoаaяеeyу]', #говн[оаяеу]
...
);

самый дословный (мета-)говнокод.

из censure.php -- Функция пытается определить наличие мата (нецензурных, матерных слов) в html-тексте.
http://forum.dklab.ru/viewtopic.php?t=17146

OlgaWolga, 05 Сентября 2009

Комментарии (9)

C++ / Говнокод #1769

+49.9

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43

bool __fastcall TForm1::ImportFile(AnsiString FName, int Direction)
{
bool StatusUpload;
StatusUpload = true;

TStringList *Inside;
Inside = new TStringList;
Inside->LoadFromFile(FName);
if (Direction == 1)
{
  // ...
  date_made = StrToSqlDate(Inside->Strings[1]);
  ops = Inside->Strings[0];

  Inside->Delete(0);
  Inside->Delete(0);
  Inside->Delete(0);

  int Iterations = (Inside->Count/5);
  for (int i=0; i<Iterations; i++)
  {
    AnsiString A, B, C, D, E;
    A = Inside->Strings[0].Trim();
    B = Inside->Strings[1].Trim();
    C = Inside->Strings[2].Trim();
    D = Inside->Strings[3].Trim();
    E = Inside->Strings[4].Trim();
    for (int j=0; j<5;j++)
       Inside->Delete(0);

    if (A == "1.1.2")
      E='1';

    if (A == "4.13" || A == "4.13" || A == "4.13")
	  E='20';
    C = RemoveZPT(C);
	ADOQuery1->SQL->Text = "INSERT INTO dnevnik_inside (row_id, ops, date_made, code, sign, val, val_init, direction, type) VALUES ("
      + row_id + ", '" + ops + "', '" + date_made + "', '" + A + "', '"
      + B + "', " + C + ", " + C + ", '" + D +"', '"+E+"')";
    ADOQuery1->ExecSQL();
  }
  Trans->CommitRetaining();
}

Собсно, название функции говорит само за себя :)
П.С. Коллега, не обижайся!

labutinpa, 05 Сентября 2009

Комментарии (15)

C++ / Говнокод #1768

+66.8

1
2
3
4
5
6
7
8
9

PChar GetFileContent(PChar FileName)
{
...
};

static inline signed int GetFileSize(PChar FileName)
{
  return(((signed int)(strlen(GetFileContent(FileName)))));
};

Всё гениальное - просто...

Говногость, 04 Сентября 2009

Комментарии (26)

C# / Говнокод #1767

+132.5

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32

foreach ( DataRow dr in dsWorkPolicyTa.Tables[0].Rows)
{
	if(strMasterNumber==dr["MAST_NO"].ToString() && strMasterSequence == dr["MAST_SEQ"].ToString() && strSectionSequence == dr["SEQ"].ToString() && strPremiumCurrency == dr["PREMIUM_CURRENCY"].ToString())
	{
		if (( strPremiumCurrency.CompareTo(dr["PREMIUM_CURRENCY"].ToString())==0)&& ( dr["ENDORSEMENT_NUMBER"].ToString().CompareTo(dsWorkPolicyTa.Tables[0].Rows[marker]["ENDORSEMENT_NUMBER"].ToString()) == 0 )&& (strEntrySurrogate==dr["EYAFNB"].ToString()) && ( dr["E2E0CD"].ToString().CompareTo(dsWorkPolicyTa.Tables[0].Rows[marker]["E2E0CD"].ToString()) == 0 ))
		{
			//Unauthorised/Policy Premium
			if((dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM"]!= System.DBNull.Value)&& (dr["PREMIUM"] != System.DBNull.Value)&& (dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM"].ToString()!= "")&& (dr["PREMIUM"].ToString() != ""))
				dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM"]=Convert.ToDouble(dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM"])+ Convert.ToDouble(dr["PREMIUM"]);
			else
				dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM"]=dr["PREMIUM"];
			if((dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM_BASE"]!= System.DBNull.Value)&& (dr["PREMIUM_BASE"] != System.DBNull.Value)&& (dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM_BASE"].ToString()!= "")&& (dr["PREMIUM_BASE"].ToString() != ""))
				dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM_BASE"]=Convert.ToDouble(dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM_BASE"])+ Convert.ToDouble(dr["PREMIUM_BASE"]);
			else
				dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM_BASE"]=dr["PREMIUM_BASE"];
			if((dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM_OURSHARE"]!= System.DBNull.Value) && (dr["PREMIUM_OURSHARE"] != System.DBNull.Value)&& (dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM_OURSHARE"].ToString()!= "") && (dr["PREMIUM_OURSHARE"].ToString() != ""))
				dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM_OURSHARE"]=Convert.ToDouble(dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM_OURSHARE"])+ Convert.ToDouble(dr["PREMIUM_OURSHARE"]);
			else
				dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM_OURSHARE"]=dr["PREMIUM_OURSHARE"];
			if((dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM_BASE_OURSHARE"]!= System.DBNull.Value)&& (dr["PREMIUM_BASE_OURSHARE"] != System.DBNull.Value) && (dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM_BASE_OURSHARE"].ToString()!= "")&& (dr["PREMIUM_BASE_OURSHARE"].ToString() != ""))
				dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM_BASE_OURSHARE"]=Convert.ToDouble(dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM_BASE_OURSHARE"])+ Convert.ToDouble(dr["PREMIUM_BASE_OURSHARE"]);
			else
				dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM_BASE_OURSHARE"]=dr["PREMIUM_BASE_OURSHARE"];
			//Quoted Premium
			if((dsWorkPolicyTa.Tables[0].Rows[marker]["QUOTED_PREMIUM"]!= System.DBNull.Value)&& (dr["QUOTED_PREMIUM"] != System.DBNull.Value) && (dsWorkPolicyTa.Tables[0].Rows[marker]["QUOTED_PREMIUM"].ToString()!= "")&& (dr["QUOTED_PREMIUM"].ToString() != ""))
				dsWorkPolicyTa.Tables[0].Rows[marker]["QUOTED_PREMIUM"]=Convert.ToDouble(dsWorkPolicyTa.Tables[0].Rows[marker]["QUOTED_PREMIUM"])+ Convert.ToDouble(dr["QUOTED_PREMIUM"]);
			else
				dsWorkPolicyTa.Tables[0].Rows[marker]["QUOTED_PREMIUM"]=dr["QUOTED_PREMIUM"];
			if((dsWorkPolicyTa.Tables[0].Rows[marker]["QUOTED_PREMIUM_BASE"]!= System.DBNull.Value)&& (dr["QUOTED_PREMIUM_BASE"] != System.DBNull.Value) && (dsWorkPolicyTa.Tables[0].Rows[marker]["QUOTED_PREMIUM_BASE"].ToString()!= "")&& (dr["QUOTED_PREMIUM_BASE"].ToString() != ""))
				dsWorkPolicyTa.Tables[0].Rows[marker]["QUOTED_PREMIUM_BASE"]=Convert.ToDouble(dsWorkPolicyTa.Tables[0].Rows[marker]["QUOTED_PREMIUM_BASE"])+ Convert.ToDouble(dr["QUOTED_PREMIUM_BASE"]);
			else
				dsWorkPolicyTa.Tables[0].Rows[marker]["QUOTED_PREMIUM_BASE"]=dr["QUOTED_PREMIUM_BASE"];

Кусок примера кода предоставленного кандидатом на должность программиста.
Модуль называеться "Genius".

При попытке наговнокодить все 100 строчек произошла "Ошибка компиляции кода: Такой длинный код врядли может быть смешным. Пожалуйста, ограничьтесь сотней строк и 4000 символами."
Пришлось ограничить 4000 символами, но не стало смешнее :(

OlgaWolga, 04 Сентября 2009

Комментарии (5)

C++ / Говнокод #1766

+63.1

1
2
3
4
5
6
7
8

if
(
IsFiltred
//проверка на фильтрованность массива
==
true
)
{

Нет слов.

Говногость, 04 Сентября 2009

Комментарии (15)

C++ / Говнокод #1765

+65.5

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17

...
switch( a )
{
  case false:
  {
    a=true;
  };break;
  case true:
  {
    a=false;
  };break;
  default:
  {
    a=false;
  };break;
};
...

a - буллёвная переменная.

Говногость, 04 Сентября 2009

Комментарии (14)

Java / Говнокод #1764

+68.6

01
02
03
04
05
06
07
08
09
10
11

// хрен разберешься
class  SaveOrUpdateAccountInBankStoredProcedure extends StoredProcedure {
....
  public String execute(String oid, String number, String name){
....
    Map result = super.execute(inputs);
    Iterator<Map.Entry> it = result.entrySet().iterator();
    List list = (List)((Map.Entry)it.next()).getValue();
    return  (String)((Map)list.get(0)).get(DaoConst.OID);
  }
}

Использование org.springframework.jdbc
Долго в отладчике пытался выяснить, какие объекты возвращаются в результате выполнения хранимой процедуры через этот framewaork :(

ngoro, 04 Сентября 2009

Комментарии (4)

PHP / Говнокод #1763

+160.2

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25

if($showmode == 0 || $showmode == 2):
	if($count['user'] != 0):
		if($count['user'] == 1):
		    echo '<p class="tpwhosonline">Members Online: <span>1</span></p>';
		else:
		    if($count['user'] > 1):
			    echo '<p class="tpwhosonline">Members Online: <span>' . $count['user'] . '</span></p>';
			endif;
		endif;
	else:
		echo '<p class="tpwhosonline">Members Online: <span>0</span></p>';
	endif;
	
    if($count['guest'] != 0):
		if($count['guest'] == 1):
		    echo '<p class="tpwhosonline">Guests Online: <span>1</span></p>';
		else:
		    if($count['guest'] > 1):
			    echo '<p class="tpwhosonline">Guests Online: <span>' . $count['guest'] . '</span></p>';
			endif;
		endif;
	else:
	    echo '<p class="tpwhosonline">Guests Online: <span>0</span></p>';
    endif;
endif;

и такая вот хрень еще и за бабки продается (Joomla template)

endi, 04 Сентября 2009

Комментарии (8)

C# / Говнокод #1762

+129.9

01
02
03
04
05
06
07
08
09
10
11
12
13
14

public static void ClearUsers()
        {
            var toremove = new List<string>();
            foreach (var user in SignedUsers.Values)
            {
                if (!user.IsActive)
                    toremove.Add(user.UserName);
            }
            foreach (var usr in toremove)
            {
                SignedUsers.Remove(usr);
            }

        }

Удаление юзера

DavidM, 04 Сентября 2009

Комментарии (20)

Си / Говнокод #1761

+133

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70

//рекурсивная процедура создающая структуру меню
byte form_hash(phasharray vrarr, WORD* ni, byte* ne)
{
	byte ct=MainMenu[(*ni)].numSubMenus, ict;
	phasharray	vrarrcr;

	if((*ni)++==0) pmain=vrarr; // если первый проход - запоминаем ссылку на главное меню
	for (ict=0; ict<ct; ict++)
	{
		vrarr->hash[ict].pitem=(tmenuitem*)RTKGetBuffer(&menuPool);
		//инициализация строки элемента
		scopy0(((*((*vrarr).hash[ict].pitem)).menustr), MainMenu[*ni].nameMenu);
		if (MainMenu[*ni].numSubMenus!=0)
		{
			(*((*vrarr).hash[ict].pitem)).pnextlev=RTKGetBuffer(&hashMenuPool);// создание хэша
			vrarrcr=(phasharray)(*((*vrarr).hash[ict].pitem)).pnextlev;//вводим новую переменную для облегчения доступа к созданной структуре
			(*vrarr).hash[ict].quantity=(byte)MainMenu[*ni].numSubMenus;//запоминаем количество элементов в меню уровнем выше
			(*vrarrcr).pPrevLev=vrarr; // ссылка на предыдущий элемент
			(*((*vrarr).hash[ict].pitem)).fpmenu=true;
			if (MainMenu[*ni].numSubMenus==MENU_FUNC) // вызов процедуры динамического формирования меню
			{
				menupr_params.vrarr=vrarrcr;
				menupr_params.phashprev=vrarr;
				menupr_params.ni=ni;
				menupr_params.ne=ne;
				menupr_params.ict=ict;
				(MainMenu[*ni].pproc)(); 
				(*ni)++;
			} else 
			{
				if(MainMenu[*ni].pproc) // если исключение (перед входом в подменю необходимо выполнить функцию)
				{
					changeArr[(*ne)++]=vrarrcr;
					(*((*vrarr).hash[ict].pitem)).pexec=MainMenu[*ni].pproc;
					(*((*vrarr).hash[ict].pitem)).fpmenu=false;
				};
				// исправляем значение элементов
				(*(vrarr)).hash[ict].quantity=form_hash(vrarrcr, ni, ne);
			};
			// проверка сформированного меню
			switch ((*(vrarr)).hash[ict].quantity)
			{
			case MENU_NO: // нет подменю
				// освободить эл-т
				RTKFreeBuffer(&hashMenuPool, vrarrcr); 
				(*((*(vrarr)).hash[ict].pitem)).pexec=*menu_refresh;
				(*((*(vrarr)).hash[ict].pitem)).fpmenu=false;
				break;
			case MENU_NOEL: // не формировать эл-т
				// освободить эл-т
				RTKFreeBuffer(&hashMenuPool, vrarrcr); 
				RTKFreeBuffer(&menuPool, vrarr->hash[ict].pitem);
				ict--;
				ct--;
				break;
			case MENU_FUNC: // нет подменю, нормальная функция
				RTKFreeBuffer(&hashMenuPool, vrarrcr); 
				(*(vrarr)).hash[ict].quantity=0; 
				//(*((*vrarr).hash[ict].pitem)).fpmenu=false;
				break;
			}
		} else // заполнение элемента
		{
			(*((*vrarr).hash[ict].pitem)).pexec=MainMenu[*ni].pproc;
			(*((*vrarr).hash[ict].pitem)).fpmenu=false;
			(*ni)++;
		};
	};
	return ct;
};

Эх, бурная была молодость :-)

Barmaglot, 04 Сентября 2009

Комментарии (26)

Говнокод: по колено в коде.

PHP / Говнокод #1770

C++ / Говнокод #1769

C++ / Говнокод #1768

C# / Говнокод #1767

C++ / Говнокод #1766

C++ / Говнокод #1765

Java / Говнокод #1764

PHP / Говнокод #1763

C# / Говнокод #1762

Си / Говнокод #1761