メソッドチェイン内の最後のゲッターがヌルでないかチェックしてください。

Question

コードでは、チェーンメソッドがたくさんあります。たとえば、obj.getA().getB().getC().getD()です。メソッドgetD()がnullではないが、その前にすべての以前のgetterをチェックする必要があるかどうかを調べるヘルパークラスを作成したいと思います。私はこの方法でそれを行うことができます。

try { 
    obj.getA().getB().getC().getD(); 
} 
catch (NullPointerException e) { 
    // some getter is null 
} 

か（「愚か」である）

if (obj!null && obj.getA()!=null && obj.getA().getB()!=null && ...) { 
    obj.getA().getB().getC().getD(); 
} 
else { 
    // some getter is null 
} 

を、私は私のコードでtry{} catch()を使用して、それを毎回チェックする必要はありません。この目的のための最良の解決策は何ですか？

私は最高のだろうと思います。

1. obj.getA().getB().getC().getD().isNull() - この目的のために、私はisNull()メソッドが含まれているいくつかのインターフェイスを実装例えば、私のゲッターのすべてを変更する必要があります。
2. NullObjectHelper.isNull(obj.getA().getB().getC().getD()); - これは最高です（私はそうだと思いますが）どのようにこれを実装するのですか？

Answer 1

Optionパターンで結果を得ることができます。これにより、メソッドのシグネチャを変更することが強制されますが、基本的には、メソッドがあるタイプのTを返す場合はnull以外の値が返され、Option<T>を返す場合は値Tまたはnullが返されます。

Java 7にはnull安全性という機能がありましたが、最終リリースから削除されました。あなたは何ができる：

obj?.getA()?.getB()?.getC()?.getD() 

をので、あなたが安全にそれを行うだろう。また、Javaの8はOptionalと呼ばれる機能が追加されます。

実際に今すぐ使用したい場合は、Null Objectパターンを試してください。つまり、nullを返す代わりに、のデフォルト値の値を返すことができ、NullPointerExceptionをトリガーしません。あなたはユーティリティがそれをしたい場合は、いくつかの機能インタフェースでそれをラップし、それを呼び出すことができます：しかし、あなたはあなたのゲッターにいくつかの変更

class Object { 
    A getA() { 
    // ... 
    return a == null ? A.NULL : a; 
    } 
} 

class A { 
    static A NULL = new A(); // some default behaviour 
    B getB() { 
    if (this == NULL) return B.NULL; 
    // ... 
    return b == null ? B.NULL : b; 
    } 
} 

EDITを追加する必要があります。

static boolean isNullResult(Callable call) throws Exception { 
    try { 
     return call.call() == null; 
    } catch (NullPointerException npe) { 
     return true; 
    } 
} 

使用方法は、次のようになります。

isNullResult(new Callable<Integer>() { 
    @Override 
    public Integer call() throws Exception { 
     return new A().getB().getC().getInt(); 
    } 
}); 

既存の機能すでに述べた

Answer 2

として、真の解決策は、リファクタリングあるを変更する必要はありません。一方

、あなただけの機能では、あなたの最初の回避策包むことができます：発信者サイトで

static D getD(MyClass obj) { 

    try { 
     return obj.getA().getB().getC().getD(); 
    } 
    catch (NullPointerException e) { 
     return null; // Or even better, some default D 
    } 
} 

を：

D d = getD(obj); 

少なくとも、あなたが発信者をゴミ箱にする必要はありませんtry-catchブロックを使用します。 getX()呼び出しの一部がnullを返し、dがnull.になると、エラーを何らかの形で処理する必要があります。ラッパー機能でデフォルトのDを返すのが最適です。

---

私は中間getX()のいずれかがnullを返した場合、あなたの質問の最後にリスト2つのオプションが役立つだろうどのように表示されません。 NullPointerExceptionが表示されます。

Answer 3

あなたはゲッターチェーンにnullを処理するためにOptional.isPresentとOptional.orElseのようなメソッドを使用できるJava 8の通り：

boolean dNotNull = Optional.ofNullable(obj) 
       .map(Obj::getA) 
       .map(A::getB) 
       .map(B::getC) 
       .map(C::getD) 
       .isPresent(); 

これは、このアプローチの欠点NullPointerExceptionがをキャッチすることが好ましいものののためのオブジェクトの割り当てでありますオプションインスタンス。

このオーバーヘッドなしで同様の動作を行う独自の静的メソッドを記述することが可能である：

boolean dNotNull = Nulls.isNotNull(obj, Obj::getA, A::getB, B::getC, C::getD); 

サンプル実装については、Nullifierタイプhereを参照してください。

ないアプローチは、ネストされた場合-nullでないチェックより大きいランタイム効率を持っている可能性があります。