java - 共有リスナーメンバー変数をvolatileとして宣言する必要がありますか？

Question

私は、独自のスレッドでいくつかの計算を行い、その結果をリスナーに報告する単純なクラスを持っています。いずれかの時点で

class Calculator extends Thread { 
    protected Listener listener; 

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

    public void run() { 
     while (running) { 
      ... do something ... 

      Listener l = listener; 

      if (l != null) { 
       l.onEvent(...); 
      } 
     } 
    } 
} 

彼は一定の期間のためのイベントを望んでいない場合、ユーザーはsetListener（ヌル）を呼び出すことができます。したがって、のrun（）関数では、私はNullPointerExceptionに入ることができないので、リスナーのコピーを作成します。！= nullの後にリスナーがnullに設定された場合に起こる可能性がある条件チェックが成功しました。私の場合は、これを同期するための正しい選択だと私は信じています。

私の質問は、ここでリスナーメンバー変数をvolatileとして宣言する必要がありますか？私は揮発性について多くのことを読んできましたが、すべての例はオブジェクトではなく、基本データ型（boolean、int、...）を対象としています。だから、私はオブジェクトが揮発性であると宣言されるべきかどうかはわかりません。私は揮発性と宣言しなければならないと信じているので、スレッドは常にメンバ変数の最新バージョンを持っていますが、わかりません。

ありがとうございます！

Answer 1

はい。 Calculatorスレッドが別のスレッドによって設定された新しい値を保証するためには、変数をvolatileにする必要があります。

しかし、volatileは非常に低レベルのメカニズムであり、クライアントコードで使用されることはめったにありません。このシナリオでjava.util.concurrent.AtomicReferenceを使用することを検討することをお勧めします。これにより、これらのことが期待どおりに機能することが確認されます。

Answer 2

この方法を使用すると、setListener(null)が返された後にリスナーがイベント通知を受信しないことが保証されなくなります。次のように実行は続行できます

Listener l = listener; // listener != null at this point 

// setListener(null) executes here 

if (l != null) { 
    l.onEvent(...); 
} 

をあなたはそれが未登録になった後はイベントがリスナーに掲載されないことを保証する必要がある場合は、あなたがsynchronizedブロックを使用する必要があります。 listenerをvolatileとすることは役に立ちません。コードは、代わりに次のようになります。

public synchronized void setListener(Listener l) { 
    listener = l; 
} 

public void run() { 
    while (running) { 
     ... do something ... 

     synchronized (this) { 
      if (listener != null) { 
       listener.onEvent(...); 
      } 
     } 
    } 
} 

あなたが​​すべての時間の費用を避けたい場合は、あなたがこれを行うことができます：

if (listener != null) { 
    synchronized (this) { 
     if (listener != null) { 
      listener.onEvent(...); 
     } 
    } 
} 

をこれはあなたが後にイベントを欠場する若干のリスクを実行しますnull以外のリスナーを設定します。 listenerをvolatileとすると、それはおそらく修正されます。