瞭解和解決ConcurrentModificationException的

Question

我處理間歇，難以重現ConcurrentModificationException中的遺留代碼塊：

class LegacyCode { 
    private final WeakHashMap<A, B> mItems = new WeakHashMap<A, B>(); 

    public void someWork() { 
     final List<A> copy = new LinkedList<A>(); 

     for (final A item : mItems.keySet()) { 
      copy.add(item); 
     } 

     for (final A item : copy) { 
      item.someMethod(); 
     } 
    } 

    public void addItem(final A item) { 
     mItems.put(item, new B()); 
    } 

    public void removeItem(final A item) { 
     mItems.remove(item); 
    } 
} 

的CME被拋出：

for (final A item : mItems.keySet()) { 
    copy.add(item); 
} 

我不完全確定爲什麼我們以這種方式創建copy。 CME被拋出，因爲在for-each循環正在運行時調用addItem(A)或removeItem(A)。

問題

1. 是我爲什麼CME被拋出正確的認識？

2. 我會盡量避免CME如果我更換了，每個循環：

   final List<A> copy = new LinkedList<A>(mItems.keySet());

3. 將這種變化等同於for-each循環，我們將取代？據我所知，這兩個片段創建mItems.keySet()在copy的淺拷貝。

Answer 1

我的理解是爲什麼CME被拋出正確嗎？

絕對。這正是發生的情況。

我會盡量避免CME如果我更換for-each循環：

final List<A> copy = new LinkedList<A>(mItems.keySet()); 

不，你不會的，因爲LinkedList<A>構造將有一個類似的循環。所以你說這個改變將等於我們將要替換的for-each循環是正確的。

就解決這個問題而言，Java標準庫中的WeakHashMap類沒有現成的併發替換。您可以通過使addItem和removeItem同步，並在構建copy的循環周圍添加一個同步塊來解決這個問題。您也可以查看third-party collections解決此問題，而無需在代碼中使用​​。