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

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

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

1. PHP (5698)
2. C++ (2733)
3. Куча (2326)
4. JavaScript (2027)
5. C# (1925)
6. Java (1501)
7. Си (1095)
8. Pascal (647)
9. Python (582)
10. 1C (532)
11. SQL (431)
12. ActionScript (292)
13. Objective C (202)
14. bash (197)
15. Perl (194)
16. Assembler (146)
17. Ruby (142)
18. VisualBasic (138)
19. Lua (49)
20. Go (29)
21. Swift (27)
22. Haskell (26)
23. Kotlin (13)

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

   +173

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

   NNLayer::~NNLayer()
   {
   	// call Initialize(); makes sense if you think
   	
   	Initialize();
   }

   Отсюда: http://www.codeproject.com/KB/library/NeuralNetRecognition.aspx

   Запостил: eigenein, 16 Мая 2011

   Tweet

   Комментарии (28) RSS

   • Lure Of Chaos 16.05.2011 23:17 # 0

     есть ли жизнь после смерти? )

     Ответить

   • absolut 17.05.2011 08:34 # 0

     похоронный марафет

     Ответить

   • JeremyW 17.05.2011 09:32 # 0

     Пытаемся надышаться перед смертью?

     Ответить

   • TarasB 17.05.2011 10:22 # 0

     А почему столь короткий деструктор описан не прямо внутри класса?

     Ответить

   • ctm 17.05.2011 10:50 # 0

     интересно, что внутри Initialize()
     хотя название то еще :(

     Ответить

     • eigenein 17.05.2011 11:49 # 0

       void NNLayer::Initialize()
       {
       	VectorWeights::iterator wit;
       	VectorNeurons::iterator nit;
       	
       	for( nit=m_Neurons.begin(); nit<m_Neurons.end(); nit++ )
       	{
       		delete *nit;
       	}
       	
       	for( wit=m_Weights.begin(); wit<m_Weights.end(); wit++ )
       	{
       		delete *wit;
       	}
       	
       	m_Weights.clear();
       	m_Neurons.clear();
       	
       	m_bFloatingPointWarning = false;
       }

       Ответить

       • guest 17.05.2011 11:58 # +2

         DeInitialize?

         Ответить

         • eigenein 17.05.2011 12:01 # +1

           Тоже нет :) Ибо
           

           NNLayer::NNLayer() :
           label( _T("") ), m_pPrevLayer( NULL )
           {
           	Initialize();
           }
           
           NNLayer::NNLayer( LPCTSTR str, NNLayer* pPrev /* =NULL */ ) :
           label( str ), m_pPrevLayer( pPrev )
           {
           	Initialize();
           }

           Ответить

           • guest 17.05.2011 12:31 # −1

             Синий цвет.

             Ответить

             • TarasB 17.05.2011 13:33 # +2

               Красный цвет.

               Ответить

               • guest 17.05.2011 13:37 # 0

                 Хуй пойми что

                 Ответить

                 • absolut 17.05.2011 13:50 # +5

                   RGB вообще-то

                   Ответить

                   • bugmenot 17.05.2011 18:22 # −1

                     ХГБ

                     Ответить

                     • absolut 17.05.2011 20:45 # 0

                       Храни Государя, Боже ?

                       Ответить

           • Lure Of Chaos 17.05.2011 14:34 # 0

             _T("") радует ))

             Ответить

             • absolut 17.05.2011 15:23 # 0

               стандартное ms-style обрамление для получения строк юникода/аскии в зависимости от настроек проекта

               Ответить

               • Lure Of Chaos 17.05.2011 15:40 # +4

                 еще один мелкософт-костыль )

                 Ответить

                 • koodeer 17.05.2011 18:07 # −3

                   Почему костыль? Это удобно!

                   Ответить

                   • carsten 19.06.2011 14:08 # 0

                     в каком месте удобно? "ansi"/unicode - чисто виндовое различие (вне винды большинство api используют в основном напрямую char*, которое обычно utf8), да и то под виндой ansi-функции давно deprecated уж лет десять, наверное.
                     если пишете код под win95 - удобно, правда (даже Win98 и то имеет Unicode Layer).

                     Ответить

           • ctm 18.05.2011 06:43 # +4

             назвали бы Clear() тогда или что-то в этом духе - все было бы понятно

             Ответить

   • carsten 19.06.2011 14:01 # +1

     бесит c++ отсутствием constructor chaining

     Ответить

     • guest 19.06.2011 20:24 # +1

       Можете рассказать поподробнее?

       Ответить

       • absolut 21.06.2011 23:19 # +1

         вероятно имеется в виду вызов одного конструктора из другого

         Ответить

         • guest 21.06.2011 23:44 # 0

           Согласен, неприятно.

           Ответить

     • guest 19.06.2011 21:24 # +1

       typical-vb-c#-user.jpg

       Ответить

   • guest8 08.04.2019 20:47 # −999

     показать все, что скрытоvanished

     Ответить

   • guest8 09.04.2019 10:38 # −999

     показать все, что скрытоvanished

     Ответить

   • guest8 09.04.2019 17:56 # −999

     показать все, что скрытоvanished

     Ответить

   Добавить комментарий

   Ошибка компиляции комментария:

   1. Гости могут высказаться только во вторник, пятницу или субботу

   Семь раз отмерь — один отрежь, guest!

   А не использовать ли нам bbcode?

   • [b]жирный[/b] — жирный
   • [i]курсив[/i] — курсив
   • [u]подчеркнутый[/u] — подчеркнутый
   • [s]перечеркнутый[/s] — перечеркнутый
   • [blink]мигающий[/blink] — мигающий
   • [color=red]цвет[/color] — цвет (подробнее)
   • [size=20]размер[/size] — размер (подробнее)
   • [code=<language>]some code[/code] (подробнее)

   Проверочный код: *