Abrufen eines Wertes, ohne dass Java auf null überprüft werden muss

Oft finde ich mich beim Abrufen eines Wertes aus einer Datenhierarchie auf Null überprüft, um NullPointerExceptions zu vermeiden, die meiner Meinung nach fehleranfällig sind und viel Boilerplate erfordern.

Ich habe eine sehr einfache Routine geschrieben, mit der ich die Nullprüfung beim Abrufen eines Objekts überspringen kann ...

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

Ich benutze es ein bisschen so ...

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

Das obige führt dazu, dass ich ein Room-Objekt oder eine Null erhalte, ohne alle übergeordneten Ebenen überprüfen zu müssen.

Was haltet ihr von dem oben genannten? Schaffe ich ein problematisches Muster? Gibt es Ihrer Meinung nach einen besseren Weg, dies zu tun?

Ihre Lösung ist sehr klug. Das Problem, das ich sehe, ist die Tatsache, dass Sie nicht wissen, warum Sie eine haben null? War es, weil das Haus keine Zimmer hatte? War es, weil die Stadt keine Häuser hatte? War es, weil das Land keine Städte hatte? War es, weil es nullaufgrund eines Fehlers eine 0-Position in der Sammlung gab, selbst wenn sich Häuser auf Position 1 und höher befanden?

Wenn Sie die NonPEKlasse in großem Umfang verwenden, treten schwerwiegende Fehlerbehebungsprobleme auf. Ich denke, es ist besser zu wissen, wo genau die Kette gebrochen ist, als im Stillen nulleinen Fehler zu finden, der einen tieferen Fehler verbergen könnte.

Auch verletzt dies das Gesetz von Demeter : country.getTown().getHouses().get(0).getLivingRoom(). Wenn Sie gegen einen guten Grundsatz verstoßen, müssen Sie in den meisten Fällen unorthodoxe Lösungen implementieren, um das Problem zu lösen, das durch einen Verstoß gegen diesen Grundsatz verursacht wird.

Meine Empfehlung ist, dass Sie es mit Vorsicht verwenden und versuchen, den Konstruktionsfehler zu beheben, der dazu führt, dass Sie im Zug Wrack-Antimuster haben müssen (damit Sie es nicht NonPEüberall verwenden müssen). Andernfalls können Fehler auftreten, die nur schwer zu erkennen sind.

Die Idee ist gut, wirklich gut. Da es die Java 8- OptionalTypen gibt, finden Sie eine ausführliche Erläuterung unter Java Optional type . Ein Beispiel für das, was Sie gepostet haben, ist

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

Und weiter.

Ihre Methode funktioniert gut genug für den beabsichtigten Zweck, obwohl sie nulls zurückgibt, wenn Sie ein NullPointerExceptionschlechtes Design hören.

Vermeiden Sie nulls, wenn Sie können, und geben Sie sie nur weiter, wenn sie etwas darstellen oder eine besondere Bedeutung haben, und geben Sie sie nur zurück, wenn sie etwas darstellen / bedeuten - andernfalls sollten Sie a werfen NullPointerException. Dies vermeidet Fehler und Verwirrung. Wenn ein Objectnicht sein sollte null, NullPointersollte ein geworfen werden. Wenn ein Objekt vorhanden sein kann, wird beim nullÜbergeben nichts schief gehen. Andernfalls funktioniert die obige Methode.

Ich kann deinen Schmerz fühlen, aber die vorgeschlagene Lösung ist eine schlechte Idee.

• Wenn einer der Getter aus einem anderen Grund eine NPE auslöst, werden Sie diese ignorieren.
• Es besteht die Gefahr, dass dieses innere Lambda zu schrecklichem Code wird. Wenn es zum Beispiel eine neue Anforderung gibt, eine spezielle Konstante zurückzugeben, wenn es keine Häuser in der Stadt gibt, kann ein fauler Programmierer die Lamda verlängern und alles in Hülle und Fülle lassen NoNPE.get.
• Wie bereits erwähnt, Optional.mapist es das , wonach Sie suchen.
• Der Aufwand für das Erstellen einer neuen Instanz von NullPointerException ist häufig erheblich. Es sind viele Mikrosekunden, besonders wenn Ihr Call-Stack immer größer wird. Es ist schwer vorherzusagen, wo Ihr Dienstprogramm letztendlich verwendet wird.

Als Randnotiz NoNPEInterfaceist ein Duplikat von java.util.function.Supplier.

In einigen Fällen können Sie die Verwendung von Ausdrucksauswertungs-Utils in Betracht ziehen, die in vielen Frameworks vorhanden sind (zum Beispiel: EL, SpEL):

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