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

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

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

1. PHP (5708)
2. C++ (2739)
3. Куча (2379)
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 (200)
15. Perl (194)
16. Assembler (146)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (34)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

1. C# / Говнокод #14575

   +134

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

   Random random = new Random(Environment.TickCount);
   for (int i = 0; i < random.Next(380, 480); i++)
   {
           Marshal.AllocHGlobal(random.Next(0x4400, 0xd400));
           Thread.Sleep(1);
   }

   Из исходников подрядчика...
   Заказчиков надо садить на иглу правильно

   Stewie, 14 Февраля 2014

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

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

   +135

   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

   if (currentItem.Type.ActiveProperty != null)
   {
   	Type controltype = currentControl.GetType();
   	PropertyInfo pinfo = controltype.GetProperty(currentItem.Type.ActiveProperty.Name);
   
   	bool enabledtrue = (currentItem.Type.ActiveProperty.Name.ToLower() != "readonly");
   
   
   	if ((r1 != 2) || _page.ReadOnly)
   		enabledtrue = !enabledtrue;
   
   	pinfo.SetValue(currentControl, enabledtrue, null);
   }

   enabledtrue как пример интуитивно-понятного названия переменной

   Gocoder, 14 Февраля 2014

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

3. Pascal / Говнокод #14572

   +96

   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
   99. 99

   if movcamn[1] then begin
        //
     if movcams[ 2] then begin  if Nmr= 2 then goto m1;
     if movcams[ 3] then begin  if Nmr= 3 then goto m1;
     if movcams[ 4] then begin  if Nmr= 4 then goto m1;
     if movcams[ 5] then begin  if Nmr= 5 then goto m1;
     if movcams[ 6] then begin  if Nmr= 6 then goto m1;
     if movcams[ 7] then begin  if Nmr= 7 then goto m1;
     if movcams[ 8] then begin  if Nmr= 8 then goto m1;
     if movcams[ 9] then begin  if Nmr= 9 then goto m1;
     if movcams[10] then begin  if Nmr=10 then goto m1;
     if movcams[11] then begin  if Nmr=11 then goto m1;
     if movcams[12] then begin  if Nmr=12 then goto m1;
     if movcams[13] then begin  if Nmr=13 then goto m1;
     if movcams[14] then begin  if Nmr=14 then goto m1;
     if movcams[15] then begin  if Nmr=15 then goto m1;
     if movcams[16] then begin  if Nmr=16 then goto m1;
     if movcams[17] then begin  if Nmr=17 then goto m1;
     if movcams[18] then begin  if Nmr=18 then goto m1;
     if movcams[19] then begin  if Nmr=19 then goto m1;
     if movcams[20] then begin  if Nmr=20 then goto m1;
   
     m1:   Summs1:=0;        movcams[1]:=False;   movcamn[1]:=False;
         movcam[1,1]:=False;  movcam[1,2]:=False;  movcam[1,3]:=False;  movcam[1,4]:=False;
   
           Summs2:=0;        movcams[2]:=False;   movcamn[2]:=False;
         movcam[2,1]:=False;  movcam[2,2]:=False;  movcam[2,3]:=False;  movcam[2,4]:=False;
         if Nmr=2 then goto m2;  // <---
   
           Summs3:=0;        movcams[3]:=False;   movcamn[3]:=False;
         movcam[3,1]:=False;  movcam[3,2]:=False;  movcam[3,3]:=False;  movcam[3,4]:=False;
         if Nmr=3 then goto m2;  // <---
   
           Summs4:=0;         movcams[4]:=False;   movcamn[4]:=False;
         movcam[4,1]:=False;  movcam[4,2]:=False;  movcam[4,3]:=False;  movcam[4,4]:=False;
         if Nmr=4 then goto m2;  // <---
   
           Summs5:=0;        movcams[5]:=False;   movcamn[5]:=False;
         movcam[5,1]:=False;  movcam[5,2]:=False;  movcam[5,3]:=False;  movcam[5,4]:=False;
         if Nmr=5 then goto m2;  // <---
   
           Summs6:=0;        movcams[6]:=False;   movcamn[6]:=False;
         movcam[6,1]:=False;  movcam[6,2]:=False;  movcam[6,3]:=False;  movcam[6,4]:=False;
         if Nmr=6 then goto m2;  // <---
   
           Summs7:=0;        movcams[7]:=False;   movcamn[7]:=False;
         movcam[7,1]:=False;  movcam[7,2]:=False;  movcam[7,3]:=False;  movcam[7,4]:=False;
         if Nmr=7 then goto m2;  // <---
   
           Summs8:=0;        movcams[8]:=False;   movcamn[8]:=False;
         movcam[8,1]:=False;  movcam[8,2]:=False;  movcam[8,3]:=False;  movcam[8,4]:=False;
         if Nmr=8 then goto m2;  // <---
   
           Summs9:=0;        movcams[9]:=False;   movcamn[9]:=False;
         movcam[9,1]:=False;  movcam[9,2]:=False;  movcam[9,3]:=False;  movcam[9,4]:=False;
         if Nmr=9 then goto m2;  // <---
   
                //
           Summs10:=0;        movcams[10]:=False;   movcamn[10]:=False;
         movcam[10,1]:=False;  movcam[10,2]:=False;  movcam[10,3]:=False;  movcam[10,4]:=False;
         if Nmr=10 then goto m2;  // <---
   
           Summs11:=0;        movcams[11]:=False;   movcamn[11]:=False;
         movcam[11,1]:=False;  movcam[11,2]:=False;  movcam[11,3]:=False;  movcam[10,4]:=False;
         if Nmr=11 then goto m2;  // <---
   
           Summs12:=0;        movcams[12]:=False;   movcamn[12]:=False;
         movcam[12,1]:=False;  movcam[12,2]:=False;  movcam[12,3]:=False;  movcam[10,4]:=False;
         if Nmr=12 then goto m2;  // <---
   
           Summs13:=0;        movcams[13]:=False;   movcamn[13]:=False;
         movcam[13,1]:=False;  movcam[13,2]:=False;  movcam[13,3]:=False;  movcam[10,4]:=False;
         if Nmr=13 then goto m2;  // <---
   
           Summs14:=0;        movcams[14]:=False;   movcamn[14]:=False;
         movcam[14,1]:=False;  movcam[14,2]:=False;  movcam[14,3]:=False;  movcam[10,4]:=False;
         if Nmr=14 then goto m2;  // <---
   
           Summs15:=0;        movcams[15]:=False;   movcamn[15]:=False;
         movcam[15,1]:=False;  movcam[15,2]:=False;  movcam[15,3]:=False;  movcam[10,4]:=False;
         if Nmr=15 then goto m2;  // <---
   
           Summs16:=0;        movcams[16]:=False;   movcamn[16]:=False;
         movcam[16,1]:=False;  movcam[16,2]:=False;  movcam[16,3]:=False;  movcam[10,4]:=False;
         if Nmr=16 then goto m2;  // <---
   
           Summs17:=0;        movcams[17]:=False;   movcamn[17]:=False;
         movcam[17,1]:=False;  movcam[17,2]:=False;  movcam[17,3]:=False;  movcam[10,4]:=False;
         if Nmr=17 then goto m2;  // <---
   
           Summs18:=0;        movcams[18]:=False;   movcamn[18]:=False;
         movcam[18,1]:=False;  movcam[18,2]:=False;  movcam[18,3]:=False;  movcam[10,4]:=False;
         if Nmr=18 then goto m2;  // <---
   
           Summs19:=0;        movcams[19]:=False;   movcamn[19]:=False;
         movcam[19,1]:=False;  movcam[19,2]:=False;  movcam[19,3]:=False;  movcam[10,4]:=False;
     m2:
     end; end; end; end; end; end; end; end; end; end;   // 20 штук
     end; end; end; end; end; end; end; end; end; end;

   Попалось в одном проекте, который приходилось расчищать.
   Особенно позабавило заботливое "// 20 штук" в конце.

   qb60, 14 Февраля 2014

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

4. JavaScript / Говнокод #14571

   +151

   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

   $('.catalogue-list .in-cart a, .items-slider .in-cart a').click(function()
   	{
   		var src_pic = $(this).parent('div').parent().children('.pic_ajax').children().children().attr('src');
   		var name_pic = $(this).parent('div').parent().children('.pic').children().children().attr('alt');
   		var colors = $(this).parent('div').children('.colors_values').children('input');
   		var colors_for_ajax = "";
   		$.each( colors, function() {
   			colors_for_ajax = colors_for_ajax+$(this).val()+',';
   		});
   		
   		if(colors_for_ajax)
   			var url=$(this).attr('href'); 
   		$.fancybox({
   					type		: 'ajax',
   					href		: '/catalogue/ajax_add_to_cart_popup.php?NAME='+encodeURI(name_pic)+'&SRC='+encodeURI(src_pic)+'&COLORS='+encodeURI(colors_for_ajax)+'&URL_ADD='+encodeURI(url),
   					fitToView	: false,
   					width		: '336px',
   					height		: '234px',
   					autoSize	: false,
   					autoDimensions: false,
   					openEffect	: 'fade',
   					closeEffect	: 'none',
   					margin		: 0,
   					padding		: 0
   					});
   		if(!colors_for_ajax)
   		{
   			$(this).parent('.in-cart').find('.ajax-load').show('fast');
   			var url=$(this).attr('href');
   			$('.top-line .block .cart').load(url,'',onAjaxCart);
   		}
   		return false;
   	});

   Осталось от прошлого разраба в проекте. Аяксовое добавление товара в корзину...
   
   Зачем получать имя,свойства,картинку из ID элемента?! Я лучше это передам параметрами.

   kopoBko, 14 Февраля 2014

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

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

   +153

   1. 1

   $rec_area=sql2arr2('select om,alr from omolog where omvis=1');

   названия полей и таблицы

   govnomes, 14 Февраля 2014

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

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

   +159

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

   class Z {
      public function __toString() { return 'zzzzzzzz'; }
   }
   
    $zz = array();
    $zz[call_user_func_array('sprintf', array('%s', new Z))] = 1;

   нахуй так жить?

   __proto__, 13 Февраля 2014

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

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

   +151

   1. 1
   2. 2
   3. 3

   error_reporting( 0 );
   
   unset( $x[new XSLTProcessor()][new RecursiveArrayIterator()] );

   __proto__, 13 Февраля 2014

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

8. Perl / Говнокод #14557

   −151

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

   if( my $row = &dbselectrow( $sql ) )
           {
                   if( $row -> { 'ofid' } == 9782 ) # moscow
                   {
                           if( $row -> { 'jt' } =~ /Начальник /i )
                           {
                                   &subscribe_to_management( $row -> { 'email' } );
                           }
                   }
           }

   Автоматическая подписка сотрудника руководящей должности на список рассылки для руководителей.

   kainwinterheart, 13 Февраля 2014

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

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

   +138

   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

   private function toDBFields($fields){
   			$db_fields = array();
   			if ($fields){
   				foreach($fields as $field){
   					switch ($field){
   
                           case 'p_bits':
                               if (!array_search('b1',$db_fields))	$db_fields[] = 'b1';
                               break;
                           case 'n2':
                               if (!array_search('b1',$db_fields))	$db_fields[] = 'b1';
                               break;
                           case 'n1':
                               if (!array_search('b1',$db_fields))	$db_fields[] = 'b1';
                               break;
   						case 'cntdf':
   							if (!array_search('cntdf',$db_fields))	$db_fields[] = 'cntdf';
   							break;
   						case 'first_pckg':
   						
   							break;
   						case 'az':
   							if (!array_search('az',$db_fields))		$db_fields[] = 'az';
   							break;
   						case 'akb_v':
   							//if (!array_search('b2',$db_fields))		$db_fields[] = 'b2';
   							if (!array_search('b3',$db_fields))		$db_fields[] = 'b3';
   							break;
   						case 'p1code':
                           case 'ignition_ap1':
   							if (!array_search('b4',$db_fields))		$db_fields[] = 'b4';
   							if (!array_search('b6',$db_fields))		$db_fields[] = 'b6';
   							break;
   						case 'fuel_p2':
                           case 'fuel_p2_debug':
                           case 'p2code':
   							if (!array_search('b5',$db_fields))		$db_fields[] = 'b5';
   							if (!array_search('b6',$db_fields))		$db_fields[] = 'b6';
   							break;
                           case 'fuel_p3':
                           case 'fuel_p3_debug':
   						case 'p3code':
                           case 'p3code_debug':
   							if (!array_search('b7',$db_fields))		$db_fields[] = 'b7';
   							if (!array_search('b6',$db_fields))		$db_fields[] = 'b6';
   							break;
                           case 'fuel_p3p4_2drt':
                               if (!array_search('b8',$db_fields))		$db_fields[] = 'b8';
   							if (!array_search('b7',$db_fields))		$db_fields[] = 'b7';
   							if (!array_search('b6',$db_fields))		$db_fields[] = 'b6';
   							break;
                           case 'fuel_p2p3':
                               if (!array_search('b5',$db_fields))		$db_fields[] = 'b5';
   							if (!array_search('b7',$db_fields))		$db_fields[] = 'b7';
   							if (!array_search('b6',$db_fields))		$db_fields[] = 'b6';
   							break;
                           case 'fuel_p4':
                           case 'p4code_debug':
   						case 'p4code':
   							if (!array_search('b8',$db_fields))		$db_fields[] = 'b8';
   							if (!array_search('b6',$db_fields))		$db_fields[] = 'b6';
   							break;
   					}
   				}
   			}
   
   			return $db_fields;

   Вот такой кейс который используется в системе для перевода типа переменной к названию поле в таблице ....

   Deimm, 13 Февраля 2014

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

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

    +154

    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

    <?php
    session_start();
    include("database.php");
    include("filter.php");
    $filter = new Filter();
    isset($_GET['cat']) ? $id = $filter->Symbols($_GET['cat']) : exit("Вы зашли на страницу без параметра!");
    	
    if ( !isset($_POST['class']) && empty($_POST['class']) )
    {
    	include('header.php');
    	echo '
    		<title>Title</title>
    	';
    	include('slider.php');
    	echo '
    	<div id="Content">
    	';
    }
    
    $query_amount = $mysqli->query("SELECT COUNT(id) FROM `page_film` WHERE `genre_film`='$id' ");
    $amount_film = $query_amount->fetch_row();
    
    $query = $mysqli->query("SELECT * FROM `page_film` WHERE `genre_film`='$id' ORDER BY `id` DESC ");
    
    if ( $amount_film[0] > 0 )
    	while( $row = $query->fetch_array(MYSQLI_ASSOC) )
    		echo '
    			<div class="block-film">
    				<a class="block-link-image" href="../films/film.php?id='.$row['id'].'">
    					<img src="'.$row['poster'].'" />
    				</a>
    				<div class="block-film-text">
    					<p class="film-name">
    						<a href="../films/film.php?id='.$row['id'].'">'.$row['title'].'</a>
    					</p>
    					<p class="film-mini-name">
    						<a href="../films/film.php?id='.$row['id'].'">'.$row['desc'].'</a>
    					</p>
    					<p class="film-genre"><span>'.$row['genre'].'</span></p>
    					<p class="film-description">'.$row['description'].'</p>
    				</div>
    			</div>
    		';
    
    if ( !isset($_POST['class']) && empty($_POST['class']) )
    {
    	echo '	
    	</div>
    </body>
    </html>
    	';
    }
    ?>

    alexkor, 13 Февраля 2014

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

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

• 2109
• …
• 1297
• 1296
• 1295
• 1294
• 1293
• 1292
• 1291
• 1290
• 1289
• …
• 1