36

Под большой вложенностью имеется ввиду код вида:

param = foo.getSomthing1().getSomthig2().getSomthing3() ...

При этом в каждом вызове может вылететь NullPointerException. Хорошим примером является сгенерированные классы по XSD схеме. Мне нужно вытащить и присвоить некоторое значение, и если на пути встретится хоть одна отсутствующая нода (null в Java) - мне нужно присвоить null.

Один из вариантов, это if вида:

if (foo != null && foo.getSomthing1() != null && ....) {
    param = foo.getSomthing1().getSomthig2().getSomthing3() ...
}

При этом код становится нечитабельным. И простой конвертер объекта, сгенеренного по XSD схеме класса в плоскую Java структуру выглядит ужасно.

Есть вариант поместить в try catch :) Но это будет выглядеть ещё хуже, не говоря уже о производительности.

Слышал про операторы ?., ?:, ?[], но заставить их работать локально у меня не получилось, а так было бы интересным решением:

param = foo?.getSomthing1()?.getSomthig2()?.getSomthing3() ...

UPD: вариант с ?. был взят из неавторитетного источника. Имелся ввиду вероятно C#.

Подскажите, какие ещё могут быть способы?

4
  • Не тянет на полноценный ответ, зато полностью решает проблему - переходите на Scala.
    – user177221
    Commented 12 дек. 2016 в 16:23
  • 3
    ?. - это C#. Переходи на шарп и используй :)
    – Qwertiy
    Commented 14 дек. 2016 в 22:27
  • @Qwertiy, да я уже понял, натыкался в контексте Java, говорили мол с 1.8 работает. Источник был не авторитетный:)
    – pegoopik
    Commented 15 дек. 2016 в 4:16
  • ?. - это groovy. groovy-lang.org/operators.html#_safe_navigation_operator Код на java является так же кодом на groovy. Плюс в groovy есть удобные классы для работы с xml и html. Commented 27 дек. 2016 в 11:03

8 ответов 8

32
+50

Есть такой вариант, но только для Java 8:

private Optional<String> get(First first) {
    return Optional.ofNullable(first.getSecond())
            .map(Second::getThird)
            .map(Third::getFourth)
            .map(Fourth::getString);
}

Допустим у нас есть класс First, который в свою очередь содержит в себе поле с классом Second, который содержит Third, который содержит Fourth в котором нужное нам поле, пусть это будет String.

Итого, чтобы получить string из Fourth, имея экземпляр класса First, нам нужна следующая конструкция String someString = first.getSecond().getThird().getFourth().getString().

Имея вышеописанный метод, мы делаем следующее: String someString = get(first).orElse(null);

Если один из вложенных классов окажется null, то мы просто получим someString = null, либо вместо null можно указать default значение.

Remark

Речь идет об использовании специального класса Optional введенного в Java 8. Для Java 7 можно воспользоваться Google Guava - там тоже есть аналогичный класс Optional

5
  • Например вам нужно вытащить некий String он находится в first.getSecond().getThird().getFourth().getString то это будет выглядеть как String test = get(first).orElse(null); Commented 8 дек. 2016 в 11:50
  • Подробнее расписал в ответе, выложил пример bitbucket.org/snippets/n0c1ip/77x6G Commented 12 дек. 2016 в 6:09
  • Пример лучше бы в сам ответ добавить. )
    – Nick Volynkin
    Commented 12 дек. 2016 в 7:48
  • Для 8 Java замечательно, ну а если 7 ая java или ниже?
    – Arch
    Commented 12 дек. 2016 в 7:55
  • 1
    @JAVAvladuxa написать свою монаду
    – Artem
    Commented 12 дек. 2016 в 8:40
13

Можно использовать Optional и ifPresent:

К примеру, если есть такой класс:

public class TestClass{

    public Optional<TestClass> getSomthing1(){
        return Optional.of(this);
    }

    public Optional<TestClass> getSomthing2(){
        return Optional.of(this);
    }
}

То использовать можно будет так:

TestClass test = new TestClass();
test.getSomthing1().ifPresent(t -> t.getSomthing2());

То есть, мы возвращаемое значение заворачиваем в Optional.

P.S. это с Java 8 только доступно

1
  • Интересно, спасибо. С xsd это не проканает, а вообще любопытно.
    – pegoopik
    Commented 8 дек. 2016 в 5:14
7

Есть шаблон проектирования, который называется Null object

Данный шаблон проектирования рекомендуется использовать, когда:

  • Объект требует взаимодействия с другими объектами. Null Object не устанавливает нового взаимодействия — он использует уже установленное взаимодействие.
  • Какие-то из взаимодействующих объектов должны бездействовать
  • Требуется абстрагирование «общения» с объектами, имеющими NULL-значение.

Также на эту тему можно посмотреть видео с примерами от Егора Бугаенко (eng).

Основной недостатком такого подхода - количество созданных Null object классов.

1
  • Ну вот только он не нужен при простой проверке на null. Как его использовать для выше поставленной цели?
    – Arch
    Commented 12 дек. 2016 в 8:06
4

Если решитесь использовать try/catch, можно создать вспомогательный static метод:

public class Util {
  public static <T> T nullIfNPE(Supplier<T> f) {
      try {
          return f.get();
      } catch (NullPointerException e) {
          return null;
      }
  }
}

Используете так:

String param = Util.nullIfNPE(() -> foo.getSomthing1().getSomthig2());

А если импортировать статически этот метод, то даже так:

String param = nullIfNPE(() -> foo.getSomthing1().getSomthig2());

Вполне лаконично =)

5
  • Стоит отметить, что само наличие try/catch блока почти не влияет на производительность, а вот обработка возникающих исключений - влияет. Т.е. если в лямбде (Supplier'е) NPE будет редко выбрасываться - с производительностью все будет ок.
    – ygun
    Commented 14 дек. 2016 в 13:26
  • 2
    Использование исключений для обыкновенного control flow наверное всё же не самая лучшая идея.
    – VladD
    Commented 15 дек. 2016 в 8:17
  • 2
    ура созданию и выбросу стэка на каждый чих
    – etki
    Commented 16 дек. 2016 в 8:17
  • @Etki, в вопросе описана конкретная ситуация с автосгенеренными классами, которые мы трогать не можем. Данный ответ полностью решает проблему нечитабельности, хоть и страдает в производительности. Ваш сарказм неуместен, по мне, так это лучший ответ применительно к описанной в вопросе проблеме.Лучший по голосам ответ описанную проблему не решает.
    – pegoopik
    Commented 29 дек. 2016 в 5:38
  • @pegoopik Хорошо, ждем вопроса про производительность
    – etki
    Commented 29 дек. 2016 в 11:25
2

Тернарный оператор работает следующим образом.

ComplexityObjectRef cObj;

String res = cObj.getFirst() == null ? "message for this cond" : 
cObj.getFirst().getSecond() == null ? "message for this cond" : 
cObj.getFirst().getSecond().getThird() == null ? "message for this cond" : 
cObj.getFirst().getSecond().getThird().getResult();

Кода много для такой проверки, но он вполне читабельный и понятный, и использование тернарного опретора для java 7 и ниже вполне приемлимый вариант.

1
  • Громоздко. Вариант с простым if, описанный в вопросе выглядит читабельнее и занимает меньше места.
    – pegoopik
    Commented 12 дек. 2016 в 8:18
1

Написал небольшую обертку, которая решает проблему NullPointerException

public static class NPEChecker<T> {
    public final T value;

    public NPEChecker(T value) {
        this.value = value;
    }

    public <R> NPEChecker<R> invoke(Function<T, R> function) {
        if (value == null)
            return new NPEChecker<>(null);

        R result = function.apply(value);
        return new NPEChecker<>(result);
    }
}

С ее помощью, конструкция вида:

AnnotatedType[] result = Solution.class
                .getAnnotatedInterfaces()
                .getClass()
                .getAnnotatedInterfaces();

Преобразуется в следующее:

AnnotatedType[] result = new NPEChecker<>(Solution.class)
            .invoke(Class::getAnnotatedInterfaces)
            .invoke(AnnotatedType[]::getClass)
            .invoke(Class::getAnnotatedInterfaces)
            .value;
5
  • А чем это принципиально отличается от ответа с Optional?
    – Roman
    Commented 13 янв. 2017 в 22:53
  • @Roman, тем, что не требует вмешательства в класс Solution.
    – pegoopik
    Commented 14 янв. 2017 в 0:28
  • @pegoopik Так и с Optional не требуется: AnnotatedType[] result = Optional.of(Solution.class) .map(Class::getAnnotatedInterfaces) .map(AnnotatedType[]::getClass) .map(Class::getAnnotatedInterfaces) .orElse(null);
    – Roman
    Commented 14 янв. 2017 в 3:04
  • @Roman принципиально ничем, просто решил запились свою реализацию.
    – Artem
    Commented 14 янв. 2017 в 4:57
  • @Roman, не тот ответ с Optional посмотрел.Но и от принятого ответа отличается тем, что решение универсально. Не нужно на каждый чих создавать отдельную функцию, типа private Optional<String> get(First first).
    – pegoopik
    Commented 14 янв. 2017 в 6:36
-2

Я, конечно, могу ошибаться, или не понять сути вопроса, но нашёл решение попроще предложенных:

public class MainClass {

    public static void main(String[] args) {
        String param;
        try {
            param= String.valueOf(foo.q().w().e().r().t());
        } catch (Exception ignored) { 
            param= "null"; 
        }
        System.out.println(param);
    }

    private MainClass q() {
        return new MainClass();
    }

    private MainClass w() {
        return new MainClass();
    }

    private MainClass e() {
        return new MainClass();
    }

    private MainClass r() {
        return null;
    }

    private MainClass t() {
        return new MainClass();
    }

}

Масса кода, чтоб было готовая проверка при необходимости, но, по сути, решение таково:

String param;
try {
    param = String.valueOf(foo.q().w().e().r().t());
} catch (Exception ignored) {
    param = "null";
}
0
-2

Пожалуй, самый простой и понятный способ:

try {
    param = foo.getSomething1().getSomething2()...;
} catch (NullPointerException e) {
    param = null;
}

Надеюсь, что это решило вашу проблему.

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.