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

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

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

1. PHP (5706)
2. C++ (2739)
3. Куча (2360)
4. JavaScript (2030)
5. C# (1928)
6. Java (1502)
7. Си (1115)
8. Pascal (648)
9. Python (583)
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 / Говнокод #8684

   +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
   24. 24
   25. 25

   public function selectItems( $id, $withStrictment = 'no' /* $withStrictment = no, retail, discount */ ) {
   		
   		if( $withStrictment == 'discount' ) $onlyWithRetailPriceSql = " and items.articul in ( select articul from items_discount ) ";
   		if( $withStrictment == 'retail' ) $onlyWithRetailPriceSql = " and items.articul in ( select articul from retailPrice ) ";
   	
   		$a = database::select("	SELECT items.* FROM items WHERE items.id = '$id' and items.deleted = 'no' ");
   		$a = $a[0];
   
   		$this->data = database::select("SELECT items.*, 
   										IF(DATEDIFF(now(),created) <= 30, 'Новинка! ', '') as fresh,
   										( items.quantity - coalesce(view_pendeditemsquantity.sum,0) ) as quantity,
   										items_comments.comment as dopDescr, items_comments.comment as comment, items_discount.priceAsUSD as discountPrice,
   										items_packing.packing,
   										concat(items.name, ' - ', items.quantity) as name
   										FROM items 
   										LEFT JOIN view_pendeditemsquantity ON items.articul = view_pendeditemsquantity.articul
   										LEFT JOIN items_comments ON items.articul = items_comments.articul
   										left join items_discount on items.articul = items_discount.articul
   										left join items_packing on items.articul = items_packing.articul
   										WHERE items.group_id in ( select articul from items where is_group = '1' and name = '{$a['name']}' and group_id = '' )
   										  and items.deleted = 'no' 
   										  $onlyWithRetailPriceSql 
   										order by items.quantity desc, items.name asc
   										");
   	}

   guzik, 30 Ноября 2011

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

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

   +186

   1. 1

   $__language = ($__language == 'en')?'ru':'lv';

   мозговынос

   jokz, 30 Ноября 2011

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

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

   +177

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

   if (preg_replace("/[\D]+/" , '', $word)) {
    $word = str_replace('.','',$word);
    $word = str_replace(',','',$word);
    $word = str_replace(' ','',$word);
    $word = str_replace('$','',$word);
    $word = str_replace('*','',$word);
   }

   Увидев первую строчку я уж обрадовался, оказалось рано.

   Daemonium, 29 Ноября 2011

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

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

   +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

   try {
       $newUserId = $users->insert($data);
   } catch (Exception $e) {
       if ($e->getCode() == '23000') {
           $this->customRedirect(
               array(
                   'notice',
                   'User already exists',
               )
           );
           return;
       } else {
           throw new Exception(null, null, $e);
       }
   }

   Быстрая проверка на уникальность.

   grudzinsky, 29 Ноября 2011

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

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

   +167

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

   if (is_file($GLOBALS['fldr']."modules/sell/images/items/small_".str_replace("<img src=http://www.site.ru/modules/sell/images/items/small_", "", str_replace(" border=0>","",$M['icon'])))) {
      echo "<a  href='".$GLOBALS['fldr']."marki/models/object/current/index.php?id=".$M['id']."'>".replace_icon($M['icon'])."</a>";
     }
   
   function replace_icon($var) {
    return str_replace("<img src=http://www.site.ru/modules/sell/images/items/", "<img src=".$GLOBALS['fldr']."modules/sell/images/items/?img_name=", $var);
   }

   Мы легких путей не ищем...

   Daemonium, 29 Ноября 2011

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

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

   +146

   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

   if ($url=='/news/index.html')
   {
   	$data = $this->dbf->execute('select * from news where status=1 order by date_add desc,id desc limit 10');
   	$object->smarty->assign('news_body',$data);
   	$object->smarty->assign('title','Новости');
   } elseif(preg_match('@/news/([0-9]+?)/@i',$url,$out)){
   			$data = $this->dbf->execute('select * from news where status=1 and id='.intval($out[1]).' order by date_add desc limit 1');
   			$object->smarty->assign('news_body',$data);
   			$object->smarty->assign('title','Новости');			
   } elseif($url=='/search/') {
   	$data = array();
   			
   	if ($_POST['search']!='' && strlen($_POST['search'])>6)
   	{
   		$data = $this->dbf->execute('select * from pages where content like "%'.mysql_real_escape_string($_POST['search'],$this->dbh->conn).'%" limit 10');
   	}
   	$object->smarty->assign('search',$data);
   	$object->smarty->assign('title','Поиск');
   	$object->smarty->assign('news',$this->dbh->execute('select * from news where status=1 order by date_add desc limit 3'));
   			
   } else {
   	$data = $this->dbf->execute('select * from pages where link="'.mysql_real_escape_string($url,$this->dbh->conn).'"');
   	$this->data = $data[0];
   	$object->smarty->assign('title',$this->data['title']);
   	$object->smarty->assign('news',$this->dbh->execute('select * from news where status=1 order by date_add desc limit 3'));
   }

   laplay, 29 Ноября 2011

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

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

   +147

   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

   function rating() {
   
    @session_start();
    session_register ("set_ms");
    if (isset($_SESSION['set_ms'])) {
     return false; // старый посетитель
    } else {
     @session_start();
     session_register ("set_ms");
     $_SESSION['set_ms'] = rand(10000, 99999);
     return true; // новый посетитель
    };
   
   
   }

   Daemonium, 29 Ноября 2011

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

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

   +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

   $menu_string =  "Список устройств=sp_ustr.php~Список пользователей=sp_cl.php~Список параметров=sp_p_param.php~Список компаний=sp_comp.php~Список ролей=sp_rol.php~Список параметров отчетов=sp_repparam.php~Список наименований  датчиков=sp_sensors.php~Список групп=sp_groups.php~Список пользовательских настроек=spn_cl.php~Просмотр активаций=sp_activ.php~Выбрать организацию=vb_org.php~Экспресс-отчеты=sp_report.php".$st;
   
   $menu = explode("~",$menu_string);
   //~Оповещение=soobsh.php~Добавить лицензию=D_lic.php~Выгрузка данных=dat_out.php~Отчеты=otch.php~Експресотчеты=sp_report.php~Параметры датчиков=sp_param.php~Удаленное управление=connect.php~Активация программы=d_activ.php~Параметры отчетов=sp_repparam.php
   for ($i=0; $i < count($menu); $i++) {
   	$menu_1 = explode("=",$menu[$i]);
   	if (2 == count($menu_1)) {
   		echo "<a href=",$menu_1[1],">",$menu_1[0],"</a><br>";
   	} else {
   		echo "<a href=index-",$i+1,".php>",$menu_1[0],"</a><br>";
   	};
   };

   нужно сделать на сайте простое статическое меню.
   вот как, элегантно было решено сделать =)))

   burlak, 29 Ноября 2011

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

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

   +171

   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 (isset($_POST['password'])) {
     $password = $_POST['password'];
     if ($password == '') exit("Вы не ввели пароль");
     if (strlen($password) < 3) 
       exit ("Пароль должен состоять не менее чем из 3 символов.");
     $password = md5($password);
     $password = $password."b3p6f";
     $result2 = mysql_query("UPDATE users SET password='$password' WHERE email='$old_email'");
     if ($result2 == 'TRUE')
       echo "<html>
               <head>
                 <meta http-equiv='Refresh' content='5; URL=page.php?id=".$_SESSION['id']."'>
               </head>
             <body>
               Ваш пароль изменен! Вы будете перемещены через 5 сек. Если не хотите ждать, то
               <a href='page.php?id=".$_SESSION['id']."'>
                 нажмите сюда.
               </a>
             </body>
           </html>";
   }

   Код из серии "Шоколадные шедевры"

   SunnyMagadan, 29 Ноября 2011

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

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

    +176

    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

    $query = "
    			SELECT `ord`.*,`cus`.`existing_phone_number`,`cus`.`nova_jednotka`,
    				`cus`.`order_phone`,`cus`.`custom_delivery_date`, `cus`.`delivery_address`,
    				`cus`.`invoice_address`, `cus`.`invoice_street`, `cus`.`invoice_street_number`,
    				`cus`.`invoice_city`, `cus`.`invoice_zip`
    			FROM ".(!empty($_GET["copy_view"]) && $_GET["copy_view"] == 1 ? "`orders`" : "`".$this->table."`")." AS `ord`,
    				`products`, `navigace`, `customers` AS `cus`
    			WHERE `ord`.`order_id` = ".$order_id."
    				AND `ord`.`id_produktu_db` = `products`.`id`
    				AND `products`.`navigace_polozka_id` = `navigace`.`id`
    				AND `cus`.`id` = `ord`.`customers_id`
    			union all
    			select
    				null, null, null, null, null, null, null, null, null, null,
    				null, null, null, null, null, null, null, null, null, null,
    				null, null, null, null, null, null, null, null, null, null,
    				null, null, null, null, null, 
                                    '', tp.description, tp.product_id, tp.activation_price, tp.activation_price_EUR,
    				0, tor.pocet_kusu, tp.name, null, null, null, null, null, null, null,
    				null, null, null, null, null, null, null, null, null, null,
    				null, null, null, null, null, null, null, null, null, null,
    				null, null, null, null, null, null, null, null, null, null,
    				null, null, null, null, null, null, null, null, null, null,
    				null, null, null, null, null, null, null, null, null, null, 
                                    null, null, null, null, null, null, null, null, null, null, null, null, null
    			from tarifs_prop tp
    			inner join tarifs_orders tor on tp.id = tor.tarif_prop_id
    			where tor.order_id = ".$order_id."
    			union all
    			select
    				null, null, null, null, null, null, null, null, null, null,
    				null, null, null, null, null, null, null, null, null, null,
    				null, null, null, null, null, null, null, null, null, null,
    				null, null, null, null, null, '', b.description, b.product_id, b.activation_price, b.activation_price_EUR,
    				0, tor.pocet_kusu, b.name, null, null, null, null, null, null, null,
    				null, null, null, null, null, null, null, null, null, null,
    				null, null, null, null, null, null, null, null, null, null,
    				null, null, null, null, null, null, null, null, null, null,
    				null, null, null, null, null, null, null, null, null, null,
    				null, null, null, null, null, null, null, null, null, null, 
                                    null, null, null, null, null, null, null, 
                                    null, null, null, null, null, null
    			from balicky b
    			inner join tarifs_orders tor on b.id = tor.tarif_balicek_id
    			where tor.order_id = ".$order_id."
    			ORDER BY `user_id` desc";

    а филе нам кабанам, надо продукты из разных таблиц объединить в одну, ну и фто што таблицы разного размера. Зато когда добавляешь новый столбец в таблицу, нужно вручную считать сколько раз нужно добавить null :D

    Lameaux, 29 Ноября 2011

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

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

• 571
• …
• 303
• 302
• 301
• 300
• 299
• 298
• 297
• 296
• 295
• …
• 1