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

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

   Всего: 16

2. C++ / Говнокод #18257

   +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

   #include <cstdio>
   
   class A
   {
   public:
   	void sayHello()
   	{
   		printf("Hello!");
   	}
   };
   
   int main(int argc, char **argv)
   {
   	A *a = 0;
   	a->sayHello();
   	return 0;
   }

   Что выведет этот код?

   hdkeeper, 29 Мая 2015

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

3. C++ / Говнокод #17440

   +54

   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

   // Блок кода с выходом по break
   #define BLOCK for (int _count = 1; _count--;)
   
   bool Result::commit()
   {
   	BLOCK {
   		mysql_query(db->mysql, "BEGIN");
   
   		if (!saveBasePart())
   			break;
   
   		if (getCategory() != NULL) {
   			if (!saveCustomValues())
   				break;
   		}
   
   		mysql_query(db->mysql, "COMMIT");
   		return true;
   	}
   
   	mysql_query(db->mysql, "ROLLBACK");
   	return false;
   }

   hdkeeper, 14 Января 2015

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

4. C++ / Говнокод #16908

   +43

   1. 1
   2. 2
   3. 3

   for each (ToolStripMenuItem ^item in пользователиToolStripMenuItem->DropDownItems) {
   	item->Enabled = true;
   }

   Это C++, и это работает!

   hdkeeper, 22 Октября 2014

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

5. Pascal / Говнокод #16869

   +93

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

   if (CurrentPlugin=nil) and (CurrentPlugin.FileName<>'InfoRTF.dll') and (CurrentPLugin.FileName<>'Preview.dll')  then begin
       if not ServerStoped then begin
           uPluginControl.StopSession();
       end;
       if ClientLoaded then begin
           ClientLoaded:=false;
           StartDemo();
       end;
   end;

   Смеяться на первой же строчке.
   Привет Алексу.

   hdkeeper, 17 Октября 2014

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

6. Lua / Говнокод #16820

   −93

   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

   function SAEController:GetPointVariableValue(WayNet,CurPoint)
     local waypointVars = {};
     for i = 0, WayNet:GetPointsCount() - 1 do
        local Link = WayNet:GetPoint(i);
       local PlaySound1 = Link:GetVariableValue("PlaySound1");
       local PlaySound2 = Link:GetVariableValue("PlaySound2");
       local PlaySound3 = Link:GetVariableValue("PlaySound3");
       
       local Effect1 = Link:GetVariableValue("Effect1");
       local Effect2 = Link:GetVariableValue("Effect2");
       local Effect3 = Link:GetVariableValue("Effect3");
       
       if PlaySound1 and CurPoint == Link then
         waypointVars.PlaySound1 = PlaySound1;  
       end;
       if PlaySound2 and CurPoint == Link then
         waypointVars.PlaySound2 = PlaySound2;  
       end;
       if PlaySound3 and CurPoint == Link then
         waypointVars.PlaySound3 = PlaySound3;  
       end;
       
       if Effect1 and CurPoint == Link then
         waypointVars.Effect1 = Effect1;  
       end;
       if Effect2 and CurPoint == Link then
         waypointVars.Effect2 = Effect2;  
       end;
       if Effect3 and CurPoint == Link then
         waypointVars.Effect3 = Effect3;  
       end;
     end;
     return waypointVars;
   end;

   Кусок игровой логики на Lua. Функция проверяет значение переменных в текущей точке пути моба.
   Мы перебираем все точки пути и сравниваем с текущей точкой моба, но это не нужно, так как именно текущая точка в функцию и передаётся!

   hdkeeper, 08 Октября 2014

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

7. JavaScript / Говнокод #14402

   +156

   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

   function search(url,param)
                   {
                           $("#rightContent").hide()
                           setBlock("Загрузка")
                           loadLeftMenu()
                           $("#content").load("/search", function(response, status, xhr){
                               $("#Subscription").show()
                                   queryStr=param.split("&")
   
                                   if (queryStr.length==2){
                                           str=(queryStr[0]).replace("querystr=","")
                                           $("#querystr").val(decodeURI(str.replace(/\+/g," ")))
                                           region=((queryStr[1]).replace("region=","")).split(",")
                                           replaceItemInArray(region,"59","59,81")
                                           removeFromArray(region,'81')
                                           replaceItemInArray(region,"75","75,80")
                                           removeFromArray(region,'80')
                                           updateMultiselect("#region", region, "region")
                                   }
                                   if (queryStr.length==3){
                                           str=(queryStr[1]).replace("querystr=","")
                                           $("#querystr").val(decodeURI(str.replace(/\+/g," ")))
                                           region=((queryStr[2]).replace("region=","")).split(",")
                                           replaceItemInArray(region,"59","59,81")
                                           removeFromArray(region,'81')
                                           replaceItemInArray(region,"75","75,80")
                                           removeFromArray(region,'80')
                                           updateMultiselect("#region", region, "region")
                                   }
                                   if (queryStr.length==4){
                       str=(queryStr[0]).replace("querystr=","")
                       $("#querystr").val(decodeURI(str.replace(/\+/g," ")))
   
                       region=((queryStr[1]).replace("region=","")).split(",")
   
                       if (queryStr[2]=='fz94=true'){
                         $('#fz94').attr('checked','checked')
                       }else{
                         $('#fz94').removeAttr('checked')
                       }
   
                       if (queryStr[3]=='fz223=true'){
                         $('#fz223').attr('checked','checked')
                       }else{
                         $('#fz223').removeAttr('checked')
                       }
                                           
                                           replaceItemInArray(region,"59","59,81")
                                           removeFromArray(region,'81')
                                           replaceItemInArray(region,"75","75,80")
                                           removeFromArray(region,'80')
                                           
                                           updateMultiselect("#region", region, "region")
                                   }
                   if (queryStr.length==5){
                       str=(queryStr[1]).replace("querystr=","")
                       $("#querystr").val(decodeURI(str.replace(/\+/g," ")))
   
                       region=((queryStr[2]).replace("region=","")).split(",")
   
                       if (queryStr[3]=='fz94=true'){
                           $('#fz94').attr('checked','checked')
                       }else{
                           $('#fz94').removeAttr('checked')
                       }
   
                       if (queryStr[4]=='fz223=true'){
                           $('#fz223').attr('checked','checked')
                       }else{
                           $('#fz223').removeAttr('checked')
                       }
   
                       replaceItemInArray(region,"59","59,81")
                       removeFromArray(region,'81')
                       replaceItemInArray(region,"75","75,80")
                       removeFromArray(region,'80')
   
                       updateMultiselect("#region", region, "region")
                   }
   
   // Ну и так далее

   Разбор строки запроса

   hdkeeper, 23 Января 2014

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

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

   +167

   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

   <?foreach($arResult["ITEMS"] as $arItem):?>
   	<p class="news-item" id="<?=$this->GetEditAreaId($arItem['ID']);?>">
   	<?if($arParams["DISPLAY_PICTURE"]!="N" && is_array($arItem["PREVIEW_PICTURE"])):?>
   		<?if(!$arParams["HIDE_LINK_WHEN_NO_DETAIL"] || ($arItem["DETAIL_TEXT"] && $arResult["USER_HAVE_ACCESS"])):?>
   			<p class="align-center"><a href="<?=$arItem["DETAIL_PAGE_URL"]?>"><img class="preview_picture" border="0" src="<?=$arItem["PREVIEW_PICTURE"]["SRC"]?>" alt="<?=$arItem["NAME"]?>"  height="252px" width="189px"  title="<?=$arItem["NAME"]?>" style="float:left" /></a></p>
   		<?else:?>
   			<p class="align-center"><img class="preview_picture" border="0" src="<?=$arItem["PREVIEW_PICTURE"]["SRC"]?>" height="252px" width="189px"  alt="<?=$arItem["NAME"]?>" title="<?=$arItem["NAME"]?>" style="float:left" /></p>
   		<?endif;?>
   	<?endif?>
   		<?if($arParams["DISPLAY_DATE"]!="N" && $arItem["DISPLAY_ACTIVE_FROM"]):?>
   			<span class="news-date-time"><?echo $arItem["DISPLAY_ACTIVE_FROM"]?></span>
   		<?endif?>
   		<?if($arParams["DISPLAY_NAME"]!="N" && $arItem["NAME"]):?>
   			<?if(!$arParams["HIDE_LINK_WHEN_NO_DETAIL"] || ($arItem["DETAIL_TEXT"] && $arResult["USER_HAVE_ACCESS"])):?>
   				<a href="<?echo $arItem["DETAIL_PAGE_URL"]?>"><b><?echo $arItem["NAME"]?></b></a><br />
   			<?else:?>
   			<?endif;?>
   		<?endif;?>
   		<?if($arParams["DISPLAY_PREVIEW_TEXT"]!="N" && $arItem["PREVIEW_TEXT"]):?>
   			<?echo $arItem["PREVIEW_TEXT"];?>
   		<?endif;?>
   		<?if($arParams["DISPLAY_PICTURE"]!="N" && is_array($arItem["PREVIEW_PICTURE"])):?>
   		<?endif?>
   		<?foreach($arItem["FIELDS"] as $code=>$value):?>
   			<p class="Parad"><?=GetMessage("IBLOCK_FIELD_".$code)?>:&nbsp;<?=$value;?></p>
   <?endforeach;$a=0;?>
   
   		<?foreach($arItem["DISPLAY_PROPERTIES"] as $pid=>$arProperty):?>
   			<?$b=0;?>
   		<?if(is_array($arProperty["DISPLAY_VALUE"])):?>
   	<?else:?>
   		<? if($a==7) { if($b!=1){ ?>
   			<? $a=$a+1;$b=1;} }?>
   			<? if($a==0) { if($b!=1){ $s=$arProperty["DISPLAY_VALUE"];?>  
   					<?if($sity_shop!=$s) 
   					{?>
   						<?=$arProperty["DISPLAY_VALUE"]; 
   						$i=$i+1;
   						$sity_shop = $s;?><br>
   					<?}?>  
   			<? $a=$a+1;$b=1;} }?>
   			<?if($a==2) { if($arItem["PROPERTIES"]["NEW"]["VALUE"]=='Y') {?>
   								<a href=""><img src="/bitrix/templates/parad/img/other/new-salon.png"/></a>
   			<? $a=$a+1;$b=1; }?> </p> <? }?>
   			<?if($a==5) { if($b!=1){ if($arItem["PROPERTIES"]["MASTERCARD"]["VALUE"]=='Y') {?>
   			<? }}$a=$a+1;$b=1; } ?>
   			<?if($a==6) { if($b!=1){ if($arItem["PROPERTIES"]["VISA"]["VALUE"]=='Y') {?>
   							<? }?>
   			<? }$a=$a+1;$b=1; } ?>
   			<?if($a==1) { if($b!=1){ ?>
   						<a href="<?echo $arItem['DETAIL_PAGE_URL'];?>"><?=$arProperty["DISPLAY_VALUE"];?></a>
   			<? $a=$a+1;$b=1;} }?>
   			<?if($a!=2) { if($b!=1){ ?>
   			<? $a=$a+1;$b=1;} }?>
   			<?endif?>
   	<?endforeach;?>
   <?endforeach;?>

   Шаблончик для битрикса, вторая половина доставляет своей магией чисел.

   hdkeeper, 28 Октября 2011

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

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

   +158

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

   if (условие)
    echo 'FILE TO DELETE '.$file;
   }
   
   // затем делается
   
   if(условие)
    echo 'FILE TO DELETE '.$file;
    unlink($file);
   }

   ...и после запуска заметить, что скобочка закрывает не этот иф.

   hdkeeper, 12 Мая 2011

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

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

    +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

    if ($arResult["CurrentStep"] == 1)
    	include($_SERVER["DOCUMENT_ROOT"].$templateFolder."/step1.php");
    elseif ($arResult["CurrentStep"] == 2)
    	include($_SERVER["DOCUMENT_ROOT"].$templateFolder."/step2.php");
    elseif ($arResult["CurrentStep"] == 3)
    	include($_SERVER["DOCUMENT_ROOT"].$templateFolder."/step3.php");
    elseif ($arResult["CurrentStep"] == 4)
    	include($_SERVER["DOCUMENT_ROOT"].$templateFolder."/step4.php");
    elseif ($arResult["CurrentStep"] == 5)
    	include($_SERVER["DOCUMENT_ROOT"].$templateFolder."/step5.php");
    elseif ($arResult["CurrentStep"] >= 6)
    	include($_SERVER["DOCUMENT_ROOT"].$templateFolder."/step6.php");

    Из проекта на битриксе.

    hdkeeper, 08 Февраля 2011

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

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

    +162

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

    if(!$RISK && in_array($USER->GetID(), array("1", "14"))){
    	$RECCURING = true;
    }else{
    	$RECCURING = false;
    }

    Подписка на оплату. Магические числа 1 и 14.

    hdkeeper, 28 Января 2011

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

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

• 2
• 1