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

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

    • А в джаве готового метода для этого нет?
      Ответить
    • if ()
              return true;
          else
              return false;

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

          Аналогию для Java сами найдёте. Здесь вообще иф не нужен, потому что действия программы не ветвятся, достаточно вернуть одно значение.
          Ответить
          • классика говна.

            надо
            public static boolean isEmpty(CharSequence str) {
                return (str == null || str.length() == 0);
            }
            Ответить
            • Скобки тоже не нужны.
              Ответить
            • Может лучше
              if(str == null) throw FuckYouException()
              return str.length() == 0
              Либо вообще
              return str.length() == 0 // ибо нехуй
              Ответить
      • Ну это уж совсем классика :)
        А вот в шарпе есть String.IsNullOrEmpty...
        Правда, я ещё не в курсе, кто такой CharSequence - это ж не string?
        Ответить
        • CharSequence - это даже не класс. Это интерфейс. Строка таки его реализует
          Ответить
    • Школьники? блять, правильный код, не будет проблем с разными jvm, по крайне мере в java машине от IBM желательно писать именно такой код
      Ответить

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