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

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

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

1. PHP (5703)
2. C++ (2739)
3. Куча (2337)
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 / Говнокод #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)

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

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

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

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

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

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

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

   +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
   24. 24
   25. 25
   26. 26
   27. 27
   28. 28
   29. 29
   30. 30
   31. 31

   // Записываем просмотр профайла
   MySQL::QueryInsert(
   	'profile_views', 
   	array(
   		'profile_id' => (int)$user['id'], 
   		'visitor' => (int)User::$current['id'], 
   		'date' => (int)$cur_time, 
   		'hidden' => (int)User::$current['hidden']
   		)
   	);
   // Смотрим последнюю запись в просмотрах
   $prev_visit = MySQL::FetchAll(
   	MySQL::Query(
   		'SELECT `visitor`,`date` 
   		   FROM profile_views 
   		  WHERE profile_id='.(int)$user['id'].' AND `date`<'.(int)$cur_time.
             ' AND `hidden`='.(int)User::$current['hidden'].' 
          ORDER BY `date` DESC 
             LIMIT 1'
           )
       );
   // если юзер сам посмотрел свой профиль - удаляем только что вставленную запись
   if ($prev_visit[0]['visitor'] == User::$current['id']) {
       MySQL::Query(
   	    'DELETE FROM profile_views 
   	      WHERE profile_id='.(int)$user['id'].' 
   	        AND `date`='.(int)$prev_visit[0]['date'].
             ' AND `hidden`='.(int)User::$current['hidden'].' 
               AND visitor='.(int)User::$current['id']
       );
   };

   Комментарии мои. Сука, я потихоньку начинаю ненавидеть прибалтийских программеров

   Sc0rp10, 28 Ноября 2011

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

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

   +147

   1. 1

   $field_value = trim( $field_value, ' \n"' );

   odmin, 28 Ноября 2011

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

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

    +165

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

    @jfredys 23-Mar-2011 01:07 
     I was looking for trimming all the elements in an array, I found this as the simplest solution: 
    
    <?php 
     array_walk($ids, create_function('&$val', '$val = trim($val);')); 
    ?>

    array_map? не, не слышал.

    DrFreez, 27 Ноября 2011

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

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

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