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

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

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

1. PHP (5712)
2. C++ (2746)
3. Куча (2407)
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 (46)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

1. Лучший говнокод

   В номинации:

   • Лучший рейтинг
   • Самый комментируемый

   За время:

   • за сегодня
   • за неделю
   • за месяц
   • за всё время

2. Pascal / Говнокод #3163

   +98

   1. 1

   dmauto.tout.FieldByName('nomer').AsString:=       inttostr(inc(strtoint(dmauto.tout.FieldByName('nomer').AsString)));

   max4d, 05 Мая 2010

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

3. VisualBasic / Говнокод #3090

   −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
   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

   ''' <summary>
   	''' Функция взятия запроса как инструкции t/sql; рекордсет возвращается в виде oZap(,).
   	''' </summary>
   	''' <param name="eConnectString">Строка подключения к БД из списка.</param>
   	''' <param name="sText">Инструкция t/sql.</param>
   	''' <param name="oErr">Необязательная ошибка.</param>
   	''' <param name="oZap">Необязательный двумерный массив, содержащий рекордсет ― 1-ю таблицу датасета.</param>
   	''' <param name="iTimeout">Необязательное число секунд таймаута запроса.</param>
   	''' <returns>Число затронутых строк в таблице БД; если ноль, то нормально ещё, а если -1, то запрос не прошёл, если -2, то соединение не установилось.</returns>
   	''' <remarks>
   	''' </remarks>
   	''' <history>
   	''' 	[Max Pro]	11.04.2007	Created
   	''' 	[Max Pro]	08.08.2008	Реабилитировал OleDb.
   	''' 	[Max Pro]	08.04.2009	Оптимизировал как надо ReDim Preserve и переполнение Integer.
   	''' 	[Max Pro]	27.08.2009	Исправил ошибку ReDim Preserve ― было больше на 1 элемент.
   	''' </history>
   	''' -----------------------------------------------------------------------------
   	Public Function iZap(ByVal eConnectString As ConnectString, ByVal sText As String, Optional ByRef oErr As Exception = Nothing, Optional ByRef oZap(,) As Object = Nothing, Optional ByVal iTimeout As Integer = -1) As Integer
   		'#region "var"
   		Dim iX As Integer
   		Dim lY As Long
   		Dim lYTemp As Long
   		Dim i As Integer
   		Dim iZapTemp As Integer
   		'var zap
   		Dim sConnectString As String = sConStrFromEnum(eConnectString)
   		Dim oConnection
   		Dim oCommand
   		Dim oReader
   		'#end region 'var
   		'#region ini
   		If sConnectString Like "*sqloledb*" Then
   			oConnection = New OleDbConnection
   			oCommand = New OleDbCommand
   		Else
   			oConnection = New SqlConnection
   			oCommand = New SqlCommand
   		End If
   		'#end region 'ini
   		iZap = -2
   		oErr = New Exception
   		oConnection.ConnectionString = sConnectString
   		Try
   			'попытка открытия соединения
   			oConnection.Open()
   			If oConnection.State Then iZap = -1 'if <>0
   			'присвоение параметров команды
   			If iTimeout >= 0 Then oCommand.CommandTimeout = iTimeout
   			oCommand.CommandType = CommandType.Text
   			oCommand.Connection = oConnection
   			oCommand.CommandText = sText
   			'попытка выполнения команды
   			oReader = oCommand.ExecuteReader()
   			iZap = 0
   			iX = oReader.FieldCount - 1
   			lY = -1
   			lYTemp = 255
   			ReDim oZap(iX, lYTemp)
   			While oReader.Read()
   				lY += 1
   				If lY < Integer.MaxValue Then				 '2 147 483 647-1(заменил <= на < чтобы число элементов iZap не зашкалило).
   					iZapTemp = lY + 1
   					If lY > lYTemp Then
   						lYTemp += 256
   						ReDim Preserve oZap(iX, lYTemp)
   					End If
   					For i = 0 To iX
   						If IsDBNull(oReader(i)) Then oZap(i, lY) = Nothing Else oZap(i, lY) = oReader(i)
   					Next i
   				End If
   			End While
   			oReader.Close()
   			oReader = Nothing
   			oCommand = Nothing
   			iZap = iZapTemp
   			ReDim Preserve oZap(iX, iZapTemp - 1)
   			If lY >= Integer.MaxValue Then
   				oErr = New Exception(lY.ToString("# ##0") & " строк >=" & Integer.MaxValue & " (Integer.MaxValue), но массив заполнен до предела Int32 и iZap=Integer.MaxValue.")
   			End If
   		Catch oError As Exception
   			oErr = oError
   		Finally
   			If IsNothing(oConnection) = False Then
   				oConnection.Close()
   				oConnection = Nothing
   			End If
   		End Try
   	End Function	  'iZap

   А так Максим Прохоров выполняет запросы. слабо?

   ursus, 23 Апреля 2010

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

4. C++ / Говнокод #3071

   +908

   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

   template <typename Handler>
     void start_connect_op(socket_type descriptor,
         per_descriptor_data&, Handler handler)
     {
       boost::asio::detail::mutex::scoped_lock lock(mutex_);
       if (!shutdown_)
       {
         boost::shared_ptr<bool> completed(new bool(false));
         connect_handler_wrapper<Handler> wrapped_handler(
             descriptor, completed, *this, handler);
         bool interrupt = write_op_queue_.enqueue_operation(
             descriptor, wrapped_handler);
         interrupt = except_op_queue_.enqueue_operation(
             descriptor, wrapped_handler) || interrupt;
         if (interrupt)
           interrupter_.interrupt();
       }
     }

   Как дико зафрагментировать кучу?!
   Столько хороших идей и так много говнокода в реализации.
   //boost::shared_ptr<bool> completed(new bool(false)); - это один из шедевров кривокодирования!!!

   mock, 22 Апреля 2010

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

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

   +84

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

   public void setPhones(List<PhoneNumber> phones) throws Exception {
       this.phones = phones;
       this.phones.clear();
       this.phones.addAll(phones);
   }

   Вот такой метод нашёл сегодня в коде.

   asolntsev, 21 Апреля 2010

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

6. C++ / Говнокод #3055

   +57.2

   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

   t1->addAnimation(new QPropertyAnimation(widget, "geometry"));
       t1->addAnimation(new QPropertyAnimation(p1, "pos"));
       t1->addAnimation(new QPropertyAnimation(p2, "pos"));
       t1->addAnimation(new QPropertyAnimation(p3, "pos"));
       t1->addAnimation(new QPropertyAnimation(p4, "pos"));
       t1->addAnimation(new QPropertyAnimation(p5, "pos"));
       t1->addAnimation(new QPropertyAnimation(p6, "pos"));
       t1->addAnimation(new QPropertyAnimation(p1, "rotation"));
       t1->addAnimation(new QPropertyAnimation(p2, "rotation"));
       t1->addAnimation(new QPropertyAnimation(p3, "rotation"));
       t1->addAnimation(new QPropertyAnimation(p4, "rotation"));
       t1->addAnimation(new QPropertyAnimation(p5, "rotation"));
       t1->addAnimation(new QPropertyAnimation(p6, "rotation"));
       t1->addAnimation(new QPropertyAnimation(p1, "opacity"));
       t1->addAnimation(new QPropertyAnimation(p2, "opacity"));
       t1->addAnimation(new QPropertyAnimation(p3, "opacity"));
       t1->addAnimation(new QPropertyAnimation(p4, "opacity"));
       t1->addAnimation(new QPropertyAnimation(p5, "opacity"));
       t1->addAnimation(new QPropertyAnimation(p6, "opacity"));
   
       QAbstractTransition *t2 = state2->addTransition(button, SIGNAL(clicked()), state3);
       t2->addAnimation(new QPropertyAnimation(box, "geometry"));
       t2->addAnimation(new QPropertyAnimation(widget, "geometry"));
       t2->addAnimation(new QPropertyAnimation(p1, "pos"));
       t2->addAnimation(new QPropertyAnimation(p2, "pos"));
       t2->addAnimation(new QPropertyAnimation(p3, "pos"));
       t2->addAnimation(new QPropertyAnimation(p4, "pos"));
       t2->addAnimation(new QPropertyAnimation(p5, "pos"));
       t2->addAnimation(new QPropertyAnimation(p6, "pos"));
       t2->addAnimation(new QPropertyAnimation(p1, "rotation"));
       t2->addAnimation(new QPropertyAnimation(p2, "rotation"));
       t2->addAnimation(new QPropertyAnimation(p3, "rotation"));
       t2->addAnimation(new QPropertyAnimation(p4, "rotation"));
       t2->addAnimation(new QPropertyAnimation(p5, "rotation"));
       t2->addAnimation(new QPropertyAnimation(p6, "rotation"));
       t2->addAnimation(new QPropertyAnimation(p1, "opacity"));
       t2->addAnimation(new QPropertyAnimation(p2, "opacity"));
       t2->addAnimation(new QPropertyAnimation(p3, "opacity"));
       t2->addAnimation(new QPropertyAnimation(p4, "opacity"));
       t2->addAnimation(new QPropertyAnimation(p5, "opacity"));
       t2->addAnimation(new QPropertyAnimation(p6, "opacity"));
   
       QAbstractTransition *t3 = state3->addTransition(button, SIGNAL(clicked()), state1);
       t3->addAnimation(new QPropertyAnimation(box, "geometry"));
       t3->addAnimation(new QPropertyAnimation(widget, "geometry"));
       t3->addAnimation(new QPropertyAnimation(p1, "pos"));
       t3->addAnimation(new QPropertyAnimation(p2, "pos"));
       t3->addAnimation(new QPropertyAnimation(p3, "pos"));
       t3->addAnimation(new QPropertyAnimation(p4, "pos"));
       t3->addAnimation(new QPropertyAnimation(p5, "pos"));
       t3->addAnimation(new QPropertyAnimation(p6, "pos"));
       t3->addAnimation(new QPropertyAnimation(p1, "rotation"));
       t3->addAnimation(new QPropertyAnimation(p2, "rotation"));
       t3->addAnimation(new QPropertyAnimation(p3, "rotation"));
       t3->addAnimation(new QPropertyAnimation(p4, "rotation"));
       t3->addAnimation(new QPropertyAnimation(p5, "rotation"));
       t3->addAnimation(new QPropertyAnimation(p6, "rotation"));
       t3->addAnimation(new QPropertyAnimation(p1, "opacity"));
       t3->addAnimation(new QPropertyAnimation(p2, "opacity"));
       t3->addAnimation(new QPropertyAnimation(p3, "opacity"));
       t3->addAnimation(new QPropertyAnimation(p4, "opacity"));
       t3->addAnimation(new QPropertyAnimation(p5, "opacity"));
       t3->addAnimation(new QPropertyAnimation(p6, "opacity"));

   http://qt.gitorious.org/qt/qt/blobs/4.7/examples/animation/states/main.cpp
   
   Пример использования Qt animation framework: образцовый быдлокод.

   Lockal, 19 Апреля 2010

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

7. PHP / Говнокод #3027

   +143.8

   1. 1

   foreach ($invoises as $invoiceId => $invoice) {

   Вот так и работаем с инвойсами...

   DiSH, 16 Апреля 2010

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

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

   +126

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

   try
   {
        ....
        com.ExecuteNonQuery();
   }
   catch (Exception ex) { }
           finally
           {
               con.Close();
     }

   ASP.NET
   Вот так вот обрабатываем ошибки, за которые получаем ПИЗДЫ.
   Интересно а откуда ж нам об них узнатьтаа :)))

   otvet_popravkodon, 13 Апреля 2010

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

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

   +121.8

   1. 1

   int lvl = Convert.ToInt32(reader[0].ToString());

   ASP.NET
   Из текста это получается лучше сделать
   reader = SqlDataReader

   otvet_popravkodon, 13 Апреля 2010

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

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

    +120

    1. 1
    2. 2

    foreach(char Enter in Environment.NewLine)
                  this.Add(Enter);

    Говногость, 10 Апреля 2010

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

11. PHP / Говнокод #2976

    +146.4

    1. 1

    $where = $this->getAdapter()->quoteInto('idUser=?', (int)$this->idUser);

    надёжная защита от инъекций )))

    paranoid, 09 Апреля 2010

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

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

• 1979
• …
• 896
• 895
• 894
• 893
• 892
• 891
• 890
• 889
• 888
• …
• 1