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

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

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

1. PHP (5708)
2. C++ (2740)
3. Куча (2389)
4. JavaScript (2034)
5. C# (1930)
6. Java (1503)
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 (148)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (37)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

1. Список говнокодов пользователя ShuraKotov

   Всего: 6

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

   +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

   TCHAR aS[22];
            m_pCurrentLine->mainPortStatistics.TOs = 0;
   	_stprintf(aS, _T("%d"), m_pCurrentLine->mainPortStatistics.TOs);
   	SetDlgItemText(IDC_EDIT_MAIN_TIMEOUTS, aS);
   	m_pCurrentLine->mainPortStatistics.err_CE_MODE = 0; 
   	_stprintf(aS, _T("%d"), m_pCurrentLine->mainPortStatistics.err_CE_MODE);
   	SetDlgItemText(IDC_EDIT_MAIN_MODE, aS);
   	m_pCurrentLine->mainPortStatistics.err_CE_BREAK = 0; 
   	_stprintf(aS, _T("%d"), m_pCurrentLine->mainPortStatistics.err_CE_BREAK);
   	SetDlgItemText(IDC_EDIT_MAIN_BREAK, aS);
   	m_pCurrentLine->mainPortStatistics.err_CE_FRAME = 0; 
   	_stprintf(aS, _T("%d"), m_pCurrentLine->mainPortStatistics.err_CE_FRAME);
   	SetDlgItemText(IDC_EDIT_MAIN_FRAME, aS);
   	m_pCurrentLine->mainPortStatistics.err_CE_RXOVER = 0; 
   	_stprintf(aS, _T("%d"), m_pCurrentLine->mainPortStatistics.err_CE_RXOVER);
   	SetDlgItemText(IDC_EDIT_MAIN_RXOVER, aS);
   	m_pCurrentLine->mainPortStatistics.err_CE_TXFULL = 0; 
   	_stprintf(aS, _T("%d"), m_pCurrentLine->mainPortStatistics.err_CE_TXFULL);
   	SetDlgItemText(IDC_EDIT_MAIN_TXFULL, aS);
   	m_pCurrentLine->mainPortStatistics.err_CE_OVERRUN = 0;
   	_stprintf(aS, _T("%d"), m_pCurrentLine->mainPortStatistics.err_CE_OVERRUN);
   	SetDlgItemText(IDC_EDIT_MAIN_OVERRUN, aS);
   	m_pCurrentLine->mainPortStatistics.err_CE_RXPARITY = 0;
   	_stprintf(aS, _T("%d"), m_pCurrentLine->mainPortStatistics.err_CE_RXPARITY);
   	SetDlgItemText(IDC_EDIT_MAIN_RXPARITY, aS);

   каждому элементу структуры mainPortStatistics присваивается значение 0, затем каждый раз форматируется строка и засылается в функцию SetDlgItemText. Вменяемый человек пользуется для обнуления структуры функцией memset а затем ставит текст "0" во все controls. А говнокодер думает ,что чем больше дерьмокода развести - тем круче

   ShuraKotov, 02 Марта 2011

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

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

   +162

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

   CString sLineNumber = _T("Line");
   	for(int lineIndex = 0; lineIndex < 255; lineIndex++)
   	{
   		sLineNumber += (_itot(lineIndex + 1, temp, 10));
   		...
   		sLineNumber = _T("Line");
   	}

   Build the string variable in loop, instead of use CString::Format(...)

   ShuraKotov, 21 Февраля 2011

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

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

   +146

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

   void CDAMN::ControlDAMNAmount(DWORD newValue)
   {
   	if(newValue != 0)
   		controlDAMNAmount = newValue;
   }
   
   DWORD CDAMN::ControlDAMNAmount()
   {
   	return controlDAMNAmount;
   }

   Автор тот же
   Две функции с одним именем. Одна изменяет внутреннее поле (член класса)- другая возвращает его значение. Использовать приставки get_ и set_ недосуг как-то. Да и называть внутренние переменные класса так, чтобы было понятно, что они не глобальные (с приставкой “m_”) тоже облом. Хайтек-шмайтек…

   ShuraKotov, 14 Февраля 2011

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

5. Си / Говнокод #5663

   +135

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

   //...
   	iInitCommDevicePeriod = GetPrivateProfileInt(extraTunungLine, _T("InitCommDevicePeriod"), INIT_COMM_DEVICE_TIMER_PERIOD / 1000, extraFilePath) * 1000;
   	iReconnectSocketPeriod = GetPrivateProfileInt(extraTunungLine, _T("ReconnectSocketPeriod"), CONN_WITH_SOCK_LOST_TIMER_PERIOD / 1000, extraFilePath) * 1000;
   	m_uiCheckLoopPeriod = GetPrivateProfileInt(extraTunungLine, _T("CheckLoopPeriod"), CCC_LINE_CHECK_LOOP_TIMER_PERIOD / 1000, extraFilePath) * 1000;	
   //....

   Автор тот же...
   мистическая инициализация таймеров.
   зачем делить default value на 1000 перед посылкой в функцию, чтобы потом результат умножить на 1000. Может, чтобы время было в целых секундах?

   ShuraKotov, 13 Февраля 2011

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

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

   +162

   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

   //...
   	CString signature(pMsg->clipEvent.signature);
   	int indx = signature.Find('-');
   	if(!indx)
   		signature.Delete(indx);
   	indx = signature.Find('-');
   	pEvent->eventData.signature.doorNum = _ttoi(signature.Left(indx));
   	signature.Delete(0, indx+1);
   	indx = signature.Find('-');
   	pEvent->eventData.signature.eventType = _ttoi(signature.Left(indx));
   	signature.Delete(0, indx+1);
   	indx = signature.Find('-');
   	pEvent->eventData.signature.eventID = _ttoi(signature.Left(indx));	
   	signature.Delete(0, indx+1);
   	indx = signature.Find('-');
   	pEvent->eventData.signature.itemNumber = _ttoi(signature.Left(indx));	
   	signature.Delete(0, indx+1);
   	indx = signature.Find('-');
   	CString dateTimeStr;
   	dateTimeStr	= signature.Mid(5, 2);
   	dateTimeStr += '/';
   	dateTimeStr += signature.Mid(8, 2);
   	dateTimeStr += '/';
   	dateTimeStr += signature.Left(4);
   	dateTimeStr += ' ';
   	dateTimeStr += signature.Mid(11, 2);
   	dateTimeStr += ':';
   	dateTimeStr += signature.Mid(14, 2);
   	dateTimeStr += ':';
   	dateTimeStr += signature.Mid(17, 2);
   	pEvent->eventData.signature.eventDateTime = DEM_getTimeDateStampFromTimeDateString(dateTimeStr);
   	signature.Delete(0, 20);
   	pEvent->eventData.signature.cameraIndex = _ttoi(signature);
   //...

   инициализация числовых переменных из строки с заранее известным форматом...
   так как автор "кончал на MFC", с функцией sscanf западло было работать

   ShuraKotov, 09 Февраля 2011

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

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

   +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
   38. 38
   39. 39
   40. 40
   41. 41
   42. 42

   // говно-код - вступление
                           KEY_ACCELERATOR_DATA acceleratorData;
                           WORD id; int i, j, k;
   
                           acceleratorData.line = 255;
                           acceleratorData.address = 255;
                           acceleratorData.sensors = 255;
                           acceleratorData.action = 255;
                           KeyAcceleratorDataList tempList;
   
                           // говно-код - квинтэссенция
                           for(i = 1; i < listWholeString.ItemsCount(); i++) 
                           {
                                       k = 0;
                                       strCurrSensor = listWholeString[i];
                                       if((k = strCurrSensor.Find(_T('-'), k)) <= 0)
                                                   break;
                                       acceleratorData.line = (BYTE)_ttoi(strCurrSensor.Left(k));
                                       // ну кто блядь это поймет??? Кто???!!!
                                       j = k;                            
                                       if((k = strCurrSensor.Find(_T('-'), k + 1)) <= 0)
                                                   break;
                                       acceleratorData.address = (BYTE)_ttoi(strCurrSensor.Mid(j + 1, k - (j + 1)));
   
                                       j = k;                            
                                       if((k = strCurrSensor.Find(_T('-'), k + 1)) <= 0)
                                                   break;
                                       acceleratorData.sensors = (BYTE)_ttoi(strCurrSensor.Mid(j + 1, k - (j + 1)));
   
                                       if((k = strCurrSensor.Find(_T('-'), k)) <= 0)
                                                   break;
                                       acceleratorData.action = (BYTE)_ttoi(strCurrSensor.Right(strCurrSensor.GetLength() - (k + 1)));
                                       
   
                                       if((acceleratorData.line == 255) || (acceleratorData.address == 255) || 
                                         (acceleratorData.sensors == 255) || (acceleratorData.action == 255))
                                                   break;
                                       tempList.push_back(acceleratorData);
                           }           
                           mapKeyAccelerators.operator[](id) = tempList;
               }
   // Аут… Тушите свет и сидите, где сидели… А я нахуй сам пойду…

   парень "кончал на компьютер" и даже получил сертификат Microsoft по MFC
   лучше бы он кончал на дорожного рабочего...

   ShuraKotov, 08 Февраля 2011

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