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

• carsten 14 лет ago # 0

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

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

  
  
  но на полпути забыли.

  Ответить

  • rat4 14 лет ago # 0 ⇈

    Это был бы уже isNullOrEmpty()

    Ответить

  • mychka 14 лет ago # 0 ⇈

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

    Ответить

    • wvxvw 14 лет ago # 0 ⇈

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

      Ответить

      • roman-kashitsyn 14 лет ago # 0 ⇈

        Полюбому нужен mixin

        Ответить

        • TarasB 14 лет ago # 0 ⇈

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

          Ответить

          • carsten 14 лет ago # 0 ⇈

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

            Ответить

            • bormand 14 лет ago # 0 ⇈

              if (NullComparatorFactory.getInstance().cre ateNullComparator(nullable).compareWithN ull()) { ... }

              Ответить

      • bormand 14 лет ago # 0 ⇈

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

        Ответить

        • roman-kashitsyn 14 лет ago # 0 ⇈

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

          Ответить

      • mychka 14 лет ago # 0 ⇈

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

        Ответить

    • tir 14 лет ago # 0 ⇈

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

      Ответить

      • mychka 14 лет ago # 0 ⇈

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

        Ответить