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

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

+11

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

#include <QVector>
#include <QString>

template< class TYPE> 
  class call
  {
    public:
      call( TYPE& vl ): value(vl){}
      TYPE& value;
  };
template< class TYPE> 
  call<TYPE> all( TYPE& vl ) 
    { return call<TYPE>(vl); }

template< class TYPE_OUT, class TYPE> 
  void operator << ( TYPE_OUT& out, call<TYPE>& in )
{
  TYPE& value = in.value;
  for( TYPE::iterator itr = value.begin() ; itr!=value.end(); itr++)
    out << *itr;
}

//**********************************************************************

QVector<int> arr;
QVector<QString> arrStr;
class Cstream{} stream;

void operator << ( Cstream& , const int value)
{
  printf("%i\n",value);
}
void operator << ( Cstream& , const QString& value)
{
  printf("%s\n",value.toAscii().data());
}


int main(int argc, char *argv[])
{
  for( int i=0; i<6;i++)
    arr.push_back(i);
  arrStr << "asd" << "sadas" << "sadsa";

  stream << all(arr);
  stream << all(arrStr);
return 0;
}

http://www.gamedev.ru/flame/forum/?id=171504
> @При копирование на говнокод сохранять авторство.

LispGovno, 11 Января 2013

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

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

+23

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

struct S* gpS;

struct S
{
   // this class has no user-defined default ctor
   void *operator new (size_t size, void*p, int i)
   {
      ((S*)p)->i = i;   // ordinarily, should not initialize
                        // memory contents inside placement new
      return p;
   }
   int i;
};

Код с сайта майкрософт.

LispGovno, 09 Января 2013

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

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

+10

1
2
3
4
5
6
7
8

for (int i = 0; i < 15; i++) {
    // Прикольное место, надо прокомментировать
    // Если наша функция Fork() вернула true, то мы
    // в дочернем процессе и форкаться больше не надо
    // Форканье - это задача родителя
    // Дети этим заниматься не должны
    if (Fork()) break;
}

Создание дочерних процессов. Вот такой костыль. Есть идеи, как улучшить?

kafeman, 07 Января 2013

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

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

+14

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

bool Item::IsBoundByEnchant() const
{
    // Check all enchants for soulbound
    for (uint32 enchant_slot = PERM_ENCHANTMENT_SLOT; enchant_slot < MAX_ENCHANTMENT_SLOT; ++enchant_slot)
    {
        if (enchant_slot > PRISMATIC_ENCHANTMENT_SLOT || enchant_slot < PROP_ENCHANTMENT_SLOT_0)    // not holding enchantment id
            continue;

        if (uint32 enchant_id = GetEnchantmentId(EnchantmentSlot(enchant_slot)))
            if (SpellItemEnchantmentEntry const* enchantEntry = sSpellItemEnchantmentStore.LookupEntry(enchant_id))
                if (enchantEntry->slot & ENCHANTMENT_CAN_SOULBOUND)
                    return true;
    }

    return false;
}

Взял из эмулятора World of WarCraft Trinity 4.3.4 ветка.
Все говно в 6 строчке
PERM_ENCHANTMENT_SLOT = 0
MAX_ENCHANTMENT_SLOT = 15

PRISMATIC_ENCHANTMENT_SLOT = 6
PROP_ENCHANTMENT_SLOT_0 = 10

Heisenberg, 03 Января 2013

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

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

+19

1
2
3
4
5
6
7

template<typename T>
static json_string _itoa(T val) json_nothrow {  
        /*...*/
        long value = (long)val;
    
        /*...*/
}

Продолжаю бороться с поддержкой 64-битных чисел в libjson :/

myaut, 30 Декабря 2012

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

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

+23

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

#include <iostream>
using namespace std;
 
class Class {
public:
    explicit Class(int a) : m_int(a) {}
    
    int get() const { return m_int; }
    
    void swapThis(int a) {
        delete this;
        Class **thisptr = reinterpret_cast<Class**>(&a);
        thisptr--;
        *thisptr = new Class(a);
        cout << "this: " << this << endl
             << "that: " << *thisptr << endl;
    }
private:
    int m_int;
};
 
int main() {
    Class *s = new Class(13);
    s->swapThis(42);
    cout << s->get() << endl;
    delete s;
    
    return 0;
}

Что творит хакингкостылинг + инлайн методы.
http://ideone.com/5Kyitw

Elvenfighter, 29 Декабря 2012

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

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

+24

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

#include <iostream>
#define n 1
int main()
{
using namespace std;
cout << (n < 0 ? 1 : 0)+
((n << 1) < 0 ? 1 : 0)+
((n << 2) < 0 ? 1 : 0)+
((n << 3) < 0 ? 1 : 0)+
((n << 4) < 0 ? 1 : 0)+
((n << 5) < 0 ? 1 : 0)+
((n << 6) < 0 ? 1 : 0)+
((n << 7) < 0 ? 1 : 0)+
((n << 8) < 0 ? 1 : 0)+
((n << 9) < 0 ? 1 : 0)+
((n << 10) < 0 ? 1 : 0)+
((n << 11) < 0 ? 1 : 0)+
((n << 12) < 0 ? 1 : 0)+
((n << 13) < 0 ? 1 : 0)+
((n << 14) < 0 ? 1 : 0)+
((n << 15) < 0 ? 1 : 0)+
((n << 16) < 0 ? 1 : 0)+
((n << 17) < 0 ? 1 : 0)+
((n << 18) < 0 ? 1 : 0)+
((n << 19) < 0 ? 1 : 0)+
((n << 20) < 0 ? 1 : 0)+
((n << 21) < 0 ? 1 : 0)+
((n << 22) < 0 ? 1 : 0)+
((n << 23) < 0 ? 1 : 0)+
((n << 24) < 0 ? 1 : 0)+
((n << 25) < 0 ? 1 : 0)+
((n << 26) < 0 ? 1 : 0)+
((n << 27) < 0 ? 1 : 0)+
((n << 28) < 0 ? 1 : 0)+
((n << 29) < 0 ? 1 : 0)+
((n << 30) < 0 ? 1 : 0)+
((n << 31) < 0 ? 1 : 0)
<< endl;
return 0;
}

Для заданного пятизначного целого числа подсчитать количество его нулей.
Прямо с экзамена. Преподаватель катается по полу. Не шучу.

LispGovno, 25 Декабря 2012

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

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

+24

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

#ifndef JSON_SPIRIT_READER_TEMPLATE
#define JSON_SPIRIT_READER_TEMPLATE

//          Copyright ***
// Distributed under the MIT License, see accompanying file LICENSE.txt

// json spirit version 4.05

#if defined(_MSC_VER) && (_MSC_VER >= 1020)
# pragma once
#endif

//some classes

#endif

Сначала проверим многократное включение через макросы, а потом # pragma once - чтобы наверняка!

Try, 24 Декабря 2012

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

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

+16

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

#include <iostream>
using namespace std;

struct T{int a;T(){cout<<"T"<<this<<endl;}~T(){cout<<"~T"<<this<<endl;}};
struct D:public T{int b;D(){cout<<"D"<<this<<endl;}~D(){cout<<"~D"<<this<<endl;}};

int main() {
   T* a = new D[2];
   cout<<"mission failure"<<endl;
   delete[] a;   
   return 0;
}

Писал тест специально для Тараса:
Виртуальный деструктор в базовых классах нужен:
*Для вызова деструкторов всех потомков. Иногда это не нужно, тк поля структуры тривиальны. Согласен с Тарасом.
*Для правильного выбора перегруженного оператора delete. Создал одним менеджером памяти, а удалил в другой - это плохо. Но также бывает редко и только в специфичных проектах.
Но вот то, что демонстрируется в этом примере - думаю чего-нибудь такого создатели той библиотеки, что ковырял Тарас, и опасались.
Не просто было вызван деструктор только для предков, забыв о деструкторах потомков, но и:
*Деструктор предков был вызван для this не по тем адресам, где расположены объекты.
Не удивлюсь, если:
*В некоторых кулхацкерских реализациях менеджеров памяти будет удален буфер меньшего размера и куча попортится или оставшаяся не удаленная часть массива в куче просто утекет.
http://liveworkspace.org/code/2X3MR0$8

LispGovno, 22 Декабря 2012

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

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