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

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

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

1. PHP (5706)
2. C++ (2739)
3. Куча (2363)
4. JavaScript (2030)
5. C# (1928)
6. Java (1502)
7. Си (1116)
8. Pascal (648)
9. Python (584)
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 / Говнокод #9793

   +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

   switch (count($params)) {
   	case 0:
   		return $this->{$method}();
   	case 1:
   		return $this->{$method}($params[0]);
   	case 2:
   		return $this->{$method}($params[0], $params[1]);
   	case 3:
   		return $this->{$method}($params[0], $params[1], $params[2]);
   	case 4:
   		return $this->{$method}($params[0], $params[1], $params[2], $params[3]);
   	case 5:
   		return $this->{$method}($params[0], $params[1], $params[2], $params[3], $params[4]);
   	default:
   		return call_user_func_array(array(&$this, $method), $params);
   	break;
   }

   cakephp framework v 1.3

   57uff3r, 28 Марта 2012

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

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

   +154

   1. 1

   if (!empty($params[$fieldName.'_need_group']) ? true : false) $errors[$fieldName] = 2;

   Просто очень тщательная проверка, если true то true, а если false то false....

   nikll, 28 Марта 2012

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

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

   +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
   36. 36
   37. 37
   38. 38
   39. 39
   40. 40
   41. 41
   42. 42
   43. 43
   44. 44
   45. 45

   if($type == 1){
   	if(!is_nan($predpolag_srok)){
   		if($predpolag_srok<=7 AND $predpolag_srok>0 ){ $real_srok =6; }
   		elseif($predpolag_srok<=10 AND $predpolag_srok>7){ $real_srok =9; }
   		elseif($predpolag_srok<=18 AND $predpolag_srok>10){ $real_srok =12; }
   		elseif($predpolag_srok<=30 AND $predpolag_srok>18){ $real_srok =24; }
   		elseif($predpolag_srok<=36 AND $predpolag_srok>30){ $real_srok =36; }
   		else{$real_srok =36;}
   	}else{
   		$real_srok =36;
   	}
   	}elseif($type == 2){
   		if(!is_nan($predpolag_srok)){
   		if( $predpolag_srok<=18 AND $predpolag_srok>0 ){ $real_srok =12; }
   		elseif($predpolag_srok<=30 AND $predpolag_srok>18){ $real_srok =24; }
   		elseif($predpolag_srok<=36 AND $predpolag_srok>30){ $real_srok =36; }
   		elseif($predpolag_srok<=72 AND $predpolag_srok>36){ $real_srok =60; }
   		elseif($predpolag_srok>72){ $real_srok =84; }
   		else{$real_srok =60;}
   		}else{
   		$real_srok =60;
   	}
   	}elseif($type == 7){
   		if(!is_nan($predpolag_srok)){
   		if( $predpolag_srok<=18 AND $predpolag_srok>0 ){ $real_srok =12; }
   		elseif($predpolag_srok<=36 AND $predpolag_srok>18){ $real_srok =36; }
   		elseif($predpolag_srok<=72 AND $predpolag_srok>36){ $real_srok =60; }
   		elseif($predpolag_srok>72){ $real_srok =84; }
   		else{$real_srok =84;}
   		}else{
   		$real_srok =84;
   	}
   	}elseif($type == 5){
   	//echo $predpolag_srok;
   		if(!is_nan($predpolag_srok)){
   		if( $predpolag_srok<=18 AND $predpolag_srok>0 ){ $real_srok =12; }
   		elseif($predpolag_srok<=36 AND $predpolag_srok>18){ $real_srok =36; }
   		elseif($predpolag_srok<=90 AND $predpolag_srok>36){ $real_srok =60; }
   		elseif($predpolag_srok<=120 AND $predpolag_srok>90){ $real_srok =90; }
   		elseif($predpolag_srok>90){ $real_srok =120; }
   		else{$real_srok =120;}
   		}else{
   		$real_srok =120;
   	}
   	}

   вот так весело определяется срок кредита в одной интересной системе

   nikll, 28 Марта 2012

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

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

   +151

   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

   $query=mysql_query("SELECT `harak`
   FROM `contacts`
   WHERE db_name=`MSSQL2008R2`");
   $row=mysql_fetch_array($query);
   echo "$row['harak']";
   $a[0]=array($sybd=$_POST['sybd']); 
   $a[1]=array($sybd1=$_POST['sybd1']);
   $a[2]=array($sybd2=$_POST['sybd2']);
   $a[3]=array($sybd3=$_POST['sybd3']);
   $a[4]=array($sybd4=$_POST['sybd4']);
   $a[5]=array($harak=$_POST['harak']);
   $a[6]=array($harak1=$_POST['harak1']);
   $a[7]=array($harak2=$_POST['harak2']);
   $a[8]=array($harak3=$_POST['harak3']);
   $a[9]=array($harak4=$_POST['harak4']);
   $a[10]=array($harak5=$_POST['harak5']);
   $a[11]=array($harak6=$_POST['harak6']);
   $a[12]=array($harak7=$_POST['harak7']);
   $a[13]=array($harak8=$_POST['harak8']);
   $a[14]=array($harak9=$_POST['harak9']);
   if (isset ($_POST['sybd']) or isset ($_POST['sybd1']) or isset ($_POST['sybd2'])or isset ($_POST['sybd3']) or isset ($_POST['sybd4'])
   or isset ($_POST['harak']) or isset ($_POST['harak1']) or isset ($_POST['harak2']) or isset ($_POST['harak3']) or isset ($_POST['harak4']) or isset ($_POST['harak5'])
   or isset ($_POST['harak6']) or isset ($_POST['harak7']) or isset ($_POST['harak8']) or isset ($_POST['harak9'])) 
   {
   echo "<table border=1>";
   echo"<tr><td></td>";
   foreach ($a[0] as $sybd) {
   echo "<p><td>".$sybd."</td></p>";
   }
   
   foreach ($a[1] as $sybd) {
   echo "<p><td>".$sybd1."</td></p>";
   }
   foreach ($a[2] as $sybd) {
   echo "<p><td>".$sybd2."</td></p>";
   }
   foreach ($a[3] as $sybd) {
   echo "<p><td>".$sybd3."</td></p>";
   }
   foreach ($a[4] as $sybd) {
   echo "<p><td>".$sybd4."</td></tr></p>";
   }
   foreach ($a[5] as $sybd) {
   echo "<p><tr><td>".$harak."</td></p>";
   echo "<p><td>nnn</td></tr></p>";
   }
   
   foreach ($a[6] as $sybd) {
   echo "<p><tr><td>".$harak1."</td></p>";
   echo "<p><td>ffjfj</td></tr></p>";
   }
   foreach ($a[7] as $sybd) {
   echo "<p><tr><td>".$harak2."</tr></td></p>";
   }
   foreach ($a[8] as $sybd) {
   echo "<p><tr><td>".$harak3."</tr></td></p>";
   }
   foreach ($a[9] as $sybd) {
   echo "<p><tr><td>".$harak4."</tr></td></p>";
   }
   foreach ($a[9] as $sybd) {
   echo "<p><tr><td>".$harak5."</tr></td></p>";
   }
   foreach ($a[10] as $sybd) {
   echo "<p><tr><td>".$harak6."</tr></td></p>";
   }
   foreach ($a[11] as $sybd) {
   echo "<p><tr><td>".$harak7."</tr></td></p>";
   }

   Однокурсница с колхоза делает дипломную работу, попросила помочь кинув кусок кода >_<

   frant, 27 Марта 2012

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

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

   +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

   /* добавленный код: */
   public function preDispatch() {
      $id = (int) $this->_request->id;
      if ($id > 0) {
           $this->_item = doctrine("Item")->findOneById($id);
           if (!is_object($this->_item)) {
               throw new PageException_NotFound;
           }
       }
   }
   
   /* выброшенный код: */
   public function indexAction() {
       $itemId = (int) $this->_request->id;
       $item = doctrine("item")->findOneBy(array("id" => $itemId, "owner" => $this->_owner->getId()));
       if (is_object($item)) {
       /* ... */
       }
   }

   Вот такой рефакторинг. Теперь кто хочет - может редактировать все, что не лень подставить в запрос.

   Wivern, 27 Марта 2012

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

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

   +156

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

   public function session_set_lifetime($time){
     /*NOT USED with cookies, can be used with sessions*/
     return null;
   }

   тотже мастер класа что и тут:
   http://govnokod.ru/9765/
   http://govnokod.ru/9777/

   fafik91, 27 Марта 2012

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

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

   +160

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

   public function return_array_element($array, $element){
     /*Return the value of specific array element*/
     return $array[$element];
   }

   тотже мастер класа что и тут:
   http://govnokod.ru/9765/

   fafik91, 27 Марта 2012

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

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

   +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

   function get_domain ($url)
   {
   	preg_match("/^(http:\/\/)?([^\/]+)/i",
       $url, $matches);
   	$host = $matches[2];
   	 
   	// get last two segments of host name
   	preg_match("/[^\.\/]+\.[^\.\/]+$/", $host, $matches);
   	return $matches[0];
   }
   function get_host ($url)
   {
   	preg_match("/^(http:\/\/)?([^\/]+\.[^\/]+)\/?/i",
       $url, $matches);
   	$host = $matches[2];
   	return $host;
   }

   я ошибаюсь, или всё это можно сделать с помощью parse_url?

   roman-kashitsyn, 27 Марта 2012

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

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

   +159

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

   ...
    function count_answers($answer_array){
     /*Count the array*/
     $count_ressult = 0;
     foreach(@$realarray as $key=>$value)
     {
       $count_ressult++;
     }
     return $count_ressult;
   }
   ...

   count($array)? не, не слышал!

   fafik91, 26 Марта 2012

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

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

    +148

    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

    ###########################################################################
    //из родительских метакурсов моего субкурса вывести те дисциплины
    //у которых есть задание
    function get_my_task_ch($subcourse_id) {
    $get_my_metacourse_ch_result = mysql_query("
    SELECT mdl_course.id, mdl_course.fullname
    FROM mdl_course, mdl_course_meta
    WHERE mdl_course.id = mdl_course_meta.parent_course
    AND mdl_course_meta.child_course = $subcourse_id
    ");  //соединить  с табличками в которых задания
    if(mysql_num_rows($get_my_metacourse_ch_result) > 0){
    		echo "<table border='1' bordercolor='#E1E1E1' style='get_my_task_ch'>";
    		echo iconv("windows-1251", "UTF-8", "<tr><th>Дисциплина</th><th>Задание</th></tr>"); 
    		 
    	$get_my_metacourse_ch_row = mysql_fetch_array($get_my_metacourse_ch_result);
    		do{
    
    			//дальше смотрим упоминаеться ли у нас этот курс в табличке mdl_course_modeules
    			$c_m_result = mysql_query("
    			SELECT mdl_course_modules.id, mdl_course_modules.module, mdl_modules.name FROM mdl_course_modules, mdl_modules
    			WHERE mdl_course_modules.course = $get_my_metacourse_ch_row[id]
    			AND mdl_course_modules.module = mdl_modules.id");
    			if(mysql_num_rows($c_m_result) > 0) {
    				$c_m_row = mysql_fetch_array($c_m_result);
    				echo "<tr><td><a href ='course/view.php?id=$get_my_metacourse_ch_row[id]'>$get_my_metacourse_ch_row[fullname]</a></td><td>";
    				do {
    					//дальше смотрим в табличку самого материала и забираем от туда необходимыйе поля
    					$task_table = mdl_.$c_m_row['name'];
    					$task_course = $get_my_metacourse_ch_row['id'];
    					$task_link = "http://localhost/moodle/moodle/mod/".$c_m_row['name']."/view.php?id=".$c_m_row['id']; //идишник берем из табличкм курс-модуль
    					$task_result = mysql_query("SELECT * FROM $task_table WHERE course = $task_course ");
    					if($task_num = mysql_num_rows($task_result) > 0) {
    					$task_row = mysql_fetch_array($task_result);
    					echo "<a href = '{$task_link}'>{$task_row[name]} ({$c_m_row['name']})</a><br/>\n";
    					
    					}
    					 
    				}
    				while($c_m_row = mysql_fetch_array($c_m_result));
    				echo"</td></tr>";
    			}
    		}
    		while($get_my_metacourse_ch_row = mysql_fetch_array($get_my_metacourse_ch_result));
    		echo "</table>";
    	}
    
    }

    ацкий писдец

    mailparen, 26 Марта 2012

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

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

• 571
• …
• 334
• 333
• 332
• 331
• 330
• 329
• 328
• 327
• 326
• …
• 1