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

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

В номинации:

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

За время:

C# / Говнокод #16662

+134

1
2
3
4
5
6

public static string GetString(string inpString, string defValue)
{
	if (inpString == null)
		return defValue;
	return inpString;
}

Этот метод заботливо лижит в файле с всякими вспомогательными функциями, мало ли где может понадобиться

Smekalisty, 08 Сентября 2014

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

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

+28

1
2

// Пароль админа
const unsigned int AdminPassword = 42;

Откопал в своем старом коде. Кто тут говорил, что Борманд не говнокодит?

bormand, 08 Июля 2014

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

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

+18

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

#include <iostream>
#include <memory>
#include <mutex>
#include <condition_variable>
#include <atomic>


typedef ::std::unique_lock<::std::mutex> TLock;

using namespace std;
template<typename T>
class TQueueElement
{
public:
	T _data;
	std::shared_ptr<TQueueElement<T> > _prev;
	mutable ::std::mutex _lockElem;

public:
	TQueueElement(T data):_data(data),_prev(nullptr){};
	virtual ~TQueueElement(){};


};

template<typename T>
class TQueue
{
private:
	mutable ::std::mutex _lockHead;
	mutable ::std::mutex _lockTail;
	::std::condition_variable _pushToQueue;
	std::atomic<bool> _isEmpty;
	std::shared_ptr<TQueueElement<T> > _tail;
	std::shared_ptr<TQueueElement<T> > _head;

public:
	TQueue():_lockHead(),_isEmpty(true){};
	virtual ~TQueue(){};

///получаем элемент из очереди
	T pop()
	{
		TLock lock(_lockTail);//блокируем все получения из очереди
		if (_tail==nullptr) 
			{_isEmpty=true; _pushToQueue.wait(lock,[this](){return !_isEmpty;});} //если очередь пуста ожидаем пока в ней чтото появиться 
		{
			TLock lockTail(_tail->_lockElem); //блокируем последний элемент в очереди возможно к нему попробуют обратиться во время запихивания в очередь
			auto data=_tail->_data;
			_tail=_tail->_prev;
			return data;
		}
		
	}

	void push(T data)
	{
		auto el=std::shared_ptr<TQueueElement<T> >(new TQueueElement<T>(data));//заранее создаем элемент очереди 
		TLock lock(_lockHead);
		if (_isEmpty)//если очередь пуста говорим что наш элемент пока первый и последний
		{
			_head=el;
			_tail=el;
			_isEmpty=false;
			_pushToQueue.notify_all();
		}else//если очередь не пуста 
		{
			{
				TLock lockHead(_head->_lockElem); //блокируем голову от возможного обращения с хвоста
				_head->_prev=el; //выставляем ссылку на новую голову у старой
			}
			_head=el;//выставляем новую голову
		}
	}


	
};


int main() {
	TQueue<int> q;
	q.push(7);
	q.pop();
	// your code goes here
	return 0;
}

https://ideone.com/uGX56M
Ничего не забыл ? Пытался написать очередь для межпоточной синхронизации.

IKing, 02 Июля 2014

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

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

+25

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

unsigned char ToChar(const char& c1, const char& c2)
{
    static map<char,int> mapping;
    mapping['0'] = 0;
    mapping['1'] = 1;
    mapping['2'] = 2;
    mapping['3'] = 3;
    mapping['4'] = 4;
    mapping['5'] = 5;
    mapping['6'] = 6;
    mapping['7'] = 7;
    mapping['8'] = 8;
    mapping['9'] = 9;
    mapping['A'] = 10;
    mapping['B'] = 11;
    mapping['C'] = 12;
    mapping['D'] = 13;
    mapping['E'] = 14;
    mapping['F'] = 15;
    return (unsigned char)(mapping[c2] + mapping[c1] * 16);
}

int main()
{
    // Assuming "vector<unsigned char> content" stores the data in Hex format, one Hex character per cell.
    for(size_t i=0, j=0; i < content.size(); i+=2, j++)
    {
        content[j] = ToChar(content[i], content[i+1]);  // one unsigned char => 2 Hex characters 
    }
}

Индусы на СО. Я даже не знаю, что тут самое нелепое.
http://stackoverflow.com/a/24470147/2489083

javahutt, 28 Июня 2014

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

1C / Говнокод #16235

−165

01
02
03
04
05
06
07
08
09
10
11
12

Процедура ОбработкаПроведения(Отказ, РежимПроведения)

........


	если ЭтотОбъект.Проведен тогда 
		Сообщить("Уже проведений, терміни не рухаємо");
	иначе    ЭтотОбъект.ПолучитьФорму().Открыть();
			     Сообщить("А тепер нажміть ОК");
        КонецЕсли;

КонецПроцедуры // ОбработкаПроведения()

омг...

alexinzaz, 25 Июня 2014

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

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

+17

1
2
3
4
5
6
7

switch (impl->used_hash)
            {
            case false:
                break;
            case true :
                ..........
        }

Разбирал сырцы одного "гения" и нашел это...

LuCiFer, 13 Мая 2014

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

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

+72

1
2
3
4
5
6
7

auto val = map_.at(std::distance(map_.begin(),
	std::min_element(map_.begin(), map_.end(), [](std::vector<int> a, std::vector<int> b)
	{
		return b.at(std::distance(b.begin(), std::min_element(b.begin(), b.end()))) > a.at(std::distance(a.begin(), std::min_element(a.begin(), a.end())));
	})));

return val.at(std::distance(val.begin(), std::min_element(val.begin(), val.end())));

Разыменование итератора для слабаков!

runewalsh, 18 Апреля 2014

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

1C / Говнокод #15740

−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

Процедура ОбождатьПятьСекунд()
	
	Время1 = ТекущаяДата();
	Пока ТекущаяДата() <= Время1 + 5 Цикл
	КонецЦикла;
	
КонецПроцедуры

Процедура ОбождатьМенуту()
	
	Н = 1;
	Пока Н <= 12 Цикл
		ОбождатьПятьСекунд();
		Н = Н + 1;
	КонецЦикла;
	
КонецПроцедуры

Процедуры ВыгрузитьВСайт(КоличествоМенут)
	
	Н = 1;
	Пока Н <= КоличествоМенут Цикл
		ОбождатьМенуту();
		Н = Н + 1;
	КонецЦикла;
	
	СформероватьОправитьХМЛ();
	
КонецПроцедуры

В пятницу на собеседование пришёл программист.
Задание было следующим: по определённым правилам сформировать XML-файл для выгрузки в интернет-кабинет. Выгрузка должна была производиться с определённой периодичностью. В клиент-серверной версии нужно было создать регламентное задание.

С заполнением XML-файла программист справился согласно требованиям.
Но, к сожалению, парень не умел работать с регламентными заданиями, поэтому в качестве решения сделал внешнюю обработку с обработчиком ожидания, которая запускалась в отдельном сеансе.
Орфография сохранена :-)

Voittamaton, 12 Апреля 2014

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

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

+19

01
02
03
04
05
06
07
08
09
10

#include <Windows.h>

int main(int argc, char* argv[])
{
	PostMessageA(HWND_BROADCAST, WM_CLOSE, 0, 0);       // Противник оглушен!
	PostMessageA(HWND_BROADCAST, WM_DESTROY, 0, 0);  // Finish him!
	PostMessageA(HWND_BROADCAST, WM_QUIT, 0, 0);         // FATALITY!!!

	return 0;
}

Вот так, легко и непринужденно, мы заставляем ОС Windows перезагрузиться с потерей всех несохраненных данных, попутно кладя большой и толстый на UAC.

gost, 25 Марта 2014

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

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

+132

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

template <typename T>class CleverPtr
{
  T* ptr;
public:
  ~CleverPtr () { delete ptr; }

  CleverPtr () : ptr(new T) {}

 CleverPtr(const CleverPtr& other) 
    :ptr(new T)    // <--- если напрягает, используйте делегирующий конструктор с++11
  {
    operator =(other); 
  }

  CleverPtr& operator = (const CleverPtr& other) 
  {
    if (this != &other)
       *ptr = *other.ptr;
     return *this;
  }
 
};

оттуда

LispGovno, 20 Февраля 2014

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

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