java - 不可修改集合中的 ConcurrentModificationException

Question

我在下面有这段代码，我通过执行以下行得到了 ConcurrentModificationException：

filterCardsToDevice(getCollection());

编码：

private List<MyClass> filterCardsToDevice(Collection<MyClass> col) {
    final List<MyClass> newList = new ArrayList<MyClass>();

    for (MyClass myObj : col) {
        long id = myObj.getId();
        if (id < 0 || id > 0xFFFFFFFFl) {
            // just a log here
        } else {
            newList.add(myObj);
        }
    }

    return newList;
}

private final Map<Long, MyClass> map = new HashMap<Long, MyClass>();

public Collection<MyClass> getCollection() {
    synchronized (map) {
        return Collections.unmodifiableCollection(map.values());
    }
}

堆栈是：

at java.util.HashMap$HashIterator.nextEntry(HashMap.java:841)                 
at java.util.HashMap$ValueIterator.next(HashMap.java:871)                 
at java.util.Collections$UnmodifiableCollection$1.next(Collections.java:1010)

正好在 foreach 行：

for (MyClass myObj : col) {

我不明白为什么会发生此错误，因为我没有修改列表。

Answer 1

请注意，这Collections.unmodifiable*不是复制集合数据，而只是将原始集合包装在一个特殊的包装器中。所以如果你修改原始集合，你会得到这个错误。

---

如果你想创建真正独立的不可修改的集合实例：

Collections.unmodifiableCollection(new ArrayList<>(map.values()));

Answer 2

map迭代时，您必须在另一个线程中更新col. map#values和Collections.unmodifiableCollection返回现有数据结构的视图，所以您正在迭代的（这由您的堆栈跟踪见证）是您的map.