java - Java の Null チェックとリスナー

Question

次のようなコードをよく見かけます。

Listener mListener;

public void setListener(Listener listener){
   mListener=listener;
}

public void fooFunction(){
   ...
   ...
   if (mListener!=null){
      mListener.notifyFoo();
   }
}

私の質問は次のとおりです。null チェックと notifyFoo() の間で別のスレッドが setListener(null) を呼び出した場合はどうなりますか? それは可能ですか？またはコンパイラはそれをアトミックにしますか

Answer 1

ブロックを同期化することでそれを回避できます

public synchronized void setListener(Listener listener) {
    ....
}

public synchronized void fooFunction() {
    ....
}

Answer 2

ここでの同期はやり過ぎです。チェックする前に、メンバー変数のコピーを取得するだけですnull。

Listener mListener;

public void setListener(Listener listener){
   mListener=listener;
}

public void fooFunction(){
   ...
   ...
   Listener listener = mListener;
   if (listener!=null){
      listener.notifyFoo();
   }
}