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

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• Oтзывы

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

1. PHP (5708)
2. C++ (2739)
3. Куча (2378)
4. JavaScript (2032)
5. C# (1929)
6. Java (1502)
7. Си (1117)
8. Pascal (648)
9. Python (591)
10. 1C (539)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (200)
15. Perl (194)
16. Assembler (146)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (34)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

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

   В номинации:

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

   За время:

   • за сегодня
   • за неделю
   • за месяц
   • за всё время

2. Куча / Говнокод #9933

   +132

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11
   12. 12
   13. 13
   14. 14
   15. 15
   16. 16
   17. 17
   18. 18
   19. 19
   20. 20
   21. 21
   22. 22

   <asp:Label></asp:Label>
   <br>
   <asp:TextBox></asp:TextBox>
   <br>
   <br>
   <br>
   <br></br>
   <asp:Label></asp:Label>
   <br>
   <asp:TextBox></asp:TextBox>
   <br>
   <br>
   <br>
   <br></br>
   <asp:Button />
   <br></br>
   </br>
   </br>
   </br>
   </br>
   </br>
   </br>

   новый сотрудник "наваял" страницу. кто его надоумил использовать закрывающий тег </br>, да еще и по такому странному принципу - неизвестно...

   _vinka, 12 Апреля 2012

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

3. Perl / Говнокод #9736

   −126

   1. 1

   if (!defined $login || (defined $login && length($login)<3 && length($login>16))) {

   PSIAlt, 22 Марта 2012

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

4. C++ / Говнокод #9726

   +995

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11
   12. 12
   13. 13
   14. 14
   15. 15
   16. 16
   17. 17
   18. 18
   19. 19
   20. 20
   21. 21
   22. 22
   23. 23
   24. 24
   25. 25
   26. 26
   27. 27
   28. 28
   29. 29
   30. 30
   31. 31
   32. 32
   33. 33
   34. 34
   35. 35
   36. 36
   37. 37
   38. 38
   39. 39
   40. 40
   41. 41
   42. 42
   43. 43
   44. 44
   45. 45
   46. 46
   47. 47
   48. 48
   49. 49
   50. 50
   51. 51
   52. 52
   53. 53
   54. 54
   55. 55
   56. 56
   57. 57
   58. 58
   59. 59
   60. 60
   61. 61
   62. 62
   63. 63
   64. 64
   65. 65
   66. 66
   67. 67
   68. 68
   69. 69
   70. 70
   71. 71
   72. 72
   73. 73
   74. 74
   75. 75
   76. 76
   77. 77
   78. 78

   Для Каждого ТекСтрокаТовары Из ЭтотОбъект.Товары Цикл
   		
         //Товары в резерве на складах
         Если ЗначениеЗаполнено(ЭтотОбъект.ЗаказПокупателя) Тогда
   
           ЗапросРезерв = Новый Запрос;
           ЗапросРезерв.УстановитьПараметр("Период", КонецДня(ЭтотОбъект.Дата));
           ЗапросРезерв.УстановитьПараметр("Склад", ЭтотОбъект.Склад);
           ЗапросРезерв.УстановитьПараметр("Номенклатура", ТекСтрокаТовары.Номенклатура);
   	ЗапросРезерв.УстановитьПараметр("Цвет", ТекСтрокаТовары.Цвет);
   	ЗапросРезерв.УстановитьПараметр("Регистратор", ЭтотОбъект.ЗаказПокупателя.Ссылка);
   	ЗапросРезерв.Текст = "ВЫБРАТЬ
   	|	ТоварыВРезервеНаСкладах.Номенклатура,
   	|	isnull(ТоварыВРезервеНаСкладах.Количество, 0) КАК Количество,
   	|	isnull(ТоварыВРезервеНаСкладах.Размер42, 0) КАК Размер42,
   	|	isnull(ТоварыВРезервеНаСкладах.Размер44, 0) КАК Размер44,
   	|	isnull(ТоварыВРезервеНаСкладах.Размер46, 0) КАК Размер46,
   	|	isnull(ТоварыВРезервеНаСкладах.Размер48, 0) КАК Размер48,
   	|	isnull(ТоварыВРезервеНаСкладах.Размер50, 0) КАК Размер50,
   	|	isnull(ТоварыВРезервеНаСкладах.Размер52, 0) КАК Размер52,
   	|	isnull(ТоварыВРезервеНаСкладах.Размер54, 0) КАК Размер54,
   	|	isnull(ТоварыВРезервеНаСкладах.Размер56, 0) КАК Размер56,
   	|	isnull(ТоварыВРезервеНаСкладах.Размер58, 0) КАК Размер58,
   	|	isnull(ТоварыВРезервеНаСкладах.Размер60, 0) КАК Размер60
   	|ИЗ
   	|	РегистрНакопления.ТоварыВРезервеНаСкладах КАК ТоварыВРезервеНаСкладах
   	|ГДЕ
   	|	ТоварыВРезервеНаСкладах.ВидДвижения = ЗНАЧЕНИЕ(ВидДвиженияНакопления.Приход)
   	|	ТоварыВРезервеНаСкладах.Период <= &Период
   	|	И ТоварыВРезервеНаСкладах.Регистратор = &Регистратор
   	|	И ТоварыВРезервеНаСкладах.Склад = &Склад
   	|	И ТоварыВРезервеНаСкладах.Номенклатура = &Номенклатура
   	|	И ТоварыВРезервеНаСкладах.Цвет = &Цвет";
   	ТЗРезерв = ЗапросРезерв.Выполнить().Выгрузить();
   			
   	Если ЗапросРезерв.Выполнить().Пустой() = Истина Тогда
   	Иначе
   		ЗапросСнятие = Новый Запрос;
   		ЗапросСнятие.УстановитьПараметр("Период", КонецДня(ЭтотОбъект.Дата));
   		ЗапросСнятие.УстановитьПараметр("Склад", ЭтотОбъект.Склад);
   		ЗапросСнятие.УстановитьПараметр("Номенклатура", ТекСтрокаТовары.Номенклатура);
   		ЗапросСнятие.УстановитьПараметр("Цвет", ТекСтрокаТовары.Цвет);
   		ЗапросСнятие.УстановитьПараметр("ДокументРезерва", ЭтотОбъект.ЗаказПокупателя.Ссылка);
   		ЗапросСнятие.Текст = "ВЫБРАТЬ
   		|	ТоварыВРезервеНаСкладах.Номенклатура,
   		|	isnull(ТоварыВРезервеНаСкладах.Количество, 0) КАК Количество,
   		|	isnull(ТоварыВРезервеНаСкладах.Размер42, 0) КАК Размер42,
   				.....
   		|	isnull(ТоварыВРезервеНаСкладах.Размер60, 0) КАК Размер60			
   		|ИЗ
   		|	РегистрНакопления.ТоварыВРезервеНаСкладах КАК ТоварыВРезервеНаСкладах
   		|ГДЕ
   		|	ТоварыВРезервеНаСкладах.ВидДвижения = ЗНАЧЕНИЕ(ВидДвиженияНакопления.Расход)
   		|	И ТоварыВРезервеНаСкладах.ДокументРезерва = &ДокументРезерва
   		|	И ТоварыВРезервеНаСкладах.Период = &Период
   		|	И ТоварыВРезервеНаСкладах.Склад = &Склад
   		|	И ТоварыВРезервеНаСкладах.Номенклатура = &Номенклатура
   		|	И ТоварыВРезервеНаСкладах.Цвет = &Цвет";
   		ТЗСнятие = ЗапросСнятие.Выполнить().Выгрузить();
   				
   		//Товары в резерве на складах
   		Движение = Движения.ТоварыВРезервеНаСкладах.Добавить();
   		Движение.ВидДвижения = ВидДвиженияНакопления.Расход;
   		Движение.Период = Дата;
   		Движение.Склад = ЭтотОбъект.Склад; 
   		Движение.Номенклатура = ТекСтрокаТовары.Номенклатура;
   		Движение.ДокументРезерва = ЭтотОбъект.ЗаказПокупателя.Ссылка;
   		Движение.Цвет = ТекСтрокаТовары.Цвет;
   		//42
   		Если ТЗРезерв.Итог("Размер42") - ТЗСнятие.Итог("Размер42") > 0 Тогда
   			Если ТЗРезерв.Итог("Размер42") - ТЗСнятие.Итог("Размер42") > ТекСтрокаТовары.Количество Тогда
   				Движение.Размер42 = ТекСтрокаТовары.Размер42;
   			Иначе
   				Движение.Размер42 = ТЗРезерв.Итог("Размер42") - ТЗСнятие.Итог("Размер42");
   			КонецЕсли;
   		КонецЕсли;      
   		...
                   //И так до 60го размера

   Нашел у себя эту простыню. Чтобы вычислить остатки по резерву - делается запрос по движения со знаком плюс, потом запрос с движениями по минус.
   Далее все это дело выгружается в две таблицы значений и высчитывается разность колонок. Вместо простого запроса по регистру остатков) Ну и много чего интересного)

   ErrorEd88, 21 Марта 2012

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

5. C# / Говнокод #9699

   +108

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11

   public static string RandomWord(int Number)
   {    
   	Random RandomIndex = new Random();
   	string[] ArrayLetter = {"q", "w", "e", "r", "t", "y", "u", "i", "o", "p", "a", "s", "d", "f", "g", "h", "j", "k", "l", "z", "x", "c", "v", "b", "n", "m"};
   	string OutputValue = null;
   	for (int i = 0; i < Number; i++)
   	{
   	OutputValue += ArrayLetter[RandomIndex.Next(0, 25)];
   	}
   	return OutputValue;
   }

   Метод выводит случайное слово блин.

   KusokGovna, 17 Марта 2012

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

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

   +106

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11
   12. 12
   13. 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)

7. C# / Говнокод #9443

   +960

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11
   12. 12
   13. 13
   14. 14
   15. 15

   private static void WriteErrLog(string FileName, string errors)
   {
       try
       {
           string name = Environment.CurrentDirectory + "\\log\\" + FileName.Replace("80020_", "");
           name = name.Replace(".xml", ".txt");
           if (File.Exists(name))
               name = name.Replace(".txt", "_" + DateTime.Now.ToString("yyyy-MM-dd hh_mm_ss") + ".txt");                
           using (StreamWriter f = new StreamWriter(name, true, Encoding.GetEncoding(1251)))
           {
               f.Write(errors);
           }
       }
       catch { }
   }

   Продолжаем...

   invi, 16 Февраля 2012

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

8. PHP / Говнокод #9429

   +159

   1. 1
   2. 2
   3. 3

   exit;
   break;
   return false;

   Вот так истинные индусы завершают выполнение скрипта. Чтоб уж наверняка.
   Только die(); не хватает.

   ruslan441, 15 Февраля 2012

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

9. SQL / Говнокод #9386

   −121

   1. 1
   2. 2
   3. 3
   4. 4
   5. 5

   CASE st.testmode_order_active
   WHEN NULL THEN '1'
   WHEN '1' THEN '0'
   WHEN '0' THEN '1'
   as testmode_product_active

   кусок sql запроса с реального проекта. логика в БД одна, в приложении - обратная.

   codename, 10 Февраля 2012

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

10. PHP / Говнокод #9307

    +167

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45

    <p class="last_pay">
                            <?if($arResult["history"]["PROPERTIES"]["payment10_date"]["VALUE"]):?>
                                 <?=$arResult["history"]["PROPERTIES"]["payment10_date"]["VALUE"]?> Произведен платеж на ... бла бла
                            <?else:?>
                                <?if($arResult["history"]["PROPERTIES"]["payment9_date"]["VALUE"]):?>
                                     <?=$arResult["history"]["PROPERTIES"]["payment9_date"]["VALUE"]?> Произведен платеж на сумму ... 
                                <?else:?>
                                    <?if($arResult["history"]["PROPERTIES"]["payment8_date"]["VALUE"]):?>
                                         <?=$arResult["history"]["PROPERTIES"]["payment8_date"]["VALUE"]?> Произведен платеж на сумму ... 
                                    <?else:?>
                                        <?if($arResult["history"]["PROPERTIES"]["payment7_date"]["VALUE"]):?>
                                             <?=$arResult["history"]["PROPERTIES"]["payment7_date"]["VALUE"]?> Произведен платеж на сумму  ... 
                                        <?else:?>
                                            <?if($arResult["history"]["PROPERTIES"]["payment6_date"]["VALUE"]):?>
                                                 <?=$arResult["history"]["PROPERTIES"]["payment6_date"]["VALUE"]?> Произведен платеж на сумму  ...
                                            <?else:?>
                                                <?if($arResult["history"]["PROPERTIES"]["payment5_date"]["VALUE"]):?>
                                                     <?=$arResult["history"]["PROPERTIES"]["payment5_date"]["VALUE"]?> Произведен платеж на сумму  ...
                                                <?else:?>
                                                    <?if($arResult["history"]["PROPERTIES"]["payment4_date"]["VALUE"]):?>
                                                         <?=$arResult["history"]["PROPERTIES"]["payment4_date"]["VALUE"]?> Произведен платеж на сумму  ... 
                                                    <?else:?>
                                                        <?if($arResult["history"]["PROPERTIES"]["payment3_date"]["VALUE"]):?>
                                                             <?=$arResult["history"]["PROPERTIES"]["payment3_date"]["VALUE"]?> Произведен платеж на сумму  ... 
                                                        <?else:?>
                                                            <?if($arResult["history"]["PROPERTIES"]["payment2_date"]["VALUE"]):?>
                                                                 <?=$arResult["history"]["PROPERTIES"]["payment2_date"]["VALUE"]?> Произведен платеж на сумму  ... 
                                                            <?else:?>
                                                                <?if($arResult["history"]["PROPERTIES"]["payment1_date"]["VALUE"]):?>
                                                                     <?=$arResult["history"]["PROPERTIES"]["payment1_date"]["VALUE"]?> Произведен платеж на сумму  ... 
                                                                <?else:?>
                                                                    Вы пока не производили платежи
                                                                <?endif;?>
                                                            <?endif;?>
                                                        <?endif;?>
                                                    <?endif;?>
                                                <?endif;?>
                                            <?endif;?>
                                        <?endif;?>
                                    <?endif;?>
                                <?endif;?>
                            <?endif;?>
                        <?endif;?>
     
                        </p>

    красиво

    SuperChel, 03 Февраля 2012

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

11. PHP / Говнокод #9302

    +157

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    51. 51
    52. 52
    53. 53
    54. 54
    55. 55
    56. 56
    57. 57
    58. 58
    59. 59
    60. 60
    61. 61
    62. 62
    63. 63

    <ul>
        <?
        $AchievmentPoints = $opts->getAchievmentPoints();
        $dost='';
        $stat='';
        ?>
    
                
                
        <? foreach ($achievments as $id => $item): ?>
    <?
    
        
        ?>
         <?$dost.='<li style="width:100%;clear:both;" class="table1"><div class="table">
             <div style="width:35px;float:left;display:table-cell;text-align: right">'.$item->getImg().'
                  </div><div style="float:left;"><div><span class="caption">'.$item->caption.'
                     </span> <span class="small minor" style="font-size:14px;">('.$item->level.')
                         </span></div><div class="minor small" style="width:100%">'.$item->subtext.':
                             <span  class="ach"><span>'.$item->count.'</span><span>/'.$item->nextLevelCount().'
                                 </span></span></div></div></div><div style="height:40px;"></div></li>';?>
        <? endforeach; ?>
    <?
    $stat.='
        <li style="width:100%;clear:both;" class="table1"> 
        <p style="padding-top: -2px;margin-top: 7px;"></p>
        <div class="table">
            <div style="width:35px;float:left;display:table-cell;text-align: right">
                        <img width="16" height="16" src="/images/icons/achive_on.png" alt="o"/>  
                    </div>
            <div style="float:left;">
                <span class="caption">Получено</span>
                <span  class="ach">
                                <span>('.$getAchievmentPoints.')</span>
                            </span>
            </div></div></li>
    <li style="width:100%;clear:both;" class="table1"> <div class="table">
            <div style="width:35px;float:left;display:table-cell;text-align: right">
                        <img width="16" height="16" src="/images/icons/achive_off.png" alt="o"/>  
                    </div>
            <div style="float:left;">
                <span class="caption" >Не получено</span>
                            <span  class="ach">
                                <span>('.(count($achievments) - $AchievmentPoints).')</span>
                            </span>
            </div></div></li>
            <p style="padding-top: -2px;margin-top: 7px;"></p>
            <div class="eventlight" style="height:2px;width:100%;"></div>
    ';
    echo $stat.$dost;
    ?>
    <li style="width:100%;clear:both;" class="table1"> <div class="table">
            <div style="width:35px;float:left;display:table-cell;text-align: right">
                        <img width="16" height="16" src="/images/icons/achive_off.png" alt="o"/>  
                    </div>
            <div style="float:left;">
                <span class="caption" >Всего достижений:</span>
                            <span  class="ach">
                                <span>(<?= $AchievmentPoints;?>)</span>
                            </span>
            </div></div></li>
     
        </ul>

    Мой первый гомнокод :)

    kolko91, 02 Февраля 2012

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

• ← влево
• вправо →

• 1972
• …
• 1232
• 1231
• 1230
• 1229
• 1228
• 1227
• 1226
• 1225
• 1224
• …
• 1