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

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

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

1. PHP (5700)
2. C++ (2734)
3. Куча (2331)
4. JavaScript (2028)
5. C# (1927)
6. Java (1501)
7. Си (1099)
8. Pascal (647)
9. Python (582)
10. 1C (534)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (198)
15. Perl (194)
16. Assembler (146)
17. Ruby (142)
18. VisualBasic (138)
19. Lua (49)
20. Go (29)
21. Swift (27)
22. Haskell (26)
23. Kotlin (13)

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

   +69

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

   public static boolean isEmpty(CharSequence str) {
       if (str == null || str.length() == 0)
           return true;
       else
           return false;
   }

   Я считаю это странным
   Нашёл в android.text.TextUtils

   Запостил: jonasas, 07 Июля 2014

   Tweet

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

   • Qwertiy 07.07.2014 23:59 # 0

     А в джаве готового метода для этого нет?

     Ответить

     • Мистер Хэнки 09.07.2014 04:46 # 0

       неа, слишком простой же

       Ответить

   • guest 08.07.2014 00:15 # 0

     if ()
             return true;
         else
             return false;

     
     За это нам препод замечания делал.

     Ответить

     • brutushafens 08.07.2014 00:25 # 0

       cleaned

       Ответить

       • inkanus-gray 08.07.2014 01:18 # 0

         Товарищ, любой паскалист скажет, что «дважды два четыре» будет равно true, потому что в Паскале для этого есть специальный тип данных, называемый boolean.
         
         Аналогию для Java сами найдёте. Здесь вообще иф не нужен, потому что действия программы не ветвятся, достаточно вернуть одно значение.

         Ответить

         • kegdan 08.07.2014 01:32 # 0

           классика говна.
           
           надо
           

           public static boolean isEmpty(CharSequence str) {
               return (str == null || str.length() == 0);
           }

           Ответить

           • bormand 08.07.2014 05:28 # +1

             Скобки тоже не нужны.

             Ответить

             • kegdan 08.07.2014 05:49 # −1

               Не придерайся
               
               в руби вообще
               
               return str||str.lenght()

               Ответить

               • bormand 08.07.2014 05:55 # 0

                 В крестах вообще тупо - return !str.empty();

                 Ответить

                 • kegdan 08.07.2014 05:57 # 0

                   В крестах все тупо

                   Ответить

                 • kegdan 08.07.2014 06:16 # 0

                   посмотрел - таки в руби просто
                   return !str==""

                   Ответить

                   • kegdan 08.07.2014 06:25 # 0

                     арр!
                     
                     !str||str.empty?
                     
                     не выебнулся(

                     Ответить

                   • bormand 08.07.2014 07:04 # +1

                     > return !str==""
                     return str!=""

                     Ответить

                     • kipar 08.07.2014 09:05 # 0

                       И return тоже не нужен

                       Ответить

                       • bormand 08.07.2014 09:26 # 0

                         Тогда и !="" не нужен. В условии и без него прокатит (но не в жабе).

                         Ответить

                         • kipar 08.07.2014 10:48 # 0

                           Не, в смысле если функция из одной строчки то return не нужен.
                           

                           def is_empty str
                             str!=""
                           end

                           Ответить

           • guest 06.12.2014 01:42 # 0

             Может лучше
             if(str == null) throw FuckYouException()
             return str.length() == 0
             Либо вообще
             return str.length() == 0 // ибо нехуй

             Ответить

     • Qwertiy 08.07.2014 00:30 # 0

       Ну это уж совсем классика :)
       А вот в шарпе есть String.IsNullOrEmpty...
       Правда, я ещё не в курсе, кто такой CharSequence - это ж не string?

       Ответить

       • kegdan 08.07.2014 01:15 # 0

         CharSequence - это даже не класс. Это интерфейс. Строка таки его реализует

         Ответить

   • guest 22.07.2014 14:54 # 0

     Школьники? блять, правильный код, не будет проблем с разными jvm, по крайне мере в java машине от IBM желательно писать именно такой код

     Ответить

     • bormand 22.07.2014 15:04 # +2

       > именно такой код
       Включая строки 3-5?

       Ответить

       • kegdan 22.07.2014 15:09 # +1

         IBM же

         Ответить

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

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

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

   Семь раз отмерь — один отрежь, guest!

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

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

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