Pobieranie wartości bez konieczności zerowania w Javie

Wiele razy sprawdzam, czy sprawdzam wartość zerową podczas pobierania wartości z jakiejś hierarchii danych, aby uniknąć wyjątków NullPointerExcept, które uważam za podatne na błędy i wymagają dużo dodatkowej analizy.

Napisałem bardzo prostą procedurę, która pozwala mi pominąć sprawdzanie wartości zerowej podczas pobierania obiektu ...

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();
    }
}

Używam tego trochę tak ...

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

Powyższe skutkuje otrzymaniem obiektu pokoju lub wartości zerowej, bez konieczności sprawdzania wszystkich poziomów nadrzędnych.

Co sądzisz o powyższym? Czy tworzę problematyczny wzór? Czy Twoim zdaniem jest na to lepszy sposób?

Twoje rozwiązanie jest bardzo inteligentne. Problem, który widzę, polega na tym, że nie wiesz, dlaczego masz null? Czy to dlatego, że dom nie miał pokoi? Czy to dlatego, że miasto nie ma domów? Czy to dlatego, że kraj nie miał miast? Czy to dlatego, że byłnull kolekcja w pozycji 0 z powodu błędu, nawet jeśli domy znajdują się w pozycjach 1 i wyższych?

Jeśli użyjesz NonPEklasy extensibe , będziesz mieć poważne problemy z debugowaniem. Myślę, że lepiej wiedzieć, gdzie dokładnie łańcuch jest zerwany, niż po cichu zdobyćnull co może kryć głębszy błąd.

Narusza to również prawo Demeter :country.getTown().getHouses().get(0).getLivingRoom() . Częściej niż nie, naruszenie jakiejś dobrej zasady powoduje, że musisz wdrożyć niekonwencjonalne rozwiązania, aby rozwiązać problem spowodowany naruszeniem tej zasady.

Radzę , abyś używał go ostrożnie i starał się rozwiązać usterkę projektową, która powoduje, że musisz ponieść atak na wrak pociągu (abyś nie musiał używać NonPEwszędzie). W przeciwnym razie możesz mieć błędy, które będą trudne do wykrycia.

Pomysł jest w porządku, naprawdę dobry. Ponieważ Optionaltypy Java 8 istnieją, szczegółowe objaśnienie znajduje się w części Opcjonalne Java . Przykładem tego, co opublikowałeś, jest

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

I dalej.

Twoja metoda działa wystarczająco dobrze, zgodnie z jej przeznaczeniem, choć zwraca nulls, gdy NullPointerExceptionbrzmi jak zły projekt.

Staraj się unikać nulls, kiedy możesz, i mijaj je tylko wtedy, gdy coś reprezentują lub mają specjalne znaczenie i zwracaj je tylko wtedy, gdy coś reprezentują / znaczą coś - w przeciwnym razie powinieneś rzucić NullPointerException. Pozwala to uniknąć błędów i zamieszania. Jeśli Objectnie powinno być null, NullPointernależy rzucić. Jeśli obiekt może być, nullwtedy nic nie pójdzie źle po przekazaniu go. W przeciwnym razie powyższa metoda działa.

Czuję twój ból, ale proponowane rozwiązanie to zły pomysł.

• Jeśli jeden z pobierających rzuci NPE z innego powodu, zignorujesz go.
• Istnieje ryzyko, że wewnętrzna lambda przerodzi się w okropny kod. Na przykład, jeśli istnieje nowy wymóg zwrócenia specjalnej stałej, gdy w mieście nie ma domów, leniwy programista może przedłużyć lamdę, pozostawiając wszystko owinięte NoNPE.get.
• Jak już wspomniano, Optional.mapwłaśnie tego szukasz.
• Kara za utworzenie nowej instancji NullPointerException jest często znacząca. To wiele mikrosekund, zwłaszcza gdy Twój stos połączeń staje się większy. Trudno przewidzieć, gdzie ostatecznie zostanie użyte Twoje narzędzie.

Na marginesie, NoNPEInterfacejest duplikatem java.util.function.Supplier.

W niektórych przypadkach możesz rozważyć użycie narzędzi do oceny wyrażeń, które są obecne w wielu ramach (na przykład: EL, SpEL):

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