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

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

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

1. PHP (5706)
2. C++ (2739)
3. Куча (2363)
4. JavaScript (2030)
5. C# (1928)
6. Java (1502)
7. Си (1116)
8. Pascal (648)
9. Python (586)
10. 1C (538)
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. Swift (27)
22. Haskell (26)
23. Kotlin (14)

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

   +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

   $mes_for_mail=str_replace("[code]","",$mes_for_mail);
   $mes_for_mail=str_replace("[/code]","",$mes_for_mail);
   
   $mes_for_mail=str_replace("[b]","",$mes_for_mail);
   $mes_for_mail=str_replace("[/b]","",$mes_for_mail);
   
   $mes_for_mail=str_replace("[i]","",$mes_for_mail);
   $mes_for_mail=str_replace("[/i]","",$mes_for_mail);
   
   $mes_for_mail=str_replace("[u]","",$mes_for_mail);
   $mes_for_mail=str_replace("[/u]","",$mes_for_mail);
   
   $mes_for_mail=str_replace("[s]","",$mes_for_mail);
   $mes_for_mail=str_replace("[/s]","",$mes_for_mail);
   
   $mes_for_mail=str_replace("[tt]","",$mes_for_mail);
   $mes_for_mail=str_replace("[/tt]","",$mes_for_mail);
   
   $mes_for_mail=str_replace("[sup]","",$mes_for_mail);
   $mes_for_mail=str_replace("[/sup]","",$mes_for_mail);
   
   $mes_for_mail=str_replace("[sub]","",$mes_for_mail);
   $mes_for_mail=str_replace("[/sub]","",$mes_for_mail);
   
   $mes_for_mail=str_replace("[code]","",$mes_for_mail);
   $mes_for_mail=str_replace("[/code]","",$mes_for_mail);
   
   $mes_for_mail=str_replace("[font color=FF0000]","",$mes_for_mail);
   $mes_for_mail=str_replace("[font color=0000FF]","",$mes_for_mail);
   $mes_for_mail=str_replace("[font color=008080]","",$mes_for_mail);
   $mes_for_mail=str_replace("[/font]","",$mes_for_mail);

   я плакол)

   TBoolean, 15 Декабря 2011

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

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

   +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

   public function deletePubs() {
           $id = $this->core->request->parameters["id"];
           if (!empty($id)) {
           	$id++; $id--;
               $this->core->database->exec(
               "delete
   		FROM lists
   		WHERE id=".$id
               );
           }
           header("Location: /admin/about/listAbout");
       }

   Не знаю что автор хотел сделал кодом на 4-ой строке. Предполагаю, что таким образом он приводит переменную $id к целочисленному типу.

   alxkolm, 15 Декабря 2011

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

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

   +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

   <?
   	function lang($lang) {
   
   		$languages = array('ru', 'en', 'lv');
   	
   		foreach($languages as $lang_arr) {
   	                // Проверка, есть ли такой язык в массиве
   			if($lang == $lang_arr) {
   				// Меняем язык
   			}
   		}
   
   	}
   ?>

   in_array(); ?
   
   Не, не слышал... ;)

   killro0000, 15 Декабря 2011

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

4. Куча / Говнокод #8841

   +147

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

   <cfif ITEM_ACTIVE>
   <cfset red = "black">
   <cfelse>
   <cfset red = "red">
   </cfif>
   <font color=#red#>#ITEM_NAME#</font>

   Капитан Очевидность в смятении.

   7ion, 15 Декабря 2011

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

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

   +181

   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

   <!--Начало сценария
   
   var res="23342142"
   
   function check_me()
   {
   	var count=0
   	with(document.test)
   	{
   		if (!Q1[0].checked&&!Q1[1].checked&&!Q1[2].checked&&!Q1[3].checked) {count+=1};
   		if (!Q2[0].checked&&!Q2[1].checked&&!Q2[2].checked&&!Q2[3].checked) {count+=1};
   		if (!Q3[0].checked&&!Q3[1].checked&&!Q3[2].checked&&!Q3[3].checked) {count+=1};
   		if (!Q4[0].checked&&!Q4[1].checked&&!Q4[2].checked&&!Q4[3].checked) {count+=1};
   		if (!Q5[0].checked&&!Q5[1].checked&&!Q5[2].checked&&!Q5[3].checked) {count+=1};
   		if (!Q6[0].checked&&!Q6[1].checked&&!Q6[2].checked&&!Q6[3].checked) {count+=1};
   		if (!Q7[0].checked&&!Q7[1].checked&&!Q7[2].checked&&!Q7[3].checked) {count+=1};
   		if (!Q8[0].checked&&!Q8[1].checked&&!Q8[2].checked&&!Q8[3].checked) {count+=1};
   		if (count>0) {alert("Вы выполнили не все задания. Проверьте себя.")} 
   		else answer()
   	}
   }
   
   function control(k,f1,f2,f3,f4,f5,f6,f7,f8)
   {
   	if (k==1&&f1.checked) return true;
   	if (k==2&&f2.checked) return true;
   	if (k==3&&f3.checked) return true;
   	if (k==4&&f4.checked) return true;
   	if (k==5&&f5.checked) return true;
   	if (k==6&&f6.checked) return true;
   	if (k==7&&f7.checked) return true;
   	if (k==8&&f8.checked) return true;
   	return false;
   }
   
   function answer()
   {
   	answ="";
   	with(document)
   	{
   		answ+=control(res.charAt(0),test.Q1[0],test.Q1[1],test.Q1[2],test.Q1[3])?"1":"0"
   		answ+=control(res.charAt(1),test.Q2[0],test.Q2[1],test.Q2[2],test.Q2[3])?"1":"0"
   		answ+=control(res.charAt(2),test.Q3[0],test.Q3[1],test.Q3[2],test.Q3[3])?"1":"0"
   		answ+=control(res.charAt(3),test.Q4[0],test.Q4[1],test.Q4[2],test.Q4[3])?"1":"0"
   		answ+=control(res.charAt(4),test.Q5[0],test.Q5[1],test.Q5[2],test.Q5[3])?"1":"0"
   		answ+=control(res.charAt(5),test.Q6[0],test.Q6[1],test.Q6[2],test.Q6[3])?"1":"0"
   		answ+=control(res.charAt(6),test.Q7[0],test.Q7[1],test.Q7[2],test.Q7[3])?"1":"0"
   		answ+=control(res.charAt(7),test.Q8[0],test.Q8[1],test.Q8[2],test.Q8[3])?"1":"0"
   		showResult();
   	}
   }
   
   function showResult()
   {
   	var nok=0;
   	var i,s;
   	for (i=0;i<answ.length;i++) {nok+=answ.charAt(i)=="1"?1:0;}
                     if(nok<4) s='"НЕУДОВЛЕТВОРИТЕЛЬНО"';
                     if(nok==4) s='"УДОВЛЕТВОРИТЕЛЬНО"';
   	if(nok>4 && nok<6) s='"УДОВЛЕТВОРИТЕЛЬНО"';
   	if(nok==6) s='"ХОРОШО"';
   	if(nok>6 && nok<8) s='"ХОРОШО"';
   	if(nok==8) s='"ОТЛИЧНО"';
   	document.test.s1.value="Количество правильных ответов "+nok+". Ваша оценка "+s+". Загляните в окно рядом с номером задания. Если ответ правильный, то там (+). Если Вы ошиблись, там (-).";
   	with(document.test)
   	{
   		if (answ.charAt(0)=="1") {T1.value=" +"} else {T1.value=" -"};
   		if (answ.charAt(1)=="1") {T2.value=" +"} else {T2.value=" -"};
   		if (answ.charAt(2)=="1") {T3.value=" +"} else {T3.value=" -"};
   		if (answ.charAt(3)=="1") {T4.value=" +"} else {T4.value=" -"};
   		if (answ.charAt(4)=="1") {T5.value=" +"} else {T5.value=" -"};
   		if (answ.charAt(5)=="1") {T6.value=" +"} else {T6.value=" -"};
   		if (answ.charAt(6)=="1") {T7.value=" +"} else {T7.value=" -"};
   		if (answ.charAt(7)=="1") {T8.value=" +"} else {T8.value=" -"};
   	}
   }

   http://www.junior.ru/wwwexam/t_gig.htm
   Тест по информатике.

   Govnocoder#0xFF, 15 Декабря 2011

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

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

   +145

   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

   <script type = "text/javascript">
   <!--
   var m3_u = (location.protocol == "https:"?
     "https://bk55.ru/openx/www/delivery/ajs.php": "http://bk55.ru/openx/www/delivery/ajs.php");
   var m3_r = Math.floor(Math.random()*99999999999);
   if(!document.MAX_used) document.MAX_used = ",";
   document.write("<scr" + "ipt type = 'text/javascript' src = '" + m3_u);
   document.write("?zoneid=8&amp;source=main");
   document.write("&amp;cb=" + m3_r);
   if(document.MAX_used != ",") document.write("&amp;exclude=" + document.MAX_used);
   document.write(document.charset? "&amp;charset=" + document.charset:
     (document.characterSet? "&amp;charset=" + document.characterSet: ""));
   document.write("&amp;loc=" + escape(window.location));
   if(document.referrer) document.write("&amp;referer=" + escape(document.referrer));
   if(document.context) document.write("&context=" + escape(document.context));
   if(document.mmm_fo) document.write("&amp;mmm_fo=1");
   document.write("'><\/scr" + "ipt>");
   -->                                                                // ajax не нужен)))))
   </script>

   Использование баннерного движка OpenX на сайте bk55.ru. Всего на странице около десятка таких кусков, которые отличаются только значением zoneid. Жесть)))

   shlyapa, 15 Декабря 2011

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

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

   +142

   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

   <?php
   // страницы, содержимое которых надо получить
   $urls = array('www.yandex.ru', 'www.google.ru', 'www.mail.ru', 'www.rambler.ru');
    
   $rtasks = array(); // задачи чтения
   $wtasks = array(); // задачи записи
   $results = array(); // результаты
    
   foreach ($urls as $url) {
   	// открываем отдельный сокет
   	$sh = socket_create(AF_INET, SOCK_STREAM, SOL_TCP);
   	if (!$sh) continue;
    
   	// таймаут для чтения
   	socket_set_option($sh, SOL_SOCKET, SO_RCVTIMEO, array("sec" => 10, "usec" => 0));
   	// таймаут для записи
   	socket_set_option($sh, SOL_SOCKET, SO_SNDTIMEO, array("sec" => 10, "usec" => 0));
   	// задаем неблокирующий режим сокетов
   	socket_set_nonblock($sh);
   	// определяем ip хоста
   	$ip = gethostbyname($url);
   	// соединяемся
   	socket_connect($sh, $ip, 80);
    
   	// добавляем в задачи для записи
   	$wtasks[$url] = $sh;
   }
    
   // продолжаем, пока есть задачи для записи или чтения
   while ($wtasks || $rtasks) {
   	// массив для сокетов с возможностью чтения
   	$rtasks_ = $rtasks;
   	// массив для сокетов с возможностью записи
   	$wtasks_ = $wtasks;
    
   	// ждем результатов из сокетов
   	$n = socket_select($rtasks_, $wtasks_, $e=null, 10);
   	if ($n > 0) {
   		// сокеты, доступные для записи
   		foreach ($wtasks_ as $sh) {
   			// ищем урл страницы по дескриптору сокета в массиве задач записи
   			$url = array_search($sh, $wtasks);
   			// удаляем из задач записи
   			unset($wtasks[$url]);
   			// добавляем в задачи чтения
   			$rtasks[$url] = $sh;
   			// формируем http-заголовки
   			$headers  = "GET / HTTP/1.0\r\n";
   			$headers .= "Host: ".$url."\r\n";
   			$headers .= "User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; ru; rv:1.9.0.6) Gecko/2009011913 MRA 5.3 (build 02557) Firefox/3.0.6\r\n";
   			$headers .= "Accept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8\r\n";
   			$headers .= "Accept-Language: ru,en-us;q=0.7,en;q=0.3\r\n";
   			$headers .= "Accept-Charset: windows-1251,utf-8;q=0.7,*;q=0.7\r\n";
   			$headers .= "\r\n";
    
   			// записываем в сокет
   			if (socket_write($sh, $headers) === false) fclose($sh);
   		}
    
   		// сокеты, доступные для чтения
   		foreach ($rtasks_ as $sh) {
   			// ищем урл страницы по дескриптору сокета в массиве задач чтения
   			$url = array_search($sh, $rtasks);
   			if (!$url) continue;
   			// считываем результат из сокета
   			$result = '';
   			while ($r = socket_read($sh, 1024)) $result .= $r;
   			// закрываем сокет
   			socket_close($sh);
   			// удаляем из задач чтения
   			unset($rtasks[$url]);
   			// заносим html в массив результатов
   			$results[$url] = $result;
   		}
   	}
   	else {
   		break;
   	}
   }
   ?>

   a_e, 15 Декабря 2011

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

8. PHP / Говнокод #8837

   +163

   1. 1

   if(!($_POST[$_POST[parameter_value]])){$parameter_value=" ";}else{$parameter_value=$_POST[$_POST[parameter_value_cash]];}

   shl, 15 Декабря 2011

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

9. Java / Говнокод #8836

   +144

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

   private static final BigDecimal $150 = BigDecimal.valueOf(150);
   private static final BigDecimal $500 = BigDecimal.valueOf(500);
   private static final BigDecimal $751 = BigDecimal.valueOf(751);
   
   // rule example
   all(
       paymentOfType(COMPREHENSIVE, greaterOrEqual($500)),
       noPaymentsButComprehensive
   ).mapTo(result(ACTIVITY_TYPE_ACC, COMPREHENSIVE_OVER_500))

   Код мой. Есть таблица правил (в Excel), по которой нужно классифицировать некие объекты. Все правила собраны в одном классе и представляют из себя конструкции наподобие той, что в топике.
   У меня с остальной частью команды возник конфликт по поводу наименования констант: все считают, что называть константы как $500 - сущее зло, а правильно называть MAX_AMOUNT_FOR_COMP_OVER_500. Я считаю, что $500 - это 500 долларов, блеать, а константа нужна только в одном месте и для читаемости. Мне по сути пофигу, как называется константа, интересно узнать ваше мнение.
   наболело

   roman-kashitsyn, 15 Декабря 2011

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

10. C# / Говнокод #8835

    +133

    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

    /// <summary>
            /// Abs function
            /// </summary>
            private static object Abs(List<Expression> p)
            {
                return Math.Abs(p[0]);
            }
    
            /// <summary>
            /// Acos function
            /// </summary>
            private static object Acos(List<Expression> p)
            {
                return Math.Acos(p[0]);
            }
    
            /// <summary>
            /// Asin function
            /// </summary>
            private static object Asin(List<Expression> p)
            {
                return Math.Asin(p[0]);
            }
    
            /// <summary>
            /// Atan function
            /// </summary>
            private static object Atan(List<Expression> p)
            {
                return Math.Atan(p[0]);
            }
    
            /// <summary>
            /// Atan2 function
            /// </summary>
            private static object Atan2(List<Expression> p)
            {
                return Math.Atan2(p[0], p[1]);
            }

    Кусок кода от "Капитана Очевидность"

    govnokoder_, 15 Декабря 2011

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

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

• 2105
• …
• 866
• 865
• 864
• 863
• 862
• 861
• 860
• 859
• 858
• …
• 1