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

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

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

1. PHP (5706)
2. C++ (2739)
3. Куча (2371)
4. JavaScript (2030)
5. C# (1928)
6. Java (1502)
7. Си (1117)
8. Pascal (648)
9. Python (586)
10. 1C (539)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (198)
15. Perl (194)
16. Assembler (146)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Go (31)
21. Haskell (29)
22. Swift (27)
23. Kotlin (14)

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

   +145

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

   <?php
   $names = array("Иван","Петр","Семен"); 
   if ($names[0]=="Иван"): 
   ?>
   <?php
   Привет, Ваня!
   endif; ?>

   Вот такому коду учат на intuit.ru

   Ded_Maksim, 23 Августа 2010

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

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

   +157

   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

   Часть 1
   =====================================
   foreach ($_POST['massive'] as $key => $val)
   {
   @mysql_query("INSERT INTO table VALUES (NULL,'".$val."'");
   } 	
   ======================================
   Часть 2
   ======================================
   if (!empty($_POST['massive'])/*&&is_array($_POST['massive']) для любителей пожестче=)*/)
   {
   //Так мы точно знаем что нам что-то передали(не выскочит нотис).
   //Далее надо сформировать правильный запрос. Никогда, НИКОГДА нельзя делать кучу инсертов циклом, т.к. скорее всего это закончиться смертью сервера.
   //А сделать как-нить так
   
   foreach ($_POST['massive'] as $key => $val)
   {
    $query.="('".mysql_real_escape_string($val)."'),"; //Cделаем заготовку для запроса, попутно параметр экранируется, что бы злобные хакеры не подсунули нам кавычку и прочую гадость.
   }
   $query= 'INSERT INTO table VALUES'.substr($query, 0, -1).';';//Обрежем последнюю лишнюю запятую, и в итоге получим запрос вида INSERT INTO table VALUES (),(),()..();
   if (!mysql_query($query)) //Скажу по секрету, что ставить собаку это моветон.
   {
    print "Упс, что-то сломалось";
    print mysql_error();
   }else {print "Вот и все"}
   ==========================================

   Вобщем, увидел на одном форуме часть 1.
   Будучи в хорошем настроении решил поправить. И Выдал часть 2, прямо вот в форуме. Думал отправить.. Но попахивает же, да?

   vitaly, 23 Августа 2010

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

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

   +104

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

   $a = !empty($_POST['a'])?$_POST['a']:null;
      if (empty($_POST['submit']) || !is_numeric($a)) {
        } else {
           echo "Прошол!";

   Vasiliy, 23 Августа 2010

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

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

   +171

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

   (
   	$hours >= 21 && $minutes >= 0	
   	or $hours == 0 && $minutes >= 0
   	or $hours == 1 && $minutes >= 0
   	or $hours == 2 && $minutes >= 0
   	or $hours == 3 && $minutes >= 0
   	or $hours == 4 && $minutes >= 0
   	or $hours == 5 && $minutes >= 0
   	or $hours == 6 && $minutes >= 0
   	or $hours == 7 && $minutes >= 0
   	or $hours == 8 && $minutes >= 0
   )

   http://webcodes.ru/publ/1-1-0-3524

   uWeb, 23 Августа 2010

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

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

   +165

   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
   
   $luser=$this->get_authorized_user();
   if (!$luser) retuen -101;
   
   // ...
   
     if (!$lpwd) 
     {
      $db->disconnect();
      return -102;
     }
   
   // ...
   
    if (!is_array($array)){$db->disconnect();return -103;}
     
    if (0 == count($array)){$db->disconnect();return -104;}
   
   // ...
   
    if (!is_array($array)){$db->disconnect();return -105;}
     
    if (0 == count($array)){$db->disconnect();return -106;}

   орфография и правописание сохранены
   
   вот таких отрицательных "статусов" по всему проекту просто миллион. Только одному автору известно, что они значат.
   
   и работает это все с неправильно написанным "return" в первом блоке

   user654321, 23 Августа 2010

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

6. JavaScript / Говнокод #4070

   +165

   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

   // Обновление надписи "Мои покупки"
   var file_f_basket = "/personal/cart/basket.php";
   
   //удаление пробелов, возврата каретки
   function trim(string)
   {
   	return string.replace(/(^\s+)|(\s+$)/g, "");
   }
   
   function BasketDeleteByID(id) {
   	BX.showWait();
   	jQuery.get(file_f_basket,{action:"DELETE",ID:id}, 
   	 function(data) 
   			{
   				if (trim(data) == "Success") 
   				{
   					var old = document.getElementById("record_" + id);
   					old.parentNode.removeChild(old);
   					SaleBasketUpdateTotal();
   				}
   				else if (trim(data) == "Empty")
   				{
   					var old = document.getElementById("goods");
   					old.parentNode.removeChild(old);
   					var old = document.getElementById("basket");
   					var mes = old.parentNode;
   					old.parentNode.removeChild(old);
   					var text = document.createTextNode("");
   					text.nodeValue = "Ваша корзина пуста.";
   					var elem = document.createElement("font");
   					elem.appendChild(text);
   					document.body.appendChild(elem);
   					elem.className="errortext";
   					mes.appendChild(elem);
   				}
   				
   				BasketUpdateLine();
   				BX.closeWait();
   				}
   				
   	);
   }
   function SaleBasketUpdateTotal() {
   
       var meForm   = document.getElementById('goods');
   
       var spanArray  = meForm.getElementsByTagName('span');
   
       var allSumm = 0;
   
       for (j = 0; j < spanArray.length; j++) {
           if (spanArray[j].className == "goodtotal")
              allSumm = allSumm + parseFloat(spanArray[j].innerHTML);
       }
   
       document.getElementById('total_sum').innerHTML = allSumm;
   }
   /*
    * Входные параметры функции:
    *    quant     - количество товара
    *    prise     - стоимость за единицу
    *    updElemId - идентификатор элемента, в котором требуется обновить данные (по конкретному товару)
    **/
    function SaleBasketUpdateTotalById(quant, price, updElemId)
    {
          
        var anum = /(^\d+$)|(^\d+\.\d+$)/;
        if (!anum.test(quant)) {
          
            alert('Введенное значение не является числом!');
            return;
        }
          
        goodSum = quant * price;
     
        document.getElementById(updElemId).innerHTML = goodSum;
   
   	 SaleBasketUpdateTotal();     
    }
    function isNumKeyPressed(_this,_event) {
   		if (!_event) _event = event;
   		var q = _this;
   		
   		if ((_event.keyCode > 8 || _event.keyCode < 57) & (_event.keyCode != 0)) return true;
   		if ((_event.charCode < 8 || _event.charCode > 57)) return false;
   		if (q.value.length >= 2) return false;	
   	}
   
   function BasketUpdateLine()
   {
   	jQuery.get(file_f_basket, {action: "COUNT"}, function(data)
   			{
   				if (parseInt(trim(data)) > 0 )
   					jQuery("#basket_line").html("<a href='/personal/cart/' class='basket-line'>Мои покупки (" + trim(data) + ")</a>");
   				else
   					jQuery("#basket_line").html("Мои покупки");
   			}
   	);

   Это мой гавнокодище... хыххы

   Ded_Maksim, 23 Августа 2010

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

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

   +175

   1. 1
   2. 2

   sleep(10);
   header('Location: http://example.com');

   Называется "Редирект через 10 секунд".
   Собственно автору требовалось это: <meta http-equiv="refresh" content="10; url=index.php">.

   Uchkuma, 23 Августа 2010

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

8. VisualBasic / Говнокод #4068

   −100

   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

   Case vIP(1)
            Case "1"  v_empirumserver="evempr"
   ....
     ещё где-то 100 таких-же строчек
   ....
            Case "160"    v_empirumserver = "svcf1501"
            Case "161"
               Select Case vIP(2)
               	Case "17"
   		End Select
   		    v_empirumserver = "svnrw040"          
            Case "163"
   		    v_empirumserver = "svrg5008" 
   ....
     ещё где-то 100 таких-же строчек
   ....
     End Select

   Только что увидел.....
   Это кусок с скрипта входу в систему на корпоративном домене.
   убило

   Select Case vIP(2)

   зачем оно здесь????

   adli__82, 23 Августа 2010

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

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

   +145

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

   <?php
   
   $x = true;
   
   if($x == 1) print "Говнокод - гамносайт";
   elseif($x == 2) print "Все завсегдатые данного сайта - пидоры, причем тупые";
   elseif($x == 3) print "Спасибо за внимание";
   
   ?>

   Сосем мой йух :)

   sergylens, 23 Августа 2010

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

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

    +164

    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

    <?php
     // Получение страницы напрямую или через проксю
    function getfile($GrabURL)
    {
     global $proxy_name,$proxy_port;
    
     if($proxy_name!="")
     {
      $f = urldecode($GrabURL);
      $proxy_fp = fsockopen($proxy_name, $proxy_port,$errno,$errstr);
      if (!$proxy_fp)
       return false;
      $out = "GET ".$f." HTTP/1.0\r\n\r\n";
      fwrite($proxy_fp, $out);
      $x = "";
      stream_set_timeout($OpenFile, 10);
      stream_set_blocking($OpenFile, 0);
      while (!feof($proxy_fp))
      {
       $RetrieveFile = fread($proxy_fp,8192) or last;
       $x = $x.$RetrieveFile;
      }
      fclose($proxy_fp);
     }
     else
     {
    
     $OpenFile = fopen($GrabURL, "rb");
    
     if(!$OpenFile) return "";
    
       $i = 0;
       $x = "";
       stream_set_timeout($OpenFile, 10);
       stream_set_blocking($OpenFile, 0);
       while (!feof($OpenFile))
        {
         $RetrieveFile = fread($OpenFile, 8192);
         $x=$x.$RetrieveFile;
        }
       fclose($OpenFile);
      }
     return $x;
    }
    ?>

    Аналог file_get_contents. По просьбам из #4065

    mr.The, 23 Августа 2010

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

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

• 2107
• …
• 402
• 401
• 400
• 399
• 398
• 397
• 396
• 395
• 394
• …
• 1