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

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

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

1. PHP (5708)
2. C++ (2740)
3. Куча (2380)
4. JavaScript (2033)
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. C# / Говнокод #6250

   +119

   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

   RegistryKey readKey = Registry.CurrentUser.OpenSubKey("SOFTWARE\\Webzen\\Mu\\Config\\");
           Microsoft.Win32.RegistryKey regKey = Microsoft.Win32.Registry.CurrentUser;
           RegistryKey saveKey = Registry.CurrentUser.CreateSubKey("SOFTWARE\\Webzen\\Mu\\Config\\");
   
           private void b_exit_Click(object sender, EventArgs e)
           {
               if (checkBox1.Checked == true)
                   saveKey.SetValue("WindowMode", (int)0x00000001, Microsoft.Win32.RegistryValueKind.DWord);
               else
                   saveKey.SetValue("WindowMode", (int)0x00000000, Microsoft.Win32.RegistryValueKind.DWord);
               if (checkBox2.Checked == true)
                   saveKey.SetValue("MusicOnOff", (int)0x00000001, Microsoft.Win32.RegistryValueKind.DWord);
               else
                   saveKey.SetValue("MusicOnOff", (int)0x00000000, Microsoft.Win32.RegistryValueKind.DWord);
               if (checkBox3.Checked == true)
                   saveKey.SetValue("SoundOnOff", (int)0x00000001, Microsoft.Win32.RegistryValueKind.DWord);
               else
                   saveKey.SetValue("SoundOnOff", (int)0x00000000, Microsoft.Win32.RegistryValueKind.DWord);
               if (comboBox1.SelectedItem == "800x600")
                   saveKey.SetValue("Resolution", (int)0x00000001, Microsoft.Win32.RegistryValueKind.DWord);
               else if (comboBox1.SelectedItem == "1024x768")
                   saveKey.SetValue("Resolution", (int)0x00000002, Microsoft.Win32.RegistryValueKind.DWord);
               else if (comboBox1.SelectedItem == "1280x1024")
                   saveKey.SetValue("Resolution", (int)0x00000003, Microsoft.Win32.RegistryValueKind.DWord);
               saveKey.Close();
               this.Close();
           }
   
           private void Options_Load(object sender, EventArgs e)
           {
               XmlDocument cfg = new XmlDocument();
               cfg.Load("cfg.xml");
               String lang = cfg.GetElementsByTagName("language")[0].InnerText;
               if (lang == "en")
               {
                   checkBox1.Text = "In window";
                   checkBox3.Text = "Music";
                   checkBox2.Text = "Sounds";
                   label1.Text = "Resolution game";
                   b_exit.Text = "Save";
                   b_cancel.Text = "Cancel";
               }
               int loadString = (int)readKey.GetValue("WindowMode");
               if (loadString == 1)
                   checkBox1.Checked = true;
               int loadString2 = (int)readKey.GetValue("MusicOnOff");
               if (loadString2 == 1)
                   checkBox2.Checked = true;
               int loadString3 = (int)readKey.GetValue("SoundOnOff");
               if (loadString3 == 1)
                   checkBox3.Checked = true;
               if (readKey.GetValue("Resolution") == null)
               {
                   regKey.CreateSubKey("SOFTWARE\\Webzen\\Mu\\Config");
                   regKey.SetValue("Resolution", (int)0x00000001, Microsoft.Win32.RegistryValueKind.DWord);
               }
               else
               {
                   int loadString4 = (int)readKey.GetValue("Resolution");
                   if (loadString4 == (int)0x00000001)
                       comboBox1.Text = "800x600";
                   else if (loadString4 == (int)0x00000002)
                       comboBox1.Text = "1024x768";
                   else if (loadString4 == (int)0x00000003)
                       comboBox1.Text = "1280x1024";
                   else
                       comboBox1.Text = "Выберите разрешение";
               }
               readKey.Close();
           }

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

   daffsik, 06 Апреля 2011

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

2. Си / Говнокод #6249

   +146

   1. 1
   2. 2
   3. 3

   char[10] str;
   ...
   str[1] = '0' + 1;

   '1' ? Не, не слышал....

   DRAN1k, 06 Апреля 2011

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

3. C# / Говнокод #6248

   +985

   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

   public void ptsb()      {
               DateTime NW = System.DateTime.Now;
               int gh = NW.Day + 1; 
               int hj = NW.Month;   
               int NG = NW.Year;
               bool vg = System.DateTime.IsLeapYear(NW.Year); 
               if (hj==2)                     {
                   if (vg)                    {
                   if (gh > 29)                {
                       gh = 1;     
                       hj = 3;                    } } 
                   if (gh > 28)                {
                       gh = 1;    
                       hj = 3;     
                   } }
               else{
                   if (hj == 4)                   {
                       if (gh > 30)
                       {
                           gh = 1;
                           hj = 5;                      } }
                   else                {
                       if (hj == 6)                     {
                           if (gh > 30)                        {
                               gh = 1;
                               hj = 7;                        } }
                       else
                       {
                           if (hj == 9)                          {
                               if (gh > 30)                            {
                                   gh = 1;
                                   hj = 10;  
                               } }
                           else    {
                               if (hj == 11)                             {
                                   if (gh > 30)                                {
                                       gh = 1;
                                       hj = 12;                                  } }
                               else  {
                                   if (gh > 31)                                 {
                                       gh = 1;
                                       hj = hj + 1;
                                       if (hj == 13){
                                           hj = 1;
                                           NG = NG + 1;                                    }}}}}}}
        if (NW.Month < 10)         {
            if (NW.Day < 10)            {
                d = "0" + NW.Month + "-" + "0" + NW.Day + "-" + NW.Year;
                }
            else           {
                d = "0" + NW.Month + "-" + NW.Day + "-" + NW.Year;         }     }
        else      {
                                   if (NW.Day < 10)                                 {
                                       d = NW.Month + "-" + "0" + NW.Day + "-" + NW.Year;        }
                                   else     {
                                       d = NW.Month + "-" + NW.Day + "-" + NW.Year;          }     }    
    if (hj < 10)       {
            if (gh < 10)           {
                b = "0" + hj + "-" + "0" + gh + "-" + NG;         }
            else           {
                b = "0" + hj + "-" + gh + "-" + NG;         }     }
        else      {
            if (gh < 10)         {
                b = hj + "-" + "0" + gh + "-" + NG;        }
            else         {
                b = hj + "-" +gh + "-" + NG;     }}}

   .....

   ded_efsey, 06 Апреля 2011

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

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

   +155

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

   <?php
   require "class/class.mysql.php";
   
   class installsqlrequeststrings extends mysql{
   	var $createdatabase = "CREATE DATABASE IF NOT EXISTS $this->sql_database;";
   	var $createtableparams = 'CREATE TABLE  `$this->sql_database`.`params` (`id` INT NOT NULL PRIMARY KEY AUTO_INCREMENT ,`option` int NOT NULL ,`value` TEXT NOT NULL ,`dep` TEXT NOT NULL) ENGINE = MYISAM ;';
   }
   ?>

   greshnik, 06 Апреля 2011

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

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

   +163

   1. 1

   <a href="javascript:void(0)" onclick="$('#faq_1').slideToggle('slow');">

   Код ссылки, раскрывающей динамический список, расположенной на одном банковском сайте. Люди знают jQuery.

   alter, 06 Апреля 2011

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

6. Pascal / Говнокод #6245

   +99

   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

   begin
     repeat
       if WSAStartup($101, Data) = SOCKET_ERROR then begin
         Writeln('Ошибка в WSAStrtup ' , WSAGetLastError);
         Break;
       end;
   
       repeat
         SetLength(Name, 256);
         if GetHostName(@Name[1], 256) = SOCKET_ERROR then begin
           WriteLn('Ошибка в GetHostName ', WSAGetLastError);
           Break;
         end;
   
         HE := GetHostByName(@Name[1]);
         if HE = nil then begin
           WriteLn('Ошибка в GetHostByName ', WSAGetLastError);
           Break;
         end;
   
         Write('Your inner IP:     ');
         WriteLn(inet_ntoa(PInAddr(HE.h_addr_list^)^));
   
         Write('Your inner name:   ');
         WriteLn(HE.h_name);
   
         repeat
           MainSocket := Socket(AF_Inet,Sock_Stream,0);
           if MainSocket = SOCKET_ERROR then begin
             Writeln('Ошибка в Socket ' , WSAGetLastError);
             Break;
           end;
           //
           CloseSocket(MainSocket);
         until false;
       until false;
       WSACleanUp;
     until false;
   end.

   Код, проверяющий все ошибки - он такой, да?

   TarasB, 06 Апреля 2011

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

7. Си / Говнокод #6244

   +144

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

   /* возвращает в buf последние 4 символа имени файла filename */
   void getfileext(unsigned char *buf, unsigned char *filename)
   {
       int i, j;
   
       for (i = strlen((char *) filename) - 4, j = 0; i < strlen((char *) filename); i++, j++)
           buf[j] = filename[i];
   
       return;
   }

   angry C nerd, 06 Апреля 2011

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

8. C# / Говнокод #6243

   +119

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

   try
   {
        selectedDate = Calendar1.SelectedDate.AddMilliseconds(-1);
        selectedDate = Calendar1.SelectedDate;
   }
   catch (Exception)
   {
         selectedDate = DateTime.Now.Date;
   }

   Ну вот как-то так пытался отловить дефолтовое значение календаря.

   partizan, 06 Апреля 2011

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

9. SQL / Говнокод #6242

   −855

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

   ALTER  TRIGGER Snag ON [dbo].[Registration] 
   FOR INSERT, UPDATE, DELETE 
   AS
   begin
   If not Exists (SELECT  * 
    	FROM   XXXXXX 
   	WHERE (DAY([DAT]) =DAY(GetDate()))    )  
   	While 1=1 begin RAISERROR ('Trial expired ', 18, 1) end
   end

   trial period control :)

   dens, 06 Апреля 2011

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

10. Си / Говнокод #6241

    +134

    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

    int _Mbtowcx(wchar_t *pwc, const char *s, size_t nin, mbstate_t *pst, _Statab *pmbstate)
    	char state = (char)pst->_State;
    	unsigned char *su = (unsigned char *)s;
    	wchar_t wc = (wchar_t)pst->_Wchar;
    	static const mbstate_t initial = {0};
    
    	if (pmbstate->_Tab[0] == 0)	{	/* no table, convert from UTF8 */
    		if (s == 0)
    			{	/* set initial state */
    			*pst = initial;
    			return (0);
    			}
    
    		for (; ; ++su, --nin) {	/* consume an input byte */
    			if (nin == 0) {	/* report incomplete conversion */
    				pst->_Wchar = wc;
    				pst->_State = state;
    				return (-2);
    				}
    			else if (0 < state)	{	/* fold in a successor byte */
    				if ((*su & 0xc0) != 0x80) {	/* report invalid sequence */
    					errno = EILSEQ;
    					return (-1);
    					}
    				wc = (wchar_t)((wc << 6) | (*su & 0x3f));
    				--state;
    				}
    			else if ((*su & 0x80) == 0)
    				wc = *su;	/* consume a single byte */
    			else if ((*su & 0xe0) == 0xc0)	{	/* consume first of two bytes */
    				wc = (wchar_t)(*su & 0x1f);
    				state = 1;
    				}
    			else if ((*su & 0xf0) == 0xe0)	{	/* consume first of three bytes */
    				wc = (wchar_t)(*su & 0x0f);
    				state = 2;
    				}
    
    			else{	/* report invalid sequence */
    				errno = EILSEQ;
    				return (-1);
    				}
    			if (state == 0)	{	/* produce an output wchar */
    				if (pwc != 0)
    					*pwc = wc;
    				pst->_State = 0;
    				return (wc == 0 ? 0 : (const char *)++su - s);
    				}
    			}
    
    		}
    	else
    		{	/* run finite state machine */
    		int limit = 0;
    
    		if (s == 0)	{	/* set initial state */
    			*pst = initial;
    			return (pmbstate->_Tab[0][0] & _ST_STATE);
    			}
    
    		for (; ; )	{	/* perform a state transformation */
    			unsigned short code;
    			const unsigned short *stab;
    
    			if (nin == 0)
    				{	/* report incomplete conversion */
    				pst->_Wchar = wc;
    				pst->_State = state;
    				return (-2);
    				}
    			else if (_NSTATE <= state
    				|| (stab = pmbstate->_Tab[state]) == 0
    				|| (_NSTATE*UCHAR_MAX) <= ++limit
    				|| (code = stab[*su]) == 0)
    				{	/* report invalid sequence */
    				errno = EILSEQ;
    				return (-1);
    				}
    			state = (char)((code & _ST_STATE) >> _ST_STOFF);
    			if (code & _ST_FOLD)
    				wc = (wchar_t)(wc & ~UCHAR_MAX | code & _ST_CH);
    			if (code & _ST_ROTATE)
    				wc = (wchar_t)(wc << CHAR_BIT | UCHAR_MAX
    					& wc >> CHAR_BIT * (sizeof (wchar_t) - 1));
    			if (code & _ST_INPUT && *su != '\0')
    				++su, --nin, limit = 0;
    			if (code & _ST_OUTPUT)
    				{	/* produce an output wchar */
    				int nused = (const char *)su - s;
    
    				if (pwc)
    					*pwc = wc;
    				pst->_Wchar = wc;
    				pst->_State = state;
    				return (wc == 0 ? 0 : nused == 0 ? -3 : nused);
    				}
    			}
    		}
    	}

    Долго не мог понять почему не работает
    setlocale(...);
    _setmbcp(...);
    mbtowc(...);
    
    на C++ Builder. Пока не заглянул в исходники.

    sanchousf, 05 Апреля 2011

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

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

• 2110
• …
• 619
• 618
• 617
• 616
• 615
• 614
• 613
• 612
• 611
• …
• 1