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

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

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

1. PHP (5714)
2. C++ (2747)
3. Куча (2428)
4. JavaScript (2035)
5. C# (1931)
6. Java (1503)
7. Си (1123)
8. Pascal (649)
9. Python (594)
10. 1C (541)
11. SQL (433)
12. ActionScript (292)
13. Objective C (202)
14. bash (202)
15. Perl (194)
16. Assembler (148)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (48)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

1. C# / Говнокод #10018

   +110

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

   int[] k = new int[docArr.Length];
   int index = 0;
   
   foreach (var item in docArr)
   {
         k[index] = (int)item;
         index++;
   }
   
   var l = k.ToArray().OrderByDescending(i => i);

   True сортировка массива.

   partizan, 23 Апреля 2012

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

2. C# / Говнокод #10006

   +105

   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

   public static bool IpCheck(String ip)
           {
               String [] octs = ip.Split(new char[] { '.' });
   
               try
               {
   
                   for (int n = 0; n < 4; n++)
                   {
                       byte b = Convert.ToByte(octs[n]);
                   }
   
                   return true;
               }
               catch (Exception)
               {
                   return false;
               }
           }

   Индуский валидатор IP - адреса

   dens, 21 Апреля 2012

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

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

   +958

   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

   public int RomeToArab (string str)
   {
   	int[] arabian = new int[13] { 1000, 900, 500, 400, 100, 90, 50, 40, 10, 9, 5, 4, 1 };
   	strint[] rome = new string[13] { "M", "CM", "D", "CD", "C", "XC", "L", "XL", "X", "IX", "V", "IV", "I" };
   	int n = 0;
   	int i = 0;
   	do
   	{
   		if (str.StartsWith(rome[i]))
   		{
   			n = n + arabian[i];
   			str = str.Substring(rome[i].Length, str.Length - rome[i].Length);
   		}
   		if (!str.StartsWith(rome[i])
   			i++;
   	}
   	while (i < 13);
   	return n;
   }

   aleksi, 20 Апреля 2012

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

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

   +953

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

   string[] phoneNumbers = new string[4];
   phoneNumbers[0] = (!string.IsNullOrEmpty(dirtyData.Телефон1)) ? dirtyData.Телефон1 : "";
   phoneNumbers[1] = (!string.IsNullOrEmpty(dirtyData.Телефон2)) ? dirtyData.Телефон2 : "";
   phoneNumbers[2] = (!string.IsNullOrEmpty(dirtyData.Телефон3)) ? dirtyData.Телефон3 : "";
   phoneNumbers[3] = (!string.IsNullOrEmpty(dirtyData.Телефон4)) ? dirtyData.Телефон4 : "";

   Данные хранятся в MSSQL, Половина таблиц по русски, а в некоторых таблиц поля написаны по русско-английски. Я считаю это плохим тоном. А ведь мне с этим работать в дальнейшем :(((

   nolka4, 20 Апреля 2012

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

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

   +947

   1. 1
   2. 2
   3. 3

   xg = (int)Math.Round(40 + ye[0] * 360 / 12);
                   yg = (int)Math.Round(340 - ye[1] * (decimal)2.5);
                   cur = new PointF((float)xg, (float)yg);

   Отрисовка графика.

   cgbs, 20 Апреля 2012

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

6. C# / Говнокод #9978

   +110

   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

   this.pictureBox14.BeginInit();
         this.pictureBox15.BeginInit();
         this.pictureBox15.BeginInit();
         this.pictureBox21.BeginInit();
         this.pictureBox21.BeginInit();
         this.pictureBox22.BeginInit();
         this.pictureBox22.BeginInit();
         this.pictureBox23.BeginInit();
         this.pictureBox23.BeginInit();
         this.pictureBox24.BeginInit();
         this.pictureBox24.BeginInit();
         this.pictureBox16.BeginInit();
         this.pictureBox16.BeginInit();
         this.pictureBox17.BeginInit();
         this.pictureBox17.BeginInit();
         this.pictureBox18.BeginInit();
         this.pictureBox18.BeginInit();
         this.pictureBox19.BeginInit();
         this.pictureBox19.BeginInit();
         this.pictureBox20.BeginInit();
         this.pictureBox20.BeginInit();
         this.pictureBox9.BeginInit();
         this.pictureBox9.BeginInit();
         this.pictureBox6.BeginInit();
         this.pictureBox6.BeginInit();
         this.pictureBox5.BeginInit();
         this.pictureBox5.BeginInit();
         this.pictureBox7.BeginInit();
         this.pictureBox7.BeginInit();
         this.pictureBox10.BeginInit();
         this.pictureBox10.BeginInit();
         this.panel1.SuspendLayout();
         this.panel1.SuspendLayout();
         this.SuspendLayout();
         this.SuspendLayout();
         this.panel3.Location = new Point(140, 36);
         this.panel3.Location = new Point(140, 36);
         this.panel3.Name = "panel3";
         this.panel3.Name = "panel3";
         this.panel3.Size = new Size(740, 436);
         this.panel3.Size = new Size(740, 436);
         this.panel3.TabIndex = 4;
         this.panel3.TabIndex = 4;
         this.panel3.MouseDown += new MouseEventHandler(this.panel3_MouseDown);
         this.panel3.MouseDown += new MouseEventHandler(this.panel3_MouseDown);
         this.panel3.MouseUp += new MouseEventHandler(this.panel3_MouseUp);
         this.panel3.MouseUp += new MouseEventHandler(this.panel3_MouseUp);
         this.pictureBox2.BackColor = Color.Gray;
         this.pictureBox2.BackColor = Color.Gray;
         this.pictureBox2.Location = new Point(-11, 0);
         this.pictureBox2.Location = new Point(-11, 0);
         this.pictureBox2.Name = "pictureBox2";
         this.pictureBox2.Name = "pictureBox2";
         this.pictureBox2.Size = new Size(938, 36);
         this.pictureBox2.Size = new Size(938, 36);
         this.pictureBox2.TabIndex = 1;
         this.pictureBox2.TabIndex = 1;
         this.pictureBox2.TabStop = false;
         this.pictureBox2.TabStop = false;
         this.pictureBox3.BackColor = Color.Gray;
         this.pictureBox3.BackColor = Color.Gray;
         this.pictureBox3.Location = new Point(880, 12);
         this.pictureBox3.Location = new Point(880, 12);
         this.pictureBox3.Name = "pictureBox3";
         this.pictureBox3.Name = "pictureBox3";
         this.pictureBox3.Size = new Size(36, 466);
         this.pictureBox3.Size = new Size(36, 466);
         this.pictureBox3.TabIndex = 0;
         this.pictureBox3.TabIndex = 0;
         this.pictureBox3.TabStop = false;
         this.pictureBox3.TabStop = false;
         this.pictureBox1.BackColor = Color.Gray;
         this.pictureBox1.BackColor = Color.Gray;
         this.pictureBox1.Location = new Point(-4, 472);
         this.pictureBox1.Location = new Point(-4, 472);
         this.pictureBox1.Name = "pictureBox1";
         this.pictureBox1.Name = "pictureBox1";
         this.pictureBox1.Size = new Size(938, 36);
         this.pictureBox1.Size = new Size(938, 36);
         this.pictureBox1.TabIndex = 0;

   Полная версия: http://pastebin.com/KwDzLR4s

   P4R4, 18 Апреля 2012

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

7. C# / Говнокод #9976

   +111

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

   private IEnumerable<Tuple<IList<string>, IList<string>>> GetFieldInfo(IField Field)
   {
   		foreach (var val in Field.Fields["info"].Values)
   		{
   				yield return new Tuple<IList<string>, IList<string>>(val["Id"].Values, val["ip"].Values);
   		}
   }

   beardeddev, 18 Апреля 2012

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

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

   +110

   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

   public  Operation.OperationStatus   OperState
           {
               get { return fOperState; }
               set
               {
                   fOperState  =   value;
                   // Надписи кнопок смены статуса
                   TOperation.TOperationState  
                       OperationState  =   TOperation.OperationState(value);
                   btnOldState.Text    =   OperationState.OldName;
                   btnNewState.Text    =   OperationState.NewName;
                   // Видимость кнопок смены статуса
                   switch (value)
                   {
                       case Operation.OperationStatus.New          :
                           btnOldState.Visible =   OperationState.OldName.Length > 0 && !ReadOnlyMode;
                           btnNewState.Visible =   OperationState.NewName.Length > 0 && OwnerOper.AccessInput && !ReadOnlyMode;
                           break;
   
                       case Operation.OperationStatus.DataInput    :
                           btnOldState.Visible =   OperationState.OldName.Length > 0 && OwnerOper.AccessInput && !ReadOnlyMode;
                           btnNewState.Visible =   OperationState.NewName.Length > 0 && OwnerOper.AccessCheck && !ReadOnlyMode;
                           break;
   
                       case Operation.OperationStatus.DataCheck    :
                           btnOldState.Visible =   OperationState.OldName.Length > 0 && OwnerOper.AccessCheck && !ReadOnlyMode;
                           btnNewState.Visible =   OperationState.NewName.Length > 0 && OwnerOper.AccessConfirm && !ReadOnlyMode;
                           break;
   
                       case Operation.OperationStatus.Closed       :
                           btnOldState.Visible =   OperationState.OldName.Length > 0 && OwnerOper.AccessConfirm && !ReadOnlyMode;
                           btnNewState.Visible =   OperationState.NewName.Length > 0 && !ReadOnlyMode;
                           break;
                   }
               }
           }
   public  static  TOperationState OperationState(Operation.OperationStatus pState)
           {
               TOperationState Result  =   new TOperationState();
               switch (pState)
               {
                   case Operation.OperationStatus.New          :
                       Result.OldName  =   "";
                       Result.NewName  =   "Данные введены";
                       break;
                   case Operation.OperationStatus.DataInput    :
                       Result.OldName  =   "Отменить \"Данные введены\"";
                       Result.NewName  =   "Данные проверены";
                       break;
                   case Operation.OperationStatus.DataCheck    :
                       Result.OldName  =   "Отменить \"Данные проверены\"";
                       Result.NewName  =   "Завершена";
                       break;
                   case Operation.OperationStatus.Closed       :
                       Result.OldName  =   "Отменить \"Завершена\"";
                       Result.NewName  =   "";
                       break;
               }
               return Result;
           }

   Не прям штобы очень, но шоколад есть...

   gcoder, 18 Апреля 2012

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

9. C# / Говнокод #9973

   +113

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

   StringBuilder script = new StringBuilder();
   script.Append("<script type=text/javascript>alert('Данные не сохранены');</");
   script.Append("script>");
   
   Page.ClientScript.RegisterStartupScript(this.GetType(), "catchScript", script.ToString());

   Поддерживаю проект от других разработчиков.
   Если честно казалось бы придраться не к чему, все будет работать - но почему нужно было делать такой разрыв строки и посреди тега? Это меня в ступор вгоняет. Совпадений по проекту 83 таких.

   sergey_sh, 18 Апреля 2012

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

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

    +113

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

    for (int i = 0; i < retries; i++)
    {
        if ((GetProductVersioningProvider() as IQueueProductVersion).IsVersionCommitted(version))
        {
            return true;
        }
        else
        {
            Thread.Sleep(checkDelay);
        }
    }

    Классика жанра же.

    abatishchev, 17 Апреля 2012

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

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

• 194
• …
• 96
• 95
• 94
• 93
• 92
• 91
• 90
• 89
• 88
• …
• 1