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

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

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

1. PHP (5701)
2. C++ (2737)
3. Куча (2338)
4. JavaScript (2028)
5. C# (1928)
6. Java (1502)
7. Си (1102)
8. Pascal (647)
9. Python (582)
10. 1C (534)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (198)
15. Perl (194)
16. Assembler (146)
17. Ruby (142)
18. VisualBasic (138)
19. Lua (49)
20. Go (30)
21. Swift (27)
22. Haskell (26)
23. Kotlin (13)

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

   −16

   1. 1
   2. 2
   3. 3

   void setOn(const bool &on) {
       // ...
   }

   А вот чего только не встретишь в коде...

   Запостил: glook, 29 Июня 2012

   Tweet

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

   • bormand 29.06.2012 13:11 # 0

     А еще есть bool on() const { ... } ?

     Ответить

     • defecate-plusplus 29.06.2012 13:26 # +3

       судя по стилю, должно быть const bool & getOn() const { ...

       Ответить

     • glook 29.06.2012 13:35 # +1

       И не только, есть ещё bool isOn() const { return on(); }

       Ответить

       • bormand 29.06.2012 13:57 # 0

         А класс называется On?

         Ответить

         • roman-kashitsyn 29.06.2012 13:59 # 0

           class She { };

           Ответить

         • glook 29.06.2012 14:05 # +1

           Класс носит незамысловатое название Guard. =)

           Ответить

           • guest 29.06.2012 18:48 # +3

             class Polizei: virtual public Guard{
             };
             
             bool Fire = true;
             Polizei Paul;
             
             Paul.setOn(Fire);

             Ответить

             • guest 30.06.2012 16:04 # −3

               Ааааааааааааааааа!!!!!!!!
               Ты меня зажёг, бой, я буду с тобой %))))))))

               Ответить

   • TarasB 29.06.2012 18:55 # 0

     Может это дословный перевод с Ады. Там всё по умолчанию передаётся как const& (правда для boolean Ада это компилирет в передачу значения, а не указателя, а кресты -хз)

     Ответить

     • glook 29.06.2012 18:58 # 0

       Да если бы. Просто кое-кто так написал.

       Ответить

     • bormand 29.06.2012 19:12 # +1

       Уважающие себя крестокомпиляторы для таких примитивных случаев const & тоже превращают в передачу копии.
       
       Так что не так и ужасен этот const bool & (хотя, конечно, вне шаблонов это говно).

       Ответить

   • Psionic 29.06.2012 19:17 # +1

     А нефиг тимлидам так активно мозги полоскать джуниорам, по поводу константных ссылок - вот и имеем такую дурную паранойю.

     Ответить

     • bormand 29.06.2012 19:43 # +6

       Плохие тимлиды значит, раз полоскание про константные ссылки до конца не довели.

       Ответить

   • Fai 30.06.2012 06:56 # 0

     On& setOn( const On& on ) const;

     Ответить

     • Fai 30.06.2012 06:59 # 0

       enum On {
       on_off = 0;
       on_on = 1;
       on_err = 2;
       };

       Ответить

     • bormand 30.06.2012 07:21 # +2

       Эм. А как он будет set если он const?

       Ответить

       • defecate-plusplus 30.06.2012 10:35 # +1

         внутри член mutable, либо const_cast в теле
         собственно, раз это какой то guard, то смысл есть -быть атомарным даже при константном обращении

         Ответить

       • Fai 30.06.2012 23:13 # +1

         setOn( любая_хрень ) = on_off;
         
         Ну всё-таки уровень индуизма должен зашкаливать.

         Ответить

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

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

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

   Помни, guest, за тобой могут следить!

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

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

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