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

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• 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. Java / Говнокод #1485

   +143.5

   1. 1

   hideVoting = (disabled || getVisitor().isAnonymous()) ? true : false;

   Запостил: guest, 07 Августа 2009

   Tweet

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

   • guest 07.08.2009 10:36 # 0

     Hott Dogg:
     отличный код

     Ответить

   • guest 07.08.2009 11:20 # 0

     баян типа
     (булевое выражение) ? true : false

     Ответить

     • Oleg_quadro 11.08.2009 17:56 # +1

       Код похоже Javascript, а не Java.
       disabled || getVisitor().isAnonymous() не будет возвращать true или false
       а будет
       disabled или getVisitor().isAnonymous()
       А они могут быть чему угодно, хоть строкой.
       так то.
       Так что ? true : false нужно

       Ответить

       • UNV 12.08.2009 09:22 # +2

         Чтобы переконвертить выражение в true/false в JavaScript достаточно поставить перед ним два отрицания - !!. Впрочем, это читается хуже тернарного оператора.

         Ответить

         • пъянь 16.08.2009 16:37 # 0

           код нормальный.
           именно так надо писать!!!!
           
           разделение условия и присвоения

           Ответить

   • guest 07.08.2009 11:41 # −3

     Доброгость:
     #2
     
     Баян, то баян, но я вот иногда специально так пишу, чтобы нагляднее было.

     Ответить

   • guest 07.08.2009 15:32 # +2

     [quote=Доброгость]специально так пишу, чтобы нагляднее было[/quote]
     Для так кого нагляднее?

     Ответить

   • guest 10.08.2009 17:51 # 0

     Иногда такая хрень появляется после рефакторинга. IDE-шки ещё не настолько умные, чтобы автоматически упрощать такие выражения

     Ответить

     • UNV 12.08.2009 09:24 # 0

       Идея такие выражения специально подсвечивает и предлагает заоптимизировать - убрав лишний тернарный оператор.

       Ответить

   • guest 10.08.2009 20:59 # +1

     Не вижу в этом ничего плохого

     Ответить

     • UNV 12.08.2009 09:25 # −3

       Плохо, что размывает логику. Однако более понятно для начинающих. В общем, однозначно нельзя сказать - добро или зло.

       Ответить

       • пъянь 16.08.2009 16:38 # 0

         наоборот не использование такого написания размывает логику

         Ответить

   • guest 11.08.2009 12:46 # 0

     хм... непонятно где же гавно зарыто

     Ответить

   • Bobby 05.04.2010 12:00 # −1

     Не говнокод, просто лишний оператор. А вообще, явное лучше неявного.

     Ответить

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

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

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

   Где здесь C++, guest?!

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

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

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