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

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

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

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

1. Куча / Говнокод #12403

   +136

   1. 1

   http://f.cl.ly/items/1B362r0E3T0z3E3v3A1f/Screen%20Shot%202013-01-10%20at%2011.01.45.png

   Это надо видеть.

   ArtFeel, 10 Января 2013

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

2. 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)

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

   +69

   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

   //А теперь я научу вас как правильно определять тип картинки
   $type = '';
   		
   # JPEG:
   $src = @imagecreatefromjpeg("./js/ajax_drag_drop_upload/php/files/".$f);
   if ($src !== false)
   {
   	$type = 'jpg';
   }
   else {
   	# GIF:
   	$src = @imagecreatefromgif("./js/ajax_drag_drop_upload/php/files/".$f);
   	if ($src !== false)
   	{
   		$type = 'gif';
   	}
   	else {
   		# PNG:
   		$src = @imagecreatefrompng("./js/ajax_drag_drop_upload/php/files/".$f);
   		if ($src !== false)
   		{
   			$type = 'png';
   		}
   	}	
           // А что делать с BMP? Та ну нафиг, BMP - это миф...
   }
   // Вот и все. Теперь вы знаете с каким типом картинки работаете
   // Ну а здесь уже можете писать ваш говнокод

   Очень эпичный code snippet. Посмотрите как автор ловко владеет возможностями языка PHP. В закладки!

   SunnyMagadan, 10 Января 2013

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

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

   +52

   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

   //--Корзина заказа
       $misc_basket=array();
       $misc_basket['num']=0;
       
       //--проверка наличия в корзине товаров
       $b_sum=0;
       $b_num=0;
       //var_dump($_SESSION['goods']);
       if(isset($_SESSION['goods']))
       {       
        //   echo "11";
           foreach(explode("^^^",$_SESSION['goods']) as $item)
           {
               $arr1=explode(":::",$item);
               $type="";
               if($arr1[2]=='disc')$dbs="disc";
               elseif($arr1[2]=='tire') $dbs="shina";
               else $dbs="truck_shina";
               $iddb=abs($arr1[0]);
               
              // echo "select `price` from `".PREFIX."$dbs` where `id`=$iddb";
               $res=$db->query("select `price` from `".PREFIX."$dbs` where `id`=$iddb");
               if($res->num_rows)
               {
                    $arr=$res->fetch_array();
                    $b_sum+=intval($arr1[1])*correct_double($arr['price']);
                    $b_num+=$arr1[1];
               }
           }
       }
       $misc_basket['num']=$b_num;
       $misc_basket['sum']=$b_sum;

   alex_ok, 10 Января 2013

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

5. JavaScript / Говнокод #12399

   +157

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

   $( function() { $('#brief').wysibb( window.uCoz.uwbb ); } );    
   $( function() { $('#message').wysibb( window.uCoz.uwbb ); } );    
   $( function() { $('#format_message, label[for="format_message"]').hide(); } );    
   $( function() { $('#format_brief, label[for="format_brief"]').hide(); } );

   В одну конструкцию-то стрёмно вставлять...

   TRANE73, 09 Января 2013

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

6. C++ / Говнокод #12398

   +23

   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

   struct S* gpS;
   
   struct S
   {
      // this class has no user-defined default ctor
      void *operator new (size_t size, void*p, int i)
      {
         ((S*)p)->i = i;   // ordinarily, should not initialize
                           // memory contents inside placement new
         return p;
      }
      int i;
   };

   Код с сайта майкрософт.

   LispGovno, 09 Января 2013

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

7. C++ / Говнокод #12396

   +10

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

   for (int i = 0; i < 15; i++) {
       // Прикольное место, надо прокомментировать
       // Если наша функция Fork() вернула true, то мы
       // в дочернем процессе и форкаться больше не надо
       // Форканье - это задача родителя
       // Дети этим заниматься не должны
       if (Fork()) break;
   }

   Создание дочерних процессов. Вот такой костыль. Есть идеи, как улучшить?

   kafeman, 07 Января 2013

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

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

   +127

   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

   Название: Регистрация заказчика.
   Основное действующее лицо: Менеджеру по работе с заказчиками
   Область действия: Система
   Уровень: Пользователя
   Минимальные гарантии: Система останется в рабочем состоянии
   Гарантии успеха: Клиент загеристрирован
   Триггер: Клиент обратился к менеджеру по работе с заказчиками с просьбой зарегистрироваться
   Основной сценарий:
   1. Клиент обращается к менеджеру по работе с заказчиками, менеджер открывает регистрацию, система запоминает.
   2. Менеджер по работе с заказчиками запрашивает у клиента, имя, клиент сообщает имя, менеджер вводит имя в соответвующую форму, система запоминает.
   3. Менеджер по работе с заказчиками запрашивает у клиента, отчество, клиент сообщает отчество, менеджер вводит отчество в соответвующую форму, система запоминает.
   4. Менеджер по работе с заказчиками запрашивает у клиента, фамилию, клиент сообщает фамилию, менеджер вводит фамилию в соответвующую форму, система запоминает.
   5. Менеджер по работе с заказчиками запрашивает у клиента, номер паспорта, клиент сообщает номер паспорта, менеджер вводит номер паспорта пв соответвующую форму, система подтверждает правильность номера паспорта.
   6. Менеджер по работе с заказчиками запрашивает у клиента, дату выдачи паспорта, клиент сообщает дату выдачи паспорта, менеджер вводит дату выдачи паспорта в соответвующую форму, система подтверждает.
   7. Менеджер по работе с заказчиками запрашивает у клиента, контактный номер, клиент сообщает контактный номер, менеджер вводит контактный номер в соответвующую форму, система подтверждает.
   8. Менеджер по работе с заказчиками закрывает регистрацию, система сохраняет информацию о клиенте и подтверждает успешность регистрации.
   Альтернативы:
   5.а. Введен неверный номер паспорта.
   	1. Система сообщает об ошибке
   	2. Переход к пункту 5 основного сценария.
   6.а. Клиент уже зарегистрирован.
   	1. Система сообщает об ошибке.

   Моя боль...

   myzone, 06 Января 2013

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

9. VisualBasic / Говнокод #12393

   −110

   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

   Sub WriteCategories(number, choice)
       
       strSql = ""
       strSql2 = ""
       
       If number = 1 Then
           strSql = "SELECT name, category_type_id" '... the rest is skipped
       Elseif number = 2 Or number = 3 Then
           strSql = "SELECT goco.category_order, goco.category_type_id," '... the rest is skipped
       End If
       
       If number = 1 And choice = 1 Then
           strSql2 = "SELECT gosct.category_type_id, goct.operator_id," '... the rest is skipped
           
       Set RS2 = db.Execute(strSql2)
           
       ' --- No sub categories are displayed if choice = 3
       Elseif choice <> 3 Then
           strSql2 = "SELECT goco.category_type_id, goco.sub_category_order, ..." 'the rest is skipped
           
           Set RS2 = db.Execute(strSql2)
       End If
       
       Set RS = db.Execute(strSql)
       
       ' --- CategoryListCurrent should not display selected category in categoryListMoveTo and vice versa.
       If number = 2 And choice = 1 Then
           '... the real code is skipped
           
           Do Until RS.EOF
               '... the real code is skipped
           Loop
       End If
       
       If number = 2 And choice = 2 Then
           '... the real code is skipped
           
           Do Until RS.EOF
               '... the real code is skipped
           Loop
       End If
       
       
       ' --- Loop through the recordset and print categories and sub categories.
       Do Until RS.EOF
           '... the real code is skipped
           
           If number = 2 And choice = 1 Then
               '... the real code is skipped
           End If
   
           If number = 2 And choice = 2 Then
               '... the real code is skipped
           End If
           
           If number = 3 And choice = 3 And subConfigCatId <> -1 Then
               '... the real code is skipped
           End If
           
           ' --- Print main categories
           Response.Write ("<option style=""color:"& color &""" value=""" & RS("category_type_id") & "_0"" " & tempSel & " " & disabled & ">" & RS("name") &  "</option>" &vbCrLf)
           
           If choice <> 3 Then
               ' --- Print sub categories
               RS2.MoveFirst
               If Not RS2.EOF Then
                   Do Until RS2.EOF
                       '... the real code is skipped
                       
                       If catId1 <> -1 And number = 2 And choice = 1 And subCatId1 > 0 Then
                           '... the real code is skipped
                       End If
                       
                       If catId2 <> -1 And number = 2 And choice = 2 And subCatId2 > 0 Then
                           '... the real code is skipped
                       End If
                       
                       ' --- Only print sub categories under a matching category
                       If CLng(RS2("category_type_id")) = CLng(RS("category_type_id")) Then
                           Response.Write ("<option value=""" & RS2("category_type_id") & "_" & RS2("sub_category_type_id") & """ " & tempSelSub & ">  - " & RS2("name") &  "</option>" &vbCrLf)
                       End If
                       RS2.MoveNext
                   Loop
               End If
           End If
           
           '... the real code is skipped
       Loop
   End Sub

   Один параметр числовой. Назовём его number (C) Captain Obvious.
   Второй параметр тоже числовой. Но так как имя number занято, назовём его choice.
   Именованные константы?.. Анти-паттерн «Spaghetti Code»? Не, не слышал…

   wissenstein, 06 Января 2013

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

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

    +61

    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

    /**
    	 * Convert unix timestamp into: (no leading zeros)
    	 * array( 'day' => x, 'month' => x, 'year' => x, 'hour' => x, 'minute' => x );
    	 * Written into separate static public function to allow for timezone to be used easily
    	 *
    	 * @param	integer	[$unix]	Timestamp
    	 * @return	array 	Date parts
    	 */
        static public function unixstamp_to_human( $unix=0 )
        {
        	$tmp = gmdate( 'j,n,Y,G,i', $unix );
    
        	list( $day, $month, $year, $hour, $min ) = explode( ',', $tmp );
    
        	return array( 'day'    => $day,
        				  'month'  => $month,
        				  'year'   => $year,
        				  'hour'   => $hour,
        				  'minute' => $min );
        }

    IPB 3.x
    Интересно сделали? А как бы сделали PRO :) ?

    111111, 05 Января 2013

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

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

• 2109
• …
• 1117
• 1116
• 1115
• 1114
• 1113
• 1112
• 1111
• 1110
• 1109
• …
• 1