Лучший говнокод за всё время

Лучший говнокод

В номинации:

Лучший рейтинг
Самый комментируемый

За время:

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

+106

1
2
3
4

foreach (DataRow row in this.TestsTable.Select(query))
{
    return true;
}

Все тот же крупный и дорогой проект :)

minuzZ, 17 Августа 2012

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

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

+106

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

if ( ... )
  {
    if ( ... )
      {
        if ( ... )
          {
            usleep(250000);
          }
        else
          {
            sleep( 1 );
          }
      }
    else
      {
        if ( ... )
          {
            if ( ... )
              {
                usleep( 250000 );
              }
            else
              {
                sleep( ... );
              }
          }
        else
          {
            sleep( ... );
          }
      }
  }
else
  {
    usleep( 250000 );
  }

из главного цикла одного "рил-тайм" приложения. (комментарии, етц были удалены.)

каждый раз тестеры/кастомеры жалуются что приложение работает слишком медленно или слишком быстро - появляется либо новый if со слипом, либо новый else со слипом. за два года существования, вот до этого "полного" дерева доросло. и все равно не работает как надо. :)

Dummy00001, 13 Августа 2012

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

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

+106

1

IDictionary activeProcesses = (IDictionary)(new Hashtable());

По мотивам

http://govnokod.ru/11512

Код встречен в книге

http://tinyurl.com/eai-patterns

Можно объяснить человеку, не знакомому с тонкостями C#, зачем это нужно, если Hashtable : IDictionary?

roman-kashitsyn, 01 Августа 2012

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

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

+106

1
2
3
4
5
6
7

private void ReferenceButton_Click(object sender, EventArgs e)
        {
            this.ContentBox.Controls.Remove(this.BooksPanel);
            this.ContentBox.Controls.Remove(this.LogoPanel);
            this.ContentBox.Controls.Remove(this.SearchPanel);
            this.ContentBox.Controls.Add(this.FAQPanel);
        }

Удаляем старые контролы таким образом)
Почти в 100% случаев найдется умник, который назовет любой код - говном.

Govnisti_Diavol, 11 Марта 2012

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

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

+106

1
2
3
4
5
6
7
8

/// <summary>
/// Класс реализует логику определяения валидности объекта 
/// по принципу отсутсвия несоответсвий с рангом >1
/// </summary>
public class Decider : IDecider
{
    ...
}

Жаль только, что его метод не возвращает IDecision - для полноты картины.

banderror, 01 Марта 2012

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

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

+106

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

public static boolean isNumeric(string word)
{
	char[] charArray = word.toCharArray();
	
	foreach (char letter in charArray)
	{
		if (!(letter == '0' || letter == '1' || letter == '2' || letter == '3' || letter == '4' || letter == '5' || letter == '6' || letter == '7' || letter == '8' || letter == '9'))
		{
			return false;
		}
		return true;
	}
}

За одно и подскажите правильный, оптимальный вариант этой функции

vistefan, 28 Февраля 2012

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

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

+106

1

for(int iyhx=0; iyhx<nLen; iyhx++)

"А идите вы на iyhx."

someone, 03 Февраля 2012

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

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

+106

01
02
03
04
05
06
07
08
09
10
11

public void CheckMemoStyle(bool isChecked)
{
    if (isChecked)
    {
        memoStyleLayoutCntrl.Selected = true;
    }
    else
    {
        memoStyleLayoutCntrl.Selected = false;
    }
}

memoStyleLayoutCntrl.Selected = isChecked; уже не модно?

alpha2050, 19 Января 2012

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

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

+106

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

#include "stdafx.h"
#include "math.h"
#include "stdio.h"
#include "conio.h"
 
int main()
{ 
int p;
if (p==1 || p==21 || p==31 || p==41 || p==51 || p==61 || p==71 
|| p==81 || p==91   || p==101 || p==121 || p==131 || p==141 || p==151 
|| p==161 || p==171 || p==181 || p==191 || p==201 || p==221 || p==231 
|| p==241 || p==251 || p==261 || p==271 || p==281 || p==291 || p==301
|| p==321 || p==331 || p==341 || p==351 || p==361 || p==371 || p==381
|| p==391 || p==401 || p==421 || p==431 || p==451 || p==461 || p==471 
|| p==481 || p==491 || p==501 || p==521 || p==531 || p==541 || p==551 
|| p==561 || p==571 || p==581 || p==591 || p==601 || p==621 || p==631
|| p==641 || p==651 || p==661 || p==671 || p==681 || p==691 || p==701 
|| p==721 || p==731 || p==741 || p==751 || p==761 || p==771 || p==781 
|| p==791 || p==801 || p==821 || p==831 || p==841 || p==851 || p==861
|| p==871 || p==881 || p==891 || p==901 || p==921 || p==931 || p==941
|| p==951 || p==961 || p==971 || p==981 || p==991 )
printf("Hruvna\n");
else 
{
if (p>1 && p<5 || p>21 && p<25 || p>31 && p<35 
|| p>41 && p<45 || p>51 && p<55 || p>61 && p<65
|| p>71 && p<75 || p>81 && p<85 || p>91 && p<95
||p>101 && p<105 || p>121 && p<125 || p>131 && p<135
||p>141 && p<145 || p>151 && p<155 || p>161 && p<165
||p>171 && p<175 || p>181 && p<185 || p>191 && p<195
||p>201 && p<205 || p>211 && p<215 || p>221 && p<225
||p>231 && p<235 || p>241 && p<245 || p>251 && p<255
||p>261 && p<265 || p>271 && p<275 || p>281 && p<285
||p>291 && p<295 || p>301 && p<305 || p>311 && p<315
||p>321 && p<325 || p>331 && p<335 || p>341 && p<345
||p>351 && p<355 || p>361 && p<365 || p>371 && p<375
||p>381 && p<385 || p>391 && p<395 || p>401 && p<405
||p>411 && p<415 || p>421 && p<425 || p>431 && p<435
||p>441 && p<445 || p>451 && p<455 || p>461 && p<465
||p>471 && p<475 || p>481 && p<485 || p>491 && p<495
||p>501 && p<505 || p>511 && p<515 || p>521 && p<525
||p>531 && p<535 || p>541 && p<545 || p>551 && p<555
||p>561 && p<565 || p>571 && p<575 || p>581 && p<585
||p>591 && p<595 || p>601 && p<605 || p>611 && p<615
||p>621 && p<625 || p>631 && p<635 || p>641 && p<645
||p>651 && p<655 || p>661 && p<665 || p>671 && p<675
||p>681 && p<685 || p>691 && p<695 || p>701 && p<705
||p>711 && p<715 || p>721 && p<725 || p>731 && p<735
||p>741 && p<745 || p>741 && p<745 || p>751 && p<755
||p>761 && p<765 || p>771 && p<775 || p>781 && p<785
||p>791 && p<795 || p>801 && p<805 || p>811 && p<815
||p>821 && p<825 || p>831 && p<835 || p>841 && p<845
||p>851 && p<855 || p>861 && p<865 || p>871 && p<875
||p>881 && p<885 || p>891 && p<895 || p>901 && p<905
||p>911 && p<915 || p>921 && p<925 || p>931 && p<935
||p>941 && p<945 || p>951 && p<955 || p>961 && p<965
||p>971 && p<975 || p>981 && p<985 || p>991 && p<995)
printf("Hryvni\n");
}
 
if (p>1000)
printf("Error\n");
else 
printf("Hryven\n");
getch();
return 0;
}

Цитата автора: "Дано натуральное число s (s <= 1000), определяющий денежную сумму (в гривнах). Дать для этого числа название «гривна», «гривны» или «гривен». Составил программу а она не работает" Оригинал здесь: http://www.cyberforum.ru/cpp-beginners/thread258801.html?uri=/cpp-beginners/thread258801.html

Fenom, 30 Ноября 2011

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

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

+106

1
2
3
4
5
6

if ( 0 == strcmp( executable, "" ) ||
     0 == strcmp( executable, "NONE" ) ||
     0 == strcmp( executable, "none" ) ||
     0 == strcmp( executable, "None" )  )
{
/* ... */

... ровно так документация и описывает данный параметр: либо пустая строка, либо case-insensitive строка NONE.

Dummy00001, 25 Ноября 2011

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

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