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

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

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

1. PHP (5711)
2. C++ (2742)
3. Куча (2393)
4. JavaScript (2034)
5. C# (1930)
6. Java (1503)
7. Си (1120)
8. Pascal (648)
9. Python (591)
10. 1C (540)
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 (43)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

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

   В номинации:

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

   За время:

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

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

   +158

   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

   <?php
   	public function clearCache() {
   		if(method_exists('regedit', 'getControllerHost')) {
   			$host = regedit::getControllerHost(true);
   			if(!is_null($host)){
   				$host->cacheClear();
   			} else {
   				$sDir = rtrim(mainConfiguration::getInstance()->includeParam('system.static-cache'), '/');
   				if(getServer('WINDIR') || getServer('windir')){
   					$arDirs = glob($sDir . '/*');
   					foreach($arDirs as $item){
   						$item = rtrim($item, "\\/ ") . "/";
   						exec("rd /s /q $item");
   					}
   				} else {
   					exec("rm -rf $sDir/*");
   				}
   			}
   		} else {
   			$sDir = rtrim(mainConfiguration::getInstance()->includeParam('system.static-cache'), '/');
   			if(getServer('WINDIR') || getServer('windir')) {
   				$arDirs = glob($sDir . '/*');
   				foreach($arDirs as $item) {
   					$item = rtrim($item, "\\/ ") . "/";
   					exec("rd /s /q $item");
   				}
   			} else {
   				exec("rm -rf $sDir/*");
   			}
   		}
   	}

   Найди два отличия.

   Elfet, 10 Ноября 2014

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

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

   +54

   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

   //Given a number, turns on those segments
   //If number == 10, then turn off number
   void lightNumber(int numberToDisplay) {
   
   #define SEGMENT_ON  LOW
   #define SEGMENT_OFF HIGH
   
     switch (numberToDisplay){
   
     case 0:
       digitalWrite(segA, SEGMENT_ON);
       digitalWrite(segB, SEGMENT_ON);
       digitalWrite(segC, SEGMENT_ON);
       digitalWrite(segD, SEGMENT_ON);
       digitalWrite(segE, SEGMENT_ON);
       digitalWrite(segF, SEGMENT_ON);
       digitalWrite(segG, SEGMENT_OFF);
       break;
   
     case 1:
       digitalWrite(segA, SEGMENT_OFF);
       digitalWrite(segB, SEGMENT_ON);
       digitalWrite(segC, SEGMENT_ON);
       digitalWrite(segD, SEGMENT_OFF);
       digitalWrite(segE, SEGMENT_OFF);
       digitalWrite(segF, SEGMENT_OFF);
       digitalWrite(segG, SEGMENT_OFF);
       break;
   
   
   ; и т. д.
   
   
     case 9:
       digitalWrite(segA, SEGMENT_ON);
       digitalWrite(segB, SEGMENT_ON);
       digitalWrite(segC, SEGMENT_ON);
       digitalWrite(segD, SEGMENT_ON);
       digitalWrite(segE, SEGMENT_OFF);
       digitalWrite(segF, SEGMENT_ON);
       digitalWrite(segG, SEGMENT_ON);
       break;
   
     case 10:
       digitalWrite(segA, SEGMENT_OFF);
       digitalWrite(segB, SEGMENT_OFF);
       digitalWrite(segC, SEGMENT_OFF);
       digitalWrite(segD, SEGMENT_OFF);
       digitalWrite(segE, SEGMENT_OFF);
       digitalWrite(segF, SEGMENT_OFF);
       digitalWrite(segG, SEGMENT_OFF);
       break;
     }
   }

   Управление 7-сегментным индикатором для ардуины

   govnokod3r, 25 Октября 2014

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

4. Java / Говнокод #16831

   +84

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

   private static final int INT_5 = 5;
   private static final int INT_3 = 3;
   private static final int INT_4 = 4;
   private static final int INT_6 = 6;
   private static final int INT_7 = 7;
   private static final int INT_8 = 8;

   xaoc, 09 Октября 2014

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

5. ActionScript / Говнокод #16727

   −84

   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 function isContainsHiddenId(hiddenId: String): Boolean
    		{
   -			for (var index: int = 0; index < hiddenIds.length; index++)
   +			/*for (var index: int = 0; index < hiddenIds.length; index++)
    			{
    				if (hiddenIds[index] == hiddenId)
    				{
    					return true;
    				}
    			}
   -			return false;
   +			return false;*/
   +			return hiddenIds.indexOf(hiddenId) == -1 ? false : true;

   От автора MathAndGeometryUtils, после того, как ему посоветовали использовать indexOf.

   wvxvw, 19 Сентября 2014

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

6. C++ / Говнокод #16659

   +54

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

   Exchange::Params pars = rawParams;
   for(Exchange::Params::const_iterator i = rawParams.constBegin(); i!= rawParams.constEnd(); i++){
       LOGN() << "Work with " << i.key() << "=" << i.value();
       if(m_specific.contains(i.key())){
           pars[i.key()] =
               (this->*m_specific.value(i.key())) (i.value());     //черная магия :)
       }
   }

   Наследие из большого рабочего проекта.
   Чтобы сохранить ясность ума на весь день решил не пытаться понять что оно делает.

   FlySnake, 08 Сентября 2014

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

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

   +159

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

   if (!isset($_SESSION[$_SESSION['tab_name']]['FILE_ATTRIBUTES'][$request['itemId']][$request['itemFieldId']])) {
         $_SESSION[$_SESSION['tab_name']]['FILE_ATTRIBUTES'][$request['itemId']][$request['itemFieldId']] = $request['itemFieldId'];
   } else {
         unset($_SESSION[$_SESSION['tab_name']]['FILE_ATTRIBUTES'][$request['itemId']][$request['itemFieldId']]);
   }

   И это часный проект где программисту платять большие деньги.
   А еще у автора более 5 лет опыта
   А сам сок, такого дерьма в проекте более 1000 обращений.

   vv3d0x, 29 Августа 2014

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

8. Java / Говнокод #16283

   +69

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

   public static boolean isEmpty(CharSequence str) {
       if (str == null || str.length() == 0)
           return true;
       else
           return false;
   }

   Я считаю это странным
   Нашёл в android.text.TextUtils

   jonasas, 07 Июля 2014

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

9. VisualBasic / Говнокод #16218

   −130

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

   Привет всем, помогите решить проблему.
   reached limit: cannot create any more controls for this from
   Как я понял, число контроллеров не должно превышать 255-256 в одной форме. Как этого избежать?
   Можно ли вообще создавать формы с общими переменными? Спасибо

   http://vbbook.ru/visual-basic/vvedenie-visual-basic/

   brutushafens, 24 Июня 2014

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

10. C# / Говнокод #16197

    +142

    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

    public Config(string[] str)
            {
                InitializeComponent();
                try
                {
                    textBox5.Text = str[0];
                    textBox6.Text = str[1];
                    textBox1.Text = str[2];
                    textBox2.Text = str[3];
                    textBox3.Text = str[4];
                    textBox4.Text = str[5];
                    textBox7.Text = str[6];
                    raz = str[7];
                }
                catch (Exception)
                {
                    return;
                }
            }

    Классика. Говнокод эпохи барокко. Строк мало, но пованивает.

    GreatMASTERcpp, 19 Июня 2014

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

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

    +159

    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
    84. 84
    85. 85
    86. 86
    87. 87
    88. 88
    89. 89
    90. 90
    91. 91
    92. 92
    93. 93
    94. 94
    95. 95
    96. 96
    97. 97
    98. 98
    99. 99

    <?
    function flevel($exp)
    {
    include_once "config/mysql.php";
    
    	$querylevel = "SELECT maxexp FROM level";
    	$levelquery = mysql_query($querylevel);
    	while($rowslvl = mysql_fetch_row($levelquery))
    	{
    	$levelarr[] = $rowslvl[0];
    	}
    	
    	
    	
    	switch(TRUE)
        {
    		
        case ($exp <= $levelarr[0]):
        return $level = "1";
        break;
    
        case ($exp <= $levelarr[1]):
        return $level = "2";
        break;
    
        case ($exp <= $levelarr[2]):
        return $level = "3";
        break;
    
        case ($exp <= $levelarr[3]):
        return $level = "4";
        break;
    	
    	case ($exp <= $levelarr[4]):
        return $level = "5";
        break;
    	
    	case ($exp <= $levelarr[5]):
        return $level = "6";
        break;
    	
    	case ($exp <= $levelarr[6]):
        return $level = "7";
        break;
    	
    	case ($exp <= $levelarr[7]):
        return $level = "8";
        break;
    	
    	case ($exp <= $levelarr[8]):
        return $level = "9";
        break;
    	
    	case ($exp <= $levelarr[9]):
        return $level = "10";
        break;
    	
    	case ($exp <= $levelarr[10]):
        return $level = "11";
        break;
    	
    	case ($exp <= $levelarr[11]):
        return $level = "12";
        break;
    	/* ... */
    	case ($exp <= $levelarr[42]):
        return $level = "43";
        break;
    	
    	case ($exp <= $levelarr[43]):
        return $level = "44";
        break;
    	
    	case ($exp <= $levelarr[44]):
        return $level = "45";
        break;
    	
    	case ($exp <= $levelarr[45]):
        return $level = "46";
        break;
    	
    	case ($exp <= $levelarr[46]):
        return $level = "47";
        break;
    	
    	case ($exp <= $levelarr[47]):
        return $level = "48";
        break;
    	
    	case ($exp <= $levelarr[48]):
        return $level = "49";
        break;
    	
    	case ($exp > $levelarr[49]):
        return $level = "50";
        break;
        }
    }
    ?>

    Пришел в проект по созданию веб игры, смотрю организацию проекта... Жопа там, что папки, что код, что база одно и тоже, каша. Вот нашел одну забавную функцию. Она возвращает уровень персонажа судя по кол-во XP. Вместо того что бы писать 'SELECT `level` FROM `level` WHERE `minexp` <= ' . $exp . ' AND `maxexp` > ' . $exp надо было switch .. case писать. Нету слов.

    volter9, 11 Июня 2014

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

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

• 1976
• …
• 1423
• 1422
• 1421
• 1420
• 1419
• 1418
• 1417
• 1416
• 1415
• …
• 1