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

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

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

1. PHP (5704)
2. C++ (2739)
3. Куча (2355)
4. JavaScript (2029)
5. C# (1928)
6. Java (1502)
7. Си (1113)
8. Pascal (647)
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 / Говнокод #631

   +165.5

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

   foreach($data as $k=>$v)
   {
      if(!isset($data[$k]))
      {
         $data[$k] = $v;
      }
   }

   guest, 28 Февраля 2009

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

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

   +143.5

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

   class upload {
   //...
   
       function upload($file = '', $destination = '', $permissions = '777', $extensions = '') {
   //...
           if ( ($this->parse() == true) && ($this->save() == true) ) {
             return true;
           } else {
             return false;
           }
         }
       }

   Интересно для кого конструктор возвращает результат...
   Oscommerce

   guest, 27 Февраля 2009

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

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

   +152

   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

   public function step($step = 1)
   	{
   	
   		switch ( $step )
   		{
   			case 1 :
   				return $this->step1();
   				break;
   			/*
   	         case 2 :
   	           return $this->step2();
   	         break;
   
   	         case 3 :
   	           return $this->step3();
   	         break;
   
   	         case 4 :
   	           return $this->step4();
   	         break;
   
   	         case 11 :
   	         	return $this->step11();
   	         break;
                    */
   			default :
   				return $this->step1();
   				break;
   		
   		}
   	}

   ^%)

   guest, 27 Февраля 2009

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

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

   +137.7

   1. 1
   2. 2
   3. 3
   4. 4

   foreach ($_pp as $k => $v){
               if ($v['qty'] == $_pp[$i*count($groups)]['qty']){
                   $qtys[$i][]=$v;
               }

   За такое надо кромсать

   guest, 27 Февраля 2009

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

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

   +144.1

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

   ... and p.products_group_access like '%,". $customer_group_id.",%' 
   OR
     p.products_group_access like '%,". $customer_group_id."' 
   OR
     p.products_group_access like '". $customer_group_id.",%' 
   OR
     p.products_group_access = '". $customer_group_id."')";

   REGEXP уже не рулят

   guest, 26 Февраля 2009

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

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

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

   function query_safe()
   	{
   		$time_before = $this->get_real_time();
   
   		if(!$this->connected) $this->connect(DBUSER, DBPASS, DBNAME, DBHOST);
   		
   		$args = func_get_args();
           $tmpl =& $args[0];
           $tmpl = str_replace("%", "%%", $tmpl);
           $tmpl = str_replace("?", "%s", $tmpl);
           foreach ($args as $i=>$v) {
               if(!$i) continue;
               if(is_int($v)) continue;
               $args[$i] = "'".mysql_escape_string($v)."'";
           }
           for($i=$c=count($args)-1; $i<$c+20; $i++)
               $args[$i+1] = "UNKNOWN_SQL_COMMAND_$i";
           
           $sql = call_user_func_array("sprintf", $args);
           //echo $sql;
           exit;
           
   		if(!($this->query_id = mysql_query($sql, $this->db_id) )) {
   
   			$this->mysql_error = mysql_error();
   			$this->mysql_error_num = mysql_errno();
   			if($show_error) {
   				$this->display_error($this->mysql_error, $this->mysql_error_num, $query);
   			}
   		}
   		
   		$this->MySQL_time_taken += $this->get_real_time() - $time_before;
   		$this->query_num ++;
   		return $this->query_id;
   	}

   Что же тут творится?? :)))

   guest, 26 Февраля 2009

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

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

   +151.5

   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

   function custom_print ($custom_category, $custom_template, $aviable, $custom_from, $custom_limit, $custom_cache, $do){
     global $db, $is_logged, $member_id, $xf_inited, $cat_info, $config, $user_group, $category_id, $_TIME, $lang;
   
   
     $do = $do ? $do : "main";
     $aviable = explode ('|', $aviable);
   
     if(!(in_array($do, $aviable)) AND ($aviable[0] != "global")) return "";
   
     $custom_category  = $db->safesql(str_replace(',', '|', $custom_category));
     $custom_from = intval($custom_from);
     $custom_limit = intval($custom_limit);
     $thisdate = date ("Y-m-d H:i:s", (time()+ $config['date_adjust']*60));
   
     if (intval($config['no_date'])) $where_date = " AND date < '".$thisdate."'"; else $where_date = "";
   
     $tpl = new dle_template;
     $tpl->dir = TEMPLATE_DIR;
   
     //if ($custom_cache == "yes") $config['allow_cache'] = "yes"; else $config['allow_cache'] = false;
     if ($is_logged AND ($user_group[$member_id['user_group']]['allow_edit'] AND !$user_group[$member_id['user_group']]['allow_all_edit'])) $config['allow_cache'] = false;
   
     $content = dle_cache("custom", "cat_".$custom_category."template_".$custom_template."from_".$custom_from."limit_".$custom_limit, true);
   
     if ($content) { return $content; }
     else {
   
     $allow_list = explode (',', $user_group[$member_id['user_group']]['allow_cats']);
   
     if ($allow_list[0] != "all") {
   
       if ($config['allow_multi_category']) {
   
         $stop_list = "category regexp '[[:<:]](".implode ('|', $allow_list).")[[:>:]]' AND ";
   
       } else {
   
         $stop_list = "category IN ('".implode ("','", $allow_list)."') AND ";
   
       }
   
     } else $stop_list = "";
   
     if ($user_group[$member_id['user_group']]['allow_short']) $stop_list = "";
   
     if ($cat_info[$custom_category]['news_sort'] != "") $config['news_sort'] = $cat_info[$custom_category]['news_sort'];
     if ($cat_info[$custom_category]['news_msort'] != "") $config['news_msort'] = $cat_info[$custom_category]['news_msort'];
   
       if ($config['allow_multi_category']) {
   
         $where_category = "category regexp '[[:<:]](".$custom_category.")[[:>:]]'";
   
       } else {
   
         $custom_category = str_replace ("|", "','", $custom_category);
         $where_category = "category IN ('".$custom_category."')";
   
       }
   
       $sql_select = "SELECT " . PREFIX . "_post.id, gallery, autor, date," . PREFIX . "_post.image," . PREFIX . "_post.imgtype, short_story, full_story, " . PREFIX . "_post.xfields, title, category, alt_name, " . PREFIX . "_post.comm_num, " . PREFIX . "_post.allow_comm, allow_rate, " . PREFIX . "_post.rating, " . PREFIX . "_post.vote_num, news_read, " . PREFIX . "_post.flag, " . PREFIX . "_users.fullname FROM " . PREFIX . "_post , " . PREFIX . "_users WHERE " . PREFIX . "_post.autor=" . PREFIX . "_users.name and ".$stop_list.$where_category." AND approve = '1'".$where_date." ORDER BY ".$config['news_sort']." ".$config['news_msort']." LIMIT ".$custom_from.",".$custom_limit;
   //echo $sql_select;
     include (ENGINE_DIR.'/modules/show.custom.php');
   
       if ($config['files_allow'] == "yes")
         if ( strpos( $tpl->result['content'], "[attachment=" ) !== false)
         {
           $tpl->result['content'] = show_attach($tpl->result['content'], $attachments);
         }
   
       create_cache("custom", $tpl->result['content'], "cat_".$custom_category."template_".$custom_template."from_".$custom_from."limit_".$custom_limit, true);
   
     }
     return $tpl->result['content'];
   }

   Не пишите так функции, пожалейте того, кто будет ее поддерживать после вас.

   guest, 26 Февраля 2009

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

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

   +158.5

   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

   switch($cms->type)
     {
         case 1:{
                } break;
         case 2:{
                } break;
         case 4:{
                    $cms->assign('content', array('title' => $cms->title, 'meta' => $cms->meta, 'mt' => $cms->mt));
                } break;
         case 6:{
                    $cms->assign('content', array('title' => $cms->title, 'meta' => $cms->meta, 'mt' => $cms->mt));
                } break;
     }

   Брутальное использование switch и дублирования кода :D

   guest, 26 Февраля 2009

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

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

   +146

   1. 1
   2. 2

   <table width="100%" cellspacing="0" cellpadding="0" border="0">
   <tr height="100" width="100%">

   Вот чтоб уже наверняка...

   guest, 26 Февраля 2009

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

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

    +142.5

    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

    switch ($search_keywords[$i]) {
          				case '(':
          				case ')':
          				case 'and':
          				case 'or':
          					
                    $Search->Where($search_keywords[$i]);
          					
          					break;
          				default:
          					$keyword = tep_db_prepare_input($search_keywords[$i]);
          					
                    $Search->Where("(pd.products_name LIKE '%" . tep_db_input($keyword) . "%' OR p.products_model LIKE '%" . tep_db_input($keyword) . "%' OR m.manufacturers_name LIKE '%" . tep_db_input($keyword) . "%'");               
                    if  ( isset($get->search_in_description) && ($get->search_in_description == 1) ) {
          					      $Search->Where("OR pd.products_description LIKE '%" . tep_db_input($keyword) . "%'");
          					}
          					
          					$Search->Where(")");					
          					break;
        			}

    Князь логики

    guest, 25 Февраля 2009

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

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

• 571
• …
• 31
• 30
• 29
• 28
• 27
• 26
• 25
• 24
• 23
• …
• 1