在多线程环境中遍历集合：ConcurrentModificationException

Question

我收到一个ConcurrentModificationException因为多个线程同时到达了此代码：

public void flush(Audit... audits) {
   // Copy first them on memory
   this.pendingAudits.addAll(Arrays.asList(audits));

   for (Iterator<Audit> it = this.pendingAudits.iterator(); it.hasNext();) {
       // Do something
       it.remove();
   }
}

我正在获得一个pendingAudits的迭代器迭代器，并且在其他线程可以添加其他审计的同时删除每个元素。

如何优雅地解决它？

Answer 1

一个快速的解决办法是换每次访问this.pendingAudits在synchronized ：

sychronized(this.pendingAudits) {
   this.pendingAudits.addAll(Arrays.asList(audits));

   for (Iterator<Audit> it = this.pendingAudits.iterator(); it.hasNext();) {
       // Do something
       it.remove();
   }
}

最终结果是，在任何给定时间，只有一个线程可以执行同步块。 如果频繁执行代码，则可能成为瓶颈。