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

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

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

1. PHP (5703)
2. C++ (2738)
3. Куча (2335)
4. JavaScript (2030)
5. C# (1929)
6. Java (1503)
7. Си (1102)
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 (31)
21. Swift (29)
22. Haskell (28)
23. Kotlin (15)

1. Java / Говнокод #7950

   +81

   1. 1

   item.getName().equals("")==false

   Запостил: Art, 23 Сентября 2011

   Tweet

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

   • karamba 23.09.2011 14:48 # +4

     Вот почему надо включать checkstyle на уровне репозитория.

     Ответить

     • Art 23.09.2011 14:49 # 0

       А ведь хорошая мысль...

       Ответить

       • karamba 23.09.2011 15:06 # +2

         У нас настроен при комите: http://checkstyle.sourceforge.net/
         Там много что есть, SimplifyBooleanExpression, SimplifyBooleanReturn. Одно плохо, если его не включили сразу и проект уже большой, то будут проблемы и проще не включать вообще. Иначе будут жестокие diffы, но все новые проекты только с ним и никак без него.

         Ответить

     • roman-kashitsyn 23.09.2011 15:11 # 0

       Да мне кажется, что та же Idea будет ворнинги на таких косяках давать.

       Ответить

       • Art 23.09.2011 15:13 # 0

         Аха, вот только этот код в NetBeans писался...

         Ответить

       • karamba 23.09.2011 15:34 # +1

         Нельзя полагаться только на IDE, централизованная проверка не помешает. Мне вот Eclipse не выдает( Да и всегда найдется человек, который в спешке в 18:59 "зафиксит" баг отредактировав файл в NotePad, зальет, сломает билд, и успеет свалить домой. Ведь IDE дольше открывать)
         
         К тому же Idea не показывает такие вещи, как equals без hashcode, Utils классы без private конструктора и прочие стандартные косяки, которые иногда пролетают через ревью.

         Ответить

         • roman-kashitsyn 23.09.2011 15:43 # 0

           > Нельзя полагаться только на IDE
           Тут я с вами полностью согласен. Сам довольно долго принципиально писал на Java в emacs.

           Ответить

   • roman-kashitsyn 23.09.2011 14:50 # +1

     Не самое страшное гавно, но некрасиво, да. NullPointer словить можно.

     Ответить

   • kibberpunk 18.10.2011 17:58 # 0

     isEmpty() - это для трусов)))

     Ответить

     • bugmenot 18.10.2011 18:20 # 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] (подробнее)

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