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

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

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

1. PHP (5708)
2. C++ (2740)
3. Куча (2386)
4. JavaScript (2034)
5. C# (1930)
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 (148)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (34)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

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

   В номинации:

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

   За время:

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

2. PHP / Говнокод #19690

   +2

   1. 1

   https://habrahabr.ru/post/280121/

   RestAPI в 2016 году. Отсосите, любители фреймворков и оттестированных библиотек

   loki90, 24 Марта 2016

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

3. C++ / Говнокод #19648

   +2

   1. 1
   2. 2
   3. 3
   4. 4
   5. 5
   6. 6
   7. 7
   8. 8

   /**
     * @brief Макрос вызова функции изменения имени файла лога ПИМ
     *
     * @param FILE_NAME  Имя файла лога
     *
     * @return Ничего
     */
   #define LOG_PIM_SET_FILE_NAME(FILE_NAME)  Soc::setLogFileName(FILE_NAME)

   Самый нужный макрос в мире

   cpp_best_practo, 17 Марта 2016

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

4. PHP / Говнокод #19644

   +2

   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
   79. 79
   80. 80
   81. 81
   82. 82
   83. 83

   //получаем Элемент 
   $resource = CIBlockElement::GetByID($Bid);
   if ($ob = $resource->GetNextElement())
   {
       //вытягиваем поля, свойства
       $arFields = $ob->GetFields();
       $arFields['PROPERTIES'] = $ob->GetProperties();
   	
       // удаляем все поля, нахрен мы их вообще выдергивали
       $arFieldsCopy = $arFields;
   	unset(
   		$arFieldsCopy['ID'], 
   		$arFieldsCopy['~ID'], 
   		$arFieldsCopy['IBLOCK_SECTION_ID'], 
   		$arFieldsCopy['~IBLOCK_SECTION_ID'], 
   		$arFieldsCopy['TIMESTAMP_X'], 
   		$arFieldsCopy['~TIMESTAMP_X'], 
   		$arFieldsCopy['ACTIVE_TO'], 
   		$arFieldsCopy['~ACTIVE_TO'], 
   		$arFieldsCopy['DATE_ACTIVE_TO'], 
   		$arFieldsCopy['~DATE_ACTIVE_TO'], 
   		$arFieldsCopy['PREVIEW_PICTURE'], 
   		$arFieldsCopy['~PREVIEW_PICTURE'], 
   		$arFieldsCopy['PREVIEW_TEXT'], 
   		$arFieldsCopy['~PREVIEW_TEXT'], 
   		$arFieldsCopy['DETAIL_PICTURE'], 
   		$arFieldsCopy['~DETAIL_PICTURE'], 
   		$arFieldsCopy['DETAIL_TEXT'], 
   		$arFieldsCopy['~DETAIL_TEXT'], 
   		$arFieldsCopy['XML_ID'], 
   		$arFieldsCopy['~XML_ID'], 
   		$arFieldsCopy['EXTERNAL_ID'], 
   		$arFieldsCopy['~EXTERNAL_ID'], 
   		$arFieldsCopy['TMP_ID'], 
   		$arFieldsCopy['~TMP_ID'], 
   		$arFieldsCopy['CODE'], 
   		$arFieldsCopy['~CODE'], 
   		$arFieldsCopy['SEARCHABLE_CONTENT'], 
   		$arFieldsCopy['~SEARCHABLE_CONTENT'], 
   		$arFieldsCopy['WF_LAST_HISTORY_ID'], 
   		$arFieldsCopy['~WF_LAST_HISTORY_ID'], 
   		$arFieldsCopy['WF_COMMENTS'], 
   		$arFieldsCopy['~WF_COMMENTS'], 
   		$arFieldsCopy['WF_DATE_LOCK'], 
   		$arFieldsCopy['~WF_DATE_LOCK'], 
   		$arFieldsCopy['WF_LOCKED_BY'], 
   		$arFieldsCopy['~WF_LOCKED_BY'], 
   		$arFieldsCopy['WF_NEW'], 
   		$arFieldsCopy['~WF_NEW'], 
   		$arFieldsCopy['WF_PARENT_ELEMENT_ID'], 
   		$arFieldsCopy['~WF_PARENT_ELEMENT_ID'], 
   		$arFieldsCopy['SHOW_COUNTER'], 
   		$arFieldsCopy['~SHOW_COUNTER'], 
   		$arFieldsCopy['TAGS'], 
   		$arFieldsCopy['~TAGS'], 
   		$arFieldsCopy['LOCKED_USER_NAME'], 
   		$arFieldsCopy['~LOCKED_USER_NAME'], 
   		$arFieldsCopy['SHOW_COUNTER_START'],
   		$arFieldsCopy['~SHOW_COUNTER_START']
   	);
          
        
           $arFieldsCopy['ACTIVE']='N';
   	$arFieldsCopy['ACTIVE_FROM']=date('d.m.Y');
          /*... тра тра та заполняю я поля ....*/ 
   
        $arFieldsCopy['PROPERTY_VALUES'] = [
          /* ... тра та та, 
                        тра та та,
                           заполняю свойства я  
               ...
         */
        ];
          /*
              ... Продолжая песню петь
                      удалю я все теперь 
              ...
          */
   	unset($arFieldsCopy['PROPERTIES']);
      // кульминация пьесы, добавление нового Элемента 
       $el = new CIBlockElement();
       $NEW_BID_ID = $el->Add($arFieldsCopy);
   }

   Эм.... толи память экономит, то ли оплата за код построчно ....

   BroadcastAddress, 16 Марта 2016

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

5. PHP / Говнокод #19638

   +2

   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

   //#1
    $queryo = "SELECT * FROM `tx_carmodels` WHERE  1 <= ID AND ID <= (SELECT COUNT(*) FROM `tx_carmodels`) ";
    $resulto = mysql_query($queryo);
    $sqlresos = mysql_fetch_assoc($resulto);
    foreach  ($sqlresos as $sqlreso){
           $preVendor[] = $sqlreso["vendor"];
                     //  print_r($sqlreso);
    }
    $arVendor = array_unique($preVendor);
    foreach($arVendor as $vendor) {
                       echo '<option value="'.$vendor.'">'.$vendor.'</option>';
      }
   // #2
   
   $query = "SELECT count(*) FROM `tx_carmodels`";
   $result = mysql_query($query);
   $res = mysql_fetch_array($result);
   $allinfile = $res[0];
   
        for ($i = 1; $i <= $allinfile; $i++) {
              $queryo = "SELECT * FROM `tx_carmodels` WHERE id=$i";
              $resulto = mysql_query($queryo);
              $sqlreso = mysql_fetch_assoc($resulto);
              $preVendor[] = $sqlreso["vendor"];
                   //print_r($sqlreso);
        }
        $arVendor = array_unique($preVendor);
        foreach($arVendor as $vendor) {
              $vsel = ($_GET["vendor"] == $vendor)?('selected="selected"'):('');
              echo '<option value="'.$vendor.'" '.$vsel.'>'.$vendor.'</option>';
         }

   Два перла,
   даже и коментировать нечего ....

   BroadcastAddress, 16 Марта 2016

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

6. PHP / Говнокод #19637

   +2

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

   // Статическая типизация для PHP < 7.0
   
   set_error_handler(function(){});
   
   function foo(int $bar)
   {
   	var_dump($bar);
   }
   
   foo(1); // Работает и выводит int(1)

   Недавно разгребал легасикод доставшийся по наследству. Увидел статическую типизацию там где ее не должно было быть. Проект крутится на 5.3.

   KiTE, 16 Марта 2016

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

7. PHP / Говнокод #19616

   +2

   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

   <div class="form-group">
   	<label for="inputName" class="col-sm-2 control-label">День рождения:</label>
   	<div class="col-sm-3">
   		<select class="form-control" id="day">
   			<option <?php if($explode[2] == 01) Echo "selected"; ?> value="01">01</option>
   			<option <?php if($explode[2] == 02) Echo "selected"; ?> value="02">02</option>
   			<option <?php if($explode[2] == 03) Echo "selected"; ?> value="03">03</option>
   			<option <?php if($explode[2] == 04) Echo "selected"; ?> value="04">04</option>
   			<option <?php if($explode[2] == 05) Echo "selected"; ?> value="05">05</option>
   			<option <?php if($explode[2] == 06) Echo "selected"; ?> value="06">06</option>
   			<option <?php if($explode[2] == 07) Echo "selected"; ?> value="07">07</option>
   			<option <?php if($explode[2] == 08) Echo "selected"; ?> value="08">08</option>
   			<option <?php if($explode[2] == 09) Echo "selected"; ?> value="09">09</option>
   			<option <?php if($explode[2] == 10) Echo "selected"; ?> value="10">10</option>
   			<option <?php if($explode[2] == 11) Echo "selected"; ?> value="11">11</option>
   			<option <?php if($explode[2] == 12) Echo "selected"; ?> value="12">12</option>
   			<option <?php if($explode[2] == 13) Echo "selected"; ?> value="13">13</option>
   			<option <?php if($explode[2] == 14) Echo "selected"; ?> value="14">14</option>
   			<option <?php if($explode[2] == 15) Echo "selected"; ?> value="15">15</option>
   			<option <?php if($explode[2] == 16) Echo "selected"; ?> value="16">16</option>
   			<option <?php if($explode[2] == 17) Echo "selected"; ?> value="17">17</option>
   			<option <?php if($explode[2] == 18) Echo "selected"; ?> value="18">18</option>
   			<option <?php if($explode[2] == 19) Echo "selected"; ?> value="19">19</option>
   			<option <?php if($explode[2] == 20) Echo "selected"; ?> value="20">20</option>
   			<option <?php if($explode[2] == 21) Echo "selected"; ?> value="21">21</option>
   			<option <?php if($explode[2] == 22) Echo "selected"; ?> value="22">22</option>
   			<option <?php if($explode[2] == 23) Echo "selected"; ?> value="23">23</option>
   			<option <?php if($explode[2] == 24) Echo "selected"; ?> value="24">24</option>
   			<option <?php if($explode[2] == 25) Echo "selected"; ?> value="25">25</option>
   			<option <?php if($explode[2] == 26) Echo "selected"; ?> value="26">26</option>
   			<option <?php if($explode[2] == 27) Echo "selected"; ?> value="27">27</option>
   			<option <?php if($explode[2] == 28) Echo "selected"; ?> value="28">28</option>
   			<option <?php if($explode[2] == 29) Echo "selected"; ?> value="29">29</option>
   			<option <?php if($explode[2] == 30) Echo "selected"; ?> value="30">30</option>
   			<option <?php if($explode[2] == 31) Echo "selected"; ?> value="31">31</option>
   		</select>
   	</div>
   	<div class="col-sm-4">
   		<select class="form-control" id="month">
   			<option <?php if($explode[3] == 01) Echo "selected"; ?> value="01">Января</option>
   			<option <?php if($explode[3] == 02) Echo "selected"; ?> value="02">Февраля</option>
   			<option <?php if($explode[3] == 03) Echo "selected"; ?> value="03">Марта</option>
   			<option <?php if($explode[3] == 04) Echo "selected"; ?> value="04">Апреля</option>
   			<option <?php if($explode[3] == 05) Echo "selected"; ?> value="05">Мая</option>
   			<option <?php if($explode[3] == 06) Echo "selected"; ?> value="06">Июня</option>
   			<option <?php if($explode[3] == 07) Echo "selected"; ?> value="07">Июля</option>
   			<option <?php if($explode[3] == 08) Echo "selected"; ?> value="08">Августа</option>
   			<option <?php if($explode[3] == 09) Echo "selected"; ?> value="09">Сентября</option>
   			<option <?php if($explode[3] == 10) Echo "selected"; ?> value="10">Октября</option>
   			<option <?php if($explode[3] == 11) Echo "selected"; ?> value="11">Ноября</option>
   			<option <?php if($explode[3] == 12) Echo "selected"; ?> value="12">Декабря</option>
   		</select>
   	</div>
   </div>

   Суровые челябинские программисты :)

   by133312, 13 Марта 2016

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

8. C++ / Говнокод #19591

   +2

   1. 1
   2. 2
   3. 3
   4. 4
   5. 5
   6. 6
   7. 7

   const int MOD = 1000000007;
   
   int pow(int a, int b) {
   	if (!b) return 1;
   	if (b & 1) return (pow(a, b - 1) * 1LL * a) % MOD;
   	return pow((a * 1LL * a) % MOD, b / 2);
   }

   http://ideone.com/JlfNxZ
   
   Там ещё куча всякого говна есть
   http://acm.math.spbu.ru/~kunyavskiy/cpp/

   3_dar, 08 Марта 2016

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

9. C++ / Говнокод #19588

   +2

   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

   switch (a) {
   	case 12345:
   		return 0;
   	case 14523:
   		return 1;
   	case 102543:
   		return 2;
   	case 104325:
   		return 3;
   	case 243051:
   		return 4;
   	case 245130:
   		return 5;
   	case 350214:
   		return 6;
   	case 351402:
   		return 7;
   	case 423150:
   		return 8;
   	case 425031:
   		return 9;
   	case 530412:
   		return 10;
   	case 531204:
   		return 11;
   	}
   	return -1;

   Ekke, 07 Марта 2016

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

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

    +2

    1. 1
    2. 2
    3. 3

    if(!$var1 = $var2){
        $var1 = /*запрос из базы*/;    
    }

    Не понял что это.

    Sentoki, 04 Марта 2016

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

11. C# / Говнокод #19566

    +2

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

    bool compareArrays(int[] a,int[] b)
    	{
    		if(a.Length != b.Length)
    			return false;
    
    		for(int i = 0;i<a.Length;i++)
    			if(a[i] != b[i])
    				return false;
    
    		return true;
    	}

    Нобелевская премия по программированию

    d_fomenok, 02 Марта 2016

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

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

• 1573
• …
• 106
• 105
• 104
• 103
• 102
• 101
• 100
• 99
• 98
• …
• 1