为什么我使用同步列表仍然得到 ConcurrentModificationException [重复]

Question

我在I used synchronized list, and i still get ConcurrentModificationException 找到了同样的问题，但没有找到正确答案。

这是我定义我的synchronized 列表的方式：

private List<ActionItemClickListener> actionItemClickListeners = Collections.synchronizedList(new ArrayList<ActionItemClickListener>());

这是我的使用方法：

@Override
public void onBackPressed() {
    boolean isConsume = false;
    synchronized (actionItemClickListeners) {
        //ConcurrentModificationException occur here
        for (ActionItemClickListener listener : actionItemClickListeners) {
            isConsume = isConsume | listener.onSystemBackPressed();
        }
    }
    if(!isConsume) {
        pendingFragment = null;
        pendingTag = null;
        currentFragmentTag = null;
        super.onBackPressed();
    }
}
public void addActionItemClickListener(ActionItemClickListener listener) {
        synchronized (actionItemClickListeners) {
            if (listener != null)
                actionItemClickListeners.add(listener);
        }
}

public void removeActionItemClickListener(ActionItemClickListener listener) {
        synchronized (actionItemClickListeners) {
            if (listener != null)
                actionItemClickListeners.remove(listener);
        }
}

为什么我仍然收到ConcurrentModificationException???

Answer 1

您的ConcurrentModificationException 不是线程问题。它被抛出是因为您在迭代列表时正在修改它。

for (ActionItemClickListener listener : actionItemClickListeners) {
    isConsume = isConsume | listener.onSystemBackPressed();
}

问题在于listener.onSystemBackPressed(); 正在调用removeActionItemClickListener，它试图更改您正在迭代的actionItemClickListeners。尝试更改您正在迭代的列表将导致 ConcurrentModificationException

如何解决

没有看到您的其余代码，我只能猜测如何解决此问题，但您可以使用iterator.next()、iterator.hasNext() 和iterator.remove() 分别循环和删除侦听器。您要么需要通过onSystemBackPressed() 和removeActionItemClickListener 传递iterator，这有点难看，要么让onSystemBackPressed() 返回一个布尔值，指示它是否应该删除侦听器。

final Iterator<String> iterator = actionItemClickListeners.iterator();
while (iterator.hasNext()) {
    final String listener = iterator.next();
    final boolean removeListener = listener.onSystemBackPressed();
    if (removeListener) {
        iterator.remove();
    }
    isConsume = isConsume | removeListener;
}

不幸的是，您似乎将onSystemBackPressed() 的返回值用于其他用途，因此这也可能不起作用。