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

    +77

    1. 1
    2. 2
    3. 3
    public static boolean isNull(Collection collection) {
    return collection == null;
}

    Запостил: mychka, 25 Июня 2012

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

    • ava carsten 25.06.2012 21:31 # 0

      Как будто хотели

      return (collection == null) || collection.isEmpty();


      но на полпути забыли.
      Ответить

      • ava rat4 25.06.2012 21:58 # +3

        Это был бы уже isNullOrEmpty()
        Ответить

      • ava mychka 25.06.2012 22:16 # +2

        Метод isEmpty (проверка на null или empty) есть, рядом. Метод isNull создан именно для проверки на null. Причём заточен на проверку не любых объектов, а именно коллекций :)
        Ответить

        • ava wvxvw 25.06.2012 22:46 # +3

          Жаль, что статик, а то такой хороший метод, можно было бы каждому классу подарить, ну или через интерфейс.
          Ответить

          • ava roman-kashitsyn 25.06.2012 22:47 # +1

            Полюбому нужен mixin
            Ответить

            • ava TarasB 25.06.2012 23:15 # +3

              Палюбэ нужен абстрактный метапаттерн с параметром, чтобы сравнивать объекты с нуллом.
              Ответить

              • ava carsten 26.06.2012 03:29 # +4

                Нужна фабрика компараторов, это очевидно.
                Ответить

                • ava bormand 26.06.2012 05:36 # +1

                  if (NullComparatorFactory.getInstance().cre ateNullComparator(nullable).compareWithN ull()) { ... }
                  Ответить

          • ava bormand 26.06.2012 05:34 # 0

            [ProblemFactory]
            Интерфейс Nullable, который должен заимплементить каждый из классов.
            [/ProblemFactory]
            Ответить

            • ava roman-kashitsyn 26.06.2012 22:58 # 0

              Кстати, есть аннотация Nullable, используемая для статического анализа кода.
              Ответить

          • ava mychka 26.06.2012 23:42 # +2

            Да. Жаль только, что он никогда не вернёт true :)
            Ответить

        • ava tir 26.06.2012 07:34 # +1

          то что для коллекций: возможно делали через extract method и просто не обратили внимание на тип аргумента, который подставила иде. А так как не для коллекций метод не вызывался, то и не заметили.

          Неплохо бы еще варианты использования этого метода. Если для того чтобы эксепшн кинуть (типа assert, preconditions) - то просто метод не дописали (или не дорефакторили). Если же в условиях типа if (isNull(c)) { ... }, то метод весьма сомнителен.
          Ответить

          • ava mychka 26.06.2012 22:57 # +1

            Метод использовался для проверок типа if (isNull(c)).
            После удаления метода такие места заменены на if (c == null) (чаще if (c != null))
            Ответить

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