Извлечение значения без проверки нуля в Java

Много раз, когда я выбираю значение из некоторой иерархии данных, я проверяю себя на нуль, чтобы избежать исключений NullPointerExceptions, которые, как я считаю, подвержены ошибкам и требуют большого количества шаблонов.

Я написал очень простую процедуру, которая позволяет мне пропустить проверку нуля при получении объекта ...

public final class NoNPE {

    public static <T> T get(NoNPEInterface<T> in) {
        try {
            return in.get();
        } catch (NullPointerException e) {
            return null;
        }
    }

    public interface NoNPEInterface<T> {
        T get();
    }
}

Я использую это немного как это ...

Room room = NoNPE.get(() -> country.getTown().getHouses().get(0).getLivingRoom());

Вышеприведенный результат привел к тому, что я получил объект Room или ноль, не проверяя все родительские уровни на ноль.

Что вы думаете о вышесказанном? Я создаю проблемный образец? Есть ли лучший способ сделать это по вашему мнению?

Ваше решение очень умное. Проблема, которую я вижу, в том, что вы не знаете, почему у вас есть null? Это потому, что в доме не было комнат? Было ли это потому, что в городе не было домов? Это потому, что в стране не было городов? Было ли это из- nullза ошибки в позиции 0 из-за ошибки, даже если дома находятся в позициях 1 и выше?

Если вы используете расширенный NonPEкласс, у вас будут серьезные проблемы с отладкой. Я думаю, что лучше знать, где именно разрывается цепь, чем молча получить сообщение, nullкоторое может скрывать более глубокую ошибку.

Кроме того, это нарушает закон Деметры : country.getTown().getHouses().get(0).getLivingRoom(). Чаще всего нарушение какого-то хорошего принципа заставляет вас искать неортодоксальные решения для решения проблемы, вызванной нарушением такого принципа.

Я рекомендую вам использовать его с осторожностью и попытаться устранить конструктивный недостаток, из-за которого вам придется столкнуться с антипаттерном крушения поезда (поэтому вам не придется использовать его NonPEповсюду). В противном случае у вас могут быть ошибки, которые будет трудно обнаружить.

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

Optional.ofNullable(country)
    .map(Country::getTown)
    .map(Town::Houses);

И дальше.

Ваш метод работает достаточно хорошо для его предназначения, хотя возвращает nulls, когда вы получаете NullPointerExceptionзвуки, похожие на плохой дизайн.

Старайтесь избегать nulls, когда вы можете, и передавать их только тогда, когда они представляют что-то или имеют особое значение, и возвращать их только тогда, когда они представляют / значат что-то, иначе вы должны бросить a NullPointerException. Это позволяет избежать ошибок и путаницы. Если не Objectдолжно быть null, NullPointerдолжно быть брошено. Если объект может быть, nullтогда ничто не пойдет не так, когда его передадут. В противном случае ваш метод работает выше.

Я чувствую вашу боль, но предложенное решение - плохая идея.

• Если один из получателей бросает NPE по какой-либо другой причине, вы его проигнорируете.
• Есть риск, что эта внутренняя лямбда превратится в ужасный код. Например, если есть новое требование возвращать специальную константу, когда в городе нет домов, ленивый программист может расширить lamda, оставив все в оболочке NoNPE.get.
• Как уже упоминалось, Optional.mapэто то , что вы ищете.
• Наказание за создание нового экземпляра NullPointerException часто является значительным. Это много микросекунд, особенно когда ваш стек вызовов становится больше. Трудно предсказать, где ваша утилита будет использоваться.

Как примечание стороны, NoNPEInterfaceявляется дубликатом java.util.function.Supplier.

В некоторых случаях вы можете рассмотреть возможность использования утилит для оценки выражений, которые присутствуют во многих средах (например, EL, SpEL):

evaluateProperty(country, "town.houses[0].livingRoom")