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

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

+83

1
2
3
4
5

void somefunc(bool val)
{
    assert(0 == val || 1 == val);
...
}

Многоуровневый говнокод. Во первых, индусское сравнение задом наперед. Во вторых, выражение в ассерте всегда истенно. Когда GCC возбухнул по этому поводу, то код был за-#ifdef-ен, но оставлен для других менее умных компиляторов.

alexzak, 11 Августа 2012

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

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

+23

1

QHash<QByteArray, QSharedPointer<KeywordAnalyzerResult::Statistics> >::const_iterator i = res.result()->constBegin();

Создание STL-style итератора для Qt-ных контейнеров обычно выглядит уродски.

viktorious, 10 Августа 2012

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

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

+34

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

#include "xsmell.hpp"
#include <iostream>

TAG(html,  NO_ATTRIBS);
TAG(head,  NO_ATTRIBS);
TAG(title, NO_ATTRIBS);
TAG(body,  NO_ATTRIBS);
TAG(p,     NO_ATTRIBS);
TAG(a,     ATTRIB(href));
TAG(img,   ATTRIB(src), ATTRIB(alt));

int main()
{
    using namespace xsmell;

    document doc = 
        _
        <html>_
            <head>_
                <title>"XSMELL demo"<!title>_
            <!head>_
            <body>_
                <p>"Yesssssssssssssssss!"<!p>_
                <img .src("chucknorris.png") .alt("sneezing eyes open")>_ <!img>_
            <!body>_
        <!html>
        _;

    std::cout << doc << '\n';

    return 0;
}

C++ умеет HTML не хуже этих ваших похапешечек ;)

https://bitbucket.org/edd/xsmell/src

bormand, 07 Августа 2012

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

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

+12

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

void EllipticPoint::Add(const EllipticPoint &b, const EllipticCoord &a, const EllipticCoord &p, EllipticPoint &res) {
    if (!x.IsNotZero() && !y.IsNotZero()) {
        res = b;
    } else if (!b.x.IsNotZero() && !b.y.IsNotZero()) {
        res = *this;
    } else if (x.Compare(b.x)!=0) {
        EllipticCoord tmp1, tmp2, lambda;
        b.x.Sub(x,p,tmp1); tmp1.Invert(p,tmp2);
        b.y.Sub(y,p,tmp1); tmp1.Mul(tmp2,p,lambda);
        lambda.Mul(lambda,p,tmp1);
        tmp1.Sub(x,p,tmp2); tmp2.Sub(b.x,p,res.x);
        x.Sub(res.x,p,tmp1); lambda.Mul(tmp1,p,tmp2); tmp2.Sub(y,p,res.y);
    } else if (y.Compare(b.y)==0) {
        EllipticCoord tmp1, tmp2, tmp3, lambda;
        x.Mul(x,p,tmp1); tmp1.Add(tmp1,p,tmp3); tmp1.Add(tmp3,p,tmp2); tmp2.Add(a,p,tmp1);
        y.Add(y,p,tmp2); tmp2.Invert(p,tmp3); tmp1.Mul(tmp3,p,lambda);
        lambda.Mul(lambda,p,tmp1); tmp1.Sub(x,p,tmp2); tmp2.Sub(x,p,res.x);
        x.Sub(res.x,p,tmp1); lambda.Mul(tmp1,p,tmp3); tmp3.Sub(y,p,res.y);
    } else {
        res.x.SetZero();
        res.y.SetZero();
    }
}

Из моего велосипеда четырехлетней давности.
Кусочек реализации ГОСТ Р 34.10-2001.

bormand, 03 Августа 2012

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

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

+20

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
91
92
93
94
95
96

class TCP1251ToUTF16StringConverter
{
public:
	static WideChar convert(Char Source)
	{
		WideChar Result=static_cast<unsigned char>(Source);
		const WideChar Russian_YO=static_cast<unsigned char>('Ё');
		const WideChar Russian_yo=static_cast<unsigned char>('ё');
		const WideChar RussianWide_YO=L'Ё';
		const WideChar RussianWide_yo=L'ё';
		const WideChar Russian_A=static_cast<unsigned char>('А');
		const WideChar RussianWide_A=L'А';
		const unsigned int AmountOfSymbols=0x40;
		if(Result==Russian_YO)
			return RussianWide_YO;
		if(Result==Russian_yo)
			return RussianWide_yo;
		if(Result>=Russian_A&&Result<Russian_A+AmountOfSymbols)
			return (Result-Russian_A+RussianWide_A);
		return Result;
	};
	
	static void convert(PwideChar UTF16StringDestination, PChar CP1251WinEngRusStringSource, const size_t TextLength)
	{
		assert(CP1251WinEngRusStringSource!=NULL);
		size_t i=0;
		for(;;)
		{
			if(i>=TextLength)
				break;
			assert(i<TextLength);
			Char CP1251SourceChar=CP1251WinEngRusStringSource[i];
			if(CP1251SourceChar=='\0')
				break;
			UTF16StringDestination[i]=convert(CP1251SourceChar);
			++i;			
		};
		UTF16StringDestination[i]=L'\0';
		assert(i<=TextLength);
	};
	
	static std::wstring convert(const std::string& CP1251WinEngRusStringSource)
	{
		assert(CP1251WinEngRusStringSource.c_str()!=NULL);
		std::wstring UTF16StringDestination;
		std::transform(CP1251WinEngRusStringSource.begin(), CP1251WinEngRusStringSource.end(), std::inserter(UTF16StringDestination, UTF16StringDestination.end())/*std::back_inserter(UTF16StringDestination)*//*VC 6.0 compatible*/, makePointerToFunction(convertChar));
		return UTF16StringDestination;
	};
	
private:
	static WideChar convertChar(char Source)
	{
		return convert(Source);
	};
};

template<const size_t MaxAmountOfChar>
class TCP1251ToUTF16StringInPlaceConverter
{
public:
	TCP1251ToUTF16StringInPlaceConverter(PChar CP1251WinEngRusStringSource)
	{
		STATIC_ASSERT(MaxAmountOfChar>0, MaxAmountOfChar_must_be_above_zero);
		TCP1251ToUTF16StringConverter::convert(&(_buffer[0]), CP1251WinEngRusStringSource, MaxAmountOfChar);
	};
	
	TCP1251ToUTF16StringInPlaceConverter(PChar CP1251WinEngRusStringSource, const size_t TextLength)
	{
		STATIC_ASSERT(MaxAmountOfChar>0, MaxAmountOfChar_must_be_above_zero);
		assert(TextLength<=MaxAmountOfChar);
		TCP1251ToUTF16StringConverter::convert(&(_buffer[0]), CP1251WinEngRusStringSource, TextLength);
	};
	
	void convert(PChar CP1251WinEngRusStringSource)
	{
		TCP1251ToUTF16StringConverter::convert(&(_buffer[0]), CP1251WinEngRusStringSource, MaxAmountOfChar);
	};
	
	void convert(PChar CP1251WinEngRusStringSource, const size_t TextLength)
	{
		assert(TextLength<=MaxAmountOfChar);
		TCP1251ToUTF16StringConverter::convert(&(_buffer[0]), CP1251WinEngRusStringSource, TextLength);
	};
	
	PWideChar Get(void) const
	{
		return &(_buffer[0]);
	};
	
	PwideChar Get(void)
	{
		return &(_buffer[0]);
	};
	
	wideChar _buffer[MaxAmountOfChar+1];
};

USB, 02 Августа 2012

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

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

+15

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

//file systeminfo.cpp
#include "../SystemInfoManager.h"
struct : public SystemInfoManager
{
//some atriburtes
//some methods
   void setSomeparametrs()///
{
/*....*/
}
///ect...

} System;


SystemInfoManager * instance()
{
       return &System;
}

Мое имя затрет история, но мои славные дела будут жить, пытайте меня дальше гниды из гестапо!

Psionic, 01 Августа 2012

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

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

+2

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

auto addition = [] (double a, double b) { return a + b; };

auto substruction = [] (double a, double b) { return a - b; };

auto divide = [] (double a, double b) { return a / b; };

auto multiplie = [] (double a, double b) { return a * b; };

typedef std::function<double (double, double)> DoubleFunc;

int main()
{
    std::vector<DoubleFunc> funcVector;

    funcVector.push_back(addition);
    funcVector.push_back(substruction);
    funcVector.push_back(multiplie);
    funcVector.push_back(divide);

    std::for_each(funcVector.begin(), funcVector.end(), []
    (DoubleFunc func)
    {
        std::cout << func(3.14159, 100500) << std::endl;
    });

    system("pause");
    return 0;
}

А ваш язык такое сможет:)? (я о delphi, java, etc.)
Кстати, если писать все в таком духе, то параллелиться прога будет безболезненно:), правда это уже какой-то хацкель/лисп...

suc-daniil, 30 Июля 2012

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

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

+19

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

void load_functions(void** functions[], const char* names[], ushort count)
{
	for(ushort i=0; i<count; i++)
		*functions[i]=wglGetProcAddress(names[i]);
}

...

void* const glslFunctions[]={&glCreateShader, &glShaderSource, &glCompileShader,
	&glGetShaderiv, &glGetShaderInfoLog, &glDeleteShader,
	&glCreateProgram, &glAttachShader, &glLinkProgram, &glGetProgramiv,
	&glGetProgramInfoLog, &glUseProgram, &glDeleteProgram,
	&glGetUniformLocation, &glGetActiveUniform, &glBindAttribLocation,
	&glGenProgramPipelines, &glBindProgramPipeline, &glDeleteProgramPipelines, &glUseProgramStages,
	&glProgramParameteri, &glGetProgramPipelineiv, &glValidateProgramPipeline, &glGetProgramPipelineInfoLog,

	&glProgramUniform1i, &glProgramUniform2iv, &glProgramUniform3iv, &glProgramUniform4iv,
	&glProgramUniform1ui, &glProgramUniform2uiv, &glProgramUniform3uiv, &glProgramUniform4uiv,
	&glProgramUniform1f, &glProgramUniform2fv, &glProgramUniform3fv, &glProgramUniform4fv,
	&glProgramUniform1d, &glProgramUniform2dv, &glProgramUniform3dv, &glProgramUniform4dv,
	&glProgramUniformMatrix2fv, &glProgramUniformMatrix3fv, &glProgramUniformMatrix4fv,
	&glProgramUniformMatrix2x3fv, &glProgramUniformMatrix2x4fv,
	&glProgramUniformMatrix3x2fv, &glProgramUniformMatrix3x4fv,
	&glProgramUniformMatrix4x2fv, &glProgramUniformMatrix4x3fv,
	&glProgramUniformMatrix2dv, &glProgramUniformMatrix3dv, &glProgramUniformMatrix4dv,
	&glProgramUniformMatrix2x3dv, &glProgramUniformMatrix2x4dv,
	&glProgramUniformMatrix3x2dv, &glProgramUniformMatrix3x4dv,
	&glProgramUniformMatrix4x2dv, &glProgramUniformMatrix4x3dv};

const char* glslFuncNames[sizeof(glslFunctions)/sizeof(glslFunctions[0])]=
	{"glCreateShader", "glShaderSource", "glCompileShader",
	"glGetShaderiv", "glGetShaderInfoLog", "glDeleteShader",
	"glCreateProgram", "glAttachShader", "glLinkProgram", "glGetProgramiv",
	"glGetProgramInfoLog", "glUseProgram", "glDeleteProgram",
	"glGetUniformLocation", "glGetActiveUniform", "glBindAttribLocation",
	"glGenProgramPipelines", "glBindProgramPipeline", "glDeleteProgramPipelines", "glUseProgramStages",
	"glProgramParameteri", "glGetProgramPipelineiv", "glValidateProgramPipeline", "glGetProgramPipelineInfoLog",

	"glProgramUniform1i", "glProgramUniform2iv", "glProgramUniform3iv", "glProgramUniform4iv",
	"glProgramUniform1ui", "glProgramUniform2uiv", "glProgramUniform3uiv", "glProgramUniform4uiv",
	"glProgramUniform1f", "glProgramUniform2fv", "glProgramUniform3fv", "glProgramUniform4fv",
	"glProgramUniform1d", "glProgramUniform2dv", "glProgramUniform3dv", "glProgramUniform4dv",
	"glProgramUniformMatrix2fv", "glProgramUniformMatrix3fv", "glProgramUniformMatrix4fv",
	"glProgramUniformMatrix2x3fv", "glProgramUniformMatrix2x4fv",
	"glProgramUniformMatrix3x2fv", "glProgramUniformMatrix3x4fv",
	"glProgramUniformMatrix4x2fv", "glProgramUniformMatrix4x3fv",
	"glProgramUniformMatrix2dv", "glProgramUniformMatrix3dv", "glProgramUniformMatrix4dv",
	"glProgramUniformMatrix2x3dv", "glProgramUniformMatrix2x4dv",
	"glProgramUniformMatrix3x2dv", "glProgramUniformMatrix3x4dv",
	"glProgramUniformMatrix4x2dv", "glProgramUniformMatrix4x3dv"};

load_functions((void***)glslFunctions, glslFuncNames, sizeof(glslFunctions)/sizeof(glslFunctions[0]));

...

Вот так я загружаю расширения OpenGL!

gammaker, 28 Июля 2012

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

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

+19

1

delete this;

Это написал гейдевщик.
Но когда его спросили: "Почему? Зачем?"
Он ответил: "Я создаю принципиально новую охретектуру движка. Это будет лучший двиг всех времен и народов."

HaskellGovno, 28 Июля 2012

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

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

+4

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

#include <iostream>
using std::cin;
using std::cout;
using std::endl;

#include <cmath>

double mySqrt( float, float );

int main() {
    float a, n;
    cin >> a >> n;
    
    cout << mySqrt( a, n ) << endl;
    return 0;    
}

double mySqrt( float a, float n ) {
    const double EPSILON = 0.00001;
    double precision;
    
    double x = 0, x_prev = 1.0;
    
    do {
        x = (x_prev * (n - 1.0) + a / pow( x_prev, n - 1.0 )) / n;
        precision = fabs( x - x_prev );
        
        x_prev = x;
    } while( precision > EPSILON );
    
    return x;
}

Студентота, да.
Извлечение корней.

vortexx1, 28 Июля 2012

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

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