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

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• 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. PHP / Говнокод #5414

   +166

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

   if ($redir < 0) {
   
   } else {
   
    echo $redir."\t";
        echo "
   ";

   Бывает даже и такое :(

   isergey, 27 Января 2011

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

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

   +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

   $result = mysql_query("SELECT name FROM joomlabd1.jos_favorit WHERE fav='$Uname' ORDER BY id")
   	or die("не выцепить" . mysql_error());
       $ids = mysql_query("SELECT id FROM joomlabd1.jos_favorit WHERE fav='$Uname'")
   	or die("не выцепить" . mysql_error());
   $key = mysql_query("SELECT `key` FROM joomlabd1.jos_favorit WHERE fav='$Uname'")
   	or die("не выцепить" . mysql_error());	
   $rows = mysql_num_rows($result);
    $fields = mysql_num_fields($result);
   //echo $rows;
   //echo $fields;
   
   for ($c=0; $c<$rows; $c++) {
    
        for ($cc=0; $cc<$fields; $cc++) {
       
   // дальше работа с извлеченными данными и много говнокода

   А еще можно вот так пробегать по строкам и столбцам результата запроса в БД

   isergey, 27 Января 2011

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

3. C# / Говнокод #5411

   +130

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

   private int FindIndexOfItemWithValue(object value)
   {
       for (int i = 0; i < base.Items.Count; i++)
       {
           object item = base.Items[i];
           if (InternalUtils.AreValuesEqual(this.SelectedValue, this.GetSelectedValue(item)))
           {
               return i;
           }
       }
       return -1;
   }

   Код комбобокса из System.Windows, Version=2.0.5.0 (Silverlight). Где тут ищется value - загадка природы.

   Unseen, 27 Января 2011

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

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

   +161

   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

   {if !$hard || 0 == $hard}
   	<img src="/i/level00.gif" alt="" /><img src="/i/level00.gif" alt="" /><img src="/i/level00.gif" alt="" /><img src="/i/level00.gif" alt="" /><img src="/i/level00.gif" alt="" />
   {elseif 1==$hard}
   	<img src="/i/level01_half.gif" alt="" /><img src="/i/level00.gif" alt="" /><img src="/i/level00.gif" alt="" /><img src="/i/level00.gif" alt="" /><img src="/i/level00.gif" alt="" /> 1
   {elseif 2==$hard}
       <img src="/i/level01.gif" alt="" /><img src="/i/level00.gif" alt="" /><img src="/i/level00.gif" alt="" /><img src="/i/level00.gif" alt="" /><img src="/i/level00.gif" alt="" /> 2
   {elseif 3==$hard}
       <img src="/i/level02.gif" alt="" /><img src="/i/level02_half.gif" alt="" /><img src="/i/level00.gif" alt="" /><img src="/i/level00.gif" alt="" /><img src="/i/level00.gif" alt="" /> 3
   {elseif 4==$hard}
       <img src="/i/level02.gif" alt="" /><img src="/i/level02.gif" alt="" /><img src="/i/level00.gif" alt="" /><img src="/i/level00.gif" alt="" /><img src="/i/level00.gif" alt="" /> 4
   {elseif 5==$hard}
       <img src="/i/level03.gif" alt="" /><img src="/i/level03.gif" alt="" /><img src="/i/level03_half.gif" alt="" /><img src="/i/level00.gif" alt="" /><img src="/i/level00.gif" alt="" /> 5
   {elseif 6==$hard}
       <img src="/i/level03.gif" alt="" /><img src="/i/level03.gif" alt="" /><img src="/i/level03.gif" alt="" /><img src="/i/level00.gif" alt="" /><img src="/i/level00.gif" alt="" /> 6
   {elseif 7==$hard}
       <img src="/i/level04.gif" alt="" /><img src="/i/level04.gif" alt="" /><img src="/i/level04.gif" alt="" /><img src="/i/level04_half.gif" alt="" /><img src="/i/level00.gif" alt="" /> 7
   {elseif 8==$hard}
       <img src="/i/level04.gif" alt="" /><img src="/i/level04.gif" alt="" /><img src="/i/level04.gif" alt="" /><img src="/i/level04.gif" alt="" /><img src="/i/level00.gif" alt="" /> 8
   {elseif 9==$hard}
       <img src="/i/level05.gif" alt="" /><img src="/i/level05.gif" alt="" /><img src="/i/level05.gif" alt="" /><img src="/i/level05.gif" alt="" /><img src="/i/level05_half.gif" alt="" /> 9
   {else}
       <img src="/i/level05.gif" alt="" /><img src="/i/level05.gif" alt="" /><img src="/i/level05.gif" alt="" /><img src="/i/level05.gif" alt="" /><img src="/i/level05.gif" alt="" /> 10
   {/if}

   Смарти...

   Klopp, 27 Января 2011

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

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

   +172

   1. 1
   2. 2
   3. 3

   if(!$go = false) {
   			
   }

   говнецо от самого себя, писал пока разговаривал и не вдумался что писал :DDDD

   sl1p, 27 Января 2011

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

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

   +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

   $phone = $DATABASE->select('select * from b_sale_order_props_value where ORDER_PROPS_ID=3');
           foreach($phone as $k) {
               $ph=str_replace("-", "", $k['VALUE']);
               $ph1=str_replace("(", "", $ph);
               $ph2=str_replace(")", "", $ph1);
               $ph3=str_replace(" ", "", $ph2);
               $ph4=str_replace("+7", "8", $ph3);
               $ph5=str_replace("/", ",", $ph4);
               $ph6=str_replace(";", ",", $ph5);
               $ph7=str_replace(".", "", $ph6);
               $ph8=str_replace("\\", ",", $ph7);
               $onephone = explode(",", $ph8);
               if(strlen($onephone[0])==11) {
                   if(substr($onephone[0],0,2) == "89") {
                       $arResult[$i]['PHONE']=$onephone[0];
                       $arResult[$i]['SITENAME'] ="xxxxx";
                       $i++;
                       if(!empty($onephone[1]))
                           if(substr($onephone[1],0,2) == "89"){
                               $arResult[$i]['PHONE']=$onephone[1];
                               $arResult[$i]['SITENAME'] ="xxxxx";
                               $i++;
                           }
                   }
               }
           }

   Да, не самый свежак, но..зацените, в отличии от простого копипаста строки тут кое-что новое! Во-первых не используются массивы в str_replace и во-вторых кадлый раз новая переменная) Я поржал)

   greevex, 27 Января 2011

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

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

   +163

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

   # изначальный код
   //... - здесь куча аналогичного срача из php + html
   exit ("<p>Вы ввели неверную сумму цифр с картинки на предыдущей странице. <br> <input name='back' type='button' value='Вернуться назад' onclick='javascript:self.back();'>");
   
   
   # код, который стал работать у человека, и который он посчитал выходом из ситуации
   echo "<meta http-equiv='refresh' content='3;URL=$_SERVER[HTTP_REFERER]'>"; 
   exit ("<div align='center'><div class='alert'>Не все поля были заполнены!</div></div>");

   У человека была проблема - не работала кнопка "Назад" при вводе некорректных данных. Решил он её кардинально - просто заставил редиректиться браузер обратно через <meta ..>-тег.

   UAS, 27 Января 2011

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

8. C# / Говнокод #5406

   +120

   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

   static void processCmd(string command) {
   	string[] c_args = command.Split(" ".ToCharArray());
   	switch (c_args[0]) {
   		case "beep":
   			nbr.PlayTone(4096, 500);
   			break;
   		case "exit":
   			exit_op();
   			break;
   		case "info":
   			log_ca("Info:");
   			log_ca(Application.ProductName + " " + Application.ProductVersion);
   			log_ca("listener is " + ((services_running[0]) ? "running" : "down"));
   			log_ca("updater is " + ((services_running[1]) ? "running" : "down"));
   			break;
   		case "start":
   			try {
   				switch (c_args[1]) {
   					case "updater":
   						break;
   					case "listener":
   						if (services_running[0])
   							log_ca("listener is already running");
   						else
   							start_listener();
   						break;
   					default:
   						throw new ArgumentException();
   						break;
   				}
   			}
   			catch {
   				log_ca("Usage: start <service>. Available services: listener, updater.");
   			}
   			break;
   		case "help":
   			foreach (string hs in System.IO.File.ReadAllLines("help.txt")) {
   				log_ca(hs);
   			}
   			break;
   		default:
   			log_ca("\"help\" will display all available commands");
   			break;
   		case "stop":
   			try {
   				nbr.MotorA.Brake();
   				nbr.MotorB.Brake();
   				nbr.MotorC.Brake();
   			}
   			catch { }
   			break;
   		case "run":
   			try {
   				switch(c_args[1]) {
   					case "a":
   						if(arr_motor[0]=="none")
   							log_e("Motor not found or config error");
   						else {
   							if (c_args[4] == "false")
   								nbr.MotorA = new NxtMotor(false);
   							else
   								nbr.MotorA = new NxtMotor(true);
   							nbr.MotorA.Run(Convert.ToSByte(c_args[2]), Convert.ToUInt32(c_args[3]));
   						}
   						break;
   					case "b":
   						if(arr_motor[1]=="none")
   							log_e("Motor not found or config error");
   						else {
   							if (c_args[4] == "false")
   								nbr.MotorB = new NxtMotor(false);
   							else
   								nbr.MotorB = new NxtMotor(true);
   							nbr.MotorB.Run(Convert.ToSByte(c_args[2]), Convert.ToUInt32(c_args[3]));
   						}
   						break;
   					case "c":
   						if(arr_motor[2]=="none")
   							log_e("Motor not found or config error");
   						else {
   							if (c_args[4] == "false")
   								nbr.MotorC = new NxtMotor(false);
   							else
   								nbr.MotorC = new NxtMotor(true);
   							nbr.MotorC.Run(Convert.ToSByte(c_args[2]), Convert.ToUInt32(c_args[3]));
   						}
   						break;
   					default:
   						throw new Exception();
   						break;
   				}
   			}
   			catch (Exception ex) {
   				log_ca("Usage: run <motor> <speed> <tacho> <reverse>. Example: run a 100 0 false.");
   			}
   		break;
   	}
   }

   Мой код, написано 3 года назад.

   A1mighty, 27 Января 2011

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

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

   +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

   function convert($str,$ky=''){
     if($ky=='')return $str;
     $ky=str_replace(chr(32),'',$ky);
     if(strlen($ky)<8)exit('key error');
     $kl=strlen($ky)<32?strlen($ky):32;
     $k=array();
     for($i=0;$i<$kl;$i++){
       $k[$i]=ord($ky{$i})&0x1F;
     }
     $j=0;
     for($i=0;$i<strlen($str);$i++){
       $e=ord($str{$i});
       $str{$i}=$e&0xE0?chr($e^$k[$j]):chr($e);
       $j++;
       $j=$j==$kl?0:$j;
     }
     return $str;
   }

   PHPASTA

   bugmenot, 27 Января 2011

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

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

    +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

    $_FILTER_MODULE = array(
    "A","a","B","b","C","c","D","d","E","e",
    "F","f","G","g","H","h","I","i","J","j",
    "K","k","L","l","M","m","N","n","O","o",
    "P","p","Q","q","R","r","S","s","T","t",
    "U","u","V","v","W","w","X","x","Y","y",
    "Z","z","-");
    
    $_FILTER_ACTION = array(
    "A","a","B","b","C","c","D","d","E","e",
    "F","f","G","g","H","h","I","i","J","j",
    "K","k","L","l","M","m","N","n","O","o",
    "P","p","Q","q","R","r","S","s","T","t",
    "U","u","V","v","W","w","X","x","Y","y",
    "Z","z","-");
    
    define("CHARS_LOGIN", "0123456789AaBbCcDdEeFfGgHhIiJjKkLlMmNnOoPpQqRrSsTtUuVvWwXxYyZzАаБбВвГгДдЕеЁёЖжЗзИиЙйКкЛлМмНнОоПпРрСсТтУуФфХхЦцЧчШшЩщЪъЫыЬьЭэЮюЯя-");
    define("CHARS_PASS", "0123456789AaBbCcDdEeFfGgHhIiJjKkLlMmNnOoPpQqRrSsTtUuVvWwXxYyZz");
    define("CHARS_FNAME", "AaBbCcDdEeFfGgHhIiJjKkLlMmNnOoPpQqRrSsTtUuVvWwXxYyZzАаБбВвГгДдЕеЁёЖжЗзИиЙйКкЛлМмНнОоПпРрСсТтУуФфХхЦцЧчШшЩщЪъЫыЬьЭэЮюЯя-");
    define("CHARS_LNAME", "AaBbCcDdEeFfGgHhIiJjKkLlMmNnOoPpQqRrSsTtUuVvWwXxYyZzАаБбВвГгДдЕеЁёЖжЗзИиЙйКкЛлМмНнОоПпРрСсТтУуФфХхЦцЧчШшЩщЪъЫыЬьЭэЮюЯя-");
    define("CHARS_EMAIL", "0123456789AaBbCcDdEeFfGgHhIiJjKkLlMmNnOoPpQqRrSsTtUuVvWwXxYyZz@-_.");
    $_FILTER_ID = array("0","1","2","3","4","5","6","7","8","9");
    $_FILTER_INT = array("0","1","2","3","4","5","6","7","8","9");
    
    function Filter($Str, $Filter) {
    	$NewStr = NULL;
    	$k = 0;
    	for($i = 0; $i<strlen($Str); $i++) {
    		for($n = 0; $n<sizeof($Filter); $n++) {
    			if($Str{$i} == $Filter[$n]) {
    				$NewStr[$k] = $Str[$i];
    				$k++;
    			}
    		}
    	}
    	return (string) @implode($NewStr);
    }
    
    function is_valid($value, $chars, $min_len, $max_len) {
    	$value_len = mb_strlen($value);
    	$chars_len = mb_strlen($chars);
    	$char_valid = false;
    	if($value_len > $max_len || $value_len < $min_len) {
    		return false;
    	}
    	for($i = 0; $i < $value_len; $i++) {
    		$char_valid = false;
    		for($k = 0; $k < $chars_len; $k++) {
    			if($value{$i} == $chars{$k}) {
    				$char_valid = true;
    			}
    		}
    		if($char_valid == false) {
    			return false;
    		}
    	}
    	return true;
    }

    Моё, октябрь 2009 =)

    Devzirom, 26 Января 2011

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

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

• 2109
• …
• 535
• 534
• 533
• 532
• 531
• 530
• 529
• 528
• 527
• …
• 1