Javaのスキップnullオブジェクト.equalsで返す（）

Question

私はいくつかのコードの最適化を実行しようとしていますし、オブジェクトのための私の現在のユニークなタイトルの検証は次のようになります。それはどこかんしゃくをスローするまで、

public boolean validateTitle(RawProfileConstraint rawProfileConstraint) { 
    RawProfileConstraint result = profileConstraintMapper.validateTitle(rawProfileConstraint.getTitle()); 
    return (!rawProfileConstraint.getTitle().equals(result.getTitle())); 
} 

これは動作しますデータベースは、私が検証しようとしているタイトルを含むレコードを取得することができません - それは利用可能です。この場合、resultはnullになります。 この場合、nullのハンドラを通常のif-elseなしで組み込むにはどうすればよいですか？ resultがそれを逆参照する前にnullの場合

Answer 1

単にチェック：

return result != null 
    && !rawProfileConstraint.getTitle().equals(result.getTitle()); 

または

return result == null 
    || !rawProfileConstraint.getTitle().equals(result.getTitle()); 

あなたは結果が（それぞれfalseとtrue）ときresult == nullなりたいものに依存します。

&&と||ショートなので、彼らが必要な場合を除き、右のオペランドを評価しません。

• false && anything == falseを、そうanything
• true || anything == trueを評価する必要はありません、そうする必要はありませんnullをチェック回避する1つのオプションがOptional（私を使用することですanything

Answer 2

を評価あなたは、Java 8）を使用しているF：

は

Optional.ofNullable(result).map(RawProfileConstraint::getTitle).orElse("") 

Answer 3

と

result.getTitle() 

を交換し、私は戻ってステップすることをお勧め。なぜあなたはif/elseを使うのが悪い考えだと思いますか？

はあなたが本当にAndyTが提案されているよう

public boolean validateTitle(RawProfileConstraint rawProfileConstraint) { 
RawProfileConstraint result = profileConstraintMapper.validateTitle(rawProfileConstraint.getTitle()); 
if (result == null) { 
    return false; 

return (!rawProfileConstraint.getTitle().equals(result.getTitle())); 

ようなソリューションがそんなにリターンを使用するよりも読み、「難しい」と& &だと思いますか？あるいは、Eranが提案する「1つの表現の中の4つの流暢な文」？

さらに、ifを使って...あなたの検証メソッドに概念的な問題があることも明らかになります。 nullの結果がfalse（またはtrue）を引き起こすはずです。

ストーリー・ショート：検証呼び出しのために内部的に使用している「インターフェース」を元に戻して考え直すと、コードが最大になると思います。

Answer 4

Ternary operatorが役立つことができます：

return reusult!=null?(!rawProfileConstraint.getTitle().equals(result.getTitle())):false;