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

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

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

1. PHP (5713)
2. C++ (2746)
3. Куча (2415)
4. JavaScript (2035)
5. C# (1930)
6. Java (1503)
7. Си (1122)
8. Pascal (648)
9. Python (592)
10. 1C (541)
11. SQL (433)
12. ActionScript (292)
13. Objective C (202)
14. bash (201)
15. Perl (194)
16. Assembler (148)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (47)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

1. Java / Говнокод #9665

   +70

   1. 1

   FORMAT.format(Calendar.getInstance().getTime())

   Ну в календаре-то явно точнее время.

   roman-kashitsyn, 13 Марта 2012

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

2. Java / Говнокод #9664

   +74

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

   private int getDSR(ViolationCache violation){
       int dsr = 0;
       for (StandardViolationCache standardViolation : violation.getStandardViolations()) {    	
           dsr = Integer.valueOf(standardViolation.getOrigPointAssignment()) > dsr ? Integer.valueOf(standardViolation.getOrigPointAssignment()) : dsr;
       }
       return dsr;
   }

   Изящненько...

   roman-kashitsyn, 13 Марта 2012

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

3. Java / Говнокод #9662

   +73

   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

   public Show extractDefault(Extract request) {
           CriteriaBuilder criteriaBuilder = converter.convertToDO(request);
           List<EntityDO> list = dao.extract(criteriaBuilder, 0, Byte.MAX_VALUE);
           if (list.size() != 8) {
               if (list.size() != 0) {
                   throw new AssertionError(list.size());
               }
               criteriaBuilder = createCriteriaBuilder();
               list = dao.extract(criteriaBuilder, 0, 100);
           }
           if (list.size() != 8) {
               throw new IllegalStateException("Entity not found");
           }
           return converter.convertToSystem(list);
       }

   Решил запостить код, который дергает сущность из базы как явный говнокод, но после повторного просмотра уже и не кажется таким уж ужасным. Разве что в extract передаются разные значения pagesize(3-ий аргумент) и list.size() сравнивается не понятно. А что скажет общественность?

   jericho, 13 Марта 2012

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

4. Куча / Говнокод #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)

5. Си / Говнокод #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)

6. 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)

7. 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)

8. 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)

9. 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)

10. 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)

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

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