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

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

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

1. PHP (5703)
2. C++ (2738)
3. Куча (2335)
4. JavaScript (2030)
5. C# (1929)
6. Java (1503)
7. Си (1102)
8. Pascal (649)
9. Python (585)
10. 1C (536)
11. SQL (434)
12. ActionScript (294)
13. Objective C (204)
14. bash (200)
15. Perl (196)
16. Assembler (148)
17. Ruby (144)
18. VisualBasic (140)
19. Lua (51)
20. Go (31)
21. Swift (29)
22. Haskell (28)
23. Kotlin (15)

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

   +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
   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

   <?
   include "mode.php";
   #session_start();
   ?>
   <?		$carttext .='<div class="text">';
   		$carttext .= "<form action=\"?confirm\"  method=\"post\"><table border=\"0\" width=\"100%\" cellpadding=\"3\" cellspacing=\"0\">";
   		$carttext .="<tr>";
   
   		
   		$split = explode(",",$list);
   
   		foreach ($split as $item) {
   			$check = explode("+",$item);
   			$query = mysql_query("SELECT p.id, p.urlname, p.name, p.group_join_category,  p.price, c.name, p.photo_foto, c.urlname FROM catalog_prod as p left join catalog_category as c  ON  p.group_join_category=c.id WHERE p.id=$check[0]");					
   			$row = mysql_fetch_row($query);
   			$row[10]="руб.";			
   			$add_item="";
   			
   
   			
   			$add_item.=$row[1]."/";
   			
   			$carttext .= "</tr><tr>";
   
   			
   			
   		if (!empty($row[6])) {
   
   		if(file_exists($_SERVER['DOCUMENT_ROOT'].$row[6])) {
   				$fo1 = '<img style="border:3px solid #f1f1f1;" src="/image.php/image-name.jpg?width=100&amp;cropratio=1:1&amp;image='.$row[6].'" alt="'.$row[2].'" width="100" height="100" align="left" />';}
   				else {$fo1 = '<img style="border:3px solid #f1f1f1;" src="/images/nofoto.png" alt="'.$row[2].'" width="100" height="100" align="left" />';}
   		} else {
   			$fo1 = '<img style="border:3px solid #f1f1f1;" src="/images/nofoto.png" alt="'.$row[2].'" width="100" height="100" align="left" />';
   		}	
   			
   			$carttext .= "<td valign=\"top\" width=\"100\">
   			<a href=\"/catalog/$row[7]/$add_item\">$fo1</a></td><td class=\"catlist\" valign=\"top\" width=\"100%\"><a href=\"/catalog/$row[7]/$add_item\">$row[2]</a></td>";
   			
   			
   			$carttext .= "<td valign=\"top\" class=\"catlist\" nowrap=\"nowrap\">$row[4] $row[10]</td>";
   			
   			$carttext .= '<td valign=\"top\" class="catlist"><input type="hidden" value="'.$row[4].'" name="curprice" id="'.$check[0].'_curprice" /><input type="text" maxlength="5" size="3" style="width:20px" name="'.$check[0].'_amount" value="'.$check[1].'" id="'.$check[0].'_amount" class="basketinput1" onKeypress="onlyDigit(event);" onkeyup="updateBasket('.$check[0].');" /></td>';
   
   		
   			$price = $row[4] * $check[1];
   			$price = sprintf("%0.2f", $price);
   
   			
   			$carttext .= "<td valign=\"top\" class=\"catlist\" nowrap>$price $row[10]</td>";
   			$carttext .= '<td valign=\"top\" class="catlist"><input type="button" onclick="xajax_DelFromCart('.$check[0].');" value="Удалить" class="cartsubmit1" /></td>';
   
   			if (!IsSet($total1)) {
   				$total1 = 0;
   			}
   			if (!IsSet($totalcheck1)) {
   				$totalcheck1 = 0;
   			}
   
   			$total1 = $total1+ $price;
   			
   			$totalcheck1 = $totalcheck1 + $check[1];
   			
   			$total1 = sprintf("%0.2f", $total1);
   		}	
   		
   		$carttext .= "</tr><tr>";
   		
   		
   		
   		$carttext .= "</tr><tr>";
   		
   		$carttext .= "<td colspan=\"6\"><div style=\"border-bottom: solid 1px #3e1f16; padding: 5px 0 0 0; font-size:1px;\">&nbsp;</div></td>";
   		$carttext .= "</tr><tr>";
   		
   		$carttext .= "<td width=\"100%\" colspan=\"3\" class=\"catlist1\"><b>Итого:</b></td>";
   		$carttext .= "<td class=\"catlist1\">$totalcheck1</td>";
   		$carttext .= "<td class=\"catlist1\" colspan=\"2\" nowrap><b>$total1 $value</b></td>";
   	
   		$carttext .= "</tr><tr>";
   	
   		$carttext .= '';
   		$carttext .= "<td colspan=\"6\"><div style=\"border-bottom: solid 1px #3e1f16; padding: 1px 0 0 0; font-size:1px;\">&nbsp;</div><br><input type=\"submit\" value=\"Оформить заказ\" class=\"inputsubmit1\" /></td>";
   		$carttext .= "</tr></table></form></div>";
   ?>

   Специалист формирует корзину товаров. ЗП 1500$

   mordrag, 07 Июля 2015

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

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

   +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

   class auth {
   
       var $db;
       var $settings;
   
       function __construct() {
           global $db, $settings;
           $this->db = $db;
           $this->settings = $settings;
   
           if (isset($_GET['token'])) {
   			...
   		}
   	}
   	...
   }

   Что не так? Подобие ООП не есть ООП.

   Serganbus, 07 Июля 2015

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

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

   +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

   function array_to_file($array_name, $array, $filename = 0, $file = 0)
   {
   	$level = 1;
   	if($file == 0) {
   		$level = 0;
   		$file = fopen($filename, "w");
   		if(!$file) {
   			return false;
   		}
   		fwrite($file, "<" . "?\n\$".$array_name." = ");
       }
   
       $cnt = count($array);
       $i = 0;
       fwrite($file, "\narray(\n");
       foreach($array as $key => $value)
       {
   		if($i++ != 0) {
   			fwrite($file, ",\n");
   		}
   		if(is_array($array[$key])) {
   			fwrite($file, "\"".$key."\" => ");
   			array_to_file($array_name, $array[$key], 0, $file);
   		} else  {
   			$value = addcslashes($value, "\""."\\\\");
   			fwrite($file, str_repeat(" ", ($level + 1) * 2) . "\"".$key."\" => \"".$value."\"");
   		}
       }
       fwrite($file, ")");
   
       if($level == 0) {
   		fwrite($file, ";\n?".">");
   		fclose($file);
   		return true;
       }
   }

   alex322, 02 Июля 2015

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

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

   +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

   <?php
   
   $self=$_SERVER['PHP_SELF'];
   
   if (isset($_GET['page'])) { $page=$_GET['page'];} else {$page=1;}
   
   $pages=25;
   $page_offset=5;
   $unique_links=9;
   
   if ($page != 1) { // Don't show back link if current page is first page.
   $back_page = $page-1;
   echo("<a href=\"$self?page=$back_page\">prev</a>    \n");}
   
   for ($i=1; $i <= $pages; $i++)  {
   
   if ($i == $page) {
    echo("<b>$i</b>\n");// If current page don't give link, just text.
    }  else {
   
   if (($i>$page-$page_offset) && ($i<$page+$page_offset)  && ($page>$page_offset)) {	
   echo("<a href=\"$self?page=$i\">$i</a> \n");
   } 
   elseif (($i<=$unique_links) && ($page<=$page_offset)) {
   	echo("<a href=\"$self?page=$i\">$i</a> \n");
   }
   elseif (($i>$pages-$unique_links) && ($page>$pages-$page_offset)) {
   echo("<a href=\"$self?page=$i\">$i</a> \n");	
   }
   
   }
   
   }
   
   if (!($page >= $pages) && $pages != 1) { // If last page don't give next link.
   $next_page = $page + 1;
   echo("<a href=\"$self?page=$next_page\">next</a>");}
   
   
   ?>

   Адовый аромат (хоть и рабочий), понимаю, но помогите упростить, пожалуйста. Условия внутри цикла скорее всего избыточны.
   Вкратце: "плавающий" пагинатор, 9 уникальных ссылок.

   nurfed362, 30 Июня 2015

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

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

   +167

   1. 1
   2. 2

   if ($result = mailer($data)) echo "<p>Коммерческое предложение отправлено к вам на электронную почту.</p>";
   else echo "<p>Коммерческое предложение отправлено к вам на электронную почту.</p>";//да простит меня Бог

   Ковыряю сайта заказчика. Нашел гениальный код в скрипте обработки формы.

   jankefir, 30 Июня 2015

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

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

   +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

   $vin = trim(strip_tags($_POST['param']));
   
   function parse_html_page($vin)
   {
   	$vin_check = file_get_contents("http://***.****.**/*****.html?vin=".$vin);
   	$parts = explode("<div id=\"vin-block\">", $vin_check);
   	$parts_2 = explode("<footer id=\"footer\">", $parts[1]);
   	$parts_3 = explode("vehicles.</dd></dl>", $parts_2[0]);
   	$parts_4 = explode("<div class=\"hr-bold-se\"></div>", $parts_3[0]);
   	
   	if($parts_4[2])
   	{
   	echo("<h1>Данные:</h1>".$parts_4[2]."</dd></dl></div>");
   	}
   	else { echo "<b>Данные не найдены, либо VIN-номер некорректен!</b>";}
   }

   Крутейший парсинг...

   kraldraav, 29 Июня 2015

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

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

   +141

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

   $query = mysql_query("SELECT * FROM users");
   $array = mysql_fetch_array($query);
   
   do
   {
   		echo 'name -'. $array['name'] . ' |  identif - ' . $array['id'];
   }
   while($array = mysql_fetch_array($query));

   Вывод данных из бд ...

   kraldraav, 29 Июня 2015

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

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

   +148

   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

   //----Навигация
   if ($page != 1) $pervpage = '<a href=?page=1>Первая</a> | <a href=?page='. ($page - 1) .'>Предыдущая</a> | ';
   if ($page != $total) $nextpage = ' | <a href=?page='. ($page + 1) .'>Следующая</a> | <a href=?page=' .$total. '>Последняя</a>';
   if($page - 5 > 0) $page5left = ' <a href=?page='. ($page - 5) .'>'. ($page - 5) .'</a> | ';
   if($page - 4 > 0) $page4left = ' <a href=?page='. ($page - 4) .'>'. ($page - 4) .'</a> | ';
   if($page - 3 > 0) $page3left = ' <a href=?page='. ($page - 3) .'>'. ($page - 3) .'</a> | ';
   if($page - 2 > 0) $page2left = ' <a href=?page='. ($page - 2) .'>'. ($page - 2) .'</a> | ';
   if($page - 1 > 0) $page1left = '<a href=?page='. ($page - 1) .'>'. ($page - 1) .'</a> | ';
   if($page + 5 <= $total) $page5right = ' | <a href=?page='. ($page + 5) .'>'. ($page + 5) .'</a>';
   if($page + 4 <= $total) $page4right = ' | <a href=?page='. ($page + 4) .'>'. ($page + 4) .'</a>';
   if($page + 3 <= $total) $page3right = ' | <a href=?page='. ($page + 3) .'>'. ($page + 3) .'</a>';
   if($page + 2 <= $total) $page2right = ' | <a href=?page='. ($page + 2) .'>'. ($page + 2) .'</a>';
   if($page + 1 <= $total) $page1right = ' | <a href=?page='. ($page + 1) .'>'. ($page + 1) .'</a>';
   if ($total > 1)
   {
   Error_Reporting(E_ALL & ~E_NOTICE);
   echo "<div class=\"pstrnav\">";
   echo $pervpage.$page5left.$page4left.$page3left.$page2left.$page1left.'<b>'.$page.'</b>'.$page1right.$page2right.$page3right.$page4right.$page5right.$nextpage;
   echo "</div>";
   }

   pi3141592653, 29 Июня 2015

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

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

   +141

   1. 1

   $skillusers = array($users['str']); //Ну нахера?!!

   Зачем, ну зачем оно могло понадобиццо? :-D

   VarrkaN, 27 Июня 2015

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

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

    +144

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

    //перевод даты в timestamp
    function strtotimestamp($time) {
        $tmp = explode(' ', $time);
        $date = explode('.', $tmp[0]);
        $hours = explode(':',$tmp[1]);
        $tmp = mktime($hours[0], $hours[1], 0, $date[1], $date[0], $date[2]);
        return $tmp;
    }

    когда нужно быстро сделать работающее решение, а времени на чтение гугла нет

    simplycrazy, 26 Июня 2015

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

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

• 571
• …
• 485
• 484
• 483
• 482
• 481
• 480
• 479
• 478
• 477
• …
• 1