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

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

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

1. PHP (5698)
2. C++ (2733)
3. Куча (2323)
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 / Говнокод #21142

   −51

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

   @Override
       public OrderDto doNothing(OrderDto orderDto) {
           return orderDto;
       }

   Из продакшна. 1169 строка...

   Запостил: ne_tot_nik, 08 Сентября 2016

   Tweet

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

   • huesto 08.09.2016 18:08 # −2

     > 1169 строка
     
     ебать, спасибо, это же меняет дело

     Ответить

     • void_main 08.09.2016 18:17 # −19

       показать все, что скрытоЕстественно меняет, умняша моя. Файлы больше 1000 строк - это уже говнокод.

       Ответить

       • huesto 08.09.2016 18:20 # −1

         тогда запости здеся пустой файл, и напиши что в нем 20000 строк

         Ответить

         • huesto 08.09.2016 18:21 # 0

           я к тому, что раз написал, то гони ссылку на полный файл хотябы

           Ответить

         • 1024-- 08.09.2016 18:28 # 0

           Вот же есть: http://govnokod.ru/18201#comment286878

           Ответить

           • huesto 08.09.2016 18:40 # 0

             бага ещё повторяется?

             Ответить

             • 1024-- 08.09.2016 19:20 # 0

               вроде нет, конец отрезается (если я ничего не напутал)

               Ответить

               • 1024-- 08.09.2016 19:22 # 0

                 лол... работает, только надо хитро

                 Ответить

                 • guest0 08.09.2016 19:25 # −20

                   показать все, что скрыто
                   .
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   
                   .

                   Ответить

                   • 1024-- 08.09.2016 19:35 # 0

                     <2000, коротковато

                     Ответить

                 • guest0 08.09.2016 19:27 # −20

                   показать все, что скрыто████████████████████████████████████████ ████████████████████████████████████████ ████████████████████████████████████████ ████████████████████████████████████████ ████████████████████████████████████████ ████████████████████████████████████████ ████████████████████████████████████████ ████████████████████████████████████████ ████████████████████████████████████████ ████████████████████████████████████████ ██████

                   Ответить

   • roman-kashitsyn 08.09.2016 18:15 # +2

     > @Override
     Всё правильно, есть больше одного способа ничего не делать.

     Ответить

     • barop 08.09.2016 22:17 # +1

       Скорее всего этот метод определен в интерфейсе, причем абстрактный класс использовать было нельзя, потому что API для ремоутинга или Dependency Injection умеет только интерфейсы
       
       Потому строчку
       

       @Override
           public OrderDto doNothing(OrderDto orderDto) {
               return orderDto;
           }

       
       пришлось повторить во всех девятьсот пятидесяти семи data transfer objects, которые есть в проекте.

       Ответить

       • bagor 09.09.2016 00:13 # −20

         показать все, что скрытоКакой багор )))

         Ответить

   • Elvenfighter 08.09.2016 20:47 # −1

     Так может оригинальный метод (из родительского класса) делает то, что не нужно.

     Ответить

     • besprincypniycentner 09.09.2016 07:36 # +1

       это анти-ООП, наследование нужно чтобы расширять (нет, блять, не анус расширять), а не замещать исходную реализацию

       Ответить

       • bagor 09.09.2016 08:40 # −22

         показать все, что скрытоА, может, всё-таки анус?

         Ответить

         • besprincypniycentner 09.09.2016 09:05 # 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] (подробнее)

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