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

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• 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. PHP / Говнокод #9983

   +145

   1. 1

   strpos(rtrim($_SERVER['HTTP_REFERER'], '/').'/', Router::GetPath('registration'))

   Мои глаза кровоточат.

   nkarakin, 19 Апреля 2012

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

2. Python / Говнокод #9982

   −92

   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

   def success_operation(operation):
       """помечаем операцию как оплаченную"""
   
       #получаем коллбэк и выполняем его
       if operation.close_callback:
           path, callback_name, params =\
               (lambda v, f: (len(v) > 1 and f(v[0], v[1])) or f(v[0]))(
                       (operation.close_callback or '').split('|'),
                       lambda n, k = None:
                           (lambda n: (lambda l: ['.'.join(l[0:len(l) - 1]) or 'app.billing.callbacks'] + [l[len(l) - 1]])(
                               n.split('.'))
                           )(n) + \
                           [(lambda k: not (k is None) and reduce(
                                   lambda h, t: (lambda a, p: a.setdefault(*p) and a)(
                                       h,
                                       (lambda l: (l[0].strip(), l[1].strip()))(t.split('='))
                                   ),
                                   k.split(','),
                                   {})
                            )(k) or {}])

   Человек правильно живет, но зря.

   tonatos, 19 Апреля 2012

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

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

   +159

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

   foreach ($metrics as $m)
   		{
   		eval('$data["'.$a.'"]["'.$m.'"] = $result->get'.$m.'();');
   		}

   На одном из сайтов который объясняют как работать с гугл аналикс апи.
   Нафига выполнять столько раз евал???
   
   не легче и лучше
   $key = 'get' . $m;
   $data[$a][$m] = $result->{$key}();
   евалить стопицот рас???

   3xdimon, 19 Апреля 2012

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

4. Pascal / Говнокод #9979

   +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

   function ADHasFactory(const AIID: TGUID): Boolean;
   var
     oIntf: IUnknown;
   begin
     try
       ADCreateInterface(AIID, oIntf, False);
       Result := oIntf <> nil;
     except
       Pointer(oIntf) := nil;
       Result := False;
     end;
   end;

   AnyDAC. Проверка наличия фабрики для заданного интерфейса.

   LightBlack, 18 Апреля 2012

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

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

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

   +174

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

   private function FunctionFalse()
   	{
   		return false;
   	}

   битрикс он такой

   minisot, 18 Апреля 2012

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

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. JavaScript / Говнокод #9975

   +155

   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

   var d = new Date();
       dom = d.getDate();
       month = d.getMonth();
       year = d.getYear();
       if (year < 2000) year = year - 100;
       else year = year - 2000;
   
       if (year == 12 && month == 3)
           days = 26 - dom;
       else 
           days = 0;
       if (days < 0) days = 0;
       if (days < 10) days = '0' + days.toString();
   
   
   var base = 'http://ubuntu.ru/media/countdown/1204_1/';
   
   document.write('<a href="http://ubuntu.ru/"><img id="countdownimage" src="'+base+days+'.png" width="180" height="150" border="0" alt="Ubuntu 12.04 LTS" title="Ubuntu 12.04 LTS - Уже скоро!"></a>');
   //document.write('<a href="http://ubuntu.ru/get"><img id="countdownimage" src="'+base+'here.png" width="180" height="150" border="0" alt="Ubuntu 12.04 LTS" title="Ubuntu 12.04 LTS - Уже здесь!"></a>');

   http://ubuntu.ru/media/countdown/display.js

   хуита, 18 Апреля 2012

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

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

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

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

• 2105
• …
• 959
• 958
• 957
• 956
• 955
• 954
• 953
• 952
• 951
• …
• 1