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

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

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

1. PHP (5703)
2. C++ (2740)
3. Куча (2337)
4. JavaScript (2030)
5. C# (1929)
6. Java (1503)
7. Си (1103)
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 (32)
21. Swift (29)
22. Haskell (28)
23. Kotlin (15)

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

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

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

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

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

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

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

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

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

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

    +139.1

    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

    if (isset($_GET["ADD"]))                        {$ADD=$_GET["ADD"];}
    /* вырезано http://govnokod.ru/610 */
    
    if (!isset($ADD))   {$ADD=0;}
    
    if ($ADD=="1")                  {$hh='users';           echo "Add New User";}
    if ($ADD=="1A")                 {$hh='users';           echo "Copy User";}
    if ($ADD==11)                   {$hh='campaigns';       $sh='basic';    echo "Add New Campaign";}
    if ($ADD==12)                   {$hh='campaigns';       $sh='basic';    echo "Copy Campaign";}
    if ($ADD==111)                  {$hh='lists';           echo "Add New List";}
    if ($ADD==121)                  {$hh='lists';           echo "Add New DNC";}
    if ($ADD==1111)                 {$hh='ingroups';        echo "Add New In-Group";}
    if ($ADD==1211)                 {$hh='ingroups';        echo "Copy In-Group";}
    if ($ADD==11111)                {$hh='remoteagent';     echo "Add New Remote Agents";}
    if ($ADD==111111)               {$hh='usergroups';      echo "Add New Users Group";}
    if ($ADD==1111111)              {$hh='scripts';         echo "Add New Script";}
    /* вырезано 200 строк аналогичного содержания */
    
    /* теперь внимание */
    
    if ( ($ADD>9) && ($ADD < 99998) )
            {
            ##### get scripts listing for dynamic pulldown
            $stmt="SELECT script_id,script_name from vicidial_scripts order by script_id";
            $rslt=mysql_query($stmt, $link);
            /* censored ~ 400 lines  */
    }
    if ( ( (strlen($ADD)>4) && ($ADD < 99998) ) or ($ADD==3) or (($ADD>20) and ($ADD<70)) or ($ADD=="4A")  or ($ADD=="4B") or (strlen($ADD)==12) )
            {
            ##### get server listing for dynamic pulldown
            $stmt="SELECT server_ip,server_description from servers order by server_ip";
            /* censored  ~ 1000 lines */
    }
    ######################
    # ADD=99999 display the HELP SCREENS
    ######################
    
    if ($ADD==99999)
    {
    echo "</title>\n";
    echo "</head>\n";
    echo "<BODY BGCOLOR=white marginheight=0 marginwidth=0 leftmargin=0 topmargin=0>\n";
    echo "<CENTER>\n";
    echo "<TABLE WIDTH=98% BGCOLOR=#E6E6E6 cellpadding=2 cellspacing=0><TR><TD ALIGN=LEFT><FONT FACE=\"ARIAL,HELVETICA\" COLOR=BLACK SIZE=4><B>VICIDIAL ADMIN: HELP<BR></B></FONT><FONT FACE=\"ARIAL,HELVETICA\" COLOR=BLACK SIZE=2><BR><BR>\n";
    
    
    ?>
    <B><FONT SIZE=3>VICIDIAL_USERS TABLE</FONT></B><BR><BR>
     <!-- 22Kb help screen here -->

    vicidial. Новый революционный паттерн. МВЦ это прошлый век.
    Читаем каменты, принцип будет понятен.

    guest, 25 Февраля 2009

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

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

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