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

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

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

1. PHP (5700)
2. C++ (2734)
3. Куча (2331)
4. JavaScript (2028)
5. C# (1927)
6. Java (1501)
7. Си (1099)
8. Pascal (647)
9. Python (582)
10. 1C (534)
11. SQL (432)
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. Лучший говнокод

   В номинации:

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

   За время:

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

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

   +55

   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

   <?php
   
   // ... много строк до этого
   
   echo "<table width=100% border=0>";
   if ($row = mysql_fetch_array($result))
   	for ($i = 0; $i < mysql_num_fields($result); $i++):	
   		if (mysql_field_name($result, $i) == "ID" || (mysql_field_name($result, $i) == "info") ||
   				mysql_field_name($result, $i) == "state" || (mysql_field_name($result, $i) == "date_st") ||
   				mysql_field_name($result, $i) == "date" || (mysql_field_name($result, $i) == "email") ||
   				mysql_field_name($result, $i) == "flag" || (mysql_field_name($result, $i) == "limit_time") ||
   				mysql_field_name($result, $i) == "cr_time" || mysql_field_name($result, $i) == "cr_money" ||
   				mysql_field_name($result, $i) == "flag" || (mysql_field_name($result, $i) == "info") ||
   				mysql_field_name($result, $i) == "attention" || mysql_field_name($result, $i) == "location" ||
   				mysql_field_name($result, $i) == "mailflag" || (mysql_field_name($result, $i) == "radio") ||
   				mysql_field_name($result, $i) == "radio30" || (mysql_field_name($result, $i) == "date_num") ||
   				mysql_field_name($result, $i) == "number" || mysql_field_name($result, $i) == "login_i"
   				|| mysql_field_name($result, $i) == "is_juridical")
   		{
   			if (mysql_field_name($result, $i) == "state")
   			{
   				echo "<select name=state style=\"background-color:red;color:white\"><option value=OPEN";
   				if ($row[$i] == "OPEN")
   					echo " selected ";
   				echo ">" . trans("OPEN") . "</option>\n";
   				echo "<option value=CLOSE";
   				if ($row[$i] == "CLOSE")
   					echo " selected ";
   				echo ">" . trans("CLOSE") . "</option></select\n";
   			}
   			elseif (mysql_field_name($result, $i) == "location")
   			{
   				genLeased($row[$i]);
   			}
   			elseif (mysql_field_name($result, $i) == "radio30" || mysql_field_name($result, $i) == "radio" || 
   				 mysql_field_name($result, $i) == "mailflag" )
   			
   			{
   			}
   			elseif (mysql_field_name($result, $i) == "date_st")
   			{
   				$ar = explode("-", $row[$i]);
   				echo "<b id=red>" . $ar[2] . "." . $ar[1] . "." . $ar[0] . "</b></td></tr>\n";
   			}
   			elseif (mysql_field_name($result, $i) == "date_num")
   			{
   				$ar = explode("-", $row[$i]);
   				$dnum = $ar[2] . "." . $ar[1] . "." . $ar[0];
   				
   				echo "<b>" . $dnum . "</b></td></tr><tr><td><a href=\"contracts/index.php?user_id=" . $row['ID'] . "\">контракти</a></td></tr><tr><td colspan=2><hr noshade></td></tr>\n";
   			}
   
   			elseif (mysql_field_name($result, $i) == "date")
   			{
   				$ar = explode("-", $row[$i]);
   				$row[$i] = $ar[2] . "." . $ar[1] . "." . $ar[0];
   				echo "<input type=text name=date value=\"" . $row[$i] . "\"</td></tr>\n";
   			}
   			elseif (mysql_field_name($result, $i) == "info")
   			{
   				echo "<textarea  rows=3 name=" . mysql_field_name($result, $i) . " >" . $row[$i] . "</textarea></td></tr>\n";
   			}
   			elseif (mysql_field_name($result, $i) == "attention")
   			{
   				echo "<textarea id=warn rows=3 name=" . mysql_field_name($result, $i) . " >" . $row[$i] . "</textarea></td></tr>\n";
   			}
   // ... и так обрабатываем далее все поля

   Разработчик сначала запросил результат с помощью mysql_fetch_array,
   потом ищет название каждого поля с помощью mysql_field_name, функцией trans()
   читает файлик со словарем и переводит название каждого поле.
   
   Писалось не индусами, вариант "для количества" не прокатывает.
   Поэтому я не могу даже предположить, что сподвигло автора на создание сего творения.
   Файл содержит около 1400 строчек.

   rsvasilyev, 16 Января 2013

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

3. PHP / Говнокод #12415

   +55

   1. 1
   2. 2
   3. 3

   $result["rg"][count($result["rg"])-1]["status"][] = Array();
   $c_res = count($result["rg"][count($result["rg"])-1]["status"])-1;
   $result["rg"][count($result["rg"])-1]["status"][$c_res] = $arr_source_data[$i];

   Им за число строк платят, что ли?

   clauclauclau, 14 Января 2013

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

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

   +55

   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

   $now = date("Y-m-d H:i:s");
   
   		if($time==1) { $now = date("Y-m-d H:i:s");}
   		elseif($time==2) { 
   
   			//$now = strtotime(date("Y-m-d H:i:s"));
   
   			if($type=="hour"){
   				$now = strtotime(date("Y-m-d H:i:s"))-3600; 
   			}
   			else if($type=="day"){
   				$now = strtotime(date("Y-m-d H:i:s"))-86400; 
   			}
   			else if($type=="month"){
   				$now = strtotime(date("Y-m-d H:i:s"))-(86400*30); 
   			}
   
   			$now = date("Y-m-d H:i:s",$now);
   		}
   
   $sql = ".....WHERE DATE_SUB('".$now."', INTERVAL .....";

   Между первой и второй перерывчик небольшой, ибо далее он использует NOW()...

   nonamez, 10 Января 2013

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

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

   +55

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

   function process() {
       ...
       throw new Exception( $error );
       return false;
   }

   Понятно, что сперва было без исключений, но не ясно чего постеснялись убрать ретерн фелс.

   enures, 25 Декабря 2012

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

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

   +55

   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

   function _bool($var){
     if(is_bool($var)){
       return $var;
     } else if($var === NULL || $var === 'NULL' || $var === 'null'){
       return false;
     } else if(is_string($var)){
       $var = trim($var);
       if($var=='false'){ return false;
       } else if($var=='true'){ return true;
       } else if($var=='no'){ return false;
       } else if($var=='yes'){ return true;
       } else if($var=='off'){ return false;
       } else if($var=='on'){ return true;
       } else if($var==''){ return false;
       } else if(ctype_digit($var)){
         if((int) $var)
           return true;
           else
           return false;
       } else { return true; }
     } else if(ctype_digit((string) $var)){
         if((int) $var)
           return true;
           else
           return false;
     } else if(is_array($var)){
       if(count($var))
         return true;
         else
         return false;
     } else if(is_object($var)){
       return true;// No reason to (bool) an object, we assume OK for crazy logic
     } else {
       return true;// Whatever came though must be something,  OK for crazy logic
     }
   }

   Я, конечно, понимаю, не во всех школах учат использованию массивов...

   RaZeR, 15 Декабря 2012

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

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

   +55

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

   if (trim((string) $_GET['tag'])) {
       $current = $conn->fetchAll("SELECT LOWER('" . (string) $_GET['tag'] . "') as `tag`");
       $current = $current[0]['tag'];
   }

   UnnamedUser, 09 Ноября 2012

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

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

   +55

   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

   function GetNumberOfLinks ($cat){ 
     $queue[] = intval($cat);
     $query = "select count(*) from #__jxlspp_prices  where ( 0!=0";
      while (list($key, $cat) = each($queue)) {
        $query .= " or catid = $cat";
      }
    $query = $query . " ) and published=1";
   	$database->setQuery($query);//это от Джумлы
     $result = $database->query();//и это от Джумлы
   	$val = mysql_fetch_row($result);//а это с бодуна
     return $val[0];
   
   }

   Сижу, скучаю, ковыряю очередной говносайт и, вдруг!
   Аж от души отлегло.

   virtual_cia, 07 Октября 2012

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

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

   +55

   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

   if(strlen($arItem["PROPERTIES"]["AGE"]["VALUE"])>0)
    {
           if ($arItem["PROPERTIES"]["AGE"]["VALUE"]=="1" ||
               $arItem["PROPERTIES"]["AGE"]["VALUE"]=="21" ||
               $arItem["PROPERTIES"]["AGE"]["VALUE"]=="31" ||
               $arItem["PROPERTIES"]["AGE"]["VALUE"]=="41" ||
               $arItem["PROPERTIES"]["AGE"]["VALUE"]=="51" ||
               $arItem["PROPERTIES"]["AGE"]["VALUE"]=="61" ||
               $arItem["PROPERTIES"]["AGE"]["VALUE"]=="71" ||
               $arItem["PROPERTIES"]["AGE"]["VALUE"]=="81" ||
               $arItem["PROPERTIES"]["AGE"]["VALUE"]=="91" ||
               $arItem["PROPERTIES"]["AGE"]["VALUE"]=="101"
           )
               $strAge="<b>Рекомендуемый возраст:</b> с ".$arItem["PROPERTIES"]["AGE"]["VALUE"]." года";
           else
               $strAge="<b>Рекомендуемый возраст:</b> с ".$arItem["PROPERTIES"]["AGE"]["VALUE"]." лет";
   }

   substr? не, не слышал...

   zubik, 02 Октября 2012

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

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

    +55

    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

    <?php
    class cURL
        {   
            
            private $handler;
            
            function cURL($url = null)
            {
                $this->handler = curl_init($url);
            }
            
            //  Да, я ленивый!
            function __call($name, $args)
            {
                if($name == "init") return;
                if(strstr($name, "multi")) die("Multiple cURL not supported in this class.");
                $name = "curl_$name";
                if(!function_exists($name)) die("Function $name not found.");
                array_unshift($args, $this->handler);
                $fn = new ReflectionFunction($name);
                return $fn->invokeArgs($args);
            }
            
        }

    Sarkian, 02 Октября 2012

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

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

    +55

    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

    <option <? if(isset($_POST['day']) and $_POST['day']=="01") echo "selected";  ?>   value="01">1</option>
                  <option <? if(isset($_POST['day']) and $_POST['day']=="02") echo "selected";  ?>  value="02">2</option>
                  <option <? if(isset($_POST['day']) and $_POST['day']=="03") echo "selected";  ?>  value="03">3</option>
                  <option <? if(isset($_POST['day']) and $_POST['day']=="04") echo "selected";  ?>  value="04">4</option>
                  <option <? if(isset($_POST['day']) and $_POST['day']=="05") echo "selected";  ?>  value="05">5</option>
                  <option <? if(isset($_POST['day']) and $_POST['day']=="06") echo "selected";  ?>  value="06">6</option>
                  <option <? if(isset($_POST['day']) and $_POST['day']=="07") echo "selected";  ?>  value="07">7</option>
                  <option <? if(isset($_POST['day']) and $_POST['day']=="08") echo "selected";  ?>  value="08">8</option>
                  <option <? if(isset($_POST['day']) and $_POST['day']=="09") echo "selected";  ?>  value="09">9</option>
                  <option <? if(isset($_POST['day']) and $_POST['day']=="10") echo "selected";  ?>  value="10">10</option>
                  <option <? if(isset($_POST['day']) and $_POST['day']=="11") echo "selected";  ?>  value="11">11</option>
                  <option <? if(isset($_POST['day']) and $_POST['day']=="12") echo "selected";  ?>  value="12">12</option>
                  <option <? if(isset($_POST['day']) and $_POST['day']=="13") echo "selected";  ?>  value="13">13</option>
                  <option <? if(isset($_POST['day']) and $_POST['day']=="14") echo "selected";  ?>  value="14">14</option>
                  <option <? if(isset($_POST['day']) and $_POST['day']=="15") echo "selected";  ?>  value="15">15</option>
                  <option <? if(isset($_POST['day']) and $_POST['day']=="16") echo "selected";  ?>  value="16">16</option>
                  <option <? if(isset($_POST['day']) and $_POST['day']=="17") echo "selected";  ?>  value="17">17</option>
                  <option <? if(isset($_POST['day']) and $_POST['day']=="18") echo "selected";  ?>  value="18">18</option>
                  <option <? if(isset($_POST['day']) and $_POST['day']=="19") echo "selected";  ?>  value="19">19</option>
                  <option <? if(isset($_POST['day']) and $_POST['day']=="20") echo "selected";  ?> value="20">20</option>
                  <option  <? if(isset($_POST['day']) and $_POST['day']=="21") echo "selected";  ?> value="21">21</option>
                  <option <? if(isset($_POST['day']) and $_POST['day']=="22") echo "selected";  ?>  value="22">22</option>
                  <option <? if(isset($_POST['day']) and $_POST['day']=="23") echo "selected";  ?>  value="23">23</option>
                  <option <? if(isset($_POST['day']) and $_POST['day']=="24") echo "selected";  ?>  value="24">24</option>
                  <option <? if(isset($_POST['day']) and $_POST['day']=="25") echo "selected";  ?>  value="25">25</option>
                  <option <? if(isset($_POST['day']) and $_POST['day']=="26") echo "selected";  ?>  value="26">26</option>
                  <option  <? if(isset($_POST['day']) and $_POST['day']=="27") echo "selected";  ?> value="27">27</option>
                  <option  <? if(isset($_POST['day']) and $_POST['day']=="28") echo "selected";  ?> value="28">28</option>
                  <option <? if(isset($_POST['day']) and $_POST['day']=="29") echo "selected";  ?>  value="29">29</option>
                  <option <? if(isset($_POST['day']) and $_POST['day']=="30") echo "selected";  ?>  value="30">30</option>
                  <option  <? if(isset($_POST['day']) and $_POST['day']=="31") echo "selected";  ?> value="31">31</option>

    вЫводим дни в селекте )) А как ты выводишь дни в селекте? %)

    Diwms, 13 Сентября 2012

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

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

• 1571
• …
• 340
• 339
• 338
• 337
• 336
• 335
• 334
• 333
• 332
• …
• 1