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

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

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

1. PHP (5703)
2. C++ (2740)
3. Куча (2340)
4. JavaScript (2030)
5. C# (1930)
6. Java (1503)
7. Си (1103)
8. Pascal (649)
9. Python (585)
10. 1C (536)
11. SQL (434)
12. ActionScript (294)
13. Objective C (204)
14. bash (200)
15. Perl (196)
16. Assembler (148)
17. Ruby (144)
18. VisualBasic (140)
19. Lua (51)
20. Go (32)
21. Swift (29)
22. Haskell (28)
23. Kotlin (15)

1. PHP / Говнокод #1909

   +170.4

   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

   <?php
   $mm=strlen( $titLE);
   $tit='';   
   for ($i=0;$i<=$mm;$i++)
   {
   $ss= $titLE[$i];
   switch ($ss) 
   {
   case "щ": $tit= $tit."sch";  break;
   case "ч": $tit= $tit."ch";  break;
   case "ш": $tit= $tit."sh";  break; 
   case "я": $tit= $tit."ja";  break;
   case "ю": $tit= $tit."ju";  break;
   case "ё": $tit= $tit."jo";  break;
   case "ж": $tit= $tit."zh";  break;
   case "э": $tit= $tit."e";  break;
   case "Щ": $tit= $tit."Sch";  break; 
    case "Ч": $tit= $tit."Ch";  break;
    case "Ш": $tit= $tit."Sh";  break;
    case "Я": $tit= $tit."Ja";  break;
   case "Ю": $tit= $tit."Ju";  break;
   case "Ё": $tit= $tit."Jo";  break; 
   case "Ж": $tit= $tit."Zh";  break;
   case "Э": $tit= $tit."E";  break;
   case "ь": $tit= $tit."";  break; 
   case "ъ": $tit= $tit."'";  break; 
   case "а": $tit= $tit."a";  break;
    case "б": $tit= $tit."b";  break;
   case "ц": $tit= $tit."c";  break;
   case "д": $tit= $tit."d";  break;
   case "е": $tit= $tit."e";  break; 
   case "ф": $tit= $tit."f";  break; 
   case "г": $tit= $tit."g";  break;
   case "х": $tit= $tit."h";  break; 
   case "и": $tit= $tit."i";  break; 
   case "й": $tit= $tit."j";  break;
   case "к":
        {
   
           if ($str[$i+1]=="с" ) {
           $tit= $tit."x";  
          $i=$i+1; break;}
            $tit= $tit."k";  break;
          }
   case "л": $tit= $tit."l";  break;
   case "м": $tit= $tit."m";  break;
   case "н": $tit= $tit."n";  break;
   case "о": $tit= $tit."o";  break;
   case "п": $tit= $tit."p";  break; 
   case "р": $tit= $tit."r";  break; 
   case "с": $tit= $tit."s";  break;
   case "т": $tit= $tit."t";  break;
   case "у": $tit= $tit."u";  break;
   case "в": $tit= $tit."v";  break;
   case "ы": $tit= $tit."y";  break;
   case "з": $tit= $tit."z";  break; 
   case "Ь": $tit= $tit."'";  break;
   case "Ъ": $tit= $tit."'";  break;
   case "А": $tit= $tit."A";  break;
   case "Б": $tit= $tit."B";  break; 
   case "Ц": $tit= $tit."C";  break;
   case "Д": $tit= $tit."D";  break;
    case "Е": $tit= $tit."E";  break;
   case "Ф": $tit= $tit."F";  break;
   case "Г": $tit= $tit."G";  break;
   case "Х": $tit= $tit."H";  break;
   case "И": $tit= $tit."I";  break;
   case "Й": $tit= $tit."J";  break;
   case "К":    
        {
         if ($str[$i+1]=="С" ) {
           $tit= $tit."X";  
          $i=$i+1; break;}
   
         if ($str[$i+1]=="с" ) {
           $tit= $tit."X";  
          $i=$i+1; break;}
   
          $tit= $tit."K"; break;
          }
   case "Л": $tit= $tit."L";  break;
    case "М": $tit= $tit."M";  break;
   case "Н": $tit= $tit."N";  break;
   case "О": $tit= $tit."O";  break;
   case "П": $tit= $tit."P";  break;  
   case "Р": $tit= $tit."R";  break;
   case "С": $tit= $tit."S";  break;
   case "Т": $tit= $tit."T";  break; 
    case "У": $tit= $tit."U";  break; 
    case "В": $tit= $tit."V";  break;
   case "Ы": $tit= $tit."Y";  break;
   case "З": $tit= $tit."Z";  break;
   case " ": $tit= $tit." ";  break;
   default:
           $end1= $tit;
     }
   }
   ?>

   транслитизатор!!!!!!!

   yamaha252, 29 Сентября 2009

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

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

   +153.5

   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

   /**
   	* Выполняет запрос SELECT
   	*
   	* @param string  $tables      Список таблиц, разделённый запятыми
   	* @param string  $condition   Условие для выборки (WHERE)
   	* @param string  $order       Поля для сортировки (ORDER BY)
   	* @param string  $fields      Список полей для получения
   	* @param int     $lim_rows    Максимльное количество получаемых записей
   	* @param int     $lim_offset  Начальное смещение для выборки
   	* @param string  $group       Поле для группировки
   	* @param bool    $distinct    Вернуть только уникальные записи
   	*
   	* @return array
   	*/
   	function select($tables, $condition = '', $order = '', $fields = '', $lim_rows = 0, $lim_offset = 0, $group = '', $distinct = false)
   	{
   		if (is_bool($fields) || $fields == '1' || $fields == '0' || !is_numeric($lim_rows)) {
   			# Обратная совместимость c 1.2.x
   			$desc = $fields;
   			$fields = $lim_rows ? $lim_rows : '*';
   			$lim_rows = $lim_offset;
   			$lim_offset = $group;
   			$group = $distinct;
   			$distinct = func_num_args() == 9 ? func_get_arg(8) : false;
   			$query = 'SELECT ';
   			if ($distinct) $query .= 'DISTINCT ';
   			if (!strlen($fields)) $fields = '*';
   			$tables = str_replace('`' ,'', $tables);
   			$tables = preg_replace('/([\w.]+)/i', '`'.$this->prefix.'$1`', $tables);
   			$query .= $fields." FROM ".$tables;
   			if (strlen($condition)) $query .= " WHERE $condition";
   			if (strlen($group)) $query .= " GROUP BY $group";
   			if (strlen($order)) {
   				$query .= " ORDER BY $order";
   				if ($desc) $query .= ' DESC';
   			}
   			if ($lim_rows) {
   				$query .= ' LIMIT ';
   				if ($lim_offset) $query .= "$lim_offset, ";
   				$query .= $lim_rows;
   			}
   		} else {
   			$query = 'SELECT ';
   			if ($distinct) $query .= 'DISTINCT ';
   			if (!strlen($fields)) $fields = '*';
   			$tables = str_replace('`','',$tables);
   			$tables = preg_replace('/([\w.]+)/i', '`'.$this->prefix.'$1`', $tables);
   			$query .= $fields." FROM ".$tables;
   			if (strlen($condition)) $query .= " WHERE ".$condition;
   			if (strlen($group)) $query .= " GROUP BY ".$group."";
   			if (strlen($order)) {
   				$order = explode(',', $order);
   				for($i = 0; $i < count($order); $i++) switch ($order[$i]{0}) {
   					case '+': $order[$i] = substr($order[$i], 1); break;
   					case '-': $order[$i] = substr($order[$i], 1).' DESC'; break;
   				}
   				$query .= " ORDER BY ".implode(', ',$order);
   			}
   			if ($lim_rows) {
   				$query .= ' LIMIT ';
   				if ($lim_offset) $query .= "$lim_offset, ";
   				$query .= $lim_rows;
   			}
   		}
   		$result = $this->query_array($query);
   
   		return $result;
   	}

   nbyt, 28 Сентября 2009

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

3. C# / Говнокод #1907

   +100.4

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

   public List<OrderGeneralVWJ> GetAll()
   {
        var q = from og in Context.OrderGeneralVWJs
             select new { 
                  og
             };
        return q.Select(r => r.og).ToList();
   }

   plsc_rover, 28 Сентября 2009

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

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

   +76.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

   Long orgId = 0L;
   try {
       orgId = Long.valueOf(textOrgId);
   } catch (Exception e) {
       orgId = 0L;
   }
   // ...
   Organization org = orgDAO.findById(orgId);
   if (org == null) {
       // не найдено? создать новую
       org = new Organization();
       // ...
   }

   Если с веба в поле textOrgId пришёл мусор, то создать новую организацию. Но перед этим всё равно поискать в базе несуществующий orgId=0.

   yvu, 28 Сентября 2009

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

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

   +83.5

   1. 1
   2. 2
   3. 3

   for (Person p : c) {
               return p;
           }

   yvu, 28 Сентября 2009

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

6. Java / Говнокод #1904

   −49.7

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

   /**
    * Pause length in milliseconds.
    */
    private static final int _100000 = 100000;
    /**
    * Summary pause length in milliseconds.
    */
    private static final int _500000 = _100000 * 5;

   Автор творения сказал, что так удобнее читать код:)

   intr13, 28 Сентября 2009

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

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

   +94.5

   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

   /// Из aspx файла
   
   //<asp:Repeater ID="Repeater1" runat="server">
   //    <HeaderTemplate>
   //        <table width="100%" cellspacing="5">
   //    </HeaderTemplate>
   //    <ItemTemplate>
   //        <tr>
   //            <td>
   //                ...
   //            </td>
   //            <asp:Label runat="server" Visible='<%# DataBinder.Eval(Container.DataItem, "visibleCol2") %>'>
   //                <td>
   //                    ...
   //                </td>
   //            </asp:Label>
   //            <asp:Label ID="LCol3" runat="server" Visible='<%# DataBinder.Eval(Container.DataItem, "visibleCol3") %>'>
   //                <td>
   //                    ...
   //                </td>
   //            </asp:Label>
   //        </tr>
   //    </ItemTemplate>
   //    <FooterTemplate>
   //        </table>
   //    </FooterTemplate>
   //</asp:Repeater>
   
   // Из cs файла
   
   DataTable dtres = // получаем товары
   if (dtres != null && dtres.Rows.Count > 0)
   {
       DataTable dt = new DataTable();
   	/*...*/
       
       for (int i=0;i<dtres.Rows.Count;)
       {
           DataRow dr=dtres.Rows[i];
           /*...*/
   
           int col2Index = i + 1;
           if (col2Index < dtres.Rows.Count)
           {
               /*...*/
               i++;
               int col3Index = i + 1;
               if (col3Index < dtres.Rows.Count)
               {
                   /*...*/
                   i++;
               }
               else { /*...*/ }
           }
           else { /*...*/ }
   
           dt.Rows.Add(newRow);
           i++;
       }
   
       /*...*/
   }

   Человеку нужно было сделать товары в сетке 3x6.
   От того что он сделал у меня пропал дар речи. (Чтобы тут очень много кода не бы большую (не нужную для понимания) часть заменил на "...")
   Вобщем, в двух словах, он поместил вторую и третью ячейки таблицы в серверные контролы (причем в Label), а в коде берёт по три товара и засовывает в один RepeaterItem, ну а если количество товаров не делится нацело на три, то в последний RepeaterItem засовываются пустые данные, а Label делается невидимым.
   Как-то так.
   И самое главное, я не знаю, как человеку объяснить, что так нехорошо делать, т.к он главный программист...

   Ordos, 27 Сентября 2009

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

8. Куча / Говнокод #1902

   +64.1

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

   <?php
   echo "<br><br>";
   echo "<a href='index.php'>На главную</a> |
   <a href='reg.php'>Регистрация</a> |
   <a href='stat.php'>Статистика</a><br><hr width=80%>
   <div class=info>Создано при участии gxcreator и kuprik17. Все права у них и ниипёт!<br>
   Кто спиздит код, тому просто пиздец будет,<br>
   руки оторвем и в жопу вставим!! Ясно, бля?!</div>";
   
   ?>

   Еще много интересного кода: http://www.google.com/codesearch?hl=ru&lr=&q=%D0%B6%D0%BE%D0%B F%D1%83&sbtn=%D0%9F%D0%BE%D0%B8%D1%81%D0 %BA

   Serge, 26 Сентября 2009

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

9. C# / Говнокод #1901

   +130.5

   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

   if (!passFlag)
                   {
                       switch (GetTreeNodeName)
                       {
                           case "group":
                               {
                                   if (GetTreeNode.Parent.Parent.Parent.Parent.Name != "StudySchedule")
                                   {
                                       StudentCard studentCard = new StudentCard(ExploreTree.SelectedNode.Text,
                                                                                 ExploreTree.SelectedNode.Parent.Parent.
                                                                                     Text,
                                                                                 ExploreView.SelectedItems[0]);
                                       studentCard.MdiParent = this.MdiParent;
                                       studentCard.Show();
                                       break;
                                   }
                                   else
                                   {
                                       StudySchedule schedule=new StudySchedule(ExploreView.SelectedItems[0], ExploreTree.SelectedNode.Parent.Nodes);
                                       schedule.MdiParent = this.MdiParent;
                                       schedule.Show();
                                       break;
                                   }
                               }

   Это мой говнокод, не знаю как строчку if (GetTreeNode.Parent.Parent.Parent.Parent .Name != "StudySchedule")
   написать по-нормальному)))

   FofanovIS, 26 Сентября 2009

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

10. C++ / Говнокод #1900

    +62.3

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

    void func(const char* str)
    {
      std::map<std::string, int>::iterator = my_map.find(str);
    
      .....
    }
    
    std::string str = "key";
    func(str.c_str());

    Уверен что это может найти каждый из вас в своих проектах, но может не в таком очевидном виде...

    pushkoff, 25 Сентября 2009

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

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

• 2104
• …
• 186
• 185
• 184
• 183
• 182
• 181
• 180
• 179
• 178
• …
• 1