Лучший говнокод за всё время

Лучший говнокод

В номинации:

Лучший рейтинг
Самый комментируемый

За время:

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

+7

1
2
3
4
5
6

namespace engine { namespace ui { class Console; } }

class Dummy
{
   engine::ui::Console * _ptr;
};

Решение проблемы с перекрёстными #include, когда классы должны хранить указатели друг на друга. Простое объявление class engine::ui::Console; не работает.
Не в первый раз сталкиваюсь с этой проблемой из-за примитивной системы импорта.

an0nym, 20 Июля 2013

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

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

+7

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

MashTextGen::MashTextGen()
{
	files.push_back("BarButton.cpp");
	files.push_back("CreateBar.cpp");
	files.push_back("Eventfulness.cpp");
	files.push_back("Glob.cpp");
	files.push_back("main.cpp");
	files.push_back("mashtextgen.cpp");
	files.push_back("MashWindow.cpp");
	files.push_back("Note.cpp");
	files.push_back("NotificationBar.cpp");
	files.push_back("Render.cpp");
	files.push_back("RogueWindow.cpp");
	files.push_back("WorldWindow.cpp");
	files.push_back("BarButton.h");
	files.push_back("CreateBar.h");
	files.push_back("Eventfulness.h");
	files.push_back("Glob.h");
	files.push_back("main.h");
	files.push_back("mashtextgen.h");
	files.push_back("MashWindow.h");
	files.push_back("Note.h");
	files.push_back("NotificationBar.h");
	files.push_back("Render.h");
	files.push_back("RogueWindow.h");
	files.push_back("WorldWindow.h");
	
	LoadMoreData();
	picker = 0;
}

Я так и не понял зачем этот код.
Найдено в опенсорсной игре.

x0m9k, 17 Апреля 2013

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

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

+7

01
02
03
04
05
06
07
08
09
10

class Pa || Ra || Ll || El
{  
   int f || o || o; // четвертое ядро простаивает - надо длиннее переменные заводить
   int b || a || r;
}

int m || a || i || n(int argc, CHAR* argv[])
{ 
    Parallel parallel = new Pa() || Ra() || Ll() || El ();
}

LispGovno, 05 Марта 2013

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

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

+7

001
002
003
004
005
006
007
008
009
010
011
012
013
014
015
016
017
018
019
020
021
022
023
024
025
026
027
028
029
030
031
032
033
034
035
036
037
038
039
040
041
042
043
044
045
046
047
048
049
050
051
052
053
054
055
056
057
058
059
060
061
062
063
064
065
066
067
068
069
070
071
072
073
074
075
076
077
078
079
080
081
082
083
084
085
086
087
088
089
090
091
092
093
094
095
096
097
098
099
100

template<typename T>
class Enumerable
{
public:
	Enumerable() : enumerableInProcess(false) { };
	virtual void begin() = 0;
	virtual void end() = 0;
	virtual bool enumeration(T* item) = 0;
protected:
	bool enumerableInProcess;
};
template<typename T>
class List : Enumerable<T*>
{
public:
	class ListItem
	{
	public:
		friend class List;
		T item;
		ListItem(T item) : item(item), next(nullptr), previous(nullptr)
		{
		}
	private:
		class ListItem* next;
		class ListItem* previous;
	};

	/* ... */
	
	void begin()
	{
		if(enumerableInProcess) {
			throw Exception("Error Enumerable!");
		}

		enumerableInProcess = true;
		enumerationItem = first;
	}
	bool enumeration(T** item)
	{
		if(enumerableInProcess) {
			if(enumerationItem != nullptr) {
				(*item) = &(enumerationItem->item);
				enumerationItem = enumerationItem->next;
				return (true);
			} else {
				(*item) = nullptr;
				return (false);
			}
		} else {
			throw Exception("Error Enumerable!");
		}
	}
	bool enumeration(ListItem **listItem)
	{
		if(enumerableInProcess) {
			if(enumerationItem != nullptr) {
				(*listItem) = enumerationItem;
				enumerationItem = enumerationItem->next;
				return (true);
			} else {
				(*listItem) = nullptr;
				return (false);
			}
		} else {
			throw Exception("Error Enumerable!");
		}
	}
	void end()
	{
		if(!enumerableInProcess) {
			throw Exception("Error Enumerable!");
		}
		enumerableInProcess = false;
	}
private:
	const int size;
	int count;
	ListItem *first;
	ListItem *last;
	ListItem *enumerationItem;
};
void list_t_1()
{
	List<int> list(8);
	List<int>::ListItem *item;
	list.add(1);
	list.add(2);
	list.add(4);
	list.add(8);
	list.add(16);

	list.begin();
	while(list.enumeration(&item))
	{
		printf("%i\n", (item->item));
	}
	list.end();
}

dreesto, 09 Февраля 2013

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

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

+7

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

#include <iostream>
namespace detail
{
  class CRWO;
  class CRO;
  class CWO;
  class CO;
}
typedef detail::CRWO&  CRWO;
typedef detail::CRO&  CRO;
typedef detail::CWO&  CWO;
typedef detail::CO&  CO;
class C
{
    friend class detail::CRWO; 
    friend class detail::CRO; 
    friend class detail::CWO; 
    friend class detail::CO; 
  public:
    C( int a ) : value(a) {}
    ~C() {};
    operator CRWO() { return *static_cast<detail::CRWO*>(static_cast<void*>(this)); }
    operator CRO() { return *static_cast<detail::CRO*>(static_cast<void*>(this)); }
    operator CWO() { return *static_cast<detail::CWO*>(static_cast<void*>(this)); }
    operator CO() { return *static_cast<detail::CO*>(static_cast<void*>(this)); }
  private:
    void set( int newValue ) { value = newValue; }
    int get() { return value; }
  private:
    int value;
};
class detail::CRWO
{
  public:
    void set( int newValue ) { static_cast<C*>(static_cast<void*>(this))->set( newValue ); }
    int get() { return static_cast<C*>(static_cast<void*>(this))->get( ); }
    operator ::CRO() { return *static_cast<detail::CRO*>(static_cast<void*>(this)); }
    operator ::CWO() { return *static_cast<detail::CWO*>(static_cast<void*>(this)); }
    operator ::CO() { return *static_cast<detail::CO*>(static_cast<void*>(this)); }    
  private:
    CRWO(); CRWO(const CRWO&);~CRWO();CRWO& operator=(const CRWO&);void operator&(); void operator*();
};
class detail::CWO
{
  public:
    void set( int newValue ) { static_cast<C*>(static_cast<void*>(this))->set( newValue ); }
     operator ::CO() { return *static_cast<detail::CO*>(static_cast<void*>(this)); }    
  private:
    CWO(); CWO(const CWO&);~CWO();CWO& operator=(const CWO&);void operator&(); void operator*();
};
class detail::CRO
{
  public:
    int get() { return static_cast<C*>(static_cast<void*>(this))->get( ); }
    operator ::CO() { return *static_cast<detail::CO*>(static_cast<void*>(this)); }    
  private:
    CRO(); CRO(const CRO&);~CRO();CRO& operator=(const CRO&);void operator&(); void operator*();
};
class detail::CO
{
  public:
  private:
    CO(); CO(const CO&);~CO();CO& operator=(const CO&);void operator&(); void operator*();
};int main(int argc, char *argv[])
{
C c(3);
CRWO rwo = c;
CRO ro = c;
CWO wo = c;
CO o = c;
  std::cout  << rwo.get() << std::endl;
  wo.set( 5);
  std::cout  << ro.get() << std::endl;
return 0;
}

Оттуда.

Автор требует указывать авторство при копировании.

LispGovno, 02 Февраля 2013

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

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

+7

1

TPrimitive a((TLine())), b((TCircle()));

Вынужденный говнокод.
Должно быть:

TPrimitive a(TLine()), b(TCircle());

Но это будет объявлением 2х указателей на функции.

HaskellGovno, 16 Августа 2012

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

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

+7

1
2

QRegion* region = new QRegion(*(new QRect(button->x()+5,button->y()+5,190,190)),QRegion::Ellipse);
button->setMask(*region);

panter_dsd, 22 Июля 2012

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

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

+7

1
2
3
4
5
6
7
8

std::string s1;
std::string s2;

int ret = strcmp(s1.c_str(), s2.c_str());  //А здесь мы сравниваем строки
if (ret == 0)
{
    .... 
}

pathfinder, 18 Августа 2009

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

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

+7

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

if(isMoving)
{
	std::map<int, unsigned char> dirToFrame = boost::assign::map_list_of(1, 10)(2, 5)(3, 6)(4, 7)(5, 8)(6, 9)(7, 12)(8, 11);
	if(dir >= 1 && dir <= 10)
		return dirToFrame[dir];

	throw std::string("Something very wrong1.");
}
else //if(isMoving)
{
	std::map<int, unsigned char> dirToFrame = boost::assign::map_list_of(1, 13)(2, 0)(3, 1)(4, 2)(5, 3)(6, 4)(7, 15)(8, 14);
	if(dir >= 1 && dir <= 10)
		return dirToFrame[dir];

	throw std::string("Something very wrong1.");
}

Кусок кода из open source игры. Выбирает кадр спрайта, в зависимости от направления героя. В ревизии, предшествующей этой, использовался switch.

just_geek, 18 Августа 2009

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

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

+6.9

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

#ifndef SOCHET_H
#define SOCHET_H

// Сдвигает самую младшую единицу в сторону младшего разряда
#define shiftLast1Right(x) ( (x-1)^((x^(x-1)) >> 2) )

// Дописывает 1 после самой младшей единицы
#define add1AfterLast1(x) ( x | ((x^(x-1))+1) >> 2 )

template<class T>
class Sochet
{
public:
	T value;

	//////////////////////////////////////////////////////////////////////////

	Sochet():value(0) { }
	Sochet(int n, int k) {
		firstTurn(n, k);
	}
	~Sochet() {
		value = 0;
	}

	//////////////////////////////////////////////////////////////////////////

	// Первая комбинация
	// В первоначальной ситуации все К единиц располагаются в старших битах
	void firstTurn(int n, int k) {
		value = ( ( T(1) << k ) - 1 ) << (n - k);
	}

	// Нахождение следующей комбинации
	// Возвращает false в случае последней комбинации
	bool nextTurn()
	{
		// Отлов последней комбинации
		if (value & (value+1) == 0)
			return false;
		
		// Условие по младшему биту: 1 или 0
		if (value & 1)
		{
			value >>= 1;
			nextTurn();
			value = add1AfterLast1(value << 1);
		} else
			value = shiftLast1Right(value);
		
		return true;
	}
}

#endif // SOCHET_H

Шаблон перебора всех сочетаний/выборок в много разрядных числах.
Пример перебираемых чисел для N=5, K=3:
11100
11010
11001
10110
10101
10011
01110
01101
01011
00111

// Код выглядит сочно(особенно дефайны), зато всё работает максимально быстро.
// Статья про этот алгоритм: http://k06a.blogspot.com/2009/04/blog-post_08.html

k06a, 15 Августа 2009

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

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