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

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

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

1. PHP (5704)
2. C++ (2738)
3. Куча (2348)
4. JavaScript (2028)
5. C# (1928)
6. Java (1502)
7. Си (1111)
8. Pascal (647)
9. Python (583)
10. 1C (536)
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 (30)
21. Swift (27)
22. Haskell (26)
23. Kotlin (14)

1. PHP / Говнокод #12625

   +155

   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

   // сначала читаем из настроек unix-timestampы дней, в которые начинаются кварталы
   // если же не получилось прочитать настройки, берем то, что сгенерим с помощью mktime
   // правда, в настройках значения от балды, и в mktime делаются не те даты, поэтому некоторые месяцы считаем не в свой квартал
   // но кого это волнует?
     $planning_current = Array(
                                "k1" => COption::GetOptionString("main", "req_planning_k1", mktime(0,0,0,intval(1),1, $_YEAR)),
                                "k2" => COption::GetOptionString("main", "req_planning_k2", mktime(0,0,0,intval(3),1, $_YEAR)),
                                "k3" => COption::GetOptionString("main", "req_planning_k3", mktime(0,0,0,intval(6),1, $_YEAR)),
                                "k4" => COption::GetOptionString("main", "req_planning_k4", mktime(0,0,0,intval(9),1, $_YEAR)),
                              ); 
   
   // массивы с порядковыми номерами месяцев квартала
     $first_kvartal  = array("01","02","03");
     $second_kvartal = array("04","05","06");
     $third_kvartal  = array("07","08","09");
     $fourth_kvartal = array("10","11","12");
   
   ... // здесь html с шапкой таблицы и началом первой строки
     
   foreach($first_kvartal as $key){ // $key нигде не используется
               $c_date = time();
               $c_cyc  = mktime(0,0,0, date('m',$planning_current['k1']), date('d',$planning_current['k1']), $_YEAR); // год сменить хочу
               
               if($c_cyc < $c_date) { // если квартал уже начался
                 ... // здесь html с ячейкой, в которой заявка текущего месяца показана одним способом
               } else {
                 ... // здесь html с ячейкой, в которой заявка текущего месяца показана другим способом
   			}
   }
   
   ... // здесь html, завершающий первую строку и начинающий вторую
   
   ... // и еще три абсолютно таких же цикла - для второго, третьего и четвертого кварталов

   Задача: вывести в таблицу ежемесячные заявки для года $_YEAR, причем каждый квартал в отдельной строке

   clauclauclau, 21 Февраля 2013

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

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

   +154

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

   if(!empty($_POST['action']) && !empty($_POST['product_id']))
   {
   	$product_id=$_POST['product_id']+0;
           etc...
   }

   Наткнулся вот!)

   Impossible, 21 Февраля 2013

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

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

   +155

   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
   79. 79
   80. 80
   81. 81
   82. 82
   83. 83
   84. 84
   85. 85
   86. 86
   87. 87

   function poisk_potomok_item ($id_otca, $glubina, $i_array)
                 {
                  global /*$style,*/ $kol_n, $sm, $fire;
   
                  if ($glubina==$max_vlog) return 1;
   
                  switch ($glubina)
                       {
                        case "1":
                         {
                          $base_style="menu2_level3";
                          break;
                         }
                        case "2":
                         {
                          $base_style="menu2_level4";
                          break;
                         }
                        case "3":
                         {
                          $base_style="menu2_level5";
                          break;
                         }
                        case "4":
                         {
                          $base_style="menu2_level6";
                          break;
                         }
                        case "5":
                         {
                          $base_style="menu2_level7";
                          break;
                         }
                        case "6":
                         {
                          $base_style="menu2_level8";
                          break;
                         }
                        case "7":
                         {
                          $base_style="menu2_level8";
                          break;
                         }
                       }
                  for ($j=1; $j<=$kol_n; $j++)
                     {
                      $mas5=explode(" ~ ", $i_array[$j]);
                      if ($mas5[1]==$id_otca and $mas5[2]=="1")
                        {
                         $style_pot="<table cellspacing=\"0\"><tr><td class=\"$base_style\"><a href=\"?r=item&gs=".$mas5[0]."\" class=\"menu2_2\">".$mas5[4]."</a></td></tr></table>";
   
                         for($kkkk=count($sm);$kkkk > 0; $kkkk--)
                            {
                             if ($mas5[0]==$sm[$kkkk])
                               {
                                if ($mas5[0]==$sm[1])
                                  {
                                   $style_pot="<table cellspacing=\"0\"><tr><td class=\"$base_style\"><span class=\"menu2_2a\">".$mas5[4]."</span></td></tr></table>";
                                   
                                    $new_gl=$glubina+1;
   
                                    $gi=poisk_potomok_item ($mas5[0],$new_gl, $i_array);
                                   
                                    if ($gi != "") {$style_pot.=$fire.$gi.$fire;}
                                  }
                                else
                                  {
                                 //  $style_pot="<table cellspacing=\"0\"><tr><td class=\"$base_style\"><a href=\"?r=item&gs=".$mas5[0]."\" class=\"menu2_2\">".$mas5[4]."</a></td></tr></table>";
                                   $style_pot="<table cellspacing=\"0\"><tr><td class=\"$base_style\"><span class=\"menu2_2a\">".$mas5[4]."</span></td></tr></table>";
   
                                   $new_gl=$glubina+1;
   
                                    $gi=poisk_potomok_item ($mas5[0],$new_gl, $i_array);
   
                                    if ($gi != "") {$style_pot.=$fire.$gi.$fire;}
   
                                  }
                               }
                            }
   
                         $style.=$style_pot;
   
                        }
                 }
   
                return $style;
                 }

   В одной из ведущих веб-студий Екатеринбурга, директор 5 лет назад был кодером)))
   вот одно их его творений..
   
   Обратите внимание на идентификаторы, такие как poisk_potomok_item, $id_otca, $glubina, $i_array.
   Вообще, как человек, директор - норм чувак, но кажется мне придется пересмотреть свои взгляды...
   
   Если человек называет функцию так: "poisk_potomok_item" - ему явно жить труднее остальных людей)))))

   arbora, 21 Февраля 2013

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

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

   +145

   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

   if (!$to && !$from)
           {
               $msgInstance->setStatus(1);
               $msgInstance->setDeletedTo(1);
               $msgInstance->setDeletedFrom(1);
               $incomingL = -1;
               $outcomingL = -1;
           }
           elseif (!$to){
               $msgInstance->setStatus(1);
               $msgInstance->setDeletedTo(1);
               $incomingL = -1;
               $outcomingL = 0;
           }
           elseif (!$from)
           {
               $msgInstance->setDeletedFrom(1);
               $incomingL = 0;
               $outcomingL = -1;
           }

   Оцените xD ;)

   mishanon, 19 Февраля 2013

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

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

   +155

   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

   if (true) {
               $res = mssql_query("SELECT  CAST(_Fld204 AS TEXT) AS _Fld204 FROM [dbo].[_Reference24] WHERE _Fld203='" . $login . "' ");
               $pass = mssql_fetch_array($res, MSSQL_ASSOC);
               $pass = $pass["_Fld204"];
               if ( strlen($pass) > 0 && $pass == $password ) {
                   $_SESSION["AUTH_V"] = "Y";
                   $_SESSION["LOGINSS"]= trim($login);
               }
               else {
                   $_SESSION["AUTH_V"] = "N";
               }
   
   
          }

   Кусок авторизации

   unst, 17 Февраля 2013

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

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

   +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

   public static function GetID()
       {
           $arr = array(
               '1', '2', '3', '4', '5', '6',
               '7', '8', '9', '0');
           // Генерируем пароль
           $pass = "";
           $number = 5;
           for ($i = 0; $i < $number; $i++) {
               // Вычисляем случайный индекс массива
               $index = rand(0, count($arr) - 1);
               $pass .= $arr[$index];
   
           }
           return $pass;
   
   
       }

   Получение уникального айди заказа.
   Заказ столиков в баре.

   unst, 17 Февраля 2013

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

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

   +142

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

   $arr = date("w", strtotime($date_bron));
   $date = $arr;
   if ($arr == 0)
        $date = 0;

   Система бронирования столика для бара...

   unst, 17 Февраля 2013

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

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

   +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

   <?php
   class MainController extends ModuleAdminController {
   	public $defaultAction = 'admin';
   	
   	public function actionCreate() {
   		$this->redirect('admin');
   	}
   	
   	public function actionDelete($id) {
   		$this->redirect('admin');
   	}
   	
   	public function actionView($id) {
   		$this->redirect('admin');
   	}
   }

   Модули в Yii такие коварные :(

   Diwms, 15 Февраля 2013

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

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

   +133

   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

   private function init_categories()
   	{
   		// Дерево категорий
   		$tree = new stdClass();
   		$tree->subcategories = array();
   		
   		// Указатели на узлы дерева
   		$pointers = array();
   		$pointers[0] = &$tree;
   		$pointers[0]->path = array();
   		
   		// Выбираем все категории
   		$query = $this->db->placehold("SELECT c.id, c.parent_id, c.name, c.description, c.url, c.meta_title, c.meta_keywords, c.meta_description, c.image, c.visible, c.position
   										FROM __categories c ORDER BY c.parent_id, c.position");
   											
   		// Выбор категорий с подсчетом количества товаров для каждой. Может тормозить при большом количестве товаров.
   		// $query = $this->db->placehold("SELECT c.id, c.parent_id, c.name, c.description, c.url, c.meta_title, c.meta_keywords, c.meta_description, c.image, c.visible, c.position, COUNT(p.id) as products_count
   		//                               FROM __categories c LEFT JOIN __products_categories pc ON pc.category_id=c.id LEFT JOIN __products p ON p.id=pc.product_id AND p.visible GROUP BY c.id ORDER BY c.parent_id, c.position");
   		
   		
   		$this->db->query($query);
   		$categories = $this->db->results();
   				
   		$finish = false;
   		// Не кончаем, пока не кончатся категории, или пока ниодну из оставшихся некуда приткнуть
   		while(!empty($categories)  && !$finish)
   		{
   			$flag = false;
   			// Проходим все выбранные категории
   			foreach($categories as $k=>$category)
   			{
   				if(isset($pointers[$category->parent_id]))
   				{
   					// В дерево категорий (через указатель) добавляем текущую категорию
   					$pointers[$category->id] = $pointers[$category->parent_id]->subcategories[] = $category;
   					
   					// Путь к текущей категории
   					$curr = $pointers[$category->id];
   					$pointers[$category->id]->path = array_merge((array)$pointers[$category->parent_id]->path, array($curr));
   					
   					// Убираем использованную категорию из массива категорий
   					unset($categories[$k]);
   					$flag = true;
   				}
   			}
   			if(!$flag) $finish = true;
   		}
   		
   		// Для каждой категории id всех ее деток узнаем
   		$ids = array_reverse(array_keys($pointers));
   		foreach($ids as $id)
   		{
   			if($id>0)
   			{
   				$pointers[$id]->children[] = $id;
   
   				if(isset($pointers[$pointers[$id]->parent_id]->children))
   					$pointers[$pointers[$id]->parent_id]->children = array_merge($pointers[$id]->children, $pointers[$pointers[$id]->parent_id]->children);
   				else
   					$pointers[$pointers[$id]->parent_id]->children = $pointers[$id]->children;
   					
   				// Добавляем количество товаров к родительской категории, если текущая видима
   				// if(isset($pointers[$pointers[$id]->parent_id]) && $pointers[$id]->visible)
   				//		$pointers[$pointers[$id]->parent_id]->products_count += $pointers[$id]->products_count;
   			}
   		}
   		unset($pointers[0]);
   		unset($ids);
   
   		$this->categories_tree = $tree->subcategories;
   		$this->all_categories = $pointers;	
   	}

   построение дерева категорий в платной cms simpla

   alpex, 15 Февраля 2013

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

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

    +138

    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

    $sql_ex = array();
    if(isset($this->request->get["id"]) && $this->request->get["id"] == '5')
    	$sql_ex[] = " `ex`.`num` = 5 ";
    else
    	$sql_ex[] = " `ex`.`num` != 5 ";
    
    if(isset($this->request->get["id"]) && $this->request->get["id"] == '7')
    	$sql_ex[] = " `ex`.`num` = 7 ";
    else
    	$sql_ex[] = " `ex`.`num` != 7 ";
    if(isset($this->request->get["id"]) && $this->request->get["id"] == '4')
    	$sql_ex[] = " `ex`.`num` = 4 ";
    else
    	$sql_ex[] = " `ex`.`num` != 4 ";
    if(isset($this->request->get["id"]) && $this->request->get["id"] == '10')
    	$sql_ex[] = " `ex`.`num` = 10 ";
    else
    	$sql_ex[] = " `ex`.`num` != 10 ";
    if(isset($this->request->get["id"]) && $this->request->get["id"] == '11')
    	$sql_ex[] = " `ex`.`num` = 11 ";
    else
    	$sql_ex[] = " `ex`.`num` != 11 ";
    if(isset($this->request->get["id"]) && $this->request->get["id"] == '14')
    	$sql_ex[] = " `ex`.`num` = 14 ";
    else
    	$sql_ex[] = " `ex`.`num` != 14 ";
    if(isset($this->request->get["id"]) && $this->request->get["id"] == '19')
    	$sql_ex[] = " `ex`.`num` = 19 ";
    else
    	$sql_ex[] = " `ex`.`num` != 19 ";
    if(isset($this->request->get["id"]) && $this->request->get["id"] == '20')
    	$sql_ex[] = " `ex`.`num` = 20 ";
    else
    	$sql_ex[] = " `ex`.`num` != 20 ";
    if(isset($this->request->get["id"]) && $this->request->get["id"] == '21')
    	$sql_ex[] = " `ex`.`num` = 21 ";
    else
    	$sql_ex[] = " `ex`.`num` != 21 ";
    if(isset($this->request->get["id"]) && $this->request->get["id"] == '13')
    	$sql_ex[] = " `ex`.`num` = 13 ";
    else
    	$sql_ex[] = " `ex`.`num` != 13 ";
    if(isset($this->request->get["id"]) && $this->request->get["id"] == '12')
    	$sql_ex[] = " `ex`.`num` = 12 ";
    else
    	$sql_ex[] = " `ex`.`num` != 12 ";
    if(isset($this->request->get["id"]) && $this->request->get["id"] == '6')
    	$sql_ex[] = " `ex`.`num` = 6 ";
    else
    	$sql_ex[] = " `ex`.`num` != 6 ";
    if(isset($this->request->get["id"]) && $this->request->get["id"] == '17')
    	$sql_ex[] = " `ex`.`num` = 17 ";
    else
    	$sql_ex[] = " `ex`.`num` != 17 ";
    if(isset($this->request->get["id"]) && $this->request->get["id"] == '23')
    	$sql_ex[] = " `ex`.`num` = 23 ";
    else
    	$sql_ex[] = " `ex`.`num` != 23 ";
    if(isset($this->request->get["id"]) && $this->request->get["id"] == '16')
    	$sql_ex[] = " `ex`.`num` = 16 ";
    else
    	$sql_ex[] = " `ex`.`num` != 16 ";
    $sql_extra = '';
    if (!empty($sql_ex)) $sql_extra = ' AND '.implode(" AND ", $sql_ex);

    В начале нужно было одно условие, потом еще и еще и ...

    gvncode, 14 Февраля 2013

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

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

• 571
• …
• 386
• 385
• 384
• 383
• 382
• 381
• 380
• 379
• 378
• …
• 1