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

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

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

1. PHP (5703)
2. C++ (2740)
3. Куча (2338)
4. JavaScript (2030)
5. C# (1929)
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. Java / Говнокод #11441

   +74

   1. 1
   2. 2
   3. 3

   //code...
   item.setInUse((map.getnStreamActive().equals("1") ? true : false));
   //...code

   Писал тим лид одного из вендоров проекта.
   nStreamActive - Integer

   Запостил: ingenuus, 19 Июля 2012

   Tweet

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

   • ingenuus 19.07.2012 10:50 # 0

     Нашел потому что с NullPointerException валилось

     Ответить

     • absolut 19.07.2012 10:55 # 0

       кто был null?

       Ответить

       • mrWindMak3r 19.07.2012 10:58 # 0

         Вот понараздают званий @"тимлидов", а потом грабельками говешко закидывают и флажками помечают...

         Ответить

       • ingenuus 19.07.2012 11:05 # 0

         Очевидно, map.getnStreamActive()

         Ответить

         • absolut 19.07.2012 11:11 # +2

           ну, может быть и map, и item

           Ответить

           • ingenuus 19.07.2012 11:16 # +3

             может, но это было бы не так символично.
             Написал бы "1".equals() никто бы и не заметил :)

             Ответить

   • roman-kashitsyn 19.07.2012 11:29 # 0

     Пехепешники всюду

     http://govnokod.ru/11386#comment146358

     Ответить

   • roman-kashitsyn 19.07.2012 12:41 # 0

     А что, количество активных потоков может быть только 1 или 0?

     Ответить

     • ingenuus 19.07.2012 12:54 # 0

       Это класс для маппинга mssql. Активный/неактивный.

       Ответить

       • roman-kashitsyn 19.07.2012 12:57 # 0

         тогда почему бы не isActive() ?

         Ответить

         • ingenuus 19.07.2012 13:01 # +1

           Наследство от предыдущих разработчиков. Стараемся не менять без реальной необходимости, то что уже написано. Чревато.

           Ответить

           • absolut 19.07.2012 14:00 # 0

             какое исправление было сделано в этот раз?

             Ответить

             • roman-kashitsyn 19.07.2012 14:04 # +7

               сменить обои на менее скучные

               Ответить

               • Lure Of Chaos 19.07.2012 19:57 # +5

                 да вы обои такие скучные

                 Ответить

   • Lure Of Chaos 19.07.2012 18:21 # 0

     тим не обязан уметь прожить

     Ответить

     • bormand 19.07.2012 18:42 # 0

       зато тим обязан уметь выжить

       Ответить

       • rat4 19.07.2012 19:41 # 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] (подробнее)

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