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

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

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

1. PHP (5700)
2. C++ (2734)
3. Куча (2328)
4. JavaScript (2028)
5. C# (1926)
6. Java (1501)
7. Си (1097)
8. Pascal (647)
9. Python (582)
10. 1C (532)
11. SQL (431)
12. ActionScript (292)
13. Objective C (202)
14. bash (198)
15. Perl (194)
16. Assembler (146)
17. Ruby (142)
18. VisualBasic (138)
19. Lua (49)
20. Go (29)
21. Swift (27)
22. Haskell (26)
23. Kotlin (13)

1. Список говнокодов пользователя guest

   Всего: 1419

2. C++ / Говнокод #213

   −27.3

   1. 1
   2. 2
   3. 3

   cout<<"enterX"<<endl;
        cin >>x;
   x = 0.125;

   guest, 18 Декабря 2008

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

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

   +1429.9

   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

   bool IsBukva(char symbol)
   {
   switch(symbol)
   {
   case'a':return 1;break;
   case'b':return 1;break;
   case'c':return 1;break;
   case'd':return 1;break;
   case'e':return 1;break;
   case'f':return 1;break;
   case'g':return 1;break;
   case'h':return 1;break;
   case'i':return 1;break;
   case'j':return 1;break;
   case'k':return 1;break;
   case'l':return 1;break;
   case'm':return 1;break;
   case'n':return 1;break;
   case'o':return 1;break;
   case'p':return 1;break;
   case'q':return 1;break;
   case'r':return 1;break;
   case's':return 1;break;
   case't':return 1;break;
   case'u':return 1;break;
   case'v':return 1;break;
   case'w':return 1;break;
   case'x':return 1;break;
   case'y':return 1;break;
   case'z':return 1;break;
   case'A':return 1;break;
   case'B':return 1;break;
   case'C':return 1;break;
   case'D':return 1;break;
   case'E':return 1;break;
   case'F':return 1;break;
   case'G':return 1;break;
   case'H':return 1;break;
   case'I':return 1;break;
   case'J':return 1;break;
   case'K':return 1;break;
   case'L':return 1;break;
   case'M':return 1;break;
   case'N':return 1;break;
   case'O':return 1;break;
   case'P':return 1;break;
   case'Q':return 1;break;
   case'R':return 1;break;
   case'S':return 1;break;
   case'T':return 1;break;
   case'U':return 1;break;
   case'V':return 1;break;
   case'W':return 1;break;
   case'X':return 1;break;
   case'Y':return 1;break;
   case'Z':return 1;break;
   default:return 0;
   }
   }

   Женский вариант проверки символа на букву.
   Оригинал тут: http://forum.vingrad.ru/forum/topic-240593.html

   guest, 17 Декабря 2008

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

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

   +25.9

   1. 1
   2. 2
   3. 3

   $temp = $this->template($this->modulePath."/right_frame");
   eval("\$blockData = \"$temp\";");
   return $blockData;

   Реализация механизма подключения шаблонов в PHP %)

   guest, 17 Декабря 2008

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

5. Java / Говнокод #207

   −43

   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

   if(isRegKeyRequired) {
               keyItems = new XItem [4];
               valueItems = new XItem [4][];
               totalItems = 4;
   
               keyItems[0] = new XItem();
               keyItems[0].setLabel(TextUtils.getText(280),XItem.LABEL_CONSTRAINT_CENTER);
               values = getValueItems(countryBox_rp);
               totalItems += setValues(valueItems, values,0);
               keyItems[1] = new XItem();
               keyItems[1].setLabel(TextUtils.getText(281),XItem.LABEL_CONSTRAINT_CENTER);
               values = getValueItems(phoneNoBox_rp);
               totalItems += setValues(valueItems, values,1);
               keyItems[2] = new XItem();
               keyItems[2].setLabel(TextUtils.getText(282),XItem.LABEL_CONSTRAINT_CENTER);
               values = getValueItems(regKeyBox_rp);
               totalItems += setValues(valueItems, values,2);
               keyItems[3] = new XItem();
               keyItems[3].setLabel(TextUtils.getText(283),XItem.LABEL_CONSTRAINT_CENTER);
               values = getValueItems(emailBox_rp);
               totalItems += setValues(valueItems, values,3);
           }
           else {
               keyItems = new XItem [3];
               valueItems = new XItem [3][];
               totalItems = 3;
   
               keyItems[0] = new XItem();
               keyItems[0].setLabel(TextUtils.getText(280),XItem.LABEL_CONSTRAINT_CENTER);
               values = getValueItems(countryBox_rp);
               totalItems += setValues(valueItems, values,0);
               keyItems[1] = new XItem();
               keyItems[1].setLabel(TextUtils.getText(281),XItem.LABEL_CONSTRAINT_CENTER);
               values = getValueItems(phoneNoBox_rp);
               totalItems += setValues(valueItems, values,1);
               keyItems[2] = new XItem();
               keyItems[2].setLabel(TextUtils.getText(283),XItem.LABEL_CONSTRAINT_CENTER);
               values = getValueItems(emailBox_rp);
               totalItems += setValues(valueItems, values,2);
           }

   очень трудолюбивые эти индусы. Такой код - это просто пипец особенно если учесть что приложение разрабатывается для J2ME

   guest, 16 Декабря 2008

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

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

   +24.9

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

   if(empty($real_name)) $real_name = $file_name; 		
   if(empty($real_name)) $real_name="fileid_".$id;
   
   .... 
   
   if($mode=="send") {
   	$head[] = array("link" => "customer.php?area=$area&mode=send", "title" => func_get_langvar_by_name("lbl_send_to_mail"), active=>"Y");
   }	else	{
   	$head[] = array("link" => "customer.php?area=$area&mode=send", "title" => func_get_langvar_by_name("lbl_send_to_mail"), active=>"n");
   }

   Разбирая один из скриптов на досуге, обнаружил, что некоторые программисты уж совсем с головой не дружат.
   
   1. Думаю, все тут ясно.
   2. Чувак явно не знает что такое логика... Мало того, что в одну строку ложится массив, так еще и ВЕЗДЕ строковые 'Y'/'N' использует вместо булевых труфолсов. И такого говна - тыщи 2 строк. Ы =)
   
   Не позавидую тем, кто будет дописывать / переделывать.
   ЗЫ: Скрипт - рип корпоративной системы.

   guest, 16 Декабря 2008

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

7. Java / Говнокод #205

   −37.2

   1. 1

   isRegKeyRequired = distKey == null ? false : distKey.equals("yes") ? true : false;

   Уматный ХавнАкоД коллеги из Индии. Ну просто няйсь. Смотрится куда зло*буче чем желаемый вариант (isRegKeyRequired = "yes".equals(distKey));

   guest, 16 Декабря 2008

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

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

   +29.3

   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

   for ($i=0; $i < $this->acl_count; $i++)
   		{
   			$acl =& $this->acl[$i];
   			if (strcasecmp( $aco_section_value, $acl[0] ) == 0) {
   				if (strcasecmp( $aco_value, $acl[1] ) == 0) {
   					if (strcasecmp( $aro_section_value, $acl[2] ) == 0) {
   						if (strcasecmp( $aro_value, $acl[3] ) == 0) {
   							if ($axo_section_value && $acl[4]) {
   								if (strcasecmp( $axo_section_value, $acl[4] ) == 0) {
   									if (strcasecmp( $axo_value, $acl[5] ) == 0) {
   										$acl_result = @$acl[6] ? $acl[6] : 1;
   										break;
   									}
   								}
   							} else {
   								$acl_result = @$acl[6] ? $acl[6] : 1;
   								break;
   							}
   						}
   					}
   				}
   			}
   		}

   из ядра Joomla

   guest, 16 Декабря 2008

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

9. Pascal / Говнокод #203

   +17.8

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

   Function nSetColorPair(att : integer) : integer;
   var
      i : integer;
   Begin
      bg := att div 16;
      fg := att - (bg * 16);
      While bg > 7 Do dec(bg,8);
      While fg > 7 Do dec(fg,8);
   // . . .

   Кусок модуля nCRT для FreePascal 2.2. У нормальных людей att - это байт. И вообще... Говно код!

   guest, 16 Декабря 2008

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

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

    +3.4

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

    if ((armid == "3504000036") || (armid == "3504000034"))
                {
                    if (armid == "3504000036")
                    {
                        PlaceSave = "3243100001";
                    }
                    if (armid == "3504000034")
                    {
                        PlaceSave = "2311030001";
                    }
                }

    Нет слов, одни эмоции

    guest, 16 Декабря 2008

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

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

    +26.1

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

    $res = $DB->getData($query); //считывает данные с ДБ и загоняет в пронумерованный массив
    $data = new array();
    foreach ($res as $key=>$value)
    {
    if (!$data[$key])
    $data[$key] = $value;
    };

    Все это вместо простого $data = $res;
    
    http://ithappens.ru/ (c)

    guest, 16 Декабря 2008

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

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

• 142
• …
• 22
• 21
• 20
• 19
• 18
• 17
• 16
• 15
• 14
• …
• 1