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

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

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

1. PHP (5703)
2. C++ (2740)
3. Куча (2339)
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++ / Говнокод #11644

   +39

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

   extern const TSafeFloat Pi;//Не использовать до входа в main!!! Возможно она ещё не инициализированна!!!
   //#define PI		3.14159265358979323846
   
   #ifndef M_PI
                            #define M_PI		3.14159265358979323846
   #endif

   Форматирование сохранено. Файл PiConsts.h

   HaskellGovno, 24 Августа 2012

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

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

   +27

   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

   // до рефакторинга
   if(!y) {
       if(!x) {
           if(grid[pos+1] || grid[pos+fieldWidth])
               ret = true;
       } else if(x == fieldWidth - 1) {
           if(grid[pos - 1] || grid[pos+fieldWidth])
               ret = true;
       } else {
           if(grid[pos-1] || grid[pos+1] || grid[pos+fieldWidth])
               ret = true;
       }
   } else if(y == fieldHeigth - 1) {
       if(!x) {
           if(grid[pos+1] || grid[pos-fieldWidth])
               ret = true;
       } else if(x == fieldWidth - 1) {
           if(grid[pos - 1] || grid[pos-fieldWidth])
               ret = true;
       } else {
           if(grid[pos-1] || grid[pos+1] || grid[pos-fieldWidth])
               ret = true;
       }
   } else {
       if(!x) {
           if(grid[pos+1] || grid[pos+fieldWidth] || grid[pos-fieldWidth])
               ret = true;
       } else if(x == fieldWidth - 1) {
           if(grid[pos - 1] || grid[pos+fieldWidth] || grid[pos+fieldWidth])
               ret = true;
       } else {
           if(grid[pos - 1] || grid[pos+fieldWidth] || grid[pos-fieldWidth] || grid[pos + 1])
               ret = true;
       }
   }
   
   // после рефакторинга
   bool ret = getGridPoint(x-1, y) || getGridPoint(x+1, y) || getGridPoint(x, y-1) || getGridPoint(x, y+1))

   Код одного из моих друзей. Проверяет закрашена ли хотя бы одна клеточка вокруг указанной...

   bormand, 20 Августа 2012

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

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

   +19

   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

   ??=include <iostream>
    
   template <typename T> class TSingleton: public T
   ??<
      public:
         static T& instance(void)
         ??<
            static T instance;
            return instance;
         ??>;
   ??>;
    
   class TOscillStatisticko
   ??<
      protected:
           TOscillStatisticko(void):_countNewInSmallPool(0), _countDeleteInSmallPool(0) ??<??>;
           
           friend class TSingleton <TOscillStatisticko>;
    
      private:
         int _countNewInSmallPool;
         int _countDeleteInSmallPool;
    
      public:
         void addCountNewInSmallPool(void)
         ??<
            _countNewInSmallPool++;
         ??>
    
         int countNewInSmallPool(void)
         ??<
            return _countNewInSmallPool;
         ??>
   ??>;
    
   typedef TSingleton <TOscillStatisticko> TOscillStatistic;
    
   int main() 
   ??<
       TOscillStatistic::instance().addCountNewInSmallPool();
       std::cout<<TOscillStatistic::instance().countNewInSmallPool()<<std::endl;
       std::cout<<"ko"<<std::endl;
   ??>

   http://ideone.com/dt9L9
   Ладно, раз уж я так палюсь, то придется признаться: у меня просто сломалась клавиатура, потому сегодня пришлось написать немного странно...
   Так вот вопрос:, почему, если закомментировать строку

   friend class TSingleton <TOscillStatisticko>;

   http://ideone.com/4WBGh

   HaskellGovno, 20 Августа 2012

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

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

   +38

   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

   const NMath::TLineEquation<> C_E_(C_, E_);
   		const NMath::TLineEquation<> D_A_(D_, A_);
   		const NMath::TVector<2> F_=C_E_.Intersection(D_A_);
   		TSafeFloat lpr=_state._safeDistance->Value()+_state._instrumentRadius->Value();
   		if((F_-B).Length()>lpr)
   		{
   			const NMath::TVector<2> F__=(D_+E_)/2.0;//F
   			//...
   			const NMath::TVector<2> TV=D_-E_;
   			const NMath::TVector<2> F___=PointAtDistance(B,TV,lpr, m90);//F*
   			const NMath::TVector<2> DEDir=rt90(F___-B, m90).Normalize()*10;
   			const NMath::TLineEquation<> DE(F___,F___+DEDir);
   			const NMath::TVector<2> E=DE.Intersection(C_E_);
   			const NMath::TVector<2> D=DE.Intersection(D_A_);
   			TpointerAnyCommand result;
   			result=new TLineCommand(CurrentCommand.SourceCommand(),E-C_,OnOffCorrectionEmpty);
   			_resultDestination.push(result);
   			result=new TLineCommand(CurrentCommand.SourceCommand(),D-E,OnOffCorrectionEmpty);
   			_resultDestination.push(result);
   			result=new TLineCommand(CurrentCommand.SourceCommand(),D_-D,OnOffCorrectionEmpty);
   			_resultDestination.push(result);
   		}

   Говногость, 20 Августа 2012

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

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

   +20

   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

   #include <iostream>
   #include <boost/noncopyable.hpp>
   #include <assert.h>
   
   using namespace std;
   
   class noncopyable
     {
      protected:
         noncopyable() {}
        virtual ~noncopyable() {}
      private:  // emphasize the following members are private
         noncopyable( const noncopyable& );
         virtual const noncopyable& operator=( const noncopyable& )final = delete;
     };
   
   class T: public noncopyable
   {
   public:
      const T& operator=( const T& ){return *this;}
   };
   
   int main() {
      T a;
      a=T();
      std::cout<<"kokoko"<<std::endl;
      return 0;
   }

   http://liveworkspace.org/code/15b5716e7618cc75a6881dfdd46982d5

   
   В чем смысл =delete, если запретить копировать T все равно не удается? Или это ошибка не стандарта, а реализации gcc?

   HaskellGovno, 20 Августа 2012

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

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

   +23

   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

   %:include <iostream>
    
   int main(int argc, char *argv<::>) 
   <%
       if (argc > 1 and argv<:1:> not_eq '\0') <%
           std::cout << "Hello " << argv<:1:> << '\n';
       %>
       std::cout<<"ko"<<std::endl;
   %>
    
    
   
   ??=include <iostream>
    
   int main(int argc, char *argv??(??)) 
   ??<
       if (argc > 1 and argv??(1??) not_eq '\0') ??<
           std::cout << "Hello " << argv??(1??) << '\n';
       ??>
       std::cout<<"ko"<<std::endl;
   ??>

   http://liveworkspace.org/code/150ad59b81b309bbffed963829f4fc9e
   http://liveworkspace.org/code/c64104e4272af4d89f2f74c35c5c2ee3
   Просто оставлю это здесь.

   HaskellGovno, 20 Августа 2012

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

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

   +24

   1. 1

   masko xor_eq mask;

   А вообще жаль, что эту дельфитню включили в стандарт.

   HaskellGovno, 20 Августа 2012

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

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

   +25

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

   uint32 originalDamage = damage;
   
       if (absorb && originalDamage > damage)
           *absorb += (originalDamage - damage);

   Вытащил из 1 эмулятора игры World of WarCraft

   Heisenberg, 19 Августа 2012

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

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

   +20

   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

   class NetGetPlmnListReply: public Message {
                   public:
                       virtual uint8_t command() const;
                       virtual uint8_t subcommand() const;
                       virtual RequestType requestType() const;
                       virtual bool deliver(IUnsolicitedReceiver *receiver);
   
                       static inline bool isTypeOf(Message *message) {
                           return message->command() == 8 && message->subcommand() == 4;
                       }
   
   
   
   std::string inspect() const;
   bool writeToStream(DataStream &stream);
   bool readFromStream(DataStream &stream);
   
   
   
           inline uint8_t count() const { return m_count; }
           inline void setCount(uint8_t count) { m_count = count; }
   
   
   
   
               class PlmnListItem {
               public:
   
   
   std::string inspect() const;
   bool writeToStream(DataStream &stream);
   bool readFromStream(DataStream &stream);
   
   
   
           enum Status {
   
                   Available = 2,
   
                   Current = 3,
   
                   Forbidden = 4,
   
           };
   
           inline Status status() const { return (Status) m_status; }
           inline void setStatus(Status status) { m_status = (uint8_t) status; }
   
   
   
   
   
           inline const std::vector<unsigned char> &plmn() const { return m_plmn; }
           inline void setPlmn(const std::vector<unsigned char> &plmn) { m_plmn = plmn; }
   
   
   private:
   
   
           uint8_t m_status;
   
   
   
           std::vector<unsigned char> m_plmn;
   
   
   
               };
   
   
           inline const std::vector<PlmnListItem> &plmnList() const { return m_plmnList; }
           inline void setPlmnList(const std::vector<PlmnListItem> &plmnList) { m_plmnList = plmnList; }
   
   
   
   private:
   
   
           uint8_t m_count;
   
   
   
           std::vector<PlmnListItem> m_plmnList;
   
   
   
                   };

   Один из тысячи сгенерированных классов.

   Grindars, 18 Августа 2012

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

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

    +20

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

    #include <iostream>
    #include <string>
    
    int main(void)
    {
            char arr[] = "All's well that ends well";
            std::string s(arr, 3000);
            std::cout << s << std::endl;
    return(0);
    }

    Первое что пришло на ум, когда смотрел конструкторы string.
    Собирал: g++-4.5 -Wall -Wextra -pedantic-errors
    ideone.com и liveworkspace.org фильтруют.

    sayidandrtfm, 17 Августа 2012

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

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

• 274
• …
• 125
• 124
• 123
• 122
• 121
• 120
• 119
• 118
• 117
• …
• 1