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

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

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

1. PHP (5708)
2. C++ (2739)
3. Куча (2379)
4. JavaScript (2032)
5. C# (1929)
6. Java (1502)
7. Си (1117)
8. Pascal (648)
9. Python (591)
10. 1C (539)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (200)
15. Perl (194)
16. Assembler (146)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (34)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

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

   В номинации:

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

   За время:

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

2. ActionScript / Говнокод #8063

   −117

   1. 1

   freedomShip(point.x,point.y,super.xy1,super(),super.ships[super.count]);

   И такое бывает:)

   swing1991, 02 Октября 2011

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

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

   +161

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

   <?php
   //в посте кроме этих ключей больше быть ничего не может
   if ($_POST['x'])unlink('/x.png');
   if ($_POST['y'])unlink('/y.png');
   if ($_POST['z'])unlink('/z.png');
   if ($_POST['a'])unlink('/a.png');
   if ($_POST['b'])unlink('/b.png');
   //etc
   ?>

   Ну не знал я, что можно было сделать так:
   <?php
   foreach ($_POST as $key => $value)
   {
   unlink('/'.$key.'.png');
   }
   ?>

   QarezZ, 01 Октября 2011

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

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

   +166

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

   <?php
   $x=$_GET['x'];
   $y=intval($x);
   $z=round($y);
   $a=max(0, $z);
   ?>

   Ну не знал я о совмещении функций, и не знал, что точно делает каждая из них :D

   QarezZ, 01 Октября 2011

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

5. C# / Говнокод #8041

   +134

   1. 1
   2. 2

   if (!(string.IsNullOrEmpty("")))
   ...

   HellMaster_HaiL, 30 Сентября 2011

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

6. JavaScript / Говнокод #8038

   +169

   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

   function echeck(str) {
     var at="@"
     var dot="."
     var lat=str.indexOf(at)
     var lstr=str.length
     //  var ldot=str.indexOf(dot)
     if (str.indexOf(at)==-1){
       alert("Invalid E-mail ID");
       return false;
     }
     if (str.indexOf(at)==-1 || str.indexOf(at)==0 || str.indexOf(at)==lstr){
       alert("Invalid E-mail ID");
       return false;
     }
     if (str.indexOf(dot)==-1 || str.indexOf(dot)==0 || str.indexOf(dot)==lstr){
       alert("Invalid E-mail ID");
       return false;
     }
     if (str.indexOf(at,(lat+1))!=-1){
       alert("Invalid E-mail ID");
       return false;
     }
     if (str.substring(lat-1,lat)==dot || str.substring(lat+1,lat+2)==dot){
       alert("Invalid E-mail ID");
       return false;
     }
     if (str.indexOf(dot,(lat+2))==-1){
       alert("Invalid E-mail ID");
       return false;
     }
     if (str.indexOf(" ")!=-1){
       alert("Invalid E-mail ID");
       return false;
     }
     return true;
   }

   если честно, дочитал где-то только до 15 строки

   marg, 30 Сентября 2011

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

7. ActionScript / Говнокод #8019

   −350

   1. 1

   if (event.target.parent.parent.parent.profileRow.children["2"].children["1"].text != "Мухомор")

   Предыдущего разработчика проекта обуял демон сложности. До некоторых сущностей пришлось добираться так. Чудеса практологии.

   crazy_horse, 28 Сентября 2011

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

8. JavaScript / Говнокод #8018

   +157

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

   function fakePassword() {
   	$(":input:password").each(
   		function(i) {
   			$("#" + this.id + "-hidden-password").attr("name", this.name);
   			$("#" + this.id + "-hidden-password").attr("value", this.value);
   			$(this).removeAttr("value");
   			$(this).removeAttr("name");
   		}
   	);
       return true;
   }

   Connor, 28 Сентября 2011

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

9. Си / Говнокод #7965

   +143

   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

   #include <unistd.h>
   
   	        main()
   	        {
           int i=0;
           char *cmd[] = { "ls", "-l", (char *)0 };
           char *env[] = { "HOME=/usr/home", "LOGNAME=home", (char *)0 };
           int rr;
           for(i; i<6; i++)
           {
       switch(rr=fork())
       {
   case -1:execl ("/bin/ls", "ls", "-1", (char *)0);break;
   case 0:execle ("/bin/ls", "ls", "-l", (char *)0, env);break;
   case 1:execlp ("ls", "ls", "-l", (char *)0);break;
   case 2:execv ("/bin/ls", cmd);break;
   case 3:execve ("/bin/ls", cmd, env);break;
   case 4:execvp ("ls", cmd);break;
       }
           }
   
   	        }

   - Ну это работают.
   И проверяющий поставил плюс.

   Niceblack, 24 Сентября 2011

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

10. Java / Говнокод #7954

    +85

    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

    nLen = m_aRealTexts[index].m_txtString.length();
    for (int iyhx = 0; iyhx < nLen; iyhx++) {
    	cLetter = m_aRealTexts[index].m_txtString.charAt(iyhx);
    	CTxtCharStyle pCharNode = new CTxtCharStyle(cLetter);
    	pCharNode.SetTxtFontFace(m_nFontFace);
    	m_aRealTexts[index].m_txtChars.add(pCharNode);
    }
    
    ...
    
    
    public void SetTxtFontFace(byte nType)
    {
    	String sLine, sTemp;
    	int nMark;
    	float xValue, yValue;		//笔画点位的临时坐标
    	byte bIsBigChar = 0;		//1表示ANSI字体UserArial_ansi.uft，2表示Unicode字体UserArial_unicode.uft，3表示宋体hztxt1.shx
    	m_cStroke.clear();			//清空笔画路径链表
    	
    	if(nType == 1)
    	{
    		if((int)m_cChar<0x7F || ((int)m_cChar>=0x2160&&(int)m_cChar<=0x2169))
    		{
    			//按Unicode编码顺序处理字符查找指定字库文件，避免读取不必要的字库文件
    			InputStream fInFile = this.getClass().getResourceAsStream("/TagResources/UserArial_ansi.uft");
    			
    			...
    			
    			fInFile.close();
    		}
    		else if((int)m_cChar>=0x007F && (int)m_cChar<0x2FFF)
    		{
    			//字符在UserArial_ansi.uft中未找到，看是否在UserArial_unicode.uft中
    			int nGetByte;
    			sLine = new String("");
    			sTemp = new String("");
    			//float xValue, yValue;
    			byte[] CharUnicode = new byte[2];			//一次读取两个字节，为一个字符
    			byte[] cBytes = new byte[2];
    			InputStream fInFile = this.getClass().getResourceAsStream("/TagResources/UserArial_unicode.uft");
    			while((nGetByte = fInFile.read(cBytes, 0, 2)) > 0)
    			{
    				...
    			}
    			fInFile.close();
    		}
    		else if((int)m_cChar>=0x3000)
    		{
    			//字符在西文字库UserArial.uft中未找到，是大字体，则在宋体文件hztxt1.shx中查找
    			m_cWidth = 112.0f;	//对于宋体，左下角点为坐标原点，包围盒长宽皆为127，调整量为(127-112)/2=5
    			m_cHeight = 112.0f;
    			byte nMoveDown = 5;
    			InputStream fInFile = this.getClass().getResourceAsStream("/TagResources/hztxt1.shx");
    			
    			...
    			
    			fInFile.close();
    		}
    		else if(bIsBigChar == 0)
    		{
    			//仍然没有找到字符字体，则按空格处理
    			m_cWidth = 12.0f;
    			m_cHeight = 16.0f;
    			m_cStroke.clear();
    		}
    	}
    	else if(nType == 2)
    	{
    		try
    		{
    			
    			InputStream fInFile = this.getClass().getResourceAsStream("/TagResources/hztxt1.shx");
    			
    			...
    			
    			fInFile.close();
    			
    		}
    		catch(IOException e)
    		{
    			//异常处理
    			e.printStackTrace();
    		}
    	}
    }

    Ещё один отжиг господ китайцев.
    
    Магические числа вместо энумов и закрытие файлов вне finally - это ещё полбеды. Больше всего умиляет, что при каждой отрисовке отдельно для каждой буквы файл шрифта открывается и сканируется по новой. Неудивительно, что отрисовка так тормозит. Буду переписывать это дело - загружать глифы из файла один раз и затем дёргать их из кэша.

    lucidfox, 23 Сентября 2011

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

11. Java / Говнокод #7924

    +63

    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

    private void anyFunction (Boolean param){
            if(param == null){
                throw new RuntimeException("param is null");
            }
            if(param.toString().length() == 4){
                //any code
            }else{
                if(param.toString().length() == 5){
                  //any code
                }else{
                    throw new RuntimeException("fack");
                }
            }
        }

    булеан к строке, для проверки true или false.
    Кроме того, если не true и не false то ...

    botsms, 21 Сентября 2011

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

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

• 1973
• …
• 1097
• 1096
• 1095
• 1094
• 1093
• 1092
• 1091
• 1090
• 1089
• …
• 1