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

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

   Этот поиск практически ничего не может найти! Но вы всё-таки попытайтесь, вдруг повезет.

   Поисковая строка:

   В языке:

   Во всехPHPJavaScriptSQLPerlPythonСиC++C#JavaPascalActionScriptAssemblerVisualBasicКучаRuby1CbashObjective CSwiftLuaHaskellKotlinGo

   Найдено: 72

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

   +172

   1. 1

   <td><?="ID"?>:</td>

   Пример из документации Bitrix
   http://dev.1c-bitrix.ru/api_help/main/general/admin.section/classes/cadminfilter

   Rpsl, 18 Октября 2010

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

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

   +165

   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

   function FormatCurrency($fSum, $strCurrency)
   {
   	return CurrencyFormat($fSum, $strCurrency);
   	/*
   	if (!isset($fSum) || strlen($fSum)<=0) return "";
   
   	$arCurFormat = CCurrencyLang::GetCurrencyFormat($strCurrency);
   
   	if (!isset($arCurFormat["DECIMALS"])) $arCurFormat["DECIMALS"] = 2;
   	$arCurFormat["DECIMALS"] = IntVal($arCurFormat["DECIMALS"]);
   	if (!isset($arCurFormat["DEC_POINT"])) $arCurFormat["DEC_POINT"] = ".";
   	if (!isset($arCurFormat["THOUSANDS_SEP"])) $arCurFormat["THOUSANDS_SEP"] = "\\"."xA0";
   	$tmpTHOUSANDS_SEP = $arCurFormat["THOUSANDS_SEP"];
   	eval("\$tmpTHOUSANDS_SEP = \"$tmpTHOUSANDS_SEP\";");
   	$arCurFormat["THOUSANDS_SEP"] = $tmpTHOUSANDS_SEP;
   	if (!isset($arCurFormat["FORMAT_STRING"])) $arCurFormat["FORMAT_STRING"] = "#";
   
   	$num = number_format($fSum, $arCurFormat["DECIMALS"], $arCurFormat["DEC_POINT"], $arCurFormat["THOUSANDS_SEP"]);
   
   	return str_replace("#", $num, $arCurFormat["FORMAT_STRING"]);
   	*/
   }

   1C-Bitrix,
   /bitrix/modules/catalog/include.php

   Under, 12 Октября 2010

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

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

   +158

   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

   <div class="item" style="padding-bottom:60px">
   	<a class="no-border" href="catalogue/?SECTION_ID=327&arrFilter_pf[PRODUCER]=<?=urlencode('Producer(Турция)')?>&arrFilter_pf[TOPLIVO]=<?=urlencode("Дизельное")?>&set_filter=Y"><img src="/bitrix/templates/producer/i/items/dis-generator.jpg" alt="Дизельные генераторы" title="Дизельные генераторы"/></a>
   	<a href="catalogue/?SECTION_ID=327&arrFilter_pf[PRODUCER]=<?=urlencode('Producer(Турция)')?>&arrFilter_pf[TOPLIVO]=<?=urlencode("Дизельное")?>&set_filter=Y">Дизельные генераторы</a>
   </div>
   <div class="item">
   	<a class="no-border" href="catalogue/?SECTION_ID=327&arrFilter_pf[PRODUCER]=<?=urlencode('Producer(Турция)')?>&arrFilter_pf[TOPLIVO]=<?=urlencode('Бензин А-92(95)')?>&set_filter=Y"><img src="/_producer/i/items/oil-generator.jpg" alt="Бензиновые генераторы" title="Бензиновые генераторы"/></a>
   	<a href="catalogue/?SECTION_ID=327&arrFilter_pf[PRODUCER]=<?=urlencode('Producer(Турция)')?>&arrFilter_pf[TOPLIVO]=<?=urlencode('Бензин А-92(95)')?>&set_filter=Y">Бензиновые генераторы</a>
   </div>
   <div class="item">
   	<a class="no-border" href="catalogue/?SECTION_ID=333"><img src="/bitrix/templates/producer/i/items/generator.jpg" alt="Газовые генераторы" title="Газовые генераторы"/></a>
   	<a href="catalogue/?SECTION_ID=333">Газовые генераторы</a>
   </div>
   <div class="item">
   	<a class="no-border" href="catalogue/?SECTION_ID=326"><img src="/bitrix/templates/producer/i/items/container.jpg" alt="Контейнеры" title="Контейнеры"/></a>
   	<a href="catalogue/?SECTION_ID=326">Контейнеры</a>
   </div>	
   <?
   // тут куча кода
   ?>
   <div class="item" style="padding-top:0px; margin-top:-15px">
   	<? if ($arrFilter_pf[TOPLIVO] != 'Дизельное') { ?><a class="no-border" href="?SECTION_ID=327&arrFilter_pf[PRODUCER]=<?=urlencode('Producer(Турция)')?>&arrFilter_pf[TOPLIVO]=<?=urlencode("Дизельное")?>&set_filter=Y"><? } ?><div class="product_div"><img src="/bitrix/templates/producer/i/items/dis-generator.jpg" alt="Дизельные генераторы" title="Дизельные генераторы"/></div><? if ($arrFilter_pf[TOPLIVO] != 'Дизельное') { ?></a>
   	<a href="?SECTION_ID=327&arrFilter_pf[PRODUCER]=<?=urlencode("Producer(Турция)")?>&arrFilter_pf[TOPLIVO]=<?=urlencode("Дизельное")?>&set_filter=Y"><? } ?>Дизельные генераторы<? if ($arrFilter_pf[TOPLIVO] != 'Дизельное') { ?></a><? } ?>
   </div>
   <div class="item" style="padding-top:0px; margin-top:-15px">
   	<? if ($arrFilter_pf[TOPLIVO] != 'Бензин А-92(95)') { ?><a class="no-border" href="?SECTION_ID=327&arrFilter_pf[PRODUCER]=<?=urlencode('Producer(Турция)')?>&arrFilter_pf[TOPLIVO]=<?=urlencode('Бензин А-92(95)')?>&set_filter=Y"><? } ?><div class="product_div"><img src="/bitrix/templates/producer/i/items/oil-generator.jpg" alt="Бензиновые генераторы" title="Бензиновые генераторы"/></div><? if ($arrFilter_pf[TOPLIVO] != 'Бензин А-92(95)') { ?></a>
   	<a href="?SECTION_ID=327&arrFilter_pf[PRODUCER]=<?=urlencode("Producer(Турция)")?>&arrFilter_pf[TOPLIVO]=<?=urlencode("Бензин А-92(95)")?>&set_filter=Y"><? } ?>Бензиновые генераторы<? if ($arrFilter_pf[TOPLIVO] != 'Бензин А-92(95)') { ?></a><? } ?>
   </div>
   <div class="item" style="padding-top:0px; margin-top:-15px">
   	<? if ($_GET['SECTION_ID'] != '333') { ?><a class="no-border" href="?SECTION_ID=333"><? } ?><div class="product_div"><img src="/bitrix/templates/producer/i/items/generator.jpg" alt="Газовые генераторы" title="Газовые генераторы"/></div><? if ($_GET['SECTION_ID'] != '333') { ?></a>
   	<a href="?SECTION_ID=333"><? } ?>Газовые генераторы<? if ($_GET['SECTION_ID'] != '333') { ?></a><? } ?>
   </div>
   <div class="item" style="padding-top:0px; margin-top:-15px">
   	<? if ($_GET['SECTION_ID'] != '326') { ?><a class="no-border" href="?SECTION_ID=326"><? } ?><div class="product_div"><img src="/bitrix/templates/producer/i/items/container.jpg" alt="Контейнеры" title="Контейнеры"/></div><? if ($_GET['SECTION_ID'] != '326') { ?></a>
   	<a href="?SECTION_ID=326"><? } ?>Контейнеры<? if ($_GET['SECTION_ID'] != '326') { ?></a><? } ?>
   </div>

   Это часть шаблона сайта, сделанного на битриксе. Хотя битрикс тут и не нужен, автор и без него справился бы с задачей. Без него было бы даже проще :)))
   
   Название сайта я заменил в коде на "Producer", не хочу уже позорить. Жаль что весь код не влез, там в таком же духе дальше.

   alex322, 28 Июля 2010

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

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

   +160

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

   <?$APPLICATION->IncludeFile(
   			$APPLICATION->GetTemplatePath("include_areas/contacts.php"),
   			Array(),
   			Array("MODE"=>"html")
   		);?> </div>

   Вот так индусские проггеры из Bitrix показывают контакты сайта.

   aleferov, 24 Мая 2010

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

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

   +157.1

   1. 1

   return strtoupper(strtr($str, "йцукенгшщзхъэждлорпавыфячсмитьбюё", "ЙЦУКЕНГШЩЗХЪЭЖДЛОРПАВЫФЯЧСМИТЬБЮЁ"));

   1C-Bitrix, вот говнищще-то!

   xEviL, 10 Августа 2009

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

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

   +153

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

   <?
   if ($arResult["MANAGE_PANEL"]["MESSAGES"] == "Y"):
   endif;
   ?>

   Ну вы поняли, какая CMS
   /bitrix/modules/forum/install/components/bitrix/forum/templates/.default/bitrix/forum.menu/.default/template.php
   Семёрка, 426 строка

   guest, 11 Июня 2009

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

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

   +149.2

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

   $page = preg_replace( array( "/action=.*?\\&/i", 
   "/bx_event_calendar_request=.*?\\&/i", 
   "/clear_cache=.*?\\&/i", 
   "/bitrix_include_areas=.*?\\&/i", 
   "/bitrix_show_mode=.*?\\&/i", 
   "/back_url_admin=.*?\\&/i"), "", $arParams['pageUrl']."&" );
   
   $page = preg_replace( array( "/^(.*?)\\&\$/i", "/^(.*?)\\?\$/i" ), "\$1", $page );

   как перечислить возможные слова или символы в регулярке? Нормальные программисты делают это через (word1|word2) и [&?], но в битриксе...

   guest, 19 Февраля 2009

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

9. JavaScript / Говнокод #574

   +158.9

   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

   if(this.o)
   {
       if((cur=='1')||(cur=='2'))cur='2';
       if((cur=='3')||(cur=='4'))cur='4';
       if((cur=='5')||(cur=='6'))cur='6';
       if((cur=='7')||(cur=='8'))cur='8';
   };
   
   /*
       Ну а чувакам пИсавшим эту функциию, посвящается:
   */
   if(cur & 1)
       ++cur;

   Фрагмент кода из CMS Bitrix 7

   guest, 19 Февраля 2009

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

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

    +11.6

    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

    function SetFileAccessPermission($path, $arPermissions, $bOverWrite=true)
    	{
    		CMain::InitPathVars($site, $path);
    		$DOC_ROOT = CSite::GetSiteDocRoot($site);
    
    		if(strlen($path) <= 0)
    			$path="/";
    		if(($p = bxstrrpos($path, "/"))!==false)
    		{
    			$path_file = substr($path, $p+1);
    			$path_dir = substr($path, 0, $p);
    		}
    		else
    			return false;
    
    		if($path_file=="" && $path_dir=="")
    			$path_file = "/";
    
    		$PERM = Array();
    		if(file_exists($DOC_ROOT.$path_dir."/.access.php"))
    			@include($DOC_ROOT.$path_dir."/.access.php");
    		
    		$FILE_PERM = $PERM[$path_file];
    		if(!is_array($FILE_PERM))
    			$FILE_PERM=Array();
    
    		if(!$bOverWrite && count($FILE_PERM)>0)
    			return true;
    
    		$bDiff = false;
    
    		$str="<?\n";
    		foreach($arPermissions as $group=>$perm)
    		{
    			if(strlen($perm) > 0)
    				$str.="\$PERM[\"".$path_file."\"][\"".$group."\"]=\"".str_replace("\"", "\\\"", $perm)."\";\n";
    			if(!$bDiff && $FILE_PERM[$group]!=$perm)
    				$bDiff=true;
    		}
    
    		foreach($PERM as $file=>$arPerm)
    		{
    			if(strval($file) !==$path_file)
    				foreach($arPerm as $group=>$perm)
    					$str.="\$PERM[\"".$file."\"][\"".$group."\"]=\"".str_replace("\"", "\\\"", $perm)."\";\n";
    		}
    
    		if(!$bDiff)
    		{
    			foreach($FILE_PERM as $group=>$perm)
    				if($arPermissions[$group]!=$perm)
    				{
    					$bDiff==true;
    					break;
    				}
    		}
    
    		$str.="?".">";
    		
    		$this->SaveFileContent($DOC_ROOT.$path_dir."/.access.php", $str);
    		$GLOBALS["CACHE_MANAGER"]->CleanDir("menu");
    		unset($this->FILE_PERMISSION_CACHE[$site."|".$path_dir."/.access.php"]);
    		
    		if($bDiff)
    		{
    			$db_events = GetModuleEvents("main", "OnChangePermissions");
    			while($arEvent = $db_events->Fetch())
    				ExecuteModuleEvent($arEvent, Array($site, $path), $arPermissions);
    		}
    		return true;
    	}

    MegaLolnii bitrix
    самое смешное с 32 строчки

    guest, 23 Января 2009

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

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

    +33.6

    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

    <?
     function ShowImage($strImage, $iMaxW=0, $iMaxH=0, $sParams=null, $strImageUrl="", $bPopup=false, $sPopupTitle=false, $iSizeWHTTP=0, $iSizeHHTTP=0)
     {
      global $DOCUMENT_ROOT, $DB;
    
      if(!($arImgParams = CFile::_GetImgParams($strImage, $iSizeWHTTP, $iSizeHHTTP)))
       return "";
    
      if($sParams === null || $sParams === false)
       $sParams = ' border="0" ';
    
      $iMaxW = intval($iMaxW);
      $iMaxH = intval($iMaxH);
    
      $strImage = htmlspecialchars($arImgParams["SRC"]);
      $intWidth = $arImgParams["WIDTH"];
      $intHeight = $arImgParams["HEIGHT"];
      $strAlt = $arImgParams["ALT"];
    
      if($sPopupTitle===false)
       $sPopupTitle=GetMessage("FILE_ENLARGE");
    
      $file_type = GetFileType($strImage);
      switch($file_type):
       case "FLASH":
        $iWidth = $intWidth;
        $iHeight = $intHeight;
        if($iMaxW>0 && $iMaxH>0 && ($intWidth > $iMaxW || $intHeight > $iMaxH))
        {
         $coeff = ($intWidth/$iMaxW > $intHeight/$iMaxH? $intWidth/$iMaxW : $intHeight/$iMaxH);
         $iWidth = intval(roundEx($intHeight/$coeff));
         $iHeight = intval(roundEx($intWidth/$coeff));
        }
        $strReturn = '
         <object
          classid="clsid:D27CDB6E-AE6D-11CF-96B8-444553540000"
          codebase="http://download.macromedia.com/pub/shockwave/cabs/flash/swflash.cab#version=6,0,0,0"
          id="banner"
          WIDTH="'.$iWidth.'"
          HEIGHT="'.$iHeight.'"
          ALIGN="">
           <PARAM NAME="movie" VALUE="'.$strImage.'" />
           <PARAM NAME="quality" VALUE="high" />
           <PARAM NAME="bgcolor" VALUE="#FFFFFF" />
           <embed
            src="'.$strImage.'"
            quality="high"
            bgcolor="#FFFFFF"
            WIDTH="'.$iWidth.'"
            HEIGHT="'.$iHeight.'"
            NAME="banner"
            ALIGN=""
            TYPE="application/x-shockwave-flash"
            PLUGINSPAGE="http://www.macromedia.com/go/getflashplayer">
           </embed>
         </object>
         ';
        return $bPopup? $strReturn : print_url($strImageUrl, $strReturn);
    
       default:
        $strReturn = "<img src=\"".$strImage."\" ".$sParams." width=\"".$intWidth."\" height=\"".$intHeight."\" alt=\"".htmlspecialchars($strAlt)."\" />";
        if($iMaxW > 0 && $iMaxH > 0) //need to check scale, maybe show actual size in the popup window
        {
         //check for max dimensions exceeding
         if($intWidth > $iMaxW || $intHeight > $iMaxH)
         {
          $coeff = ($intWidth/$iMaxW > $intHeight/$iMaxH? $intWidth/$iMaxW : $intHeight/$iMaxH);
          $strReturn = "<img src=\"".$strImage."\" ".$sParams." width=\"".intval(roundEx($intWidth/$coeff))."\" height=\"".intval(roundEx($intHeight/$coeff))."\" alt=\"".htmlspecialchars($strAlt)."\" />";
    
          if($bPopup) //show in JS window
          {
           if(strlen($strImageUrl)>0)
           {
            $strReturn =
             '<a href="'.$strImageUrl.'" title="'.$sPopupTitle.'" target="_blank">'.
             '<img src="'.$strImage.'" '.$sParams.' width="'.intval(roundEx($intWidth/$coeff)).'" height="'.intval(roundEx($intHeight/$coeff)).' alt="'.htmlspecialchars($sPopupTitle).'" />'.
             '</a>';
           }
           else
           {
            CFile::OutputJSImgShw();
    
            $strReturn =
             "<a title=\"".$sPopupTitle."\" onClick=\"ImgShw('".AddSlashes($strImage)."','".$intWidth."','".$intHeight."', '".AddSlashes(htmlspecialcharsex(htmlspecialcharsex($strAlt)))."'); return false;\" href=\"".$strImage."\" target=\"_blank\">".
             "<img src=\"".$strImage."\" ".$sParams." width=\"".intval(roundEx($intWidth/$coeff))."\" height=\"".intval(roundEx($intHeight/$coeff))."\" /></a>";
           }
          }
         }
        }
        return $bPopup? $strReturn : print_url($strImageUrl, $strReturn);
    
      endswitch;
    
      return $bPopup? $strReturn : print_url($strImageUrl, $strReturn);
     }
    ?>

    Уникальный фрагмент кода из CMS Bitrix 7.x.
    Судя по всему его писал индус который не проверял его работы.
    Некотором будет непонятен тонкий юмор автора этого кода, но когда пытаешься решить с его помощью конкретную задачу становится совсем не смешно а наоборот обидно, за себя и толпы тысяч людей полагающих что Bitrix решение всех его проблем логика работы в котором напрочь отсутствует так же как и здравый смысл.
    Сделаю небольшой намёк. Весь фокус во взаимной зависимости параметров
    $iMaxW=0, $iMaxH=0, $strImageUrl="", $bPopup=false

    guest, 22 Января 2009

    Комментарии (3)
• ← влево
• вправо →
• 8
• …
• 6
• 5
• 4
• 3
• 2
• 1