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

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

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

1. PHP (5703)
2. C++ (2740)
3. Куча (2339)
4. JavaScript (2030)
5. C# (1930)
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. Лучший говнокод

   В номинации:

   • Лучший рейтинг
   • Самый комментируемый

   За время:

   • за сегодня
   • за неделю
   • за месяц
   • за всё время

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

   +2

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

   $model->attributes = $_POST[$class];
   
   // костыль на ajax валидацию и отправку файлов
   $save = isset($_GET['notsave']) ? false : true;
   if ($save) {

   видимо автор кода соглашается с "хорошим" решением...

   akanit, 16 Декабря 2015

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

3. Куча / Говнокод #19188

   0

   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

   <?xml version="1.0" encoding="utf-8"?>
   <!-- Generator: Adobe Illustrator 19.1.0, SVG Export Plug-In . SVG Version: 6.00 Build 0)  -->
   <svg version="1.1" id="Layer_1" xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" x="0px" y="0px"
   	 viewBox="0 0 77.1 97.7" style="enable-background:new 0 0 77.1 97.7;" xml:space="preserve">
   <style type="text/css">
   	.st0{fill:#010101;}
   	.st1{fill:#FFC627;}
   	.st2{fill:#FFD432;}
   	.st3{fill:#231F20;}
   	.st4{fill:#FFFFFF;}
   </style>
   <g>
   	<g>
   		<g>
   			<g>
   				<path class="st0" d="M6.1,71.6c-3.3,0-6.1,2.7-6.1,6.1v13.9c0,3.3,2.7,6.1,6.1,6.1h61.3c3.3,0,6.1-2.7,6.1-6.1V77.7
   					c0-3.3-2.7-6.1-6.1-6.1H6.1z"/>
   				<path class="st1" d="M67.3,73.5H6.1c-2.3,0-4.1,1.9-4.1,4.1v13.9c0,2.3,1.8,4.1,4.1,4.1h61.3c2.3,0,4.1-1.8,4.1-4.1V77.7
   					C71.5,75.4,69.6,73.5,67.3,73.5 M69.5,91.6c0,1.2-1,2.2-2.2,2.2H6.1c-1.2,0-2.2-1-2.2-2.2V77.7c0-1.2,1-2.2,2.2-2.2h61.3
   					c1.2,0,2.2,1,2.2,2.2V91.6z"/>
   				.........
   			</g>
   		</g>
   	</g>
   	<path class="st1" d="M73.4,73.4c0-1.1,0.9-1.8,1.8-1.8c1,0,1.8,0.7,1.8,1.8c0,1.1-0.9,1.8-1.8,1.8C74.3,75.2,73.4,74.5,73.4,73.4
   		 M75.3,72c-0.8,0-1.4,0.6-1.4,1.4c0,0.8,0.6,1.4,1.4,1.4c0.8,0,1.4-0.6,1.4-1.4C76.6,72.5,76,72,75.3,72 M74.9,74.4h-0.4v-2h0.8
   		c0.5,0,0.8,0.2,0.8,0.6c0,0.4-0.2,0.5-0.5,0.5l0.5,0.9h-0.4l-0.4-0.9h-0.2V74.4z M74.9,73.2h0.3c0.3,0,0.4-0.1,0.4-0.3
   		c0-0.2-0.1-0.3-0.4-0.3h-0.3V73.2z"/>
   </g>
   </svg>

   а вы когда-нибудь заглядывали в SVG?

   makc3d, 15 Декабря 2015

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

4. C# / Говнокод #19182

   +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
   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
   88. 88
   89. 89
   90. 90
   91. 91
   92. 92
   93. 93
   94. 94
   95. 95
   96. 96
   97. 97
   98. 98
   99. 99

   public static CategoryAttribute Build(string category_name, string attribute_type, string attribute_name, object attribute_value)
           {
               try
               {
                   CategoryAttribute document_attribute;
   
                   switch(attribute_type)
                   {
                       case "string":
                       case "double_as_string":
                           document_attribute = new CategoryAttribute(category_name, attribute_name, TypeEnum.String);
                           break;
                       case "string_as_date":
                           document_attribute = new CategoryAttribute(category_name, attribute_name, TypeEnum.DateTime);
                           break;
                       case "int":
                       case "double_as_int":
                           document_attribute = new CategoryAttribute(category_name, attribute_name, TypeEnum.Integer);
                           break;
                       case "double":
                           document_attribute = new CategoryAttribute(category_name, attribute_name, TypeEnum.Double);
                           break;
                       default:
                           throw new ApplicationException("Не верно настроен тип данных атрибута '" + attribute_name + "' категории '" + category_name + "'.");
                   }
   
                   if (attribute_value == null)
                       return document_attribute;// атрибут остался неопределенным
   
                 switch (document_attribute.Type)
                 {
                     case TypeEnum.String:
                           {
                               switch (attribute_type)
                               {
                                   case "string":
                                       {
                                           if (!(attribute_value is string))
                                               throw new ApplicationException("Тип данных '" + attribute_value.GetType() + "' входного значения атрибута '" + attribute_name
                                                                              + "' категории '" + category_name + "' не может быть приведен к требуему типу данных '" +
                                                                              attribute_type + "'.");
   
                                           document_attribute.sValue = attribute_value as string;
                                       } break;
                                   case "double_as_string":
                                       {
                                           if (!(attribute_value is double))
                                               throw new ApplicationException("Тип данных '" + attribute_value.GetType() + "' входного значения атрибута '" + attribute_name
                                                                              + "' категории '" + category_name + "' не может быть приведен к требуему типу данных '" +
                                                                              attribute_type + "'.");
   
                                           document_attribute.sValue = ((double)attribute_value).ToString();
                                       } break;
                               }
                           }
                           break;
                           case TypeEnum.Integer:
                           {
                               switch (attribute_type)
                               {
                                   case "int":
                                   {
                                       if (!(attribute_value is int))
                                           throw new ApplicationException("Тип данных '" + attribute_value.GetType() + "' входного значения атрибута '" + attribute_name
                                                                          + "' категории '" + category_name + "' не может быть приведен к требуему типу данных '" +
                                                                          attribute_type + "'.");
   
                                       document_attribute.iValue = (int)attribute_value;
                                   }   break;
                                   case "double_as_int":
                                   {
                                       if (!(attribute_value is double))
                                           throw new ApplicationException("Тип данных '" + attribute_value.GetType() + "' входного значения атрибута '" + attribute_name
                                                                          + "' категории '" + category_name + "' не может быть приведен к требуему типу данных '" +
                                                                          attribute_type + "'.");
   
                                       document_attribute.iValue = (int)(double)attribute_value;
                                   }   break;
                               }
                           }
                           break;
                         case TypeEnum.Double:
                           {
                               if (!(attribute_value is float || attribute_value is double || attribute_value is int || attribute_value is long))
                                   throw new ApplicationException("Тип данных '" + attribute_value.GetType() + "' входного значения атрибута '" + attribute_name
                                       + "' категории '" + category_name + "' не может быть приведен к требуему типу данных '" + attribute_type + "'.");
   
                               document_attribute.dValue = (double)attribute_value;
                           }
                           break;
                   }
   
                   return document_attribute;
   
                   catch (Exception e)
                      {
                        //...
                      }
                 }

   Создание объекта типа CategoryAttribute.

   algore, 15 Декабря 2015

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

5. JavaScript / Говнокод #19130

   0

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

   $(".tabs_nav-item").click ->
       if($(this).hasClass("_active")) 
       else 
         $this = $(this)
         target = $(this).attr("href")
         $this.addClass("_active").siblings("._active").removeClass("_active")
         $(target).addClass("_active").siblings("._active").removeClass("_active")

   LOL
   P.S. coffeescript

   Crazyzubr, 03 Декабря 2015

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

6. C# / Говнокод #19084

   +1

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

   string str3 = Strings.Trim(ID);
    do
    {
        num2 = (short) Strings.InStr(str3, " ", CompareMethod.Binary);
        if (num2 > 0)
        {
               str3 = str3.Substring(0, num2 - 1) + Strings.Mid(str3, num2 + 1);
        }
   }
   while (num2 > 0);

   А зачем нам str3.Replace(" ", string.Empty) ?

   inickvel, 25 Ноября 2015

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

7. Куча / Говнокод #19078

   +1

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

   section.static > div[style="display:block; border: solid 1px #cfcfcf; padding : 10px"]{
       border: none !important;
       padding: 0 !important;
   }

   css-костыль!)

   Stefan_Hitskov, 24 Ноября 2015

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

8. Куча / Говнокод #19068

   +3

   1. 1

   <image_block class="inner" interactivity="no" list_entry_id="da vi uporolis">

   Fai, 23 Ноября 2015

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

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

   0

   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
   88. 88
   89. 89
   90. 90
   91. 91
   92. 92
   93. 93
   94. 94
   95. 95
   96. 96
   97. 97
   98. 98
   99. 99

   $breadcrumb = array();
   $breadcrumb[0] = new Json();
   $breadcrumb[0]->url = URL::base().'cat';
   $breadcrumb[0]->caption = 'Каталог';
   try {
   	if(isset($_GET['q'])) {
   		$qs = $_GET['q'];
   		$matches = array();
   		$count = preg_match_all("/\d+/", $qs, $matches);
   		if ($count > 1) {
   			$lc = $matches[0][0];
   			$vc = $matches[0][1];
   			list ($totalCount, $thumbnails, $lamps_orm, $formCount) = Imp::getVariants($lc, $vc, $p);
   			list ($form, $formList) = Imp::getForms($lamps_orm[0]->f->latin);
   			$this->template->title = $lamps_orm[0]->cyrillic;
   			$lampName = $lamps_orm[0]->cyrillic;
   			$breadcrumb[1] = new Json();
   			$breadcrumb[1]->url = URL::base().'cat/'.$form->c->latin;
   			$breadcrumb[1]->caption = $form->c->cyrillic;
   			$breadcrumb[2] = new Json();
   			$breadcrumb[2]->url = URL::base().'lamp/'.$form->latin;
   			$breadcrumb[2]->caption = $form->cyrillic;
   			$breadcrumb[3] = new Json();
   			$breadcrumb[3]->url = URL::base().'lamp/search/?q='.$lamps_orm[0]->code;
   			$breadcrumb[3]->caption = $lamps_orm[0]->cyrillic;
   			$breadcrumb[4] = new Json();
   			$breadcrumb[4]->url = NULL;
   			$breadcrumb[4]->caption = $thumbnails[0]->code;
   		} elseif ($count == 1) {
   			$lc = $matches[0][0];
   			list ($totalCount, $thumbnails, $lamps_orm, $formCount) = Imp::getVariants($lc, NULL, $p);
   			list ($form, $formList) = Imp::getForms($lamps_orm[0]->f->latin);
   			$this->template->title = $lamps_orm[0]->cyrillic;
   			$lampName = $lamps_orm[0]->cyrillic;
   			$breadcrumb[1] = new Json();
   			$breadcrumb[1]->url = URL::base().'cat/'.$form->c->latin;
   			$breadcrumb[1]->caption = $form->c->cyrillic;
   			$breadcrumb[2] = new Json();
   			$breadcrumb[2]->url = URL::base().'lamp/'.$form->latin;
   			$breadcrumb[2]->caption = $form->cyrillic;
   			$breadcrumb[3] = new Json();
   			$breadcrumb[3]->url = NULL;
   			$breadcrumb[3]->caption = $lamps_orm[0]->cyrillic;
   		} else {
   			list ($totalCount, $thumbnails, $lamps_orm, $formCount) = Imp::getVariants($qs, NULL, $p);
   			list ($form, $formList) = Imp::getForms($lamps_orm[0]->f->latin);
   			if (count($lamps_orm) > 1) {
   				$this->template->title = $form->cyrillic;
   				$lampName = $form->cyrillic;
   				$breadcrumb[1] = new Json();
   				$breadcrumb[1]->url = URL::base().'cat/'.$form->c->latin;
   				$breadcrumb[1]->caption = $form->c->cyrillic;
   				$breadcrumb[2] = new Json();
   				$breadcrumb[2]->url = NULL;
   				$breadcrumb[2]->caption = $form->cyrillic;
   				if ($formCount > 1) {
   					$breadcrumb = array();
   					$breadcrumb[0] = new Json();
   					$breadcrumb[0]->url = NULL;
   					$breadcrumb[0]->caption = 'Результаты поиска';
   					$this->template->title = 'Результаты поиска по запросу "'.$qs.'"';
   					$lampName = 'Результаты поиска по запросу "'.$qs.'"';
   				}
   			} else {
   				$this->template->title = $lamps_orm[0]->cyrillic;
   				$lampName = $lamps_orm[0]->cyrillic;
   				$breadcrumb[1] = new Json();
   				$breadcrumb[1]->url = URL::base().'cat/'.$form->c->latin;
   				$breadcrumb[1]->caption = $form->c->cyrillic;
   				$breadcrumb[2] = new Json();
   				$breadcrumb[2]->url = URL::base().'lamp/'.$form->latin;
   				$breadcrumb[2]->caption = $form->cyrillic;
   				$breadcrumb[3] = new Json();
   				$breadcrumb[3]->url = NULL;
   				$breadcrumb[3]->caption = $lamps_orm[0]->cyrillic;
   			}
   		}
   		$pagination = Imp::getPagination($totalCount, 16, $p, 'lamp/search/:page/?q='.$qs);
   	} else {
   		list ($form, $formList) = Imp::getForms($f);
   		if (isset($_GET['y'])) {
   			list ($totalCount, $thumbnails, $lamps_orm, $formCount) = Imp::getVariants($form, 'form', $p, array(0, 1), $_GET['y'], $_GET['m'], $_GET['d']);
   			$paginationUrlFormat = 'lamp/'.$f.'/:page/?y='.$_GET['y'].'&m='.$_GET['m'].'&d='.$_GET['d'];
   			$lampName = $form->cyrillic.' <span style="color: #666; font-style: oblique;"><small>Показаны обновления от '.D::_($_GET['y'].'-'.$_GET['m'].'-'.$_GET['d']).'.    <a href="'.URL::base().'lamp/'.$f.'" style="font-weight: normal; font-style: normal;">Показать все варианты ›</a></small></span>';
   		}
   		else {
   			list ($totalCount, $thumbnails, $lamps_orm, $formCount) = Imp::getVariants($form, 'form', $p);
   			$paginationUrlFormat = 'lamp/'.$f.'/:page';
   			$lampName = $form->cyrillic;
   		}
   		$this->template->title = $form->cyrillic;
   		$pagination = Imp::getPagination($totalCount, 16, $p, $paginationUrlFormat);
   		$breadcrumb[1] = new Json();
   		$breadcrumb[1]->url = URL::base().'cat/'.$form->c->latin;
   		$breadcrumb[1]->caption = $form->c->cyrillic;
   		$breadcrumb[2] = new Json();
   		$breadcrumb[2]->url = NULL;
   		$breadcrumb[2]->caption = $form->cyrillic;
   	}

   Кусок контроллера трехлетней давности. В данный момент готовлю это к утилизации. А, когда я это писал, мне казалось, что я крут.
   В то время на хостинге не было json_encode/json_decode, поэтому сделал класс Json. Хотя, использовал его просто так, чтобы не создавать кучу специализированных классов.

   kgm-rj, 16 Ноября 2015

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

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

    +2

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

    <?
    $key = crc32("release") ^ 0x817990;
    $header_bg = '#' . substr(base_convert(crc32($DBName) ^ $key, 10, 16), 0, 6);
    ?>
    <div id="top_header" style="background:<?= $header_bg ?>">

    На доменах с похожими названиями живут продакшн и версии для разработки. Заказчик путается при тестировании, а потом разрабы удивляются тестовым данным в рабочей базе.
    Вот так была реализована цветовая дифференциация штанов сайтов в зависимости от имени БД. Подобрать им имя базы, которое даст коллизию, что ли...

    clauclauclau, 09 Ноября 2015

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

11. PHP / Говнокод #18973

    +2

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

    if (count($not_enough_rights) == 1 and !empty($not_enough_rights['user_extra'])) {
    					// если нужен доступ только к user_extra, то дополнительного доступа не надо, но скрываем сами права user_extra
    
    				} else {
    
    					if (count(array_merge($user_rights, $EDIT_ADMIN_GROUPS)) != count($EDIT_ADMIN_GROUPS)) {
    						$GLOBALS['VALID_GROUPS'] = array('root');
    					} else {
    						$GLOBALS['VALID_GROUPS'] = array('superadmin');
    					}
    
    				}

    Если $user_rights не пустой, то ты просто админ, если пустой, то суперадмин. Не проставил в БД права - суперадмин!

    dimka3210, 05 Ноября 2015

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

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

• 1963
• …
• 86
• 85
• 84
• 83
• 82
• 81
• 80
• 79
• 78
• …
• 1