C++ — Говнокод.ру

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

+162

1
2
3
4
5
6
7
8
9

SeismicDriver* SeismicSelectionDG::seismicDriver()
{
  Exf2dMW* mw = Exf2dMW::getMW(this);
  _seismicDriver = mw->seismicDriver();
  if ( !mw || !_seismicDriver)
    return 0;

  return _seismicDriver;
}

Все тот же коллега китаец. Exf2dMW - класс главного окна приложения. Во-первых бесмыссленно проверять его на 0, потому что без него вообще бы ничего не работало, но и как видно проверяет его он все равно не там. Ну и проверка на 0 _seismicDrvier тоже лишняя. Итог никакие проверки не нужны да и вообще первую строку в методе можно перенести в конструктор а возвращать член класса _seismicDriver.

kitaec, 09 Февраля 2011

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

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++ / Говнокод #5599

+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
43
44
45
46
47
48
49
50
51
52
53
54
55
56

// Часть первая. Объявления. Exceptions.hpp

template <typename E>
struct ExeptionKeeper 
{
    E * exptn;
    ExeptionKeeper(E * ex) 
    {
        exptn = ex;
    }
    ~ExeptionKeeper()
    {
        if(exptn != NULL )
        {
            delete exptn;
        }
    }
    E * operator -> () 
    { 
        return exptn; 
    }
};

#define EX_TRY(XX) try { XX }
#define EX_CATCH(TT, NAME, XX) catch (TT *_ex) { ExeptionKeeper<TT> NAME(_ex);  XX }

/// Часть вторая. Использование. ModelsHelper.cpp

.... // код
        EX_TRY
        (
           ............
             TestParams(args); // кидает new ParamsException в случае неудачи
           ............
             TestFunctions(funclist); // Кидает new FuncsException ... 
           ............
           //// тут еще есть вызовы, кидающие исключения
           ............
        ) 
        EX_CATCH
        (   ParamsException, exp,
            GLogger(levelerror).Log("Invalid params for model ", exp->model(), " [", exp->paramlist().str(), "]");
        ) 
        EX_CATCH
        (   FuncsException, exp,
            GLogger(levelerror).Log("Invalid functions for model ", exp->model(), " [", exp->funclist().str(), "]");
        ) 
        EX_CATCH
        (   LevelsException, exp,
            GLogger(levelerror).Log("Invalid levels for model ", exp->model(), " [", exp->levelslist().str(), "]");           
        ) 
        EX_CATCH
        (    .... , exp,
            ........
        )
.... // код

И снова на тему исключений.
Прислал приятель. Какой-то софт для аэропортов.
Комменты на русском мои.

ossa, 09 Февраля 2011

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

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

+154

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
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61

//построение суффиксного массива
vector <int> getarr(string s) 
{
  //s - исходная строка
  //суффиксный массив
  vector <int> arr;
  arr.resize(s.size());
  //массив цветов
  vector <int> col;
  col.resize(s.size());
  //массив для временных данных
  vector <int> buf;
  buf.resize(s.size());
  //массив для карманов сортировки
  vector <int> buck;
  buck.resize(max(L, (int) s.size()));
  //Шаг первый - начальная сортировка
  //мы хотим отсортировать буквы строки
  //посчитаем количество всех букв
  for (int i = 0; i < (int) s.size(); i++) 
    buck[s[i]]++;
  //преобразуем массив так, чтобы каждый элемент указывал на положение в массиве первой данной буквы
  int sum = 0; 
  for (int i = 0; i < L; i++) 
  {
    sum += buck[i];
	buck[i] = sum - buck[i];
  }
  //теперь заполним массив arr: Теперь в нем суффиксы отсортированы по первой букве
  for (int i = 0; i < (int) s.size(); i++) 
    arr[buck[s[i]]++] = i;
  //теперь проставляем цвета: цвет увеличивается на 1 если следующая буква - другая
  col[arr[0]] = 0; 
  for (int i = 1; i < (int) s.size(); i++) 
    col[arr[i]] = col[arr[i-1]] + (s[arr[i]] != s[arr[i-1]]); 
  int cn = col[arr[s.size() - 1]] + 1;
  //Шаг второй - постепенное расширение подстрок
  //в начале цикла отсортированы подстроки длины l, а в конце - длины 2l
  for (int l = 1; l < (int) s.size(); l *= 2) 
  {
    //обнуляем массив buck  и заполняем для сортировки по col
    for (int i = 0; i < (int) s.size(); i++) 
      buck[i] = 0; 
    for (int i = 0; i < (int) s.size(); i++) 
      buck[col[i]]++; sum = 0; 
    for (int i = 0; i < cn; i++) 
      sum += buck[i], buck[i] = sum - buck[i];
    //строим новый массив в buf (не забываем сдвинуть указатель по модулю на l влево), затем копируем его в arr
    for (int i = 0; i < (int) s.size(); i++) 
      buf[buck[col[(arr[i] - l + s.size()) % s.size()]]++]=(arr[i] - l + s.size()) % s.size(); 
    arr = buf;
    //теперь перекрашиваем массив col: заполняем массив buf, увеличиваем цвет на единицу если один из цветов отличается, затем копируем
    buf[arr[0]] = 0; 
    for (int i = 1; i < (int) s.size(); i++) 
      buf[arr[i]] = buf[arr[i - 1]] + (col[arr[i]] != col[arr[i - 1]] || col[(arr[i] + l) % s.size()] != col[(arr[i - 1] + l) % s.size()]); 
    cn = buf[arr[s.size() - 1]] + 1; 
    col = buf;
  }
  //возвращаем результат
  return arr;
}

Это просто жуть

chexov, 09 Февраля 2011

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

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

+170

1

double* (*(*(*fp)(int*,int*,int*))[])(float*,float*,float*);

уже третий семпл из жгучей темы на форуме сайта wasm.ru... я цитирую: "Объявление указателя на функцию, принимающую 3 аргумента-указателя на int, а возвращающая указатель на массив указателей на функцию принимающую 3 аргумента-указателя на float и возвращающую указатель на double"))) http://www.wasm.ru/forum/viewtopic.php?pid=419403#p419403

ReL, 08 Февраля 2011

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

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

+165

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16

if (seismicDriver()->has3DSeismic() && !seismicDriver()->has2DSeismic() ) {
    enableSeismicTab(0, true);
    enableSeismicTab(1, false);
  }
  else if (!seismicDriver()->has3DSeismic() && seismicDriver()->has2DSeismic() ) {
    enableSeismicTab(0, false);
    enableSeismicTab(1, true);
  }
  else if (seismicDriver()->has3DSeismic() && seismicDriver()->has2DSeismic() ) {
    enableSeismicTab(0, true);
    enableSeismicTab(1, true);
  }
  else if (!seismicDriver()->has3DSeismic() && !seismicDriver()->has2DSeismic() ) {
    enableSeismicTab(0, false);
    enableSeismicTab(1, false);
  }

Этим своим первым постом предворяю серию говнокода, который пишет мой коллега, китаец, имеющий PhD по физике.

В этом сниппете советую обратить внимание на два компонента в условиях if и на расстановку true и false в соответствии с ними.

kitaec, 08 Февраля 2011

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

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

+161

1

throw &std::exception();

Из того же проекта, что и http://www.govnokod.ru/5581 .
Естественно это потом иногда удаляется по delete...

Говногость, 08 Февраля 2011

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

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)

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

+163

1

throw new TSilentException("");

Говногость, 08 Февраля 2011

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

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

+149

1
2
3
4
5
6

struct SomeData 
{
    ...
    double balls;
    ....
}

Interfere, 08 Февраля 2011

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

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

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

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

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

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

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

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

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

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

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

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