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

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

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

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

   +147

   1. 1

   date('d ', time()+3600*4);

   vdmodule, 28 Июня 2011

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

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

   +147

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

   private function _fileExists($file)
   {
       if(file_exists(self::FILE_PATH . $file)) {
           return true;
       }
       return false;
   }

   Нашел в одном из проектов.

   cartman, 28 Июня 2011

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

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

   +159

   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

   $get = dbcom('SELECT * FROM downloads ORDER BY views DESC;');
   			$i = 1;
   			while($row = mysql_fetch_array($get))
   			{
   				if($i == 1)
   				{
   					$p['v'] = $row['views'];
   				}
   				if($row['reports'] != 0)
   				{
   					$t['rep']++;
   				}
   				$t['dl'] += $row['views'];
   				$i = 0;
   			}

   DDLCMS is a COMMERCIAL grade content management system for DDL site owners.
   при 400к записях в downloads немножно все в ОЗУ не помещалось.

   peinguin, 28 Июня 2011

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

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

   +153

   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

   function read($fields = null, $id = null) {
   	$this->validationErrors = array();
   
   	if ($id != null) {
   		$this->id = $id;
   	}
   
   	$id = $this->id;
   
   	if (is_array($this->id)) {
   		$id = $this->id[0];
   	}
   
   	if ($id !== null && $id !== false) {
   		$this->data = $this->find(array($this->alias.'.'.$this->primaryKey => $id), $fields);
   		return $this->data;
   	} else {
   		return false;
   	}
   }

   И ещё cakePHP (самый-самый фреймворк, даже в говнокоде впереди планеты всей).

   zabuhailo, 28 Июня 2011

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

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

   +147

   1. 1
   2. 2
   3. 3

   if ($this->alias === null) {
   			$this->alias = (isset($alias) ? $alias : $this->name);
   		}

   Это cakePHP. Вот так вот в нём модель узнаёт свой alias.

   zabuhailo, 28 Июня 2011

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

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

   +168

   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

   function b_desc($value){
   	return ($value);
     }
   function a_id($value){
   	return $value;
   }
   function a_name($value){
   	return $value;
   }
     function b_name($value){
   	return ($value);
     }
     function b_alias($value){
   	return ($value);
     }
     function c_name($value){
   	return ($value);
     }
     function c_id($value){
   	return ($value);
     }  
     function c_level($value){
   	return ($value);
     }
     function g_id($value){
   	return ($value);
     }
     function g_name($value){
   	return ($value);
     }
      function g_desc($value){
   	return ($value);
     }
     function g_appointment($value){
   	return ($value);
     }
     function g_tech_desc($value){
   	return ($value);
     }

   Функции для отбработки значений, полученных из БД. Одно поле — одна функция. Выполняются все хором для каждой записи.

   Oonitaz, 27 Июня 2011

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

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

   +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
   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
   75. 75
   76. 76
   77. 77
   78. 78

   <?php   ////////////////////////////////
   	//Данная вставка будет обрабатывает переменные переданные post запросом;
   	//Передается с файла MenuProduction.php.
   	//и в соответсвтвии с переданной post переменной будет выставлять get переменную
   	//Page, для корректного перехода на нужную нам страницу_02.12.2009
   	/////////////////////////
   ?>
   
   <table align="center" id="Table_01" width="100%" border="0" cellpadding="0" cellspacing="0">
   	<tr>
   		<td colspan="5" width="961" height="12">
   		<div id="Layer1">
     <!--<table align="center" cellspacing="0">
   		<tr><td>  <?php include('FunctionCellOn.php'); 
         //Поясню, что делаем подключением данного php файла:
           //   В нем описана всего одна функция, которая сверяет переданный Get параметр
             // с номером страницы(см. далее).
             // Если значения равны, значит пользователь нажал на ячейку меню, следовательно,
              //необходимо обозначить, что она нажата. Поэтому далее во всех ячейках меню делаем
              //так: проверка, если совпадение есть, значит выводим заголовок ячейки меню
              //как обычный текст, иначе обозначаем его как ссылку. 
              //Минус данного подхода - получилась мешанина php-кода c html
              //Расшифровка страниц:
              //$_GET['Page'] =
               //1 - Главная
               //2 - Интерактивная карта
               //3 - Производство
               //4 - О компании
               //5 - Контакты
               
               //далее пошли не пункты меню
               //6 - Заявка на звонок
          ?>     
          		<?php   
   		 //На главную страницу будут дополнительные условия, т.к. надо
          	         //чтобы по умолчанию грузилась главная страница, то следовательно
          	         //по умолчанию надо и отобразить ячейку меню 'главная', как 'нажато 
   		 //Для этого помимо стандартной проверке на нажатие, надо еще
   		 //проверить вот что: если ни одни параметр методом Get не передан, то
   		 //соответственно грузится просто index.php, где по умолчанию
   		 //в область вывода загружаем файли соответствующий пункты меню 'главная'
   		 //Следовательно ссылку 'главная' делаем не активной    
           	 if ((CellOnCheck($_GET['Page'],1) == true) || 
   		 !isset($_GET["Page"]) && !isset($_GET["id_news"]) &&
   	         !isset($_GET["archives"]) && !isset($_GET["page"])) {
            	 echo "<Span class = \"MenuCellOn\">Главная</Span>"; 
   		 }
   		 else echo "<a href=\"http://amalker.su\">Главная</a>";
   	
         		 ?>
         </td>
   	  <td>
   	  <?php 
           	 if (CellOnCheck($_GET['Page'],4) == true) {
           	  echo "  |  <Span class = \"MenuCellOn\">О компании </Span>"; 
   		 }
   		 else echo "  |  <a href=\"index.php?Page=4\">О компании </a>";
          		?>
                  </td>
   	   <td>  |  <a href="map/map_client.php" target="_blank">Карта конструкций<img src="new.png" width="6" height="12" border="0"></a> 
         
            	      </td>
   	  <td><?php 
           	 if (CellOnCheck($_GET['Page'],3) == true) {
            	 echo "  |  <Span class = \"MenuCellOn\">Производство  </Span>"; 
   		 }
   		 else echo "  |  <a href=\"index.php?Page=3\">Производство  </a>";
          	 	?>
           	      </td>
   	   <td>
           	<?php 
           	 if (CellOnCheck($_GET['Page'],5) == true) {
            	 echo "  |  <Span class = \"MenuCellOn\">Контактная информация</Span>"; 
   	 	 }
   		 else echo "  |  <a href=\"index.php?Page=5\">Контактная информация</a>";
          		?></td>
       </tr>
   </table>--></div>

   Ужас и только.

   nethak, 26 Июня 2011

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

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

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

   $url_view = 'pages/news';// страница вида
   			$data = array(); //Создаем массив дата
   			// сегменты урл
   			$data['segment_1'] = $filter_1;
   			$data['segment_2'] = $filter_2; 
   			$limit = 10; // вывод новостей
   			$num_links = 4;  // кол-во страниц во круг текущей
   			$uri_segment = 5; // сегмент ссылки
   			$offset = ($page - 1) * $limit; // переводим page в offset
   			if(!is_numeric($filter_2)) {$filter_2 = iconv('utf-8', 'windows-1251', urldecode($filter_2));}	
   				
   			$base_url = '/page/filter/'.$filter_1.'/'.$filter_2.'/'; 
   			
   			// разбираем 1 фильтр для проверки
   			$filter_global = explode("_",$filter_1);
   			if($filter_global[0] == 'user') {$filter_1 = 'user'; $user_id = (int)$filter_global[1];}
   		
   			// описание каждого элемента фильтра
   			switch($filter_1) {
   				
   				case 'date':
   					$total = $this->paginations->count_posts_filter('news',$filter_1,$filter_2);
   					$data['materials'] = $this->paginations->list_posts_filter($limit, $offset, 'news', $filter_1, $filter_2);
   				break;
   				
   				case 'date_day':
   					$total = $this->paginations->count_posts_filter('news',$filter_1,$filter_2);
   					$data['materials'] = $this->paginations->list_posts_filter($limit, $offset, 'news', $filter_1, $filter_2);
   				break;
   				
   				case 'date_month':
   					$total = $this->paginations->count_posts_filter('news',$filter_1,$filter_2);
   					$data['materials'] = $this->paginations->list_posts_filter($limit, $offset, 'news', $filter_1, $filter_2);	
   				break;
   				
   				case 'date_year':
   					$total = $this->paginations->count_posts_filter('news',$filter_1,$filter_2);
   					$data['materials'] = $this->paginations->list_posts_filter($limit, $offset, 'news', $filter_1, $filter_2);
   				break;
   				
   				case 'rating':
   					$total = $this->paginations->count_posts_filter('news',$filter_1,$filter_2);
   					$data['materials'] = $this->paginations->list_posts_filter($limit, $offset, 'news', $filter_1, $filter_2);
   				break;
   				
   				case 'random':
   					$total = $this->paginations->count_posts_filter('news',$filter_1,$filter_2);
   					$data['materials'] = $this->paginations->list_posts_filter($limit, $offset, 'news', $filter_1, $filter_2);
   				break;
   				
   				case 'moderation':
   					$total = $this->paginations->count_posts_filter('news',$filter_1,$filter_2);
   					$data['materials'] = $this->paginations->list_posts_filter($limit, $offset, 'news', $filter_1, $filter_2);
   				break;
   				
   				case 'user':
   					$filter_1 = $filter_global[0].'_'.$filter_global[1]; // собираем обратно
   					$total = $this->paginations->count_posts_filter('news',$filter_1,$filter_2);
   					$data['materials'] = $this->paginations->list_posts_filter($limit, $offset, 'news', $filter_1, $filter_2);
   				break;
   				
   				default:
   					$total = $this->paginations->count_posts_filter('news',$filter_1,$filter_2);
   					$data['materials'] = $this->paginations->list_posts_filter($limit, $offset, 'news', $filter_1, $filter_2);
   				break;
   				
   			}
   
   		$this->auxiliary->pagination($base_url,$uri_segment,$num_links,$total,$limit);//настройки пагинации
   		$this->auxiliary->general_view($data,'title_best','description_best','keywords_best',$url_view); // общий вид

   проверьте на наличие гавно кода .
   
   Контроллер фильтра выдачи в Codeigniter

   nepster, 25 Июня 2011

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

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

   +147

   1. 1

   dirname(__FILE__)

   peinguin, 25 Июня 2011

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

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

    +147

    1. 1
    2. 2

    if(!empty(_SESSION['order']['contact']['user_id']))
        $user_id = preg_replace('/\D|\s/', '', $_SESSION['order']['contact']['user_id']);

    Радует знание регулярных выражений =)

    mitallast, 24 Июня 2011

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

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

• 571
• …
• 252
• 251
• 250
• 249
• 248
• 247
• 246
• 245
• 244
• …
• 1