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

Всего: 6

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

+159

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
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)

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

+162

1
2
3
4
5
6
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)

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

+146

01
02
03
04
05
06
07
08
09
10

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

DWORD CDAMN::ControlDAMNAmount()
{
	return controlDAMNAmount;
}

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

ShuraKotov, 14 Февраля 2011

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

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

+135

1
2
3
4
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)

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

+162

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
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)

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

+161

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
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)

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