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

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• Oтзывы

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

1. PHP (5703)
2. C++ (2740)
3. Куча (2341)
4. JavaScript (2030)
5. C# (1930)
6. Java (1503)
7. Си (1103)
8. Pascal (649)
9. Python (585)
10. 1C (536)
11. SQL (434)
12. ActionScript (294)
13. Objective C (204)
14. bash (200)
15. Perl (196)
16. Assembler (148)
17. Ruby (144)
18. VisualBasic (140)
19. Lua (51)
20. Go (32)
21. Swift (29)
22. Haskell (28)
23. Kotlin (15)

1. C++ / Говнокод #13983

   +11

   1. 1
   2. 2
   3. 3
   4. 4
   5. 5

   // This is an example of an exported function.
   SCCONNECT_API int fnSCConnect(void)
   {
   	return 42;
   }

   Не говно, но не мог промолчать!
   Вот такой шаблон функции генерирует MSVC при создании DLL.
   
   MSVC даёт ответ на самый главный вопрос жизни, вселенной и всего такого!

   Little-Horny, 21 Октября 2013

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

2. C++ / Говнокод #13971

   +2

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11
   12. 12
   13. 13
   14. 14

   class pool_exeption: public std::exception
   {
       virtual  const char* what() const throw()
       {
           return "Pool out of bounds";
       }
   }pool_out_of_bounds;
   class pool_exeption2: public std::exception
   {
       virtual  const char* what() const throw()
       {
           return "Pool block is too small";
       }
   }pool_small_block;

   А потом эти исключения кидаются по значению: throw pool_out_of_bounds;
   virtual const char* WAT()

   Yuuri, 18 Октября 2013

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

3. C++ / Говнокод #13952

   +8

   1. 1
   2. 2
   3. 3

   QRegExp re("<cite>‎(.*)</cite>");
   re.setMinimal(true);
   qDebug() << re.indexIn("<cite>http://test.com/</cite>");

   Ошибка, на фикс которой мы с другом убили почти час.

   bormand, 15 Октября 2013

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

4. C++ / Говнокод #13948

   +4

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11
   12. 12
   13. 13
   14. 14
   15. 15
   16. 16
   17. 17
   18. 18
   19. 19
   20. 20
   21. 21
   22. 22
   23. 23
   24. 24
   25. 25
   26. 26
   27. 27
   28. 28
   29. 29
   30. 30
   31. 31
   32. 32
   33. 33
   34. 34
   35. 35
   36. 36
   37. 37
   38. 38
   39. 39
   40. 40
   41. 41
   42. 42
   43. 43
   44. 44
   45. 45
   46. 46
   47. 47
   48. 48
   49. 49
   50. 50
   51. 51
   52. 52
   53. 53
   54. 54
   55. 55
   56. 56
   57. 57
   58. 58
   59. 59
   60. 60
   61. 61
   62. 62
   63. 63
   64. 64
   65. 65
   66. 66
   67. 67
   68. 68
   69. 69
   70. 70
   71. 71
   72. 72
   73. 73
   74. 74
   75. 75
   76. 76
   77. 77
   78. 78
   79. 79
   80. 80
   81. 81
   82. 82
   83. 83

   int DataBase::garbageCollector(string filename){ 
       fstream from(filename.c_str());
       fstream from0((filename+".index").c_str());
       Index x;
       Data d;
       char *a/*,*b*/;
       int oldpos = 0 ;
       bool done = false;
       from.seekg(0,ios::end);
       int dbend = static_cast<int>(from.tellp());
       from0.seekg(0,ios::end);
       int indexend = static_cast<int>(from0.tellp());
       if(dbend == BlockSize || indexend == IndexSize){
   	from.close();
   	from0.close();
   	return 0;
       }
       from.seekg(0,ios::beg);
       from0.seekg(0,ios::beg);
       while(true){ 
           from0 >> x;
           if(from0.eof())break;
           if(!x.Num()){ 
               int count = dbend - x.Pointer() - BlockSize;
               int count0 = indexend - (static_cast<int>(from0.tellp())+1);
               from.seekg(x.Pointer()+BlockSize);
               a = new char[count]; 
               from.read(a,count);
               from.seekp(x.Pointer());
               from.write(a,count);
               delete a;
               a = new char[count0];
               from0.seekp(1,ios::cur);
               int test = from0.tellp();
               from0.read(a,count0);
               from0.clear();
               from0.seekp(indexend - count0 - IndexSize,ios::beg);
               test = from0.tellp();
               from0.write(a,count0);
               test = from0.tellp();
               delete a;
               ofstream to("tmp.base",ios::trunc),to0("tmp.index",ios::trunc);
               a = new char[dbend - BlockSize];
               from.seekg(0,ios::beg);
               from.read(a,dbend - BlockSize);
               to.write(a,dbend - BlockSize);
               delete a;
               a = new char[indexend - IndexSize];
               from0.seekg(0,ios::beg);
               test = from0.tellp();
               from0.read(a,indexend - IndexSize);
               to0.write(a,indexend - IndexSize);
               delete a;
               from.close();
               from0.close();
               if(remove(filename.c_str()) || rename("tmp.base",filename.c_str())) return -1;
               if(remove((filename+".index").c_str()) || rename("tmp.index",(filename+".index").c_str())) return -1;
               oldpos = indexend - count0 - IndexSize;
               done = true;
               break;
           }
       }
       if(done){
           from0.open((filename+".index").c_str());
           while(true){ 
               from0 >> x;
               if(from0.eof())break;
               if(x.Pointer() > static_cast<unsigned>((oldpos/23)*BlockSize) ){ 
                   x.Pointer(x.Pointer()-BlockSize);
                   from0.seekp(-(IndexSize-1),ios::cur);
                   from0 << x;
                   int test = from0.tellp();
               }
               else{ 
                   from0.seekp(-(IndexSize-1),ios::cur);
                   from0 << x;
                   int test = from0.tellp();
                   from0.clear();
               }
           }
       }
       return 0;
   }

   Собираем пустые блоки

   Abbath, 15 Октября 2013

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

5. C++ / Говнокод #13947

   +6

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11
   12. 12
   13. 13
   14. 14
   15. 15
   16. 16
   17. 17
   18. 18
   19. 19
   20. 20
   21. 21
   22. 22
   23. 23
   24. 24
   25. 25
   26. 26
   27. 27
   28. 28
   29. 29
   30. 30
   31. 31
   32. 32
   33. 33
   34. 34
   35. 35
   36. 36
   37. 37
   38. 38
   39. 39
   40. 40
   41. 41
   42. 42
   43. 43
   44. 44
   45. 45
   46. 46
   47. 47
   48. 48
   49. 49
   50. 50
   51. 51
   52. 52
   53. 53
   54. 54
   55. 55
   56. 56
   57. 57
   58. 58
   59. 59
   60. 60
   61. 61
   62. 62
   63. 63
   64. 64
   65. 65
   66. 66
   67. 67
   68. 68
   69. 69
   70. 70
   71. 71
   72. 72
   73. 73
   74. 74
   75. 75
   76. 76
   77. 77
   78. 78
   79. 79
   80. 80
   81. 81
   82. 82
   83. 83
   84. 84
   85. 85
   86. 86
   87. 87
   88. 88

   int DataBase::split(fstream& from,fstream& from0,unsigned int p){
       int pos0 = static_cast<int>(from.tellp());
       char *a,*b;
       Data d,d0;
       Index x,x0;
       from.clear();
       from.seekp(0,ios::end);
       int PosEnd = static_cast<int>(from.tellp());
       PosEnd -=static_cast<int>(p);
       a = new char[PosEnd];
       from.seekp(p,ios::beg);
       from.read(a,PosEnd); 
       from.seekp(p+BlockSize,ios::beg);
       from.write(a,PosEnd); 
       from.seekp(p+BlockSize,ios::beg);
       b = a+5*RecordSize;
       from.write(b,RecordSize*5);
       for (int i = 0; i < 5; ++i){ 
           from << d;
       }
       from.seekp(p+RecordSize*5,ios::beg);
       for (int i = 0; i < 5; ++i){ 
           from << d;
       }
       from.seekg(p+RecordSize*4);
       from >> d;
       int newmax = d.Id(); 
       from0.seekp(0,ios::end);
       int indexend =static_cast<int>(from0.tellp());
       from0.seekg(0,ios::beg);
       while(true){ 
           from0 >> x;
           if(from0.eof())break;
           if(x.Pointer() == p){ 
               int oldpos = static_cast<int>(from0.tellp());
               oldpos++;
               int count = indexend - oldpos;
               char *c = new char[count];
               from0.seekg(1,ios::cur);
               from0.read(c,count);
               from0.seekp(oldpos,ios::beg);
               x0.Max(x.Max()); 
               x0.Num(5);
               x0.Pointer(p + BlockSize);
               from0 << x0;
               from0.write(c,count);
               from0.seekp(oldpos - IndexSize,ios::beg);
               x.Max(newmax); 
               x.Num(5);
               from0 << x;
               delete c;
               from.clear();
               from0.seekp(0,ios::beg);
               break;
           }
       }
       from.clear();
       bool first = false;
       while(true){ 
           from0 >> x;
           if(from0.eof())break;
           if(x.Pointer() > p ){
               if(!first){ 
                   from0.seekp(-(IndexSize-1),ios::cur);
                   from0 << x;
                   int test = from0.tellp();
                   from0.clear();
                   first = true;
                   continue;
               }
               x.Pointer(x.Pointer()+BlockSize);
               from0.seekp(-(IndexSize-1),ios::cur);
               from0 << x;
               int test = from0.tellp();
           }
           else{ 
               from0.seekp(-(IndexSize-1),ios::cur);
               from0 << x;
               int test = from0.tellp();
               from0.clear();
           }
       }
       from.clear();
       from.seekp(pos0);
       delete a;
       //delete b;
       return 0;
   }

   Нарезаем

   Abbath, 15 Октября 2013

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

6. C++ / Говнокод #13944

   +10

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11
   12. 12
   13. 13
   14. 14
   15. 15
   16. 16
   17. 17
   18. 18
   19. 19
   20. 20
   21. 21
   22. 22
   23. 23
   24. 24
   25. 25
   26. 26
   27. 27
   28. 28
   29. 29
   30. 30
   31. 31
   32. 32
   33. 33
   34. 34
   35. 35
   36. 36
   37. 37
   38. 38
   39. 39
   40. 40
   41. 41
   42. 42
   43. 43
   44. 44
   45. 45
   46. 46
   47. 47

   void ImageArea::saveConf(bool def)
   {
       static bool fp = true;
       static QString filename = QString("default.conf");
       if(!def){
           filename = QFileDialog::getSaveFileName(this, tr("Save config"), "", tr("Config files (*.conf)"));
           if(fp){
               QFile file(filename);
               if(file.open(QFile::WriteOnly)){
                   QTextStream str(&file);
                   str << crop[0].x() << " " << crop[0].y() << "\n";
                   str << crop[1].x() << " " << crop[1].y() << "\n";
                   str << square[0].x() << " " << square[0].y() << "\n";
                   str << square[1].x() << " " << square[1].y() << "\n";
                   str << square[2].x() << " " << square[2].y() << "\n";
                   QMessageBox::information(this, tr("Next step"), tr("Put 3 points then press Save again"));
               }else{
                   QMessageBox::warning(this, tr("Error"), tr("Can not open a file"));
               }
               fp = false;
           }else{
               QFile file(filename);
               if(file.open(QFile::Append)){
                   QTextStream str(&file);
                   str << square[0].x() << " " << square[0].y() << "\n";
                   str << square[1].x() << " " << square[1].y() << "\n";
                   str << square[2].x() << " " << square[2].y() << "\n";
               }else{
                   QMessageBox::warning(this, tr("Error"), tr("Can not open a file"));
               }
               fp = true;
           }
       }else{
           QFile file(filename);
           if(file.open(QFile::WriteOnly)){
               QTextStream str(&file);
               str << conf.crop[0].x() << " " << conf.crop[0].y() << "\n";
               str << conf.crop[1].x() << " " << conf.crop[1].y() << "\n";
               str << conf.square[0].x() << " " << conf.square[0].y() << "\n";
               str << conf.square[1].x() << " " << conf.square[1].y() << "\n";
               str << conf.square[2].x() << " " << conf.square[2].y() << "\n";
               str << conf.square0[0].x() << " " << conf.square0[0].y() << "\n";
               str << conf.square0[1].x() << " " << conf.square0[1].y() << "\n";
               str << conf.square0[2].x() << " " << conf.square0[2].y() << "\n";
           }
       }
   }

   Сохраняем конфиг в джва захода.

   Abbath, 14 Октября 2013

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

7. C++ / Говнокод #13942

   +13

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11
   12. 12
   13. 13
   14. 14
   15. 15
   16. 16
   17. 17
   18. 18
   19. 19
   20. 20
   21. 21
   22. 22
   23. 23
   24. 24
   25. 25
   26. 26
   27. 27
   28. 28
   29. 29

   if(ortho){
               if( abs(e->x() - line.p1().x() ) * 2 < abs( e->y() - line.p1().y() ) ){
                   y = e->y();
                   x = line.p1().x();
               }else if(abs(e->x() - line.p1().x() ) > 2 * abs( e->y() - line.p1().y() )){
                   x = e->x();
                   y = line.p1().y();
               }else if(abs(e->x() - line.p1().x() ) * 2 > abs( e->y() - line.p1().y() ) &&
                        abs(e->x() - line.p1().x() ) < abs( e->y() - line.p1().y() )){
                   if((e->x() < line.p1().x() && e->y() < line.p1().y()) || (e->x() > line.p1().x() && e->y() < line.p1().y())){
                       x = e->x();
                       y = line.p1().y() - abs(line.p1().x() - e->x());
                   }else if((e->x() > line.p1().x() && e->y() > line.p1().y()) || (e->x() < line.p1().x() && e->y() > line.p1().y())){
                       x = e->x();
                       y = line.p1().y() + abs(line.p1().x() - e->x());
                   }
               }else if(abs(e->x() - line.p1().x() ) < 2 * abs( e->y() - line.p1().y() ) &&
                        abs(e->x() - line.p1().x() ) > abs( e->y() - line.p1().y() )){
                   if((e->x() < line.p1().x() && e->y() < line.p1().y()) || (e->x() < line.p1().x() && e->y() > line.p1().y())){
                       y = e->y();
                       x = line.p1().x() - abs(line.p1().y() - e->y());
                   }else if((e->x() > line.p1().x() && e->y() > line.p1().y()) || (e->x() > line.p1().x() && e->y() < line.p1().y())){
                       y = e->y();
                       x = line.p1().x() + abs(line.p1().y() - e->y());
                   }
               }else{
                   x = e->x();
                   y = e->y();
               }

   Abbath, 14 Октября 2013

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

8. C++ / Говнокод #13914

   +8

   1. 1
   2. 2
   3. 3
   4. 4

   n = strlen(pName);
   name = new char[n + 1];
   memset(name, 0, n + 1);
   memcpy(name, pName, n);

   боянчик. std::string наверное религия не позволяет. а strdup() слишком С. oh wait...

   Dummy00001, 09 Октября 2013

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

9. C++ / Говнокод #13909

   −4

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11
   12. 12
   13. 13
   14. 14
   15. 15
   16. 16
   17. 17

   static Singleton * g_pInstance = NULL;
   
   Singleton* Singleton::sharedInstance()
   {
       if (g_pInstance)
           return g_pInstance;
       else {
           g_pInstance = new Singleton();
           if (g_pInstance->init()) {
               return g_pInstance;
           } else {
               delete g_pInstance;
               g_pInstance = NULL;
           }
           return g_pInstance;
       }
   }

   Ещё из жизни синглтонов. Даёшь больше return'ов!

   Deacon, 08 Октября 2013

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

10. C++ / Говнокод #13908

    +16

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30

    // in .h file
    class Singleton
    {
    public:
        Singleton();
        ~Singleton();
    private:
        static Singleton* m_Instance;
        friend Singleton& GetInstance();
    };
    
    inline Singleton& GetInstance()
    {
        assert(Singleton::m_Instance);
        return *Singleton::m_Instance;
    }
    
    // in .cpp file
    Singleton* Singleton::m_Instance = NULL;
    
    Singleton::Singleton()
    {
        assert(!m_Instance);
        m_Instance = this;
    }
    
    Singleton::~Singleton()
    {
        m_Instance = NULL;
    }

    Вот такую реализацию синглтона увидел в одном проекте.
    ЗЫ: Для его корректной работы, в main было написано конечно же:
    main() {
    Singleton* s = new Singleton;
    ...
    delete s;
    }

    Deacon, 08 Октября 2013

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

• ← влево
• вправо →

• 274
• …
• 158
• 157
• 156
• 155
• 154
• 153
• 152
• 151
• 150
• …
• 1