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

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• 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 (44)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

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

   В номинации:

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

   За время:

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

2. C# / Говнокод #1881

   +128.3

   1. 1

   List<estExecPeriod> lst = TEDManager.getExecPeriodByIDExecWork(budget.IdBudget);

   iley, 22 Сентября 2009

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

3. 1C / Говнокод #1876

   −92.1

   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

   Запрос.Текст = 
   "ВЫБРАТЬ
   |	СостояниеРаботниковОрганизаций.Регистратор,
   |	СостояниеРаботниковОрганизаций.ФизЛицо,
   |	СостояниеРаботниковОрганизаций.Организация,
   |	СостояниеРаботниковОрганизаций.Приказ,
   |	СостояниеРаботниковОрганизаций.Состояние,
   |	СостояниеРаботниковОрганизаций.Период
   |ИЗ
   |	(ВЫБРАТЬ
   |		СостояниеРаботниковОрганизаций.Регистратор КАК ДокументРегистратор
   |	ИЗ
   |		РегистрСведений.СостояниеРаботниковОрганизаций КАК СостояниеРаботниковОрганизаций
   |	ГДЕ
   |		СостояниеРаботниковОрганизаций.Период МЕЖДУ &ДатаНачала И &ДатаКонца
   |		И СостояниеРаботниковОрганизаций.Организация = &Организация
   |		И СостояниеРаботниковОрганизаций.ФизЛицо = &ФизЛицо
   |		И СостояниеРаботниковОрганизаций.Приказ = &Приказ) КАК ДокРегистратор
   |		ВНУТРЕННЕЕ СОЕДИНЕНИЕ РегистрСведений.СостояниеРаботниковОрганизаций КАК СостояниеРаботниковОрганизаций
   |		ПО ДокРегистратор.ДокументРегистратор = СостояниеРаботниковОрганизаций.Регистратор";
   .

   Просто выбрать движения из регистра - это не круто.

   Kopchuga, 21 Сентября 2009

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

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

   +156.1

   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

   function CompletesGetTableOld($clist, $mid =0)
   {
     if(strlen($clist) or $mid > 0)
     {
       $com = CatalogComplsCostsGet(0, 0, $clist, $mid);
       $csz = sizeof($com['id']);
       if($csz > 0)
       {
           $param = CatalogComplParamsGet();
           $psz = sizeof($param);
           $tds = '';
           
           $cVals = CatalogComplParamValsGet(implode(",", $com['cid']));
           
           echo '
   		<table cellpadding="4" cellspacing="0" border="0" width="700" class="table-comp">
   		 <tr><td width="50%" class="first"></td>
          	';
           for($i = 0; $i < $csz; $i++)
    	    {
    	        echo '<td align="center">'.$com['name'][$i].'</td>';
    	        $tds .= '<td> &nbsp; </td>';
    	    }
           echo '</tr>';
           
           $prgrnm = '';
           for($i = 0; $i < $psz; $i++)
    	    {
    	        $data = false;
    	        $html = '';
    	        if($i==0||$param[$i]['gr_name']!=$prgrnm)
    	        {
   				$html='
   				<tr>
   					<td class="first"><br><b>'.$param[$i]['gr_name'].'</b></td>';for($ii = 0; $ii < $csz; $ii++)$html.='<td class="second" > &nbsp; </td>';
   					$html.= $tds.'
   				</tr>';
    	        }
    	        $html.= '
    	        <tr>
    	          <td class="first">'.$param[$i]['name'].'</td>';
    	        
    	        for($ii = 0; $ii < $csz; $ii++)
        	    {
        	        @$val = $cVals[$com['cid'][$ii]][$param[$i]['id']];
        	        if(isset($val) and strlen($val))
        	        {
        	            if($param[$i]['type'] == 'CH') 
        	            {
        	                if($val == 1)
        	                {
        	                    $html.= '<td class="second" align="center"><img src="/images/tochka.gif" ></td>'; 
        	                    $data = true;
        	                }
        	                else $html.= '<td class="second"> &nbsp; </td>';
        	                
        	            }
        	            else 
        	            {
        	                $html.= '<td class="second" align="center">'.$val.'</td>';
        	                $data = true;
        	            }
        	        }
        	        else $html.= '<td class="second"> &nbsp; </td>';
        	    }
        	    $html.= '</tr>';
        	    
        	    if($data) 
        	    {
        	        $prgrnm = $param[$i]['gr_name'];
        	        echo $html;
        	    }
    	    }
    	    
    	    echo '
    	     <tr><td width="50%" class="first"><b>????</b></td>';
    	    for($i = 0; $i < $csz; $i++)
    	    {
    	    	$cost = intval($com['cost'][$i]);
    	        if($cost > 1000000) $cost = '<nobr>'.substr($cost, 0, -6)." ".substr($cost, 1, -3).". ".substr($cost, 4).'</nobr> ???';
    	        elseif($cost > 100) $cost = '<nobr>'.substr($cost, 0, -3)." ".substr($cost, strlen($cost)-3).'</nobr> ???';
    	        else $cost = '';
        	        
    	        echo '<td class="second" align="center" height="30">'.$cost.'</td>';
    	    }
                  
           echo '</tr>
           </table>';
            
           return true;
       }
     }
      
     return false;
   }

   обратите внимание на переменную $data. долго и упорно собираем из кусочков $html, и только если $data истинно тогда выводим $html (примерно 5-10% случаев)

   gorky, 18 Сентября 2009

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

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

   +103.9

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

   public Parameter()
               {
                   _name = new String((char[])null);
                   Value = new String((char[])null);
               }

   Вот такой забавный метод инициализации пустой строки обнаружил в чужом коде 0_о

   samuilovaa, 15 Сентября 2009

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

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

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

   // from views_bonus: grid.inc
   function theme_og_picture_grid($result, $column_count = 5) {
     $content = '<table class="og-picture-grid">';
     $count = 0;
     $total = db_num_rows($result);
     while ($user = db_fetch_object($result)) {
       $classes = 'og-picture-grid-item';
       if ($count < $column_count) {
         $classes .= ' first';
       }
       
       $item = '';
       if ($count % $column_count == 0) { 
         $content .= '<tr>'; 
       }
       $picture = theme('user_picture', $user);
       $name = theme('username', $user);
       $group_role = $user->is_admin ? t('admin') : '&nbsp;';
       $content .= "<td class=\"$classes\">$picture<div class=\"og-name\">$name</div>";
       if ($user->is_admin) {
         $txt = t('admin');
         $content .= "<div class=\"group-role\">$txt</div>"; 
       }
       $content .= "</td>\n";
   
       $count++;
       if ($count % $column_count == 0 || $count == $total) {
         $content .= '</tr>';
       }
     }
     $content .= '</table>';
     
     if ($total) {
       return $content;
     }
   }

   Меня ДУРЬпаловские финты просто бесят.
   
   типа тема

   paranoid, 14 Сентября 2009

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

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

   +102.6

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

   //Получаем путь к програме
   public static string getProgramPath()
   {
                 return Console.Title;
   }

   psina-from-ua, 12 Сентября 2009

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

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

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

   function getday($varitable){
   	$select_strings = "\t\t<option ";
   	if ($varitable=='01'){
   	$select_strings .= " selected ";}
   	$select_strings .= "value=\"01\">1</option>\n\t\t<option ";
   	if ($varitable=='02'){
   	$select_strings .= " selected ";}
   	$select_strings .= "value=\"02\">2</option>\n\t\t<option ";
   	if ($varitable=='03'){
   	$select_strings .= " selected ";}
   	$select_strings .= "value=\"03\">3</option>\n\t\t<option ";
   	if ($varitable=='04'){
   	$select_strings .= " selected ";}
   	$select_strings .= "value=\"04\">4</option>\n\t\t<option ";
   	if ($varitable=='05')
   //так далее 
   	if ($varitable=='30'){
   	$select_strings .= " selected ";}
   	$select_strings .= "value=\"30\">30</option>\n\t\t<option ";
   	if ($varitable=='31'){
   	$select_strings .= " selected ";}
   	$select_strings .= "value=\"31\">31</option>\n\t\t";
   	return $select_strings;
   }

   1ac1324, 10 Сентября 2009

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

9. PHP / Говнокод #1796

   +151.7

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

   for ($i = 0; $i < count($oldCLI); $i++) {
   	if ($oldCLI[$i] == null) {
   		unset($oldCLI[$i]);
   	}
   }

   unset($oldCLI[$i]); - это как я понимаю контрольный выстрел в голову

   Sadie, 09 Сентября 2009

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

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

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

    $query = mysql_query("SELECT w_id,title FROM bhost_weblogs
                                         WHERE owner='".$user_info['username']."'");
    while($blog = mysql_fetch_assoc($query))
    {
        $stat = mysql_query("SELECT * FROM stat WHERE blog='".$blog['w_id']."' AND user!='' AND
                                          datestamp>='$d2' ORDER BY datestamp DESC");
        $all_users = mysql_result(mysql_query("SELECT Count(blog) FROM stat
                                           WHERE blog='".$blog['w_id']."' AND datestamp>='$d2'"),0);
        $reg_users = mysql_result(mysql_query("SELECT Count(blog) FROM stat
                                           WHERE blog='".$blog['w_id']."' AND user!='' AND datestamp>='$d2'"),0);
         $unreg_users = mysql_result(mysql_query("SELECT Count(blog) FROM stat
                                            WHERE blog='".$blog['w_id']."' AND user='' AND datestamp>='$d2'"),0);
    
         echo "<center>Всего: $all_users<br />
         Зарегистрированных пользователей: $reg_users<br />
         Незарегистрированных пользователей: $unreg_users</center>";
    
         echo "<table  cellpadding='0' cellspacing='0' align='center' class='form'>";
         while($statd = mysql_fetch_assoc($stat))
         {
         echo ...;
         }
         echo "</table><br /><br />";
    }

    Очень правильная работа с базой, подумаешь пара лишних запросов.

    BabyWolf, 06 Сентября 2009

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

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

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

    foreach ( DataRow dr in dsWorkPolicyTa.Tables[0].Rows)
    {
    	if(strMasterNumber==dr["MAST_NO"].ToString() && strMasterSequence == dr["MAST_SEQ"].ToString() && strSectionSequence == dr["SEQ"].ToString() && strPremiumCurrency == dr["PREMIUM_CURRENCY"].ToString())
    	{
    		if (( strPremiumCurrency.CompareTo(dr["PREMIUM_CURRENCY"].ToString())==0)&& ( dr["ENDORSEMENT_NUMBER"].ToString().CompareTo(dsWorkPolicyTa.Tables[0].Rows[marker]["ENDORSEMENT_NUMBER"].ToString()) == 0 )&& (strEntrySurrogate==dr["EYAFNB"].ToString()) && ( dr["E2E0CD"].ToString().CompareTo(dsWorkPolicyTa.Tables[0].Rows[marker]["E2E0CD"].ToString()) == 0 ))
    		{
    			//Unauthorised/Policy Premium
    			if((dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM"]!= System.DBNull.Value)&& (dr["PREMIUM"] != System.DBNull.Value)&& (dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM"].ToString()!= "")&& (dr["PREMIUM"].ToString() != ""))
    				dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM"]=Convert.ToDouble(dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM"])+ Convert.ToDouble(dr["PREMIUM"]);
    			else
    				dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM"]=dr["PREMIUM"];
    			if((dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM_BASE"]!= System.DBNull.Value)&& (dr["PREMIUM_BASE"] != System.DBNull.Value)&& (dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM_BASE"].ToString()!= "")&& (dr["PREMIUM_BASE"].ToString() != ""))
    				dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM_BASE"]=Convert.ToDouble(dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM_BASE"])+ Convert.ToDouble(dr["PREMIUM_BASE"]);
    			else
    				dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM_BASE"]=dr["PREMIUM_BASE"];
    			if((dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM_OURSHARE"]!= System.DBNull.Value) && (dr["PREMIUM_OURSHARE"] != System.DBNull.Value)&& (dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM_OURSHARE"].ToString()!= "") && (dr["PREMIUM_OURSHARE"].ToString() != ""))
    				dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM_OURSHARE"]=Convert.ToDouble(dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM_OURSHARE"])+ Convert.ToDouble(dr["PREMIUM_OURSHARE"]);
    			else
    				dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM_OURSHARE"]=dr["PREMIUM_OURSHARE"];
    			if((dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM_BASE_OURSHARE"]!= System.DBNull.Value)&& (dr["PREMIUM_BASE_OURSHARE"] != System.DBNull.Value) && (dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM_BASE_OURSHARE"].ToString()!= "")&& (dr["PREMIUM_BASE_OURSHARE"].ToString() != ""))
    				dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM_BASE_OURSHARE"]=Convert.ToDouble(dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM_BASE_OURSHARE"])+ Convert.ToDouble(dr["PREMIUM_BASE_OURSHARE"]);
    			else
    				dsWorkPolicyTa.Tables[0].Rows[marker]["PREMIUM_BASE_OURSHARE"]=dr["PREMIUM_BASE_OURSHARE"];
    			//Quoted Premium
    			if((dsWorkPolicyTa.Tables[0].Rows[marker]["QUOTED_PREMIUM"]!= System.DBNull.Value)&& (dr["QUOTED_PREMIUM"] != System.DBNull.Value) && (dsWorkPolicyTa.Tables[0].Rows[marker]["QUOTED_PREMIUM"].ToString()!= "")&& (dr["QUOTED_PREMIUM"].ToString() != ""))
    				dsWorkPolicyTa.Tables[0].Rows[marker]["QUOTED_PREMIUM"]=Convert.ToDouble(dsWorkPolicyTa.Tables[0].Rows[marker]["QUOTED_PREMIUM"])+ Convert.ToDouble(dr["QUOTED_PREMIUM"]);
    			else
    				dsWorkPolicyTa.Tables[0].Rows[marker]["QUOTED_PREMIUM"]=dr["QUOTED_PREMIUM"];
    			if((dsWorkPolicyTa.Tables[0].Rows[marker]["QUOTED_PREMIUM_BASE"]!= System.DBNull.Value)&& (dr["QUOTED_PREMIUM_BASE"] != System.DBNull.Value) && (dsWorkPolicyTa.Tables[0].Rows[marker]["QUOTED_PREMIUM_BASE"].ToString()!= "")&& (dr["QUOTED_PREMIUM_BASE"].ToString() != ""))
    				dsWorkPolicyTa.Tables[0].Rows[marker]["QUOTED_PREMIUM_BASE"]=Convert.ToDouble(dsWorkPolicyTa.Tables[0].Rows[marker]["QUOTED_PREMIUM_BASE"])+ Convert.ToDouble(dr["QUOTED_PREMIUM_BASE"]);
    			else
    				dsWorkPolicyTa.Tables[0].Rows[marker]["QUOTED_PREMIUM_BASE"]=dr["QUOTED_PREMIUM_BASE"];

    Кусок примера кода предоставленного кандидатом на должность программиста.
    Модуль называеться "Genius".
    
    При попытке наговнокодить все 100 строчек произошла "Ошибка компиляции кода: Такой длинный код врядли может быть смешным. Пожалуйста, ограничьтесь сотней строк и 4000 символами."
    Пришлось ограничить 4000 символами, но не стало смешнее :(

    OlgaWolga, 04 Сентября 2009

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

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

• 1976
• …
• 451
• 450
• 449
• 448
• 447
• 446
• 445
• 444
• 443
• …
• 1