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

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

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

1. PHP (5711)
2. C++ (2742)
3. Куча (2393)
4. JavaScript (2034)
5. C# (1930)
6. Java (1503)
7. Си (1120)
8. Pascal (648)
9. Python (591)
10. 1C (540)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (200)
15. Perl (194)
16. Assembler (148)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (43)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

1. Лучший говнокод

   В номинации:

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

   За время:

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

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

   +159

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

   function summArray($arr) 
   {
       $summ = 0;
       foreach($arr as $a) {
           $summ += intval($a);
       }
       
       return $summ;
   }

   minisot, 05 Мая 2010

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

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

   +164

   1. 1
   2. 2
   3. 3

   <?php
   $count =300;
   define('MAXPROCESS',$count);

   $count дальше естественно нигде не используется..

   null, 05 Мая 2010

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

4. JavaScript / Говнокод #3148

   +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
   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 addElement(name, params) {
   var elem = document.createElement(name);
   if (params.href) elem.href = params.href;
   if (params.innerHTML) elem.innerHTML = params.innerHTML;
   if (params.innerText) elem.innerText = params.innerText;
   if (params.className) elem.className = params.className;
   if (params.id) elem.id = params.id;
   if (params.name) elem.name = params.name;
   if (params.type) elem.type = params.type;
   if (params.onclick) elem.onclick = params.onclick;
   if (params.onkeypress) elem.onkeypress = params.onkeypress;
   if (params.onkeydown) elem.onkeydown = params.onkeydown;
   if (params.onkeyup) elem.onkeyup = params.onkeyup;
   if (params.onmousedown) elem.onmousedown = params.onmousedown;
   if (params.onmouseup) elem.onmouseup = params.onmouseup;
   if (params.onmousewheel) elem.onmousewheel = params.onmousewheel;
   if (params.onload) elem.onload = params.onload;
   if (params.onkeydown) elem.onkeydown = params.onkeydown;
   if (params.height) elem.height = params.height;
   if (params.width) elem.width = params.width;
   if (params.align) elem.align = params.align;
   if (params.valign) elem.valign = params.valign;
   if (params.celspacing) elem.cellspacing = params.cellspacing;
   if (params.cellpadding) elem.cellpadding = params.cellpadding;
   if (params.src) elem.src = params.src;
   if (params.rel) elem.rel = params.rel;
   if (params.border) elem.border = params.border;
   if (params.onkeyup) elem.onkeyup = params.onkeyup;
   if (params.value) elem.value = params.value;
   if (params.action) elem.action = params.action;
   if (params.title) elem.title = params.title;
   vif (params.alt) elem.alt = params.alt;
   document.body.appendChild(elem);
   }

   А что такое for…in?
   P.S. там ещё был перебор всех свойств elem.style, но, увы, сюда не влезло.
   P.P.S. по началу 32-й строки можно определить, как был написан код ;)

   Infamous, 02 Мая 2010

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

5. C++ / Говнокод #3022

   +61.2

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

   FILE *f=fopen(name, "wb");
   char *text="<DMB 1.0>\n\n";
   for(int i=0;i<=10;i++){fputc((int)text, f);}
   fclose(f);

   Замечу это c++, так как это чудо находилось в классе.

   hromjo, 16 Апреля 2010

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

6. C++ / Говнокод #3019

   +913.2

   1. 1

   if (c != "$"[0])

   Без комментария

   Snake2101, 16 Апреля 2010

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

7. C++ / Говнокод #3002

   +49.8

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

   bool ok;
   double d;
   
   QLocale c(QLocale::C);
   d = c.toDouble( "1234.56", &ok );  // ok == true, d == 1234.56

   Ссылка на bool!

   mock, 14 Апреля 2010

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

8. 1C / Говнокод #2722

   −108.4

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

   Процедура ПередОткрытием(Отказ, СтандартнаяОбработка)
   	
   	ОтраженоВОперУчете           = Истина;
   	ОтражатьВБухгалтерскомУчете  = Истина;
   	ОтражатьВНалоговомУчете      = Истина;
   	
   	Если НЕ ДокументОснование = Неопределено Тогда
   		ЭлементыФормы.Проект.Значение = ДокументОснование.Проект;
   	КонецЕсли;	
   		
   КонецПроцедуры // ПередОткрытием()

   Автор хотел что бы все документы обязательно проводились по БУ УУ и НУ... И хотел что бы при вводе на основании заполнялось поле проект. Ну почти получилось:))))

   acckiy_koder, 05 Марта 2010

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

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

   +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
   23. 23
   24. 24
   25. 25
   26. 26
   27. 27
   28. 28

   function ShowButton(name, link, target,img,id,style){
   	         {/<TD align = "right" width=1>/}
   				{/<SPAN id=" %id " title=" %name " style=" %style ">/}
   				{/<TABLE style="CURSOR: pointer" height=16 cellSpacing=0 cellPadding=0 border=0>/}
   				{/<TBODY>/}
   				{/<TR height=3>/}
   					{/<TD vAlign=top width=7><IMG height=3 src=" %pPath() left_top_button" width=7 border=0></TD>/}
   					{/<TD class=spacer vAlign=top background= %pPath() bg_top_button>&nbsp;</TD>/}
   					{/<TD vAlign=top width=7><IMG height=3 src=" %pPath() right_top_button" width=7 border=0></TD></TR>/}
   				{/<TR style="CURSOR: pointer">/}
   					{/<TD class=spacer vAlign=top width=7 background= %pPath() left_button>&nbsp;</TD>/}
   					{/<TD onClick = " %link " class=buttonLink vAlign=center noWrap align=middle background= %pPath() bg_button>/}
   					if(img!=null){
   						Image(img, null, null, null, name); 
   					}else{
   						{/<a href=#> %name </a>/}	
   					}
   					{/</TD>/}
   					{/<TD class=spacer vAlign=top width=7 background= %pPath() right_button>&nbsp;</TD></TR>/}
   				{/<TR style="CURSOR: pointer" height=3>/}
   					{/<TD class=spacer vAlign=top width=7><IMG src=" %pPath() left_bot_button" border=0></TD>/}
   					{/<TD class=spacer background= %pPath() bg_bot_button height=3>&nbsp;</TD>/}
   					{/<TD class=spacer vAlign=top width=7><IMG src=" %pPath() right_bot_button" border=0></TD></TR>/}
   					{/</TBODY>/}
   				{/</TABLE>/}
                 {/</SPAN>/}
               {/</TD>/}
   }

   Из одной говносистемы документооборота:
   
   вот из этого километра говноразметки - получается обычная кнопочка, почти ничем не отличающаяся от input type="button"

   GeX, 04 Марта 2010

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

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

    +156.8

    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

    $q1=mysql_query("SELECT * FROM `categories` where id_p=0 and type='gallery';");
    		if($r1=mysql_fetch_array($q1))
    	     		{
    				$name=$r1['name'];
    				$id=$r1['id'];
    				echo "<tr>";
    				echo "<td colspan='2'> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp; <a href='add_articles_gallery.php?level=2&id=$id&lang=$lang&id1=$id'><b> $name </b></a> </td>";
    				echo "</tr>";
    				while($r1=mysql_fetch_array($q1))
    					{	
    						$name=$r1['name'];
    						$id=$r1['id'];
    						echo "<td colspan='2'> &nbsp;&nbsp;&nbsp;&nbsp;&nbsp; <a href='add_articles_gallery.php?level=2&id=$id&lang=$lang&id1=$id'><b> $name </b></a></td> ";
    						echo "</td>";
    						echo "</tr>";
    					}
    				}

    )))

    doremidon, 26 Января 2010

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

11. Си / Говнокод #2171

    +106.7

    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

    char *getUI_ID_AsString(MENU_ID menuId)
    {
        char *IdStr;
        	
    	if((menuId == VOLUME_ID) || (menuId == POPUPWIN_ID) || (menuId == YESNOWIN_ID) || (menuId == MAINMENU_ID) || (menuId == 
    		MODEMENU_ID) || (menuId == USBMODE_ID) || (menuId == SETTINGSMENU_ID) || (menuId == MENUPLAYMODE_ID) || (menuId == MENUEQ_ID) 
    		|| (menuId == MENUSETTING_ID) || (menuId == MENULOCK_ID) || (menuId == MENUPLAYSPEED_ID) || (menuId == MENUCONTRAST_ID) || (menuId == 
    		MENUBACKLIGHT_ID) || (menuId == MENULANGUAGE_ID) || (menuId == MENUSHUTDOWN_ID) || (menuId == MENUINFO_ID) || (menuId == 
    		MENUBEEP_ID) || (menuId == MENUFACTORYSET_ID) || (MENUUSEREQ_ID) || (menuId == SET_PREFERENCES_ID) || (menuId == 
    		MENUCONFIRM_RESET_ID) || (menuId == MENUCONFIRM_FORMAT_ID) || (menuId == MAINRECMENU_ID) || (menuId == RECSETMODE_ID) || 
    		(menuId == RECSETINPUT_ID) || (menuId == RECSETVAD_ID) || (menuId == RECSETAUTOSYNC_ID) || (menuId == NAVIGATION_ID) ||
    		(menuId == PROMPT_VOLUME_ID) || (menuId == MENUGPSAPPL_MAIN_ID) || (menuId == POWERON_CONNECTING_ID) || (menuId == 
    		HOMESCREEN_ID) || (menuId == RC_DOWNLOADING_ID) || (menuId == SCREEN_BRIGHTNESS_ID) || (menuId == ABOUT_ID) || (menuId == 
    		INSTMENU_ID) || (menuId == INST_ABOUT_ID) || (menuId == DISPL_ORIENT_ID) || (menuId == LINEOUT_ID) || (menuId == 
    		PRIVACY_MUTE_ID) || (menuId == TEST_INSTALL_ID) || (menuId == FACTORY_RESET_ID) || (menuId == CALL_DTMF_ID) ||	(menuId == 
    		NAVI_FILEMENU_ID) || (menuId == NAVI_CONFIRM_DEL_ID) || (menuId == NAVI_SELECTFOLDER_ID) || (menuId == LANG_SELECT_ID) ||
    		(menuId == DISPLAY_JPG_ID) || (menuId == DISPLAY_TEXT_ID) || (menuId == SET_DISPLAY_ID))
    	{
    	    IdStr = (char *) malloc(4);
        	strcpy(IdStr, " MENU");
    	}
    	else if( <тут примерно еще столько же условий> )
    		{
    		       IdStr = (char *) malloc(5);
    			   strcpy(IdStr, " PHONE");
    		}
    	else if( <тут примерно еще столько же условий> )
    		{
    		    strcpy(IdStr, " PBOOK");
    		}
    	else if( <тут примерно еще столько же условий> )
    		{
    		    IdStr = (char *) malloc(5);
    		    strcpy(IdStr, " MUSIC");
    		}
    	else if( <тут примерно еще столько же условий> )
    		{
    		    IdStr = (char *) malloc(3);
    		    strcpy(IdStr, " MSG");
    		}
    		
    		return IdStr;
    }

    Шедевр индусского говнокодирования. А потом с нас спрашивают, почему это у нас система нестабильно работает.
    switch, по ходу, они еще в школе не прошли, да и с математикой у них плоховато. А где-то память вообще выделять не надо - зачем?

    Scrambler, 24 Ноября 2009

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

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

• 1976
• …
• 1570
• 1569
• 1568
• 1567
• 1566
• 1565
• 1564
• 1563
• 1562
• …
• 1