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

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

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

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

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

   +61

   1. 1

   int len = dynamic_cast<GatewayRec *>(*Iter)->PrefixMatch(aliases, matchedalias, priority);

   gnugk

   Запостил: bormand, 02 Сентября 2014

   Tweet

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

   • Soul_re@ver 02.09.2014 09:16 # +1

     А потом это собирается на 64битной платформе...
     
     А зачем им указатель в инте?

     Ответить

     • bormand 02.09.2014 10:05 # +4

       > А потом это собирается на 64битной платформе...
       ... и ничего не ломается.
       
       > А зачем им указатель в инте?
       А указатель не в инте. Инт - результат вызова PrefixMatch'а над результатом динамического каста.
       
       Тут подвох в другом ;)

       Ответить

       • lucidfox 02.09.2014 10:16 # +4

         В том, что указатель не проверяется после dynamic_cast?
         
         Я сначала подумала, что он выбросит bad_cast, но потом посмотрела документацию - нет, оказывается, bad_cast кидается только со ссылками, а с указателями возвращается нулевой указатель.
         
         М-да. Крестопроблемы.

         Ответить

         • roman-kashitsyn 02.09.2014 10:23 # +6

           > М-да. Крестопроблемы.
           В чём крестопроблемы? В языке намеренно два вида каста. Ссылки не могутдолжны быть невалидными, поэтому dynamic_cast на них может сигнализировать об ошибке только исключением.
           
           Указатели могут быть nullptr, dynamic_cast для указателей возвращает nullptr. Есть даже специальная конструкция, полезная, когда ты не уверен в успешности даункаста:

           if (Derived *d = dynamic_cast<Derived *>(base)) {
               d->do_custom_stuff();
           }

           Ответить

           • lucidfox 02.09.2014 11:33 # −1

             > Ссылки не могутдолжны быть невалидными, поэтому dynamic_cast на них может сигнализировать об ошибке только исключением.
             
             Я знаю, но это не повод делать два разных поведения в зависимости от того, что кастуется.
             
             "По правильному" неверный dynamic_cast должен бросать исключение. Всегда. Проверять возвращаемое значение - это стиль сишки, ретроградство эксепшенофобов. Хотите нулевой указатель - ловите исключение и присваивайте. Ну или сначала проверяйте, прежде чем кастовать.
             
             Давайте тогда обычный new будет при нехватке памяти возвращать нулевой указатель, а new[] - кидать bad_alloc. Ну а что, разные операторы же!
             
             > Есть даже специальная конструкция, полезная, когда ты не уверен в успешности даункаста
             
             Костыльный закос под жабовский instanceof поверх костыльной технологии. И так в ваших крестах всё.

             Ответить

             • roman-kashitsyn 02.09.2014 11:43 # +3

               > бросать исключение. Всегда.
               
               Исключения - опциональная фича языка, и некоторые компиляют код с выключенными исключениями, хоть я это и не одобряю.
               
               > Костыльный закос под жабовский instanceof
               
               Ну вообще-то dynamic_cast появился до рождения самой жабы и идея скорее заимствована из тайп-свича в Simula.
               А жабий instanceof - ещё бОльшее убожество, там потом ещё один каст писать надо.

               Ответить

               • lucidfox 02.09.2014 11:50 # 0

                 > Исключения - опциональная фича языка, и некоторые компиляют код с выключенными исключениями, хоть я это и не одобряю.
                 
                 RTTI (включая dynamic_cast) тоже отключают, и что? Ретрограды должны страдать. Либо уж и исключения, и RTTI, либо ни того, ни другого.
                 
                 > Ну вообще-то dynamic_cast появился до рождения самой жабы и идея скорее заимствована из тайп-свича в Simula.
                 
                 Не сам dynamic_cast, а вот такая идиома. Или она тоже до Жабы появилась?
                 
                 > А жабий instanceof - ещё бОльшее убожество, там потом ещё один каст писать надо.
                 
                 Тут согласна, компилятор мог бы и без каста догадаться, по одному instanceof. Кстати, компилятор Ceylon так и делает. Например, такой код скомпилируется:
                 
                 

                 Object obj = <...>;
                 
                 if (is Integer obj) {
                     print(obj + 1 < 42);
                 }

                 Ответить

                 • roman-kashitsyn 02.09.2014 11:52 # 0

                   > RTTI (включая dynamic_cast)
                   СЕНСАЦИЯ! dynamic_cast можно использовать с отключенным RTTI.
                   
                   > Или она тоже до Жабы появилась?
                   Весьма вероятно.

                   Ответить

                   • lucidfox 02.09.2014 12:05 # 0

                     GCC позволяет, если может разрешить dynamic_cast на стадии компиляции. В общем случае - нет:
                     
                     http://pastebin.com/3SRktZZR

                     Ответить

                     • roman-kashitsyn 02.09.2014 12:08 # 0

                       > GCC позволяет, если может разрешить dynamic_cast на стадии компиляции
                       
                       Я это и имел в виду.
                       
                       А вообще суровые мужики пишут свой dynamic_cast, работающий за O(1), а не юзают стандартное тормозное говно.
                       Иногда нужно для реализации алгебраических типов данных, например.

                       Ответить

                       • absolut 04.09.2014 07:09 # +1

                         После упоминания суровых мужиков диалог прервался.

                         Ответить

                         • roman-kashitsyn 04.09.2014 13:11 # +1

                           А кстати, вот ещё разница есть: операторам кастов нужно всегда указатели проверять на null чтобы из нуля случайно не вычесть. А с сылками это делать не обязательно.
                           ПИРФОМАНС

                           Ответить

                         • kegdan 04.09.2014 15:24 # −2

                           боятся-с сурового сибирского мужика Борманда

                           Ответить

                           • guest 04.09.2014 23:18 # +2

                             Хватит лизать жопы, на тебя тут никто не польстится.
                             Ищи ебарей в другом месте.

                             Ответить

                             • guest 05.09.2014 08:56 # 0

                               Он забавный! Без него нам будет скучно.

                               Ответить

               • absolut 04.09.2014 07:15 # 0

                 Задумался: а что происходит с ошибочным приведением ссылок при отсутствии исключений в реализации?

                 Ответить

                 • defecate-plusplus 04.09.2014 08:14 # 0

                   приложение допустило недопустимое?
                   
                   так-то плохую ссылку сделать можно и без динамик-каста, приложение упадет при обращении к ней
                   http://ideone.com/jyWpwp

                   Ответить

                   • absolut 04.09.2014 11:35 # 0

                     >упадет при обращении к ней
                     ещё бы. ведь адрес такой ссылки нулевой:
                     http://ideone.com/ME9Mhm

                     Ответить

                     • defecate-plusplus 04.09.2014 11:50 # +2

                       насчет того, что должно быть в ней, это UB

                       Ответить

                       • absolut 04.09.2014 12:54 # +2

                         согласен, но тогда может и не упасть.

                         Ответить

                         • defecate-plusplus 04.09.2014 13:01 # +1

                           дык у тебя и dynamic_cast может иногда выдать хорошее значение

                           Ответить

                   • bormand 04.09.2014 11:46 # 0

                     УМВР ЧЯНТД? http://ideone.com/MavDCH

                     Ответить

                     • absolut 04.09.2014 11:51 # 0

                       а как же данные объекта потрогать?
                       http://ideone.com/OshNcu

                       Ответить

                 • bormand 04.09.2014 08:50 # +1

                   std::terminate() по идее. В общем то же самое, что и если бы ты его не поймал.

                   Ответить

                   • absolut 04.09.2014 11:26 # +1

                     угу. А поймать его не получится, т.к. с -fno_exceptions нельзя собрать с catch в коде.

                     Ответить

             • bormand 02.09.2014 11:48 # +2

               > Давайте тогда обычный new будет при нехватке памяти возвращать нулевой указатель, а new[] - кидать bad_alloc.
               
               А там, кстати, есть new, который возвращает нулевой указатель :)

               Ответить

               • roman-kashitsyn 02.09.2014 11:50 # +2

                 > есть new, который возвращает нулевой указатель
                 
                 ага, new (std::nothrow)

                 Ответить

                 • lucidfox 02.09.2014 11:51 # 0

                   Про nothrow я знаю. Кстати, ничто не мешало ввести аналогичный синтаксис и для dynamic_cast:
                   
                   

                   dynamic_cast<Derived *>(base, std::nothrow)

                   Ответить

                   • defecate-plusplus 02.09.2014 12:02 # +3

                     емним, наличие ровно одного параметра в *_cast - политика партии
                     этим даже страдает boost::lexical_cast, их просили добавить опционально, например, указание локали, отказались

                     Ответить

                     • 3_dar 23.11.2020 22:09 # 0

                       Не отвечай трапу.

                       Ответить

                       • CHayT 23.11.2020 22:17 # 0

                         У тебя просто трапа нормального не было.

                         Ответить

                         • guest3 23.11.2020 22:18 # 0

                           Не отвечай пожалуйста уёбку.

                           Ответить

             • TarasB 05.09.2014 09:29 # −2

               Эксепшоны с РАИИ это большой геморрой. Минуснул, короче, тупое ньюфажедрочерство.

               Ответить

               • roman-kashitsyn 05.09.2014 10:08 # +5

                 а эксепшоны без РАИИ это вообще путь в ад

                 Ответить

                 • bormand 05.09.2014 11:58 # 0

                   В особенности с GC и динамической питуизацией.

                   Ответить

                 • TarasB 05.09.2014 12:30 # 0

                   Верно. Либо геморрой, либо путь в ад. Отлично!

                   Ответить

                   • kegdan 05.09.2014 14:58 # −1

                     хотя с геморроем в рай не пускают, а в аду еще тот геморрой...

                     Ответить

         • tirinox 02.09.2014 10:35 # 0

           > подумала
           > Крестопроблемы
           O_o

           Ответить

           • roman-kashitsyn 02.09.2014 10:53 # +1

             http://govnokod.ru/8329

             Как карлсон прям

             Ответить

             • tirinox 02.09.2014 10:56 # 0

               Не ушла значит.
               Да, не. Просто необычно это :)

               Ответить

               • roman-kashitsyn 02.09.2014 11:13 # +1

                 http://www.youtube.com/watch?v=YX3iRjKj7C0&t=8m50s

                 
                 C++ is a man's language. You have to have serious cohones to sling that code around. You taste the testosterone running through every line of that code.
                 Java is more ... estrogen like language.
                 ...
                 C++ programmer is a wild hunter that stabs a bore in a heart with a spear.

                 Ответить

         • eth0 02.09.2014 16:38 # +4

           > lucidfox
           Едические рога! Оно вернулось!

           Ответить

           • 3.14159265 02.09.2014 17:18 # +3

             Практически все они возвращаются (особенно те кто громко хлюопал дверью в стиле: злые вы - я ухожу). #7736 например.
             Хотя Webkill [ненастоящий] таки вроде помер.

             Ответить

       • roman-kashitsyn 02.09.2014 10:17 # +4

         > Тут подвох в другом ;)
         непроверенный dynamic_cast?

         Ответить

   • roman-kashitsyn 02.09.2014 11:16 # +6

     Кто-то согнал ботов и минусует весёлые комменты, плюсуя занудные. Не надо так.

     Ответить

     • defecate-plusplus 02.09.2014 11:20 # +2

       кто-то балуется на переменке
       не во все школы ещё завезли глушилки сотовой связи

       Ответить

       • absolut 04.09.2014 07:13 # 0

         На моей школе так и вовсе антенна мегафона стояла. Пока директора не привлекли за незаконную установку. Уровень связи потом долго еще к прежнему варианту вернуть не могли.

         Ответить

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

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

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

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

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

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

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