Отримання значення без необхідності перевірки на 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? Це було тому, що в будинку не було кімнат? Невже місто не було будинків? Це було тому, що в країні не було містечок? Це було тому, що nullв колекції 0 було місце через помилку, навіть якщо є будинки на позиціях 1 і вище?

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

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

Моя рекомендація полягає в тому, що ви використовуєте це обережно і намагаєтеся вирішити ваду дизайну, яка змушує вас зазнати аварійної аварії (тому вам не доведеться користуватися NonPEскрізь). Інакше у вас можуть виникнути помилки, які важко буде виявити.

Ідея чудова, насправді гарна. Оскільки Optionalтипи Java 8 існують, детальне пояснення можна знайти у Java-додатковому типі . Приклад того, що ви розмістили, - це

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

І далі.

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

Спробуйте уникати nulls, коли ви можете і передавати їх лише тоді, коли вони щось представляють або мають особливе значення, і повертати їх лише тоді, коли вони щось представляють / означають - інакше вам слід кинути NullPointerException. Це дозволяє уникнути помилок і плутанини. Якщо Objectне повинно бути null, NullPointerмає бути викинута. Якщо об'єкт може бути, nullтоді при передачі його нічого не піде. Інакше ваш метод працює вище.

Я можу відчути ваш біль, але запропоноване рішення - погана ідея.

• Якщо хтось із ораторів кине NPE з іншої причини, ви проігноруєте це.
• Існує ризик, що внутрішня лямбда переросте в жахливий код. Наприклад, якщо є нова вимога повернути спеціальну константу, коли в місті немає будинків, ледачий програміст може продовжити ламду, залишаючи все, що загорнуте NoNPE.get.
• Як вже було сказано, Optional.mapце те, що ви шукаєте.
• Штраф за створення нового примірника NullPointerException часто суттєвий. Це багато мікросекунд, тим більше, що стек викликів стає більшим. Важко передбачити, де в кінцевому підсумку ваша утиліта буде використана.

Як бічна примітка, NoNPEInterfaceє дублікатом java.util.function.Supplier.

У деяких випадках ви можете скористатися утилітами оцінки виразів, які присутні у багатьох структурах (наприклад: EL, SpEL):

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