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

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

+53

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23

void nearest(double v) {
    double d = 0.1;
    double v2 = v + d;
    double v3;
    double v4;
    while (v2 != v) {
        v3 = v + d;
        v4 = v - d;
        d = d * 0.9;
        v2 = v + d;
    }
    qDebug() << QString::number(v2,'f',30) << QString::number(v3,'f',30) << QString::number(v4,'f',30);
}

void roundTest() {
    double v1 = 1.90;
    double v2 = 1.91;
    QString vr1 = QString::number((v1+v2)*0.5,'f',2);
    QString vr2 = QString::number((v1+v2)*0.500000000000000111022302462516,'f',2);
    qDebug() << vr1 << vr2;
}

#define DOUBLE_HALF 0.500000000000000111022302462516

Когда-нибудь меня за это побьют

overloop, 29 Августа 2014

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

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

+1

1
2
3
4
5
6
7

for (conn_vector::iterator i = m_connections.begin(); i != m_connections.end(); ++i) {
    if (*i == conn) {
        m_connections.erase(i);
        toDelete = *i;
        break;
    }
}

Да, С++98. Да, в std::find не умеем. Да, сегфолт. Да, моё.

Elvenfighter, 27 Августа 2014

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

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

+144

1
2
3
4
5
6
7

for (conn_vector::iterator i = m_connections.begin(); i != m_connections.end(); ++i) {
    if (*i == conn) {
        toDelete = *i;
        m_connections.erase(i);
        break;
    }
}

Да, С++98. Да, в std::find не умеем. Да, сегфолт. Да, моё.

Elvenfighter, 27 Августа 2014

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

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

+58

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23

void Model::setPosition(GLfloat x, GLfloat y, GLfloat z) {
        if (m_position == NULL) {
            m_position = (float *) malloc(3 * sizeof(float));
        }

        if (m_position != NULL) {
            m_position[0] = x;
            m_position[1] = y;
            m_position[2] = z;
        }
    }

    void Model::setRotate(GLfloat x, GLfloat y, GLfloat z) {
        if (m_rotate == NULL) {
            m_rotate = (float *) malloc(3 * sizeof(float));
        }

        if (m_rotate != NULL) {
            m_rotate[0] = x;
            m_rotate[1] = y;
            m_rotate[2] = z;
        }
    }

http://code.google.com/p/forget3d/
Во первых, умиляет умение людей взяв современный супернавороченый удобный автомат, вместо того чтоб нажать на курок и выстрелить в противника, взять этот автомат наперевес как дубинку, и идти им пиздить противника по голове. Я о сишке, слегка прикрытой крестами.
И во вторых, этот код при недостатке памяти просто нифига не делает, что чревато непонятками - вроде и повертел модель, вроде и подвинул, а оно никак не вертится и не двигается.

Pythoner, 27 Августа 2014

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

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

+53

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

void _debugPrintWaveHeader(const char *path, const WAVHEADER &source_header)
{
	FILE *debug = fopen("path", "wt");
	fprintf(debug, "chunkId    = %s\n", source_header.chunkId);
	fprintf(debug, "chunkSize  = %u\n", source_header.chunkSize);
	fprintf(debug, "format     = %s\n", source_header.format);
	fprintf(debug, "subCh1Id   = %s\n", source_header.subchunk1Id);
	fprintf(debug, "subCh1Size = %u\n", source_header.subchunk1Size);
	fprintf(debug, "audioform  = %u\n", source_header.audioFormat);
	fprintf(debug, "numChanels = %u\n", source_header.numChannels);
	fprintf(debug, "sampleRate = %u\n", source_header.sampleRate);
	fprintf(debug, "byteRate   = %u\n", source_header.byteRate);
	fprintf(debug, "blockAlign = %u\n", source_header.blockAlign);
	fprintf(debug, "bitsPerSam = %u\n", source_header.bitsPerSample);
	fprintf(debug, "subCh2Id   = %s\n", source_header.subchunk2Id);
	fprintf(debug, "subCh2Size = %u\n", source_header.subchunk2Size);
	fclose(debug);
}

Начал разгребать один говнопроект... При отладке программа всё падает и падает, падает и не может остановится. Смотрю на код - все в норме, а потом, по прошествии нескольких часов...

GreatMASTERcpp, 25 Августа 2014

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

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

+58

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

if(ext == ".wav")
		{
			if(main_header.bitsPerSample == 8)
			{
				switch(main_header.bitsPerSample)
				{
					case 8:
					{
						open_mode = n8bit_mode;
					}break;
					case 16:
					{
						open_mode = n16bit_mode;
					}break;
					default: open_mode = nNULL;
				}
....далее какото код
}

...

GreatMASTERcpp, 25 Августа 2014

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

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

+62

1
2

wchar_t c1 =
    (wchar_t)LOWORD(::CharUpperW((LPWSTR)(DWORD_PTR) MAKELONG(*i1, 0)));

Шиндовс, штроки, штрадания
http://llvm.org/bugs/show_bug.cgi?id=20712

roman-kashitsyn, 21 Августа 2014

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

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

+57

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
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90

#include "TrayIcon.h"

// ----------------------------------------------------------------------------

TrayIcon::TrayIcon(QObject* parent) : QSystemTrayIcon(parent),
                                      INTERVAL(1000),
                                      WIDTH_ICON(30), HEIGHT_ICON(30)
{
    currentDate = QDate(0, 0, 0);

    defaultIcon = QPixmap(WIDTH_ICON, HEIGHT_ICON);
    defaultIcon.fill(Qt::black);

    reminderIcon = QPixmap(WIDTH_ICON, HEIGHT_ICON);
    reminderIcon.fill(Qt::yellow);

    thousandthDayIcon = QPixmap(WIDTH_ICON, HEIGHT_ICON);
    thousandthDayIcon.fill(Qt::green);

    dialogBoxIsActive = false;

    readSettings();

    wasReminder = false;
    wasCongratulation = false;
    wasSetNewDate = false;

    slotUpdateDate();

    QMenu* contextMenu = new QMenu;

    contextMenu->addAction("Set date of birth...",
                                             this, SLOT(slotSetDateOfBirth()));

    QAction* autorunAction = contextMenu->addAction("Autorun",
                                             this, SLOT(slotSetAutorun(bool)));
    autorunAction->setCheckable(true);
    autorunAction->setChecked(autorun);

    contextMenu->addSeparator();

    contextMenu->addAction("About...", this, SLOT(slotAboutMyProgram()));

    contextMenu->addSeparator();

    contextMenu->addAction("Quit", qApp, SLOT(quit()));

    setContextMenu(contextMenu);

    QTimer* timer = new QTimer(this);
    connect(timer, SIGNAL(timeout()), SLOT(slotUpdateDate()));
    timer->start(500);
}

// ----------------------------------------------------------------------------

void TrayIcon::slotUpdateDate()
{
    if((currentDate != QDate::currentDate()) || wasSetNewDate)
    {
        currentDate = QDate::currentDate();

        daysToThousandthDay
                       = INTERVAL - dateOfBirth.daysTo(currentDate) % INTERVAL;

        wasCongratulation = false;
        wasReminder = false;
        wasSetNewDate = false;

        if((daysToThousandthDay != INTERVAL) && (daysToThousandthDay != 1))
        {
            setIcon(QIcon(defaultIcon));
            setToolTip("Until next thousandth day in " +
                             QString().number(daysToThousandthDay) + " days.");
        }
        else
        {
            if(daysToThousandthDay == 1)
            {
                if(!wasReminder)
                    reminderBeforeDay();
            }
            else
            {
                if(!wasCongratulation)
                    congratulation();
            }
        }
    }
}

Описание конструктора и одного из методов класса иконки в системном трее из программы, которая должна оповещать пользователя о каждом тысячном дне его жизни. Написано на Qt.

Полная версия:

main: http://pastebin.com/DEKiMWdb

хедер класса иконки трея: http://pastebin.com/rN9NwBPB
.cpp класса иконки трея: http://pastebin.com/uuyf5uxX

хедер класса окна ввода даты рождения: http://pastebin.com/bmfAbFwg
.cpp класса окна ввода даты рождения: http://pastebin.com/3R8BaPFC

darkKnight, 18 Августа 2014

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

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

+63

1

renderer->SetBackground(.0, .0, .0); // Background color green

Abbath, 16 Августа 2014

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

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

+42

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

int complexFunction(int flag)
{
    QMutexLocker locker(&mutex);

    int retVal = 0;

    switch (flag) {
    case 0:
    case 1:
        return moreComplexFunction(flag);
    case 2:
        {
            int status = anotherFunction();
            if (status < 0)
                return -2;
            retVal = status + flag;
        }
        break;
    default:
        if (flag > 10)
            return -1;
        break;
    }

    return retVal;
}

Пора добавлять отдельную ветку для фрейворка Qt. Это просто клад, так извратить все принципы програмирования :-). Этот код из справки к этому чуду. QMutexLocker - целый класс для того чтобы не нужно было разблокировать мьютекс при выходе из функции! Так они скоро и до сборщика мусора с неявной типизацией дойдут!
P.S. У кого есть Qq попробуйте в "коде" сборки qmake вызвать include внутри функции.

rst256, 15 Августа 2014

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

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