オブジェクトの関数を呼び出す前に、オブジェクトが null かどうかを確認して、NullPointerException.
これについて最善の方法は何ですか？これらの方法を検討しました。
Java の最適なプログラミング手法はどれですか?
// Method 1
if (foo != null) {
if (foo.bar()) {
etc...
}
}
// Method 2
if (foo != null ? foo.bar() : false) {
etc...
}
// Method 3
try {
if (foo.bar()) {
etc...
}
} catch (NullPointerException e) {
}
// Method 4 -- Would this work, or would it still call foo.bar()?
if (foo != null && foo.bar()) {
etc...
}
方法 4 が最適です。
if(foo != null && foo.bar()) {
someStuff();
}
短絡評価を使用します。つまり、 a の最初の条件logical ANDが falseの場合に終了します。
最後で最高のもの。つまり 、論理積
if (foo != null && foo.bar()) {
etc...
}
なぜなら論理的に&&
右辺が何であるかを知る必要はありません。結果は false でなければなりません
読むことを好む：Java論理演算子の短絡
NullPointerException。それは悪い習慣です。値が null でないことを確認することをお勧めします。方法 4 は、何が起こるかを明確に示し、最小限のコードを使用するため、はるかに優れています。
方法 3 はすべてのレベルで間違っています。アイテムが null である可能性があることはわかっているため、例外的な状況ではなく、チェックする必要があります。
方法 2 は、必要以上に複雑にしているだけです。
方法 1 は、コード行を追加した方法 4 です。
Java 7では、 を使用できますObjects.requireNonNull()。Objectsからのクラスのインポートを追加しjava.utilます。
public class FooClass {
//...
public void acceptFoo(Foo obj) {
//If obj is null, NPE is thrown
Objects.requireNonNull(obj).bar(); //or better requireNonNull(obj, "obj is null");
}
//...
}
方法 4 は、私が見たコードからの最も一般的なイディオムだと思います。しかし、これはいつも私には少し臭いです。foo == null は foo.bar() == false と同じであると想定しています。
それはいつも私にとって正しいとは限りません。
方法 4 は、私の好みの方法です。&& 演算子の短絡により、コードが最も読みやすくなります。方法 3 の NullPointerException のキャッチは、ほとんどの場合、単純な null チェックで十分な場合に嫌われます。
<script src="https://gist.github.com/rcvaram/f1a1b89193baa1de39121386d5f865bc.js"></script>そのオブジェクトが null でない場合、次のことを行います。
を。オブジェクトを変異させることができます (I)
b. オブジェクト (I) を変更する代わりに、何か (O) を出力として返すことができます。
c. 私たちは両方を行うことができます
この場合、オブジェクトである入力 param(I) を取得する必要がある関数を渡す必要があります。そのように取得すると、必要に応じてそのオブジェクトを変更できます。また、その関数は何か (O) である可能性があります。
オブジェクトが null の場合、次のことを行います
を。カスタマイズされた方法で例外をスローする場合があります
b. 何かを返すかもしれません。
この場合、オブジェクトは null であるため、値を提供する必要があるか、例外をスローする必要がある場合があります。
2つの例を挙げます。
public String trimValue(String s){
return s == null ? null : s.trim();
}
public void setTeacherAge(Teacher teacher, int age){
if (teacher != null){
teacher.setAge(age);
} else{
throw new RuntimeException("teacher is null")
}
}
私の説明では、値 (値は null の場合もあります) を取る汎用メソッド、オブジェクトが null でない場合に実行される関数、およびオブジェクトが null の場合に実行される別のサプライヤー関数を作成しました。
public <I, O> O setNullCheckExecutor(I value, Function<I, O> nonNullExecutor, Supplier<O> nullExecutor) {
return value != null ? nonNullExecutor.apply(value) : nullExecutor.get();
}
したがって、この汎用関数を使用した後、サンプル メソッド 1 に対して次のように実行できます。
//To Trim a value
String trimmedValue = setNullCheckExecutor(value, String::trim, () -> null);
ここでは、nonNullExecutor Function が値をトリムします (メソッド リファレンスが使用されます)。nullExecutorFunction は識別関数であるため、null を返します。
2.
// mutate the object if not null otherwise throw a custom message runtime exception instead of NullPointerException
setNullCheckExecutor(teacher, teacher -> {
teacher.setAge(19);
return null;
}, () -> {
throw new RuntimeException("Teacher is null");
});
StringUtils.isNoneEmpty("")また、check is null または empty にも使用できます。
あなたの最後の提案は最高です。
if (foo != null && foo.bar()) {
etc...
}
なぜなら：
commons apache ライブラリにアクセスできない場合は、次の方法で問題なく動作する可能性があります。
if(null != foo && foo.bar()) {
//do something
}