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

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

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

1. PHP (5708)
2. C++ (2740)
3. Куча (2382)
4. JavaScript (2033)
5. C# (1929)
6. Java (1502)
7. Си (1117)
8. Pascal (648)
9. Python (591)
10. 1C (539)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (200)
15. Perl (194)
16. Assembler (146)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (34)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

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

   В номинации:

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

   За время:

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

2. C++ / Говнокод #17208

   +56

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

   std::uint8_t octets[4];
   for(int i = 0; i < 4; ++i) {
       if(i != 0 && inp.get() != '.')
           { assert(false && "unexpected symbol"); }
       inp >> octets[i];
   }

   Два вида говна: в коде и в языке.

   Soul_re@ver, 28 Ноября 2014

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

3. Си / Говнокод #17179

   +133

   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

   if (*wpPat == L'\\')
       {
         wpCharStart=wpPat;
         if (++wpPat >= wpMaxPat) goto Error;
   
         if (*wpPat == L'x')
         {
           if (++wpPat >= wpMaxPat) goto Error;
   
           if (*wpPat == L'{')
           {
             wpStrTmp=++wpPat;
             for (;;)
             {
               if (wpPat >= wpMaxPat) goto Error;
               if (*wpPat++ == L'}') break;
             }
             if (lpREGroupItem->nGroupLen != -1 && !bClassOpen)
             {
               nPatChar=(int)hex2decW(wpStrTmp, (wpPat - 1) - wpStrTmp);
               if (nPatChar == -1)
               {
                 wpPat=wpStrTmp;
                 goto Error;
               }
               if (nPatChar <= MAXWORD)
                 lpREGroupItem->nGroupLen+=1;
               else
                 lpREGroupItem->nGroupLen+=2;
             }
           }
           else
           {
             if (wpPat + 2 > wpMaxPat)
               goto Error;
             wpPat+=2;
             if (lpREGroupItem->nGroupLen != -1 && !bClassOpen)
               ++lpREGroupItem->nGroupLen;
           }
         }
         else if (*wpPat == L'u')
         {
           if (wpPat + 5 > wpMaxPat)
             goto Error;
           wpPat+=5;
           if (lpREGroupItem->nGroupLen != -1 && !bClassOpen)
             ++lpREGroupItem->nGroupLen;
         }

   Регулярные велосипеды.
   Akelpad.

   gost, 25 Ноября 2014

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

4. Си / Говнокод #17172

   +134

   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

   //// AEM_* AkelEdit control messages
   
     if (uMsg >= WM_USER)
     {
       if (uMsg >= AEM_CANUNDO)
         goto CanUndo;
   
       //Text retrieval and modification
       if (uMsg == AEM_EXSETTEXTA)
       {
         AESETTEXTA *at=(AESETTEXTA *)lParam;
   
         return AE_SetTextAnsi(ae, at->nCodePage, at->pText, at->dwTextLen, at->nNewLine);
       }
       if (uMsg == AEM_EXSETTEXTW)
       {
         AESETTEXTW *at=(AESETTEXTW *)lParam;
   
         return AE_SetText(ae, at->pText, at->dwTextLen, at->nNewLine, FALSE);
       }
       if (uMsg == AEM_SETTEXTA)
       {
         return AE_SetTextAnsi(ae, CP_ACP, (char *)lParam, wParam, AELB_ASINPUT);
       }
       if (uMsg == AEM_SETTEXTW)
       {
         return AE_SetText(ae, (wchar_t *)lParam, wParam, AELB_ASINPUT, FALSE);
       }
       if (uMsg == AEM_APPENDTEXTA)
       {
         AEAPPENDTEXTA *at=(AEAPPENDTEXTA *)lParam;
   
         AE_AppendTextAnsi(ae, at->nCodePage, at->pText, at->dwTextLen, at->nNewLine);
         return 0;
       }
       if (uMsg == AEM_APPENDTEXTW)
       {
         AEAPPENDTEXTW *at=(AEAPPENDTEXTW *)lParam;
   
         AE_AppendText(ae, at->pText, at->dwTextLen, at->nNewLine);
         return 0;
       }
       if (uMsg == AEM_REPLACESELA)
       {
         AEREPLACESELA *rs=(AEREPLACESELA *)lParam;
   
         AE_ReplaceSelAnsi(ae, rs->nCodePage, rs->pText, rs->dwTextLen, rs->nNewLine, rs->dwFlags, rs->ciInsertStart, rs->ciInsertEnd);
         return 0;
       }
       if (uMsg == AEM_REPLACESELW)
       {
         AEREPLACESELW *rs=(AEREPLACESELW *)lParam;
   
         AE_ReplaceSel(ae, rs->pText, rs->dwTextLen, rs->nNewLine, rs->dwFlags, rs->ciInsertStart, rs->ciInsertEnd);
         return 0;
       }
       if (uMsg == AEM_GETTEXTRANGEA)
       {
         AETEXTRANGEA *tr=(AETEXTRANGEA *)lParam;
   
         return AE_GetTextRangeAnsi(ae, tr->nCodePage, tr->lpDefaultChar, tr->lpUsedDefChar, &tr->cr.ciMin, &tr->cr.ciMax, tr->pBuffer, tr->dwBufferMax, tr->nNewLine, tr->bColumnSel, tr->bFillSpaces);
       }

   И далее около сотни if'ов в таком духе.
   AkelPad

   gost, 25 Ноября 2014

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

5. Pascal / Говнокод #17165

   +88

   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

   function TCalc.getRewardItems(items : AnsiString; countItems : integer; userId : Integer; from:integer) : ISuperObject;
   var tmpFirstOdds : integer;
       strLines, extrLines : TStringList;
       strItems, strOdds, strOdds2: TIntegerList;
       i, j, tmpCountItems : integer;
       tmpOddsItems, sum : Integer;
       str: ansistring;
   begin
     Result := nil;
   
     // parse items line [id1,odd1],[id2,odd2],[id3,odd3]
     strLines := TStringList.Create;
     extrLines := TStringList.Create;
     strItems := TIntegerList.Create;
     strOdds := TIntegerList.Create;
     ExtractStrings(['[', ']'],[','], PChar(items), strLines);
     for i := 0 to strLines.Count - 1 do
     begin
       ExtractStrings([','],[' '], PChar(strLines[i]), extrLines);
       strItems.Add(StrToInt(extrLines[0]));
       strOdds.Add(StrToInt(extrLines[1]));
       extrLines.Clear;
     end;
   
     // calculate odds [a,b,c,d] as [a,a+b,a+b+c,a+b+c+d]
     sum := 0;
     strOdds2 := TIntegerList.Create;
     for i := 0 to strOdds.Count-1 do
     begin
       sum := sum + strOdds.Items[i];
       strOdds2.Add(sum);
     end;
   
     // select items
     str := '';
     for i := 0 to countItems - 1 do
     begin
       tmpFirstOdds := Random(101); // 0-100
       for j := 0 to strOdds2.Count-1 do
       begin
         if tmpFirstOdds < strOdds2.Items[j] then
         begin
           str := str + IntToStr(strItems[j]) + ',';
           break;
         end;
       end;
     end;
     str := '['+Copy(str,1,Length(str)-1)+']';
     // from monster
     if from=0 then Result := so('{"part":"battle","func":"itemsdropped","data":'+str+'}');
     // from chest
     if from=1 then Result := so('{"part":"battle","func":"itemsdroppedchest","data":'+str+'}');
     strLines.Free;
     extrLines.Free;
     strItems.Free;
     strOdds.Free;
     strOdds2.Free;
   end;

   У меня чуть глаза не вытекли от такого решения. Код по идее сообщает клиенту, что выпало из монстра после победы над ним.

   Cynicrus, 24 Ноября 2014

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

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

   +135

   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

   public void ValidateValue( ref object value ) {
               bool b;
               if ( value == null ) return;
               var typeName = BaseType.Name;
               var valueType = value.GetType();
               if ( valueType.Name == "String" ) {
                   if ( BaseType == valueType ) return;
                   if ( BaseType == Constants.Types.Boolean ) {
                       if ( string.Compare( "yes", (string) value, StringComparison.OrdinalIgnoreCase ) == 0 ) value = true;
                       else if ( string.Compare( "no", (string) value, StringComparison.OrdinalIgnoreCase ) == 0 ) value = false;
   #if !CF
                       else if ( Boolean.TryParse( value.ToString(), out b ) ) value = b;
   #else
             else if (TryParseUtility.TryParse(value.ToString(), out b)) value = b;
   #endif
                       else throw new ArgumentException( String.Format( Resources.ValueNotCorrectType, value ) );
                       return;
                   }
               }
   
   #if !CF
               if ( typeName == "Boolean"
                    && Boolean.TryParse( value.ToString(), out b ) ) {
                   value = b;
                   return;
               }
   
               UInt64 uintVal;
               if ( typeName.StartsWith( "UInt64" )
                    && UInt64.TryParse( value.ToString(), out uintVal ) ) {
                   value = uintVal;
                   return;
               }
   
               UInt32 uintVal32;
               if ( typeName.StartsWith( "UInt32" )
                    && UInt32.TryParse( value.ToString(), out uintVal32 ) ) {
                   value = uintVal32;
                   return;
               }
   
               long intVal;
               if ( typeName.StartsWith( "long" )
                    && long.TryParse( value.ToString(), out intVal ) ) {
                   value = intVal;
                   return;
               }
   
               int intVal32;
               if ( typeName.StartsWith( "Int32" )
                    && Int32.TryParse( value.ToString(), out intVal32 ) ) {
                   value = intVal32;
                   return;
               }
   #else
         if (typeName == "Boolean" && TryParseUtility.TryParse(value.ToString(), out b)) { value = b; return; }
   
         UInt64 uintVal;
         if (typeName.StartsWith("UInt64") && TryParseUtility.TryParse(value.ToString(), out uintVal)) { value = uintVal; return; }
   
         UInt32 uintVal32;
         if (typeName.StartsWith("UInt32") && TryParseUtility.TryParse(value.ToString(), out uintVal32)) { value = uintVal32; return; }
   
         long intVal;
         if (typeName.StartsWith("long") && TryParseUtility.TryParse(value.ToString(), out intVal)) { value = intVal; return; }
   
         Int32 intVal32;
         if (typeName.StartsWith("Int32") && TryParseUtility.TryParse(value.ToString(), out intVal32)) { value = intVal32; return; }
   #endif
   
               object objValue;
   #if RT
         Type baseType = BaseType.GetTypeInfo().BaseType;
   #else
               var baseType = BaseType.BaseType;
   #endif
               if ( baseType != null
                    && baseType.Name == "Enum"
                    && ParseEnum( value.ToString(), out objValue ) ) {
                   value = objValue;
                   return;
               }
   
               throw new ArgumentException( String.Format( Resources.ValueNotCorrectType, value ) );
           }

   MySql.Data 6.9.5, MySqlConnectionStringBuilder.cs

   kasthack, 24 Ноября 2014

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

7. C++ / Говнокод #17158

   +51

   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

   std::list<COptions *> COptions::m_InstanceList;
   ..........
   COptions::COptions()
   {
   	for (int i = 0; i < OPTIONS_NUM; ++i)
   		m_OptionsCache[i].bCached = FALSE;
   	m_pOptionsHelperWindow = new COptionsHelperWindow(this);
   	simple_lock lock(m_mutex);
   #ifdef _DEBUG
   	for (std::list<COptions *>::iterator iter=m_InstanceList.begin(); iter != m_InstanceList.end(); ++iter)
   		ASSERT(*iter != this);
   #endif _DEBUG
   	m_InstanceList.push_back(this);
   	m_SpeedLimits[0] = m_sSpeedLimits[0];
   	m_SpeedLimits[1] = m_sSpeedLimits[1];
   }
   
   COptions::~COptions()
   {
   	{
   		simple_lock lock(m_mutex);
   		std::list<COptions *>::iterator iter;
   		for (iter=m_InstanceList.begin(); iter != m_InstanceList.end(); ++iter) {
   			if (*iter == this)
   				break;
   		}
   
   		ASSERT(iter != m_InstanceList.end());
   		if (iter != m_InstanceList.end())
   			m_InstanceList.erase(iter);
   	}
   
   	if (m_pOptionsHelperWindow)
   		delete m_pOptionsHelperWindow;
   	m_pOptionsHelperWindow=0;
   }
   .......

   В отладке включается тяжелый героиновый бред разработчиков FileZilla

   fsmoke, 24 Ноября 2014

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

8. C++ / Говнокод #17122

   +49

   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

   void ZModulesDlg::OnDestroy(HWND hwnd)
   {
   	LVITEM lvi = { LVIF_PARAM };
   
   	if (lvi.iItem = ListView_GetItemCount(hwnd))
   	{
   		do 
   		{
   			lvi.iItem--;
   			if (ListView_GetItem(hwnd, &lvi))
   			{
   				((ZDll*)lvi.lParam)->Release();
   			}
   		} while (lvi.iItem);
   	}
   }

   zhukas, 17 Ноября 2014

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

9. PHP / Говнокод #17091

   +156

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

   // Real itemid is "contained" in public itemid in a bit tricky way. The formula is:
   
   public_itemid = itemid * 256 + anum
   
   // Quite easy, isn't it? Except where to get that anum. anum is some number assigned when you create a new post in LJ.
   
   // But in fact, you don't need it. Because it is always less than 256, we can avoid searching for it, and just use simple Math to find itemid.
   
   $anum = $public_itemid - floor($public_itemid / 256) * 256;
   $itemid = ($public_itemid - $anum) / 256;

   Прекрасный подход, то место, где вордпресс и ЖЖ сошлись воедино.
   
   http://drumrock.skipitnow.org/livejournal-tools/itemid/

   Fike, 10 Ноября 2014

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

10. Python / Говнокод #17000

    −106

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

    qdev_id, iops = _update_device_iops(instance, device_for_change)
    try:
    	qemu.volumes.set_io_throttle(controller.qemu(), qdev_id, iops)
    except Exception as e:
    	# Check if we turn off this instance? just a moment ago.
    	if "'NoneType' object has no attribute 'connected'" in e:
    		LOG.warning("kemu process seems to be killed")
    	else:
    		raise

    Метод set_io_throttle не бросает exception.
    Мы так проверяем,есть ли connection к qemu или нет.

    gmmephisto, 30 Октября 2014

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

11. Куча / Говнокод #16990

    +126

    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

    .cat_mid a{
    color: #ffa10e !important;
    }
    .cat_mid {
    padding: 0 10px;
    display: inline-block;
    font-family: georgia, "times new roman", times, serif;
    font-weight: normal;
    font-size: 14px;
    height: 64px !important;
    overflow: hidden;
    margin-bottom: -3px !important;
    padding: 10px 10px 0;
    }
    .cat_hid {
    color: #bc843f;
    padding: 0 10px;
    text-align: left; 
    z-index: 100;
    position: relative;
    display: none;
    }
    .cat_hid.not{
    display: block !important;
    padding: 8px 10px 19px;
    }
    .cat_hid.not.price{
    height: 25px;
    padding: 12px 10px 17px;
    }

    css

    UncaughtException, 30 Октября 2014

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

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

• 1973
• …
• 72
• 71
• 70
• 69
• 68
• 67
• 66
• 65
• 64
• …
• 1