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

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

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

1. PHP (5703)
2. C++ (2737)
3. Куча (2343)
4. JavaScript (2028)
5. C# (1928)
6. Java (1502)
7. Си (1108)
8. Pascal (647)
9. Python (583)
10. 1C (535)
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 (30)
21. Swift (27)
22. Haskell (26)
23. Kotlin (14)

1. C++ / Говнокод #6097

   +170

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

   transaction::~transaction()
     {
       if (db_) {
         int rc = db_->execute(fcommit_ ? "COMMIT" : "ROLLBACK");
         if (rc != SQLITE_OK)
           throw database_error(*db_);
       }
     }

   (c) http://code.google.com/p/sqlite3pp/source/browse/trunk/sqlite3pp.cpp#486
   
   пожалуй, здесь нехватает картинки в стиле Nichtlustig с подписью "лемминг делает throw в деструкторе"

   iddqd, 27 Марта 2011

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

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

   +152

   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

   BOOL NAKED CompareCurrentProcess(PCHAR pszName)
   {
       /*
           1. Found PEB
           2. Found _RTL_USER_PROCESS_PARAMETERS
           3. Found _RTL_USER_PROCESS_PARAMETERS->ImagePathName
           4. Convert UNICODE to ANSI
           5. compare strings
       */
       __asm {
           MOV        ESI,ESP                            // SAVE STACK PTR
   
           MOV        EAX,DWORD PTR FS:[0x30]            // eax == PEB
           MOV        EAX,DWORD PTR DS:[EAX+0x10]        // eax == _RTL_USER_PROCESS_PARAMETERS
           ADD        EAX,0x38                        // eax == _RTL_USER_PROCESS_PARAMETERS->ImagePathName
           
           XOR        EDX,EDX                            //
           XOR        ECX,ECX                            //
           XOR        EDI,EDI                            //
                   
           MOV        CL, BYTE PTR [EAX]                // CL = UNICODE_STRING.Length in bytes
           
           SUB        ESP,ECX                            // reserve in stack CL bytes 
                       
           ADD        EAX,4                            // EAX ptr to WCHAR ImagePathName
           MOV        EAX,DWORD PTR [EAX]                // EAX = PWCHAR
   
   next_char:
           CMP        CL,0                            // WCHAR end
           JZ        end;
   
           MOV        DL,BYTE PTR [EAX]                // DL == *(PCHAR)WCHAR
           ADD        EAX,2                            // GOTO next WIDEchar
           MOV        BYTE PTR [ESP],DL                // SAVE char in memory reserved in stack
           INC        ESP                                // pStr++
           INC        EDI                                // pStrLen++
           SUB        CL,2                            // Length--;
           jmp        next_char                        // goto_next;
   end:
           MOV        BYTE PTR [ESP],0                // *pStr = 0; null terminated
           SUB        ESP,EDI                            // pStr = (PCHAR)(pStr - pStrLen)
           
           XOR        EDX,EDX
           XOR        ECX,ECX
   
           mov        ecx,esp                            // ecx = pStr
           mov        edx,esp                            // edx = pStr
   
           //
           // HERE FOUND image file name 
   m_loop:
           cmp        edi,0
           jz        file_founded
   
           cmp        byte ptr [edx],0x5C                // '\'
           JZ        founded;
           inc        edx
           dec        edi
           jmp        m_loop
   
   founded:
           dec        edi
           mov        ecx,edx
           inc        edx
           jmp        m_loop
   
   file_founded:
           push    esi
           inc        ecx
           push    ecx
           push    dword ptr [esi+4]
           call    my_strcmp
   
           pop        esi
           mov        esp,esi
   
           ret        
       }
   }

   Источник: http://www.wasm.ru/forum/viewtopic.php?id=40652
   Что может быть плохого в функции кроме того, что эту задачу можно было бы решить тонной адекватных вариантов?

   ReL, 23 Марта 2011

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

3. C++ / Говнокод #6068

   +168

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

   bool SomeFuncrion () 
   {
   ...................................
   	if (dbAttrList.size())
   		return true;
   	else
   		return false;
   
       scroll( 0 , 0 );
   }

   А вдруг?

   panter_dsd, 23 Марта 2011

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

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

   +166

   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

   void СSomeClass::InternalRemovePath( QString qsPath, int pUIModule )
   {
     QProgressDialog * pDlg = (QProgressDialog *)pUIModule;
   ...................
   }
   .......................
   void someFunction ()
   {
   ..................
   QProgressDialog * pProgress = NULL;
   pProgress = new QProgressDialog( QObject::tr( "Deleting...") , QObject::tr( "Abort" ), 0, nTotal, NULL );
   InternalRemovePath( qsPath , (int) pProgress );
   ..................
   }

   И по коду таких вещей (приведение указателей к int при передаче в функции) очень много. Пока собрал проект, задолбался.
   Интересно, что автор курил? Хотя, нет - тут уже что-то потяжелее.

   panter_dsd, 23 Марта 2011

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

5. C++ / Говнокод #6062

   +165

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

   #define		DEFAULT_TIMEOUT				30000
   ................................................
   class CSettings //: public QSettings 
   {
   ................................................
     static QVariant             svalue( QString qs, QVariant defValue );
   ................................................
   };
   ................................................
   QString qs = QString("%1").arg( DEFAULT_TIMEOUT );
   m_nTimeout = CSettings::svalue("connection/Timeout", qs.toAscii().data()).toString().toInt();

   panter_dsd, 22 Марта 2011

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

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

   +160

   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

   #include <iostream>
   #include <string>
   using namespace std;
   int main()
   {
   	const int hz_limit = 20000;
   	string str;
   	string str_mass[] = {"a", "b", "ais", "h", "ces", "c", "his", "cis", "des",
   		"d", "dis", "es", "e", "fes", "f", "eis", "fis", "ges", "g", "gis", "as"};
   	begin:
   	while (cin>>str)
   	{
   		for (size_t ix = 0; ix != str.size(); ++ix)
   		{
   			str[ix] = tolower(str[ix]);
   		}
   		size_t f = 0;
   		for (size_t ix = 0; ix != 21; ++ix)
   		{
   			if(str == str_mass[ix])
   			{
   				f = ix;
   			}
   		}
   		if (str != str_mass[f])
   		{
   			goto begin;
   		}
   		double mass[] = {27.500, 29.135, 29.135, 30.868, 30.868, 32.703, 32.703, 34.648, 34.648,
   		36.708, 38.891, 38.891, 41.203, 41.203, 43.654, 43.654, 46.249, 46.249, 48.999, 51.913, 51.913};
   		for (; mass[f] < hz_limit; mass[f] *= 2)
   			{
   				cout<<mass[f]<<" Hz"<<endl;
   			}
   	}
   	return 0;
   }

   Моя первая прога. Выводит частоты на которых находится введенная нота...

   Extrawelt, 21 Марта 2011

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

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

   +159

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

   bool DoYouWannaRealyDoThisFunctionYesNo(T*(f)(T...), bool answer)
   {
   	return answer;
   }

   Microsoft style

   semens, 21 Марта 2011

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

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

   +159

   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

   bool Document::_Parse_Auto (char sym)
   	{
   		bool error = 0;
   
   		switch (mInfo.AutoState)
   		{
   		case 0:
   			{
   				if (is_stag (sym))
   					mInfo.AutoState = 1;
   				else
   					mInfo.AutoState = 14;
   				break;
   			}
   		case 1:
   			{
   				if (is_name (sym))
   					mInfo.AutoState = 24;
   				else if (is_delim (sym))
   					mInfo.AutoState = 2;
   				else if (is_prcom (sym))
   					mInfo.AutoState = 11;
   				else if (is_info (sym))
   					mInfo.AutoState = 25;
   				else if (is_ekey (sym))
   					mInfo.AutoState = 21;
   				else
   					error = true;
   				break;
   			}
   <100500 строк case'ов>
   		case 26:
   			{
   				if (is_etag (sym))
   					mInfo.AutoState = 0;
   				else
   					error = true;
   				break;
   			}
   		}
   
   		return error;
   	}
   
   	void Document::_Parse_React (char sym)
   	{
   		switch (mInfo.AutoState)
   		{
   		case 0:
   			{
   				switch (mInfo.NodeType)
   				{
   				case ParseInfo::preproc:
   					{
   						delete mInfo.CurrentNode;
   						mInfo.CurrentNode = mInfo.LastTextNode;
   						break;
   					}
   				case ParseInfo::info:
   					{
   						delete mInfo.CurrentNode;
   						mInfo.CurrentNode = mInfo.LastTextNode;
   						break;
   					}
   				case ParseInfo::close:
   					{
   						
   						if (mInfo.name == mInfo.CurrentNode->name)
   							mInfo.CurrentNode = mInfo.CurrentNode->parent;
   						else
   						{ mInfo.Error = 51; return; }
   
   						mInfo.name.clear ();
   						break;
   					}
   				}
   				break;
   			}
   <ещё 100500 строк case'ов>
   		case 25:
   			{
   				mInfo.NodeType = ParseInfo::info;
   				mInfo.LastTextNode = mInfo.CurrentNode;
   				mInfo.CurrentNode = new Node;
   				break;
   			}
   		}
   
   		switch (mInfo.LastAutoState)
   		{
   <и ещё 100500 строк case'ов>

   Это я в 10 классе писал XML парсер, работающий на конечном автомате, в котором было 27 состояний.

   YuraTim, 20 Марта 2011

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

9. C++ / Говнокод #6032

   +163

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

   const QColor * c = & QColor.red;
   const QColor * c = & QColor.rgb(12,12,12);
   const QColor * c = & int QColor::red () const;
   const int * c = & int QColor::red () const;

   Взято отсюда: http://www.prog.org.ru/topic_16855_0.html
   Особенно радует фраза: "Пробовал так <....> И другими способами, на которые вылетало множество ошибок".

   TarTar, 19 Марта 2011

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

10. C++ / Говнокод #6027

    +170

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

    void Draw_Pixel_Memset(UINT color,    //color = _RGB32BIT(a,r,g,b) ((b) + ((g) << 8) + ((r) << 16) + ((a) << 24))
                          UINT *buffer){ //buffer =  (UINT*) ddsd.lpSurface
      memset(buffer, color,8); 
    }

    Рисование пикселя с помощью meset()
    
    Подскажите, пожалуйста, почему при использование этой функции в 32 битном режиме точка получается какового то серого цвета.
    С другой стороны если использовать 8 битный режим то пиксель будет нужного цвета.
    
    Источник: http://www.gamedev.ru/code/forum/?id=145274

    TarasB, 19 Марта 2011

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

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

• 274
• …
• 75
• 74
• 73
• 72
• 71
• 70
• 69
• 68
• 67
• …
• 1