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

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

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

1. PHP (5705)
2. C++ (2739)
3. Куча (2357)
4. JavaScript (2030)
5. C# (1928)
6. Java (1502)
7. Си (1115)
8. Pascal (647)
9. Python (583)
10. 1C (538)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (198)
15. Perl (194)
16. Assembler (146)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Go (31)
21. Swift (27)
22. Haskell (26)
23. Kotlin (14)

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

   +70

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11
   12. 12
   13. 13
   14. 14
   15. 15
   16. 16
   17. 17
   18. 18
   19. 19
   20. 20
   21. 21
   22. 22
   23. 23
   24. 24
   25. 25
   26. 26

   protected void writeCaptionForDoubleColumns(HSSFWorkbook wb,
   		HSSFSheet sheet, HSSFRow row, HSSFCellStyle fulBorderCalignFont14,
   		Type1 obj1, Type2 obj2,
   		Type3 obj3, Type4 obj4, Type5 obj5,
   		Type6 obj6, Type7 obj7,
   		boolean writeObj6, boolean writeObj5,
   		boolean isHorizontal, String formName, String formNameCode) {
   	writeCaptionForDoubleColumns(wb, sheet, row, fulBorderCalignFont14,
   			obj1, obj2, obj3, obj4, obj5, obj6,
   			obj7, writeObj6, writeObj5, isHorizontal, formName,
   			formNameCode, true, null);
   }
   
   protected void writeCaptionForDoubleColumns(HSSFWorkbook wb,
   		HSSFSheet sheet, HSSFRow row, HSSFCellStyle fulBorderCalignFont14,
   		Type1 obj1, Type2 obj2,
   		Type3 obj3, Type4 obj4, Type5 obj5,
   		Type6 obj6, Type7 obj7,
   		boolean writeObj6, boolean writeObj5,
   		boolean isHorizontal, String formName, String formNameCode,
   		boolean nameFormCaption) {
   	writeCaptionForDoubleColumns(wb, sheet, row, fulBorderCalignFont14,
   			obj1, obj2, obj3, obj4, obj5, obj6,
   			obj7, writeObj6, writeObj5, isHorizontal, formName,
   			formNameCode, nameFormCaption, null);
   }

   описание может быть только матерным

   Запостил: evg_ever, 29 Января 2014

   Tweet

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

   • Lure Of Chaos 29.01.2014 15:53 # 0

     все имена изменены и все совпадения случайны, или так и есть в коде?
     
     не вижу тут говна, кроме излишне длинных сигнатур

     Ответить

     • evg_ever 29.01.2014 15:56 # 0

       изменены имена классов, определённых внутри проекта, и объектов этих классов
       
       по-моему, 17 аргументов - это уже всё-таки говно

       Ответить

       • Lure Of Chaos 29.01.2014 16:09 # +2

         длинные сигнатуры - классика говна.

         Ответить

       • anonimb84a2f6fd141 30.01.2014 03:29 # 0

         Type3 obj3 - так и есть в коде?

         Ответить

         • evg_ever 30.01.2014 10:12 # 0

           нет, я же написал выше "изменены имена классов, определённых внутри проекта, и объектов этих классов"

           Ответить

   • anonimb84a2f6fd141 30.01.2014 03:29 # −4

     В жаве нету именованных параметров - жаваебы соснули.

     Ответить

     • evg_ever 30.01.2014 10:11 # 0

       сам-то понял, что сказал?

       Ответить

       • anonimb84a2f6fd141 31.01.2014 23:00 # −1

         Внезапно, да. Ух какой пламеннный багет, аж 4 минуса.

         Ответить

     • Lure Of Chaos 30.01.2014 13:17 # 0

       и как они помогли бы сократить сигнатуры?

       Ответить

       • bormand 30.01.2014 13:49 # +1

         С ними пришлось бы писать одну сигнатуру на все случаи жизни, а не 18 штук с убывающим количеством аргументов, да еще и покрывающими далеко не все юзкейсы...

         Ответить

         • Lure Of Chaos 30.01.2014 13:56 # 0

           или я не понимаю, или речь о необязательных параметрах?

           Ответить

           • evg_ever 30.01.2014 13:57 # 0

             имелись ввиду параметры по умолчанию, видимо

             Ответить

             • roman-kashitsyn 30.01.2014 14:05 # 0

               в питонах к примеру можно не только указывать значения параметров по умолчанию, но и при вызове изменять умолчальные значения конкретного аргумента.

               def f(x, y=1, z=2):
                   pass
               
               f(3, z=17)

               Ответить

               • Lure Of Chaos 30.01.2014 14:13 # +1

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

                 Ответить

                 • roman-kashitsyn 30.01.2014 14:17 # +2

                   об этом писали тыщу раз, но аноним не унимается

                   Ответить

                   • laMer007 30.01.2014 14:20 # 0

                     def f(x, y=1, z=2):
                         y = 42
                     
                     f(3, z=17)
                     f(3, z=18)

                     так интереснее

                     Ответить

                     • laMer007 30.01.2014 14:30 # 0

                       В f печать y забыл перед присвоением. Так понятнее будет

                       Ответить

                   • Lure Of Chaos 30.01.2014 14:22 # 0

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

                     Ответить

                   • anonimb84a2f6fd141 31.01.2014 23:02 # 0

                     Со структурой хуже читается. И снова багет.

                     Ответить

                 • anonimb84a2f6fd141 31.01.2014 23:03 # 0

                   А необязательные параметры не именованы?

                   Ответить

                   • Lure Of Chaos 01.02.2014 00:18 # 0

                     а зачем?

                     Ответить

                     • kegdan 01.02.2014 00:31 # 0

                       для большей гибкости, мой друг

                       Ответить

                       • Lure Of Chaos 01.02.2014 00:32 # 0

                         это не нужно. либо уж порядок, либо имена. иначе будет ад.

                         Ответить

                         • kegdan 01.02.2014 00:42 # 0

                           ну а если у тебя например вводятся координаты точек в н- мерном пространстве, причем остальные не введенные точки довычисляются как ты будешь это описывать?
                           
                           

                           public Vector3D GetVector3D(Plane guidePlane = Plane.Unknown,
                                       Coord x = Coord.Unknown, 
                                       Coord y = Coord.Unknown,
                                       Coord z = Coord.Unknown)
                                   {...}

                           Ответить

                         • anonimb84a2f6fd141 01.02.2014 00:49 # 0

                           Перлопитонистододиезники твое мнение не разделяют.

                           Ответить

           • kegdan 30.01.2014 18:05 # 0

             Как в ля шарпе.
             private Orgasm Sex(Man man= vibrator, Women women = hand){...}
             
             Sex(women = Masha)
             Sex(man = anonimb84a2f6fd141)

             Ответить

             • evg_ever 30.01.2014 18:07 # 0

               try {
                   Sex();
               } catch (NoOrgasmException e) { ... }

               Ответить

               • kegdan 30.01.2014 18:13 # 0

                 try {
                 Sex();
                 } catch (NewChildException e) { ... }

                 Ответить

                 • evg_ever 30.01.2014 18:14 # +3

                   хм, ребёнок у вибратора и руки?

                   Ответить

                   • kegdan 30.01.2014 18:31 # +1

                     Ну так у тебя всякое ветвление в методе должно быть, не?
                     
                     try {
                     Sex();
                     } catch (YouMasturbateVibratorException e) { ... }

                     Ответить

                     • evg_ever 30.01.2014 18:43 # 0

                       хахахах, вот это уже ближе к истине)))

                       Ответить

                     • Lure Of Chaos 30.01.2014 19:30 # 0

                       > YouMasturbateVibratorException
                       мамка дочку спалила за отсутствием мужика?

                       Ответить

                 • Lure Of Chaos 30.01.2014 18:15 # 0

                   try {
                   ...
                   }
                   catch (AllRightException e)
                   {
                   ...
                   }

                   Ответить

             • roman-kashitsyn 30.01.2014 19:45 # 0

               > ля шарпе
               что это

               Ответить

               • bormand 30.01.2014 19:48 # 0

                 до-диез :)

                 Ответить

               • kegdan 30.01.2014 19:50 # 0

                 это по французски, ты не поймешь

                 Ответить

     • VseGovnoOdinYaKrut 30.01.2014 19:54 # 0

       А анонимы анонимно аннотировали абстракцию ануса.

       Ответить

   • roman-kashitsyn 30.01.2014 07:40 # 0

     Отчёты, XSL. Старый добрый apache.poi.
     А параметры нельзя в коллекции передавать и универсально обрабатывать? Или их нужно в хитрые места в шаблоне втыкать?

     Ответить

     • evg_ever 30.01.2014 10:13 # 0

       Увы, в хитрые места.

       Ответить

       • roman-kashitsyn 30.01.2014 10:21 # +1

         Я сталкивался с чем-то подобным. Тогда я сделал enum с типами полей, а метод рендеринга принимал Map<FieldType, ?> и выковыривал из неё поля.
         Причём портянки ифов не было, т.к. расположение полей в документе было забито в статических структурах данных.

         Ответить

         • evg_ever 30.01.2014 10:24 # 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] (подробнее)

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