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

    +161

    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
    class SomeClass
    {
    public:
    	__declspec(dllexport) SomeClass(UINT i_Width, UINT i_Height, UINT i_Lines, float i_Result, bool i_bAutoFill = false, и т.д. еще штук 5);
    	__declspec(dllexport) SomeClass::~SomeClass();
    
    	__declspec(dllexport) void setCallbackFunction(void	(*i_pCallbackFunction)(SomeClass* i_pSomeClass)) { m_pCallbackFunction = i_pCallbackFunction; };	
    	__declspec(dllexport) bool isFinished() { return m_bIsFinished; };
    	__declspec(dllexport) void clear() { m_ResultData.clear(); };
    	__declspec(dllexport) bool save(const char* i_sFilename);
    
    	...
    
    private:	
    	bool createThread();
    	void initWork();	
    	void loopWork();
    	void stepWork();
    	void exitWork();
    
    	static UINT WINAPI workProc(LPVOID lpContext);
    
    	inline bool someInlineFunction(UINT i_Index);
    	...
    	
    private:
    	HANDLE				m_ThreadHandle;
    	bool				m_bIsFinished;
    	bool				m_bThreadStopped;
    	bool				m_bThreadClosed;
    	vector<SomeType>	m_a...; 
    	string				m_sFilename;
    	
    	void				(*m_pCallbackFunction)(SomeClass* i_pSomeClass);
    	
    	...
    };

    Это краткий пересказ того, что шло вместе с DLL, только длиннее и с комментариями к каждой строке и доксигеновской докой!

    Aleskey, 30 Марта 2011

    Комментарии (6)
  2. C++ / Говнокод #6128

    +161

    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
    /////////////////////////////////////////////////////////////////////////////
    #define TWAIT(_WAIT_EXPR_, _TIMEOUT_)	TWAIT_EX(_WAIT_EXPR_, _TIMEOUT_, 0)
    #define TWAIT_(_WAIT_EXPR_, _TIMEOUT_)	TWAIT_EX_(_WAIT_EXPR_, _TIMEOUT_, 0)
    /////////////////////////////////////////////////////////////////////////////
    #define TWAIT_DECL_VARS														\
    	DWORD TWAIT_START = GetTickCount();										\
    	bool TWAIT_RET = true;													
    /////////////////////////////////////////////////////////////////////////////
    #define TWAIT_EX(_WAIT_EXPR_, _TIMEOUT_, _ABORT_)							\
    	DWORD TWAIT_START = GetTickCount();										\
    	bool TWAIT_RET = true;													\
    	do {ProcessMessage(1);													\
    		TWAIT_RET = GetTickCount() - TWAIT_START < (DWORD)(_TIMEOUT_);		\
    		if( !TWAIT_RET ) break;												\
    		if( _ABORT_ != 0 ) { TWAIT_RET = false; break; }					\
    	} while( (_WAIT_EXPR_) == 0 );
    /////////////////////////////////////////////////////////////////////////////
    #define TWAIT_EX_(_WAIT_EXPR_, _TIMEOUT_, _ABORT_)							\
    	TWAIT_START = GetTickCount();											\
    	TWAIT_RET = true;														\
    	do {ProcessMessage(1);													\
    		TWAIT_RET = GetTickCount() - TWAIT_START < (DWORD)(_TIMEOUT_);		\
    		if( !TWAIT_RET ) break;												\
    		if( _ABORT_ != 0 ) { TWAIT_RET = false; break; }					\
    	} while( (_WAIT_EXPR_) == 0 );

    lambdas, functors... макрос - это наше все... а ProcessMessage(1) тоже радует.

    Aleskey, 30 Марта 2011

    Комментарии (4)
  3. C++ / Говнокод #6126

    +179

    1. 1
    2. 2
    3. 3
    4. 4
    ClassName::~ClassName()
        {
        memset( this, 0, sizeof( *this ) );
        }

    Написано коллегой из теплой страны. Случайно нашел в коде :)

    Mintel, 29 Марта 2011

    Комментарии (8)
  4. C++ / Говнокод #6119

    +174

    1. 1
    (addNewItem) ? (isVideoAudioAttached = true) : (isVideoAudioAttached = false);

    addNewItem и isVideoAudioAttached типа bool

    psl, 29 Марта 2011

    Комментарии (3)
  5. C++ / Говнокод #6115

    +172

    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
    CString convertInt2Str(int i_Number, int i_NumDigits)
    {
    	CString str = "";
    	for(int j=1; j<i_NumDigits; j++)
    	{
    		int digits = (int) pow((float) 10, j);
    		
    		if(i_Number<digits) str += "0";			
    	}	
    	CString num;
    	num.Format("%d", i_Number);
    	return str+num;
    }

    Adding leading zeros...

    Aleskey, 29 Марта 2011

    Комментарии (25)
  6. C++ / Говнокод #6114

    +162

    1. 1
    Speed = ini.GetInt("Speed" , Speed /(1 mm_s )) * (-1 mm_s);

    Скорость задается изначально как положительное число, но в ресчетах используется ее отрицательное значение.

    Aleskey, 29 Марта 2011

    Комментарии (6)
  7. C++ / Говнокод #6108

    +162

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    for (i = 0;i<n;i++)
        {
            for (j = 0;j<m;j++)
            {
                if (j=m)
                {
                    j = 0;
                    break;
                }
         }
    }

    Мастер! Без комментариев.

    P.S. Как это ни странно, автор предпочел остаться неизвестным :)

    DimX-6600, 28 Марта 2011

    Комментарии (6)
  8. 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)
  9. 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)
  10. 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)