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

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

В номинации:

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

За время:

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

+139

1

Response.Write("<" + '/'.ToString() + "script>");

Даже не знаю, кому и зачем это понадобилось...

Ordos, 24 Февраля 2011

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

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

+139

1
2
3
4
5
6
7
8
9

FILE *ConfigFile = NULL;
char *workdir = NULL;
workdir=getenv("APP_WORKDIR");
if ((NULL!=workdir) && (NULL != (ConfigFile = fopen(strcat(workdir,"appname/config.xml"), "r"))))
{
   fclose(ConfigFile);
   workdir[strlen(workdir)-10] = '\0';
   setenv("APP_RESOURCE_DIR", workdir , 1);
}

вот так нужно указывать крутой библиотеке, любимому и незаменимому детищу нашего R&D, читать файл из другой директории.

и вот такое было закопипащено в 4 приложений....

Dummy00001, 19 Ноября 2010

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

Куча / Говнокод #4491

+139

01
02
03
04
05
06
07
08
09
10

<noindex>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
...
<noindex>

Заказчик попросил изменить шаблон. Открываю, а там это

Setti, 31 Октября 2010

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

Куча / Говнокод #4233

+139

1

<input type="hidden" class="form" TYPE="checkbox" id="xxx" value="0"  onclick="if(this.value==1) {this.value} else {this.value=1}">

И как они хотят, чтобы на скрытый элемент нажимали?)

Nerd, 12 Сентября 2010

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

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

+139

1
2
3
4
5
6
7
8

#define MIN(i1, i2) (i1 < i2 ? i1 : i2)

int mr_word_compare(const char* r1, int s1, const char* r2, int s2)
{
    int l1 = strchr(r1, ' ') - r1;
    int l2 = strchr(r2, ' ') - r2;
    return strncmp(r1, r2, MIN(l1, l2));
}

пердложенный вариант исправления #4093 (http://govnokod.ru/4093)

vayerx, 25 Августа 2010

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

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

+139

1
2
3
4

inline Gdiplus::Color colorrefToGdiColor(COLORREF col, char alpha)
{
	return (static_cast<unsigned long>(static_cast<unsigned char>((col & Gdiplus::Color::RedMask) >> Gdiplus::Color::RedShift)) <<   Gdiplus::Color::BlueShift) | (static_cast<unsigned long>(static_cast<unsigned char>((col & Gdiplus::Color::GreenMask) >> Gdiplus::Color::GreenShift)) << Gdiplus::Color::GreenShift) | (static_cast<unsigned long>(static_cast<unsigned char>((col & Gdiplus::Color::BlueMask) >> Gdiplus::Color::BlueShift)) << Gdiplus::Color::RedShift) | (static_cast<unsigned long>(alpha) << Gdiplus::Color::AlphaShift);
}

Тихо себя ненавижу. Слава Б-г'у это всё выкидывается.

Altravert, 23 Июля 2010

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

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

+139

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

int main() {
  unsigned long oct;
  int m,n,i,j;	
  int max=0;
  
  scanf("%d",&oct);
  for(i=0;i<=32/3;i++) {
    m=0;
    for(j=0;j<=2;j++) {
      n=oct & 1;
      if(n==1)
        m=m | power(2,j);
      oct=oct >> 1;  
    } 
  if(m>max)
    max=m;                   
  }
  printf("%d",max);  
  return 0;
}

int power(int x, int a) {
  int i;
  int t=x;
  if(a==0)
    t=1;
  else {  
  for(i=1;i<a;i++)
    t*=x; 
  }
  return t;  
}

Определение максимальной цифры в восьмеричной записи числа.
Особенно радует вызов функции возведения в степень для двойки, да и сама функция тоже.

movaxbx, 11 Июня 2010

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

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

+139

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

// export in csv - part of the code
// ...
            foreach (users_view _item in _users_view)
            {
                _writer.Write(String.Format("{1}{0}{2}{0}{3}{0}{4}{0}{5}{0}{6}{0}{7}{0}{8}{0}{9}{0}{10}{0}{11}{0}{12}{0}{13}{0}{14}{0}{15}{0}{16}{0}{17}{0}{18}{0}{19}{0}{20}{0}{21}{0}{22}{0}{23}{0}{24}\n",
                    AppSettingsReader.GetValue("CSVFileSeparator"),
                     @"""" + _item.title_name + @"""", @"""" + _item.first_name + @"""",
                     @"""" + _item.last_name + @"""", @"""" + _item.job_title + @"""",
                    @"""" + _item.user_type_name + @"""",
                                            
                    @""""+_item.company_name+@"""",
                    (_item.telephone != null) ? (@"""" + _item.telephone + @"""") : (""),
                    (_item.fax != null) ? (@"""" + _item.fax + @"""") : (""),
                    @"""" + _item.email + @"""",
                    (_item.account_email != null) ? (@"""" + _item.account_email + @"""") : (""),
                    @"""" + _item.site_address + @"""",
                    @"""" + _item.advertisement_source_name+@"""",

                    @"""" + _item.address_1+@"""",
                    (_item.address_2 != null) ? (@"""" + _item.address_2+@"""") : (""),
                    @"""" + _item.country_name+@"""",
                   
                    //_item.email_format_name,

                    (_item.postcode != null) ? (@"""" + _item.postcode+@"""") : (""),
                    (_item.county != null) ? (@"""" + _item.county+@"""") : (""),
                    @"""" + _item.town + @"""",
                //    (_item.is_active == false) ? ("No") : ("Yes"),

                    @"""" + _item.username+@"""",
                    @"""" + _item.password+@"""",
                    @"""" + _item.account_type_name + @"""",

                    @"""" + _item.creation_date + @"""",

                    (_item.is_newsletter_subscriber == false) ? ("No") : ("Yes"),
                    (_item.is_marketing_subscriber == false) ? ("No") : ("Yes")
                    )
                );
            }

Вот что нашел :)

bugotrep, 14 Октября 2009

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

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

+139

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

public void Recalculate(List<DateTime> weekDates,List<HistoricalRateOccurence> historicalRates) {
            UnitsTotal = MonUnits + TuesUnits + WedsUnits + ThursUnits + FriUnits + SatUnits + SunUnits;
            if (historicalRates.Count == 0) {
                PayAmount = UnitsTotal*PayRate;
            }else {
                for (int i = 0; i < weekDates.Count; i++) {
                    switch (i) {
                        case 0:
                            PayAmount += MonUnits*GetPayRateForDay(PayRate, weekDates[i], historicalRates);
                            break;
                        case 1:
                            PayAmount += TuesUnits * GetPayRateForDay(PayRate, weekDates[i], historicalRates);
                            break;                            
                        case 2:
                            PayAmount += WedsUnits * GetPayRateForDay(PayRate, weekDates[i], historicalRates);
                            break;                            
                        case 3:
                            PayAmount += ThursUnits * GetPayRateForDay(PayRate, weekDates[i], historicalRates);
                            break;                            
                        case 4:
                            PayAmount += FriUnits * GetPayRateForDay(PayRate, weekDates[i], historicalRates);
                            break;

                    }
                }
            }
            ChargeAmount = UnitsTotal * AmsBillRate;
        }

Никогда не доверяй циклу!

xrundelek, 28 Августа 2009

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

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

+139

1

dgridPFF1.Rows[1].Cells[1].Value = UtilHelper.Number2String(Convert.ToDouble(UtilHelper.StringToInvariantCulture(dgridPFF1.Rows[0].Cells[1].Value), UtilHelper.getCultureInfo()) / Convert.ToDouble(UtilHelper.StringToInvariantCulture(dgridPFF2.Rows[0].Cells[0].Value),UtilHelper.getCultureInfo()), 2); // String.Format(UtilHelper.getCultureInfo(),UtilHelper.NUMBERIC_FORMAT_2,Convert.ToDouble(dgridPFF1.Rows[0].Cells[1].Value) / Convert.ToDouble(dgridPFF2.Rows[0].Cells[0].Value, UtilHelper.getCultureInfo()));

Виетнамский jungle code :)

bugotrep, 16 Августа 2009

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

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