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

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

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

1. PHP (5706)
2. C++ (2739)
3. Куча (2360)
4. JavaScript (2030)
5. C# (1928)
6. Java (1502)
7. Си (1115)
8. Pascal (648)
9. Python (583)
10. 1C (538)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (198)
15. Perl (194)
16. Assembler (146)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Go (31)
21. Swift (27)
22. Haskell (26)
23. Kotlin (14)

1. Куча / Говнокод #9661

   +126

   1. 1
   2. 2

   http://my.sec.ru/author.cfm
   http://daily.sec.ru/search.cfm?s=%22+%2F%3E

   И это называется портал по безопасности...

   istem, 13 Марта 2012

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

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

   +127

   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

   typedef struct _xjmp_stru {
       unsigned long Ebp;
       unsigned long Ebx;
       unsigned long Edi;
       unsigned long Esi;
       unsigned long Esp;
       unsigned long Eip;
   } xjmp_stru;
   
   typedef int xjmp_buf[6];
   
   __declspec(naked)
   int __cdecl xsetjmp(xjmp_buf)
   {
   	__asm
   	{
   		mov     edx, [esp+4]
   		mov     [edx], ebp
   		mov     [edx+4], ebx
   		mov     [edx+8], edi
   		mov     [edx+12], esi
   		mov     [edx+16], esp
   		mov     eax, [esp]
   		mov     [edx+20], eax
   		xor     eax, eax
   		ret
   	}
   } 
   
   
   __declspec(naked, noreturn)
   void __cdecl xlongjmp(xjmp_buf, int)
   {
   	__asm
   	{
   		mov     edx, [esp+4]
   		mov     ebp, [edx]
   		mov     ebx, [edx+4]
   		mov     edi, [edx+8]
   		mov     esi, [edx+12]
   		mov     eax, [esp+8]
   		test    eax, eax
   		jne     __
   		inc     eax
   __:      
   		mov     esp, [edx+16]
   		add     esp, 4
   		mov     edx, [edx+20]
   		jmp     edx
   	}
   } 
   
   int dummy(xjmp_buf jbuf)
   {
   	volatile int jk = 8;
   	if (jk)
   		xlongjmp(jbuf, 2);
   	else
   		return 7;
   }
   
   int main()
   {
   	xjmp_buf jbuf;
   	if (xsetjmp(jbuf))
   	{
   		puts("excpt");
   		return -1;
   	}
   	dummy(jbuf);
   	puts("great work");
   	return 0;
   }

   экая хренотень

   63F45EF45RB65R6VR, 13 Марта 2012

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

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

   +964

   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

   namespace WindowsFormsApplication12
   {
       public partial class Form1 : Form
       {
           public Form1()
           {
               InitializeComponent();
           }
   
           private void button1_Click(object sender, EventArgs e)
           {
               if (textBox1.Text == "") textBox1.Text = "1";
               if (textBox2.Text == "") textBox2.Text = "1";
               if (textBox3.Text == "") textBox3.Text = "1";
   
               label7.Text = textBox1.Text + "x2 + " + textBox2.Text + "x + " + textBox3.Text;
   
               label4.Text = "x1 = " + Convert.ToString((Convert.ToInt32("-" + textBox2.Text) + (Math.Sqrt(Math.Pow(Convert.ToInt32(textBox2.Text), 2) - 4
                       * Convert.ToInt32(textBox1.Text) * Convert.ToInt32(textBox3.Text)))) / (2 * Convert.ToInt32(textBox1.Text)));
   
               label5.Text = "x2 = " + Convert.ToString((Convert.ToInt32("-" + textBox2.Text) - (Math.Sqrt(Math.Pow(Convert.ToInt32(textBox2.Text), 2) - 4
               * Convert.ToInt32(textBox1.Text) * Convert.ToInt32(textBox3.Text)))) / (2 * Convert.ToInt32(textBox1.Text)));
   
               label6.Text = "D = " + (Math.Pow(Convert.ToInt32(textBox2.Text), 2) - 4 * Convert.ToInt32(textBox1.Text) * Convert.ToInt32(textBox3.Text)).ToString();
           }

   Программа находит корни квадратного уравнения.

   P4R4, 12 Марта 2012

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

4. C# / Говнокод #9658

   +116

   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

   private void label1_Click(object sender, EventArgs e)
           {
               MessageBox.Show("Данная программа вычисляет\nнаибольший общий делитель.");
           }
   
           private void textBox1_TextChanged(object sender, EventArgs e)
           {
   
           }
   
           private void button1_Click(object sender, EventArgs e)
           {
               if (Convert.ToInt32(textBox1.Text) >= Convert.ToInt32(textBox2.Text))
               {
                   for (int i = Convert.ToInt32(textBox1.Text); i > 0; i--)
                   {
                       if (Convert.ToInt32(textBox1.Text) % i == 0)
                       {
                           if (Convert.ToInt32(textBox2.Text) % i == 0)
                           {
                               label2.Text = "НОД = " + Convert.ToString(i);
                               break;
                           }
                       }
                   }
               }
               else
                   for (int i = Convert.ToInt32(textBox2.Text); i > 0; i--)
                   {
                       if (Convert.ToInt32(textBox2.Text) % i == 0)
                       {
                           if (Convert.ToInt32(textBox1.Text) % i == 0)
                           {
                               label2.Text = "НОД = " + Convert.ToString(i);
                               break;
                           }
                       }
                   }
           }

   Программа с одного форума

   P4R4, 12 Марта 2012

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

5. Java / Говнокод #9657

   +74

   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

   for (Tm_RecipientConstructor recipient: m_Recipients) {
   	
   	Tm_PHB_Contact m_Contact = recipient.getContact();
   	
   	if ( m_Contact != null) {					
   		if(m_Contact.getMobile()!=null) {						
   			st.setLong(1,recipient.getEntry().getID());//nWebEntryID
   			st.setLong(2,Tm_Registry.getInstance().getAttachementTypes().getID(En_AttachementType.ARGUMENT));//NTYPE
   			st.setString(3,"mobile");//STRNAME
   			st.setLong(4,Tm_Registry.getInstance().getAttributeType().getID(En_AttributeType.STRING));//NDATATYPE
   			st.setString(5,m_Contact.getMobile());//STRVALUE
   			st.addBatch();
       		index++;
       		if (index % 5000 == 0){		    				
       			st.executeBatch();
       			st.clearBatch();				            
       			st = conn.prepareStatement(sql);
       		}
   		}
   		if(m_Contact.getName()!=null) {						
   			st.setLong(1,recipient.getEntry().getID());//nWebEntryID
   			st.setLong(2,Tm_Registry.getInstance().getAttachementTypes().getID(En_AttachementType.ARGUMENT));//NTYPE
   			st.setString(3,"first_name");//STRNAME
   			st.setLong(4,Tm_Registry.getInstance().getAttributeType().getID(En_AttributeType.STRING));//NDATATYPE
   			st.setString(5,m_Contact.getName());//STRVALUE
   			st.addBatch();
       		index++;
       		if (index % 5000 == 0){		    				
       			st.executeBatch();
       			st.clearBatch();				            
       			st = conn.prepareStatement(sql);
       		}
   		}
   		if(m_Contact.getSurName()!=null) {
   			
   			st.setLong(1,recipient.getEntry().getID());//nWebEntryID
   			st.setLong(2,Tm_Registry.getInstance().getAttachementTypes().getID(En_AttachementType.ARGUMENT));//NTYPE
   			st.setString(3,"last_name");//STRNAME
   			st.setLong(4,Tm_Registry.getInstance().getAttributeType().getID(En_AttributeType.STRING));//NDATATYPE
   			st.setString(5,m_Contact.getSurName());//STRVALUE
   			st.addBatch();
       		index++;
       		if (index % 5000 == 0){		    				
       			st.executeBatch();
       			st.clearBatch();				            
       			st = conn.prepareStatement(sql);
       		}
   		}					
   		if(m_Contact.getPatrName()!=null) {
   			
   			st.setLong(1,recipient.getEntry().getID());//nWebEntryID
   			st.setLong(2,Tm_Registry.getInstance().getAttachementTypes().getID(En_AttachementType.ARGUMENT));//NTYPE
   			st.setString(3,"second_name");//STRNAME
   			st.setLong(4,Tm_Registry.getInstance().getAttributeType().getID(En_AttributeType.STRING));//NDATATYPE
   			st.setString(5,m_Contact.getPatrName());//STRVALUE
   			st.addBatch();
       		index++;
       		if (index % 5000 == 0){		    				
       			st.executeBatch();
       			st.clearBatch();				            
       			st = conn.prepareStatement(sql);
       		}
   		}					
   		if(m_Contact.getCompanyName()!=null) {
   			
   			st.setLong(1,recipient.getEntry().getID());//nWebEntryID
   			st.setLong(2,Tm_Registry.getInstance().getAttachementTypes().getID(En_AttachementType.ARGUMENT));//NTYPE
   			st.setString(3,"company");//STRNAME
   			st.setLong(4,Tm_Registry.getInstance().getAttributeType().getID(En_AttributeType.STRING));//NDATATYPE
   			st.setString(5,m_Contact.getCompanyName());//STRVALUE
   			st.addBatch();
       		index++;
       		if (index % 5000 == 0){		    				
       			st.executeBatch();
       			st.clearBatch();				            
       			st = conn.prepareStatement(sql);
       		}
   		}
   		if(m_Contact.getComment()!=null) {
   			
   			st.setLong(1,recipient.getEntry().getID());//nWebEntryID
   			st.setLong(2,Tm_Registry.getInstance().getAttachementTypes().getID(En_AttachementType.ARGUMENT));//NTYPE
   			st.setString(3,"comments");//STRNAME
   			st.setLong(4,Tm_Registry.getInstance().getAttributeType().getID(En_AttributeType.STRING));//NDATATYPE
   			st.setString(5,m_Contact.getComment());//STRVALUE
   			st.addBatch();
       		index++;
       		if (index % 5000 == 0){		    				
       			st.executeBatch();
       			st.clearBatch();				            
       			st = conn.prepareStatement(sql);
       		}
   		}	
   	}
   }

   Функции? Не, не слышал.

   SadKo, 12 Марта 2012

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

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

   +146

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

   if($this->unpriced){
               //...............  строк 30 кода
               if($this->unpriced){
                     //................ строк 10 кода
               }                
   }

   Вот такую забавную проверку нашол в старом коде. Видать, для уверенности, или скорее всего логика менялась.

   Edd, 12 Марта 2012

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

7. Objective C / Говнокод #9655

   −83

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

   -(void)makeItemavAilableWithName:(NSString*) name{
       [[NSUserDefaults standardUserDefaults] setValue:@"YES" forKey:name];
   }
   
   -(BOOL)isItemAvaliableWithName:(NSString*) name{
       //item could be a chracter or a world
       return [[[NSUserDefaults standardUserDefaults] valueForKey:name]boolValue];
   }

   Вот так вот мы сохраняем булевые переменные в UserDefaults. Оригинальная орфография и пунктуация сохранены

   SevInf, 12 Марта 2012

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

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

   +158

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

   $uri = $_SERVER["REQUEST_URI"];
   
   // получаем урл и выбираем id
   preg_match('![?|&]p=(.*)$!',$uri,$r);
   @$_GET['p'] = $r[1];

   DarkThinker, 12 Марта 2012

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

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

   +125

   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

   #include <stdio.h>
   
   #define MAXLINE 1000 /* максимальная длина строки в потоке */
   
   int getline(char line[], int maxline);
   
   main()
   {
       int len; /* длина текущей строки */
       char line[MAXLINE]; /* текущая введённая строка */
   
       while((len = getline(line, MAXLINE)) != (-1))
           printf("%s", line);
           
       system("PAUSE");
       return 0;
   }
   
   /* getline: считывает строку в s, удаляет пробелы и знаки табуляции в конце строки, возвращает её длину. Удаляет полностью пустые строки */
   getline(char s[], int lim)
   {
                int c, i;
                
                for(i = 0; i < lim - 1 && (c = getchar()) != EOF && c != '\n'; ++i)
                      s[i] = c;
                if (c == EOF)
                   return (-1); /* конец ввода */
                else if (c == '\n') {
                     while(s[i-1] == ' ' || s[i-1] == '\t') /* "удаление" пробелов и знаков табуляции в конце строки */
                        --i;
                     if (i > 0) { /* если строка непустая */
                        s[i] = '\n';
                        ++i;
                     }
                     s[i] = '\0';
                     return i;
                }
   }

   Задача: убирать в конце каждой строки лишние пробелы и знаки табуляции, удалять полностью пустые строки.
   Решение: при считывании строки вида "abc_____\n" на выходе получаем массив символов "abc\n\0___" (пробелы в конце). Печатаем строки по мере поступления, профит.
   
   Говнокод или нормально?

   justanothernickname, 12 Марта 2012

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

10. C# / Говнокод #9650

    +961

    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

    private class LinkedAction
    {
        public Action<Action<bool>> Action { get; set; }
    
        public LinkedAction Next { get; set; }
    
        public void Execute()
        {
            Action(s =>
                {
                    if (s && Next != null) Next.Execute();
                });
        }
    }
    
    private static LinkedAction NewLinkedAction(LinkedAction link, Action<Action<bool>> action)
    {
        return new LinkedAction
        {
            Action = action,
            Next = link
        };
    }
    
    var action = GetCopytoTestActions(parameter)
        .Reverse()
        .Aggregate<Action<Action<bool>>, LinkedAction>(null, NewLinkedAction);
    action.Execute();

    abatishchev, 12 Марта 2012

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

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

• 2105
• …
• 930
• 929
• 928
• 927
• 926
• 925
• 924
• 923
• 922
• …
• 1