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

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

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

1. PHP (5703)
2. C++ (2740)
3. Куча (2337)
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 / Говнокод #4481

   +145

   1. 1
   2. 2
   3. 3
   4. 4
   5. 5

   private static final Color COLOR_GREEN = Color.GREEN;
     private static final Color COLOR_GREEN_DARK = JBoard.COLOR_GREEN;
     private static final Color COLOR_RED = Color.RED;
     private static final Color COLOR_RED_DARK = JBoard.COLOR_RED;
     private static final Color COLR_GRAY = Color.GRAY;

   упс, это я случайно )))) если вы это видите, то я это уже рефакторю ))))

   Запостил: Lure Of Chaos, 29 Октября 2010

   Tweet

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

   • seonull 29.10.2010 15:41 # +2

     на рефакторинг предлагаю позвать обфускатора http://govnokod.ru/4469 . даешь замену меджик фордс на меджик нумберс!

     Ответить

   • absolut 01.11.2010 00:03 # 0

     А как так получается, что однородные константы формируются из разных мест ?
     Кто из Color., кто из JBoard.

     Ответить

     • Lure Of Chaos 01.11.2010 01:27 # 0

       это определено в JBoard. Т.е. COLOR_GREEN =..., потом COLOR_GREEN_DARK = COLOR_GREEN

       Ответить

       • absolut 01.11.2010 09:31 # 0

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

         Ответить

         • KirAmp 01.11.2010 10:29 # 0

           Вот мне почему-то кажется, что автор кода решил сделать парочку констант (к которым он обращается в коде), но временно не стал заморачиваться с их значением...

           Ответить

           • Lure Of Chaos 01.11.2010 13:56 # 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] (подробнее)

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