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

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

В номинации:

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

За время:

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

+13

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

void User::AddFriend(User& newFriend)
{
    friends_.push_back(&newFriend);
    pDB_->AddFriend(GetName(), newFriend.GetName());
}

//...VS...

void User::AddFriend(User& newFriend)
{
    friends_.push_back(&newFriend);
    try
    {
        pDB_->AddFriend(GetName(), newFriend.GetName());
    }
    catch (...)
    {
        friends_.pop_back();
        throw;
    }
}

//...VS...

class VectorInserter//Глобальный безопасный вектороВставлятель.
{
public:
    VectorInserter(std::vector<User*>& v, User& u)
    : container_(v), commit_(false)
    {
        container_.push_back(&u);
    }
    void Commit() throw()
    {
        commit_ = true;
    }
    ~VectorInserter()
    {
        if (!commit_) container_.pop_back();
    }
private:
    std::vector<User*>& container_;
    bool commit_;
};

void User::AddFriend(User& newFriend)
{
    VectorInserter ins(friends_, &newFriend);
    pDB_->AddFriend(GetName(), newFriend.GetName());
    // Everything went fine, commit the vector insertion
    ins.Commit();
}

LispGovno, 03 Сентября 2013

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

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

+13

01
02
03
04
05
06
07
08
09
10

bool[] Inmask;
....
        private void discreteOIForm_Load(object sender, EventArgs e)
        {
            if (icpI_measure.Connect() && icpO_measure.Connect())
            {
                Inmask = new bool[1];
                Inmask[Convert.ToInt32(0)] = true;
            }
        }

Новоприбывший сотрудник откладывает вот такое. Это он сделал после совета использовать маски. Вы когда-нибудь видели массив из одного элемента? Я тоже нет.

phys-tech, 07 Мая 2013

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

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

+13

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

struct read_access_tag {};
struct write_access_tag {};
struct read_write_access_tag : read_access_tag, write_access_tag {};

template <
    typename ByteOrder,
    typename ValueType
>
std::size_t get_from(const uint8_t *src, ValueType &dst, const read_access_tag&) {
    ByteOrder::decode(src, dst);
    return sizeof(dst);
}

template <
    typename ByteOrder,
    typename ValueType
>
std::size_t put_into(ValueType src, uint8_t *dst, const write_access_tag&) {
    ByteOrder::encode(src, dst);
    return sizeof(src);
}

// ...

template <
    typename ByteOrder = default_byte_order,
    typename AccessTag = read_write_access_tag
    >
class basic_buffer {
public:
    typedef ByteOrder byte_order;
    typedef AccessTag access_tag;
    typedef typename access_traits<access_tag>::value_type value_type;
    typedef typename access_traits<access_tag>::strict_type strict_type;
    typedef typename access_traits<access_tag>::iterator iterator;
    typedef typename access_traits<access_tag>::const_iterator const_iterator;

    basic_buffer(iterator begin, iterator end)
        : begin_(begin)
        , end_(end)
        , pos_(begin)
    {}

    // ...

    template <typename T>
    basic_buffer & put(T value) {
        if (bytes_left() < sizeof(value)) throw Overflow;
        pos_ += put_into<byte_order>(value, pos_, access_tag());
        return *this;
    }

    template <typename T>
    basic_buffer & get(T &value) {
        if (bytes_left() < sizeof(value)) throw Overflow;
        pos_ += get_from<byte_order>(pos_, value, access_tag());
        return *this;
    }
}

Развитие идей из

http://govnokod.ru/12465

Изобретаем права доступа в compile time, чтобы можно было запретить писать в readonly-буфер и читать из writeonly-буфера без дупликации кода. put_into по сути не нужен (запись в readonly_buffer у меня и без этого не скомпилится), существует из соображений симметрии. Полный код здесь

https://github.com/roman-kashitsyn/encoding-binary

roman-kashitsyn, 28 Апреля 2013

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

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

+13

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

void CMainWindow::OnTimer(UINT_PTR id){
	switch(id){
	case IDT_TIMER1:
		CClientDC dc(this);
		dc.SetMapMode(MM_ISOTROPIC);
		GetClientRect(&rcClient);
		dc.SetWindowExt(CONSTANT,CONSTANT);
		dc.SetViewportExt(rcClient.right,rcClient.bottom);
		dc.SelectObject(GetStockObject(NULL_BRUSH));
		dc.SetROP2(R2_NOTXORPEN);
		wsprintf(buff,L"Score: %d       ", score);
		dc.TextOut(5,5,buff,15);
		if(!won || resized)dc.Ellipse(ptsBegin0.x-RAD,ptsBegin0.y-RAD,ptsBegin0.x+RAD,ptsBegin0.y+RAD);
		ptsBegin0.x = rand() % (CONSTANT-RAD) + RAD;
		ptsBegin0.y = rand() % (CONSTANT -RAD)+ RAD;
		dc.DPtoLP(&ptsBegin0,1);
		dc.Ellipse(ptsBegin0.x-RAD,ptsBegin0.y-RAD,ptsBegin0.x+RAD,ptsBegin0.y+RAD);
		if(!won || resized)dc.Ellipse(ptsEnd0.x-RAD,ptsEnd0.y-RAD,ptsEnd0.x+RAD,ptsEnd0.y+RAD);
		ptsEnd0.x = rand() % (CONSTANT - RAD) + RAD;
		ptsEnd0.y = rand() % (CONSTANT - RAD) + RAD;
		dc.DPtoLP(&ptsEnd0,1);
		dc.Ellipse(ptsEnd0.x-RAD,ptsEnd0.y-RAD,ptsEnd0.x+RAD,ptsEnd0.y+RAD);
		won = resized = false;
	}
}

Abbath, 18 Апреля 2013

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

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

+13

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

std::string StringUtilities::replace(const std::string& strValue,
                                     uint8_t piWhat,
                                     uint8_t piWith)
{  
  size_t len = strValue.length();
  uint8_t* lTemp = new uint8_t[len + 1];
  memset(lTemp, '\0', len + 1); //+ 1 for \0
  memcpy(lTemp, strValue.c_str(), len);
  for (size_t i = 0; i < len; i++)
  {
    if (lTemp[i] == piWhat)
      lTemp[i] = piWith;
  }
  return string( (int8_t*) lTemp );  
}

Любители велосипедов...

blackhearted, 10 Апреля 2013

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

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

+13

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

#include <iostream>
#include <vector>
#include <stdlib.h>
#include <time.h>

using std::vector;

void print_vec(const vector<int> v)
{   /* Print Vector */
    for(vector<int>::size_type i(0); i!=v.size(); ++i)
        std::cout << v[i] << (i!=v.size()-1 ? "|":"\n");
}

bool sort_vec(const vector<int> v)
{   /* Return True if vector sorted */
    bool b(true);
    for(vector<int>::size_type i(v.size()-1);i!=0;--i)
        if (v[i]<v[i-1]) {b=false;}
    return b;
}

int main()
{
    vector<int> VectorForNumber;
    const unsigned int ConstMaxElement(10);
    srand(time(NULL));
    for(vector<int>::size_type i(0);i!=ConstMaxElement;++i)
        VectorForNumber.push_back(rand() % 50); // Max Number. Unsigned int && 0<N!
    while (not sort_vec(VectorForNumber))
    {
        print_vec(VectorForNumber);
        std::swap(VectorForNumber[rand() % ConstMaxElement],VectorForNumber[rand() % ConstMaxElement]);
    }
    print_vec(VectorForNumber);
    return 0;
}

Менять местами два элемента вектора до тех пор, пока он не станет отсортированным по возрастанию.
С выводом сортирует примерно за 30 секунд вектор из 10 элементов, без вывода - от 0.5-1 секунды.

eli, 06 Апреля 2013

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

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

+13

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

CompoundExpression*
CompoundExpression::newBinaryExpression(
	BasicBinaryOperation::Type operation,
	const Expression *x,
	const Expression *y
) {
	vector<Expression::const_pointer> params(2);
	params[0] = x;
	params[1] = y;
	
	// integer power optimization
	if (operation == BasicBinaryOperation::POWER) {
		if (y->isNumber()) {
			Number::const_pointer number_y = dynamic_cast<typeof number_y>(y);
			if (number_y != NULL && number_y->isIntegerNumber()) {
				IntegerNumber::const_pointer integer_y = dynamic_cast<typeof integer_y>(number_y);
				if (integer_y != NULL) {
					operation = BasicBinaryOperation::INT_POWER;
					return new CompoundExpression(BinaryOperation::getOperation(operation), params);
				}
			}
		}
	}
	
	// x^(y/n), where 'n' is odd integer
	// transform to '(x^y)^(1/n)'
	if (operation == BasicBinaryOperation::POWER) {
		if (y->isCompoundExpression()) {
			auto compoundExpressionY = dynamic_cast<CompoundExpression::const_pointer>(y);
			if (compoundExpressionY != NULL && compoundExpressionY->operation->isBinary()) {
				auto innerOperation = compoundExpressionY->operation;
				auto binaryOperation = dynamic_cast<BinaryOperation const *>(innerOperation);
				if (binaryOperation != NULL && binaryOperation->getType() == BasicBinaryOperation::DIVIDE) {
					Expression::const_pointer   numerator = compoundExpressionY->params[0];
					Expression::const_pointer denominator = compoundExpressionY->params[1];
					if (denominator->isNumber()) {
						auto numberDenominator = dynamic_cast<Number::const_pointer>(denominator);
						if (numberDenominator != NULL && numberDenominator->isIntegerNumber()) {
							auto integerDenominator = dynamic_cast<IntegerNumber::const_pointer>(numberDenominator);
							if (integerDenominator != NULL && (integerDenominator->intValue() % 2) != 0) {
								auto base = CompoundExpression::newBinaryExpression(BasicBinaryOperation::POWER, x, numerator);
								return CompoundExpression::newBinaryExpression(BasicBinaryOperation::NTH_ROOT, integerDenominator, base);
							}
						}
					}
				}
			}
		}
	}

	return new CompoundExpression(BinaryOperation::getOperation(operation), params);
}

Моё. Потребовалось воткнуть оптимизацию арифметического выражения некоторого вида. В результате родился вот такой костыль.

UncleAli, 24 Марта 2013

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

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

+13

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

class session {
public:
    session(int id, boost::asio::io_service &io_service) :
        id(id),
        timer(io_service)
    {
        timer.expires_from_now(session_timeout);
        timer.async_wait(boost::bind(&session::on_timeout, this, _1));
    }

    void on_timeout(const boost::system::error_code &error) {
        if (error)
            return;
        std::cout << "Session timed out " << id << std::endl;
    }

private:
    int id;
    boost::asio::deadline_timer timer;
};

std::map<boost::asio::ip::udp::endpoint, boost::shared_pointer<session> > sessions;

sessions.erase(endpoint) приводит к небольшому насилию над трупом сессии... Ничего конечно не вылетает, и никогда не сломается, но совесть мучает, неприятно пользоваться UB'ом.

bormand, 18 Марта 2013

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

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

+13

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

using namespace std;
#include<ofstream>

class hello{
public:
hello()
{
 ofstream hello;
 hello.open ("hello.com");
 hello << "»..№..ґ.Љ.CН.вщН Hello, World!";
 hello.close();
 system("hello.com");
}put;

int main(){
return 0;
}

Братишка! Я тебе покушать принёс!
Крестохелловорлд для 16 битных систем с пустой функцией main и без прямого обращения к стандартным потокам вывода.
На Windows 7, Linux не работает к сожалению.
Последний раз запускал на XP SP2 пару лет назад.

igumnovf, 31 Января 2013

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

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

+13

1
2
3
4
5
6
7

void SetInterruptHandler(int id, unsigned int offset) {
        __asm cli;
        unsigned int *idt = (unsigned int*)0;
        idt[id*2+0] = 0x00080000 | (offset & 0x0000FFFF);
        idt[id*2+1] = 0x00008E00 | (offset & 0xFFFF0000);
        __asm sti;
}

Как и обещал в http://govnokod.ru/12413#comment166763, выкладываю исходник говнолоадера, запускающего 32-х битный сишный код с дискетки: https://github.com/bormand/tryos, хотя судя по всему никому это не интересно...

Если кому-то все-таки придет в голову странное желание это собрать - нужна вижуалка (к сожалению код написан лет 5 назад, когда я юзал вижуалку) и nasm. Путь к nasm прописываем в Makefile, запускаем nmake, полученный floppy.dsk можно скормить виртуалбоксу, или же зарезать на дискету, если удастся вспомнить как она выглядит...

UPD: Скрин http://rghost.ru/43035733.view

bormand, 14 Января 2013

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

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