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

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• 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 / Говнокод #222

   +37.7

   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

   <div id="navLine">
           <? if(isset($title5)) { ?>
             <a href="index.php">Главная</a>&nbsp;&raquo; 
             <a href="index.php?page=<?=$link?>"><?=htmlspecialchars($title)?></a> &nbsp;&raquo;
             <a href="index.php?page=<?=$link2?>"><?=htmlspecialchars($title2)?></a> &nbsp;&raquo;
             <a href="index.php?page=<?=$link3?>#"><?=htmlspecialchars($title3)?></a> &nbsp;&raquo; 
             <a href="index.php?page=<?=$link4?>"><?=htmlspecialchars($title4)?></a>
           <? } elseif(isset($title4)) { ?>
             <a href="index.php">Главная</a>&nbsp;&raquo; 
             <a href="index.php?page=<?=$link?>"><?=htmlspecialchars($title)?></a>&nbsp;&raquo; 
             <a href="index.php?page=<?=$link2?>"><?=htmlspecialchars($title2)?></a>&nbsp;&raquo; 
             <a href="index.php?page=<?=$link3?>"><?=htmlspecialchars($title3)?></a>
           <? } elseif(isset($title3)) { ?>
             <a href="index.php">Главная</a>&nbsp;&raquo;
             <a href="index.php?page=<?=$link?>"><?=htmlspecialchars($title)?></a>&nbsp;&raquo; 
             <a href="index.php?page=<?=$link2?>"><?=htmlspecialchars($title2)?></a>
           <? } elseif(isset($title2)) { ?>
             <a href="index.php">Главная</a>&nbsp;&raquo; 
             <a href="index.php?page=<?=$link?>"><?=htmlspecialchars($title)?></a>
           <? } else { ?>
             <a href="index.php">Главная</a>
           <? } ?>
     </div>

   guest, 19 Декабря 2008

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

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

   +48.7

   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

   <?php
       global
           $_REQUEST,
           $_SERVER,
           $_FILES,
           $PHP_SELF,
           $_GET_VARS,$_GET,
           $_POST_VARS,  
           $_POST,
           $_COOKIE,
           $HTTP_ENV_VARS,
           $HTTP_SERVER_VARS,
           $_page,
           $_text,
           $_user,
           $_cook_session,
           $_page_index,
           $_page_title,
           $parce_page;
   
       extract($_GET);
       extract($_POST);
       extract($_FILES);
   
       while (list($key123, $value123) = each($_GET))    ${$key123}=$value123;
       while (list($key123, $value123) = each($_POST))   ${$key123}=$value123;
       while (list($key123, $value123) = each($_FILES))  ${$key123}=$value123;
   
       foreach ($_POST as $key=>$value) {
   	    ${$key}=$value;
   	}
       foreach ($_GET as $key=>$value) {
   	    ${$key}=$value;
   	}
   ?>

   Цитирую весь файл, как есть

   guest, 18 Декабря 2008

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

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

   +26

   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

   protected function _buildParamsUrl($additional = array()) {
          $url = array();
          $params = $this->_request->getParams();
          $params = array_merge($params);
          foreach ($params as $var => $val) {
              if (in_array($var,$this->_neededParams) && is_scalar($val)) {
                   $url[$var] = $val;
              }
          }
          
          $url = array_merge($url,$additional);
          
          return $url;
       }

   Было нужно подставлять параметры из POST в action формы. Почему осталось в таком виде - хз... Видимо было выпито мало кофе с утра...

   guest, 18 Декабря 2008

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

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

   +25.9

   1. 1
   2. 2
   3. 3

   $temp = $this->template($this->modulePath."/right_frame");
   eval("\$blockData = \"$temp\";");
   return $blockData;

   Реализация механизма подключения шаблонов в PHP %)

   guest, 17 Декабря 2008

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

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

   +24.9

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

   if(empty($real_name)) $real_name = $file_name; 		
   if(empty($real_name)) $real_name="fileid_".$id;
   
   .... 
   
   if($mode=="send") {
   	$head[] = array("link" => "customer.php?area=$area&mode=send", "title" => func_get_langvar_by_name("lbl_send_to_mail"), active=>"Y");
   }	else	{
   	$head[] = array("link" => "customer.php?area=$area&mode=send", "title" => func_get_langvar_by_name("lbl_send_to_mail"), active=>"n");
   }

   Разбирая один из скриптов на досуге, обнаружил, что некоторые программисты уж совсем с головой не дружат.
   
   1. Думаю, все тут ясно.
   2. Чувак явно не знает что такое логика... Мало того, что в одну строку ложится массив, так еще и ВЕЗДЕ строковые 'Y'/'N' использует вместо булевых труфолсов. И такого говна - тыщи 2 строк. Ы =)
   
   Не позавидую тем, кто будет дописывать / переделывать.
   ЗЫ: Скрипт - рип корпоративной системы.

   guest, 16 Декабря 2008

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

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

   +29.3

   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

   for ($i=0; $i < $this->acl_count; $i++)
   		{
   			$acl =& $this->acl[$i];
   			if (strcasecmp( $aco_section_value, $acl[0] ) == 0) {
   				if (strcasecmp( $aco_value, $acl[1] ) == 0) {
   					if (strcasecmp( $aro_section_value, $acl[2] ) == 0) {
   						if (strcasecmp( $aro_value, $acl[3] ) == 0) {
   							if ($axo_section_value && $acl[4]) {
   								if (strcasecmp( $axo_section_value, $acl[4] ) == 0) {
   									if (strcasecmp( $axo_value, $acl[5] ) == 0) {
   										$acl_result = @$acl[6] ? $acl[6] : 1;
   										break;
   									}
   								}
   							} else {
   								$acl_result = @$acl[6] ? $acl[6] : 1;
   								break;
   							}
   						}
   					}
   				}
   			}
   		}

   из ядра Joomla

   guest, 16 Декабря 2008

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

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

   +26.1

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

   $res = $DB->getData($query); //считывает данные с ДБ и загоняет в пронумерованный массив
   $data = new array();
   foreach ($res as $key=>$value)
   {
   if (!$data[$key])
   $data[$key] = $value;
   };

   Все это вместо простого $data = $res;
   
   http://ithappens.ru/ (c)

   guest, 16 Декабря 2008

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

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

   +22

   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

   ...
       /**
        * get a concrete filename for automagically created content
        *
        * @param string $auto_base
        * @param string $auto_source
        * @param string $auto_id
        * @return string
        * @staticvar string|null
        * @staticvar string|null
        */
       function _get_auto_filename($auto_base, $auto_source = null, $auto_id = null)
   
   ...

   Ухыхы )
   Я всегда знал, что Smarty жесток.
   И контент у него получается не иначе как autoMAGICally :-)
   
   Убицца веником...

   guest, 16 Декабря 2008

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

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

   +29.3

   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

   $key_tr = $_key-1;  //// Последний ключ массива (получаем для размещения не базового поля (Media file) на строку выше положенного)	
   	
   	$new = new stdClass;
   	$new->caption = "Media File";
   	$new->name = "media";
   	$new->table = "media";
   	$new->type = "file";
   	if ($role == "transcriber")
   		$new->read_only = "read_only";
   	$fields[$key_tr.'.5'] = $new;
   	
   	
   	
   	ksort($fields);

   "на строку выше положенного"
   
   Тема грязных хаков:
   В массив 1,2,3,4...N втыркиваем элемент 10.5 (шняга-с половиной) и сортируем по возрастанию ключа. (позже в коде идёт foreach этого массива)
   В проекте наблюдаются элементы развития данного хака ввиде +0.3, +0.4 и т.п.
   
   ВНИМАНИЕ: Не рекомендуем повторять этот трюк! Опасно для здоровья мозга!

   guest, 16 Декабря 2008

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

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

    +28

    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

    $sp_filter = array(0, '', '', '', '', '', '', 1, 1, 'desc', $id);
    	
    		$data = new vtSpotlight_Show($sp_filter); 
    
    
    		$table = $data->getTable();
    		$fields = $data->getFields();
     
    (...)
    
    		if(empty($table[0]['transcriber_concat']))
    		{
    			unset($table[0]['transcriber_concat']);
    			unset($fields[5]); ///  Удаление картикни T, если поле не содержит имени транскрибера
    		}
    	
    		$spotlight = $table[0];

    паттерн на паттерне одним словом...

    guest, 16 Декабря 2008

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

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

• 571
• …
• 12
• 11
• 10
• 9
• 8
• 7
• 6
• 5
• 4
• …
• 1