回避方法対話型リスナーの場合はConcurrentModificationException

Question

異なるイベントのリスナーを保持するmEventMapを持ち、addListener（）を使用してイベントリスナー、removeListener（）およびdispatchEvent（）を登録されたリスナーに登録します。 dispatchEvent（中

public void addListener(EventListener listener) { 
    synchronized (mEventMap) { 

     List<WeakReference<EventListener<Event>>> listeners = mEventMap.get(listener.mEventClass); 
WeakReference<EventListener<Event>> listenerRef = new WeakReference<>(
       (EventListener<Event>) listener) 
     … 
     listeners.add(listenerRef); 

     … 
    } 

}

public void removeListener(EventListener listener) { 
    synchronized (mEventMap) { 
     List<WeakReference<EventListener<Event>>> listeners = mEventMap.get(listener.mEventClass); 

     … 
     if (contains(listeners, listener)) { 
       doRemove(listeners, listener); 
      } 
     … 

    } 
} 


public boolean dispatchEvent(Event event) { 
synchronized (mEventMap) { 
    List<WeakReference<EventListener<Event>>> listeners = mEventMap.get(event.getClass()); 
    ListIterator<WeakReference<EventListener<Event>>> listenerIterator = listeners.listIterator(listeners.size()); 
    … 
    while (listenerIterator.hasPrevious()) { 
      WeakReference<EventListener<Event>> listenerItem = listenerIterator.previous(); 
      EventListener<Event> listenerRef = listenerItem.get(); 
      if (listenerRef != null) { 
       listenerRef.onEvent(event); 
      } else { 
       listenerIterator.remove(); 
      } 
     } 

    … 
} 

ユースケース

EventListener<Event> mEventListener = new EventListener<Event>(
     Event.class) { 
    @Override 
    public boolean onEvent(Event event) { 
     eMgr.removeListener(mEventListener); 
     // do something 
    } 
}; 

addEventListener(mEventListener); 

）、それはループのremoveListener（にあるとき）と呼ばれ、listenerItem = listenerIteratorでConcurrentModificationExceptionを引き起こします。前（）;

質問：誰かが反復している間にmEventMapデータの変更によって引き起こされるクラッシュを避ける最良の方法は何ですか？

Answer 1

選択するアプローチは、リスナーオブジェクトに、「isRemoved」フラグを追加することです。したがって、removeListenerは、このリスナーが移動されたことだけをマークしますが、マークされていないディスパッチ・タイムにディスパッチするまで、リストに残ります。マークされたものはディスパッチ・ループの後にリストから削除されます。

Answer 2

のようなループに何かする前dispatchEventであなたのリストの安全なコピーを作成します。

List<WeakReference<EventListener<Event>>> listeners 
          = new ArrayList<>(mEventMap.get(event.getClass())); 

しかし、これまでのリスナーのための最善のアプローチは、変更されていないとして、リスナーのリストを管理するためにCopyOnWriteArrayListを使用することですそれはあまりにも頻繁に、あなたはもうsynchronized blocksを必要としないように既にスレッドセーフです。

Answer 3

リストから要素を削除する場合は、暗黙のイテレータではを削除します。

あなたが明示的にイテレータを使用してのことを解決することができ：

for (Iterator<EventListener> it = list.iterator(); it.hasNext();) { 
    EventListener el = it.next(); 
    it.remove(); 
} 

Answer 4

もう1つのアプローチは、javaのEventListenerListクラスhttps://docs.oracle.com/javase/8/docs/api/index.html?javax/swing/event/EventListenerList.htmlで構築されたものを使用することです。 javax.swingパッケージに含まれていますが、一般的なイベント通知には非常に便利です。これは配列によってサポートされ、クラスごとにリスナーを管理し、同時mod例外をスローしません。イベントはそのように行われ、発射：

Object[] listeners = listenerList.getListenerList(); 
    for(int i = listeners.length - 2; i >= 0; i -= 2){ 
     if(listeners[i] == YourListenerClass.class){ 
      ((YourListenerClass)listeners[i + 1]).yourListenerMethod(yourEvent); 
     } 

    }