for-each 迭代线程安全吗？

Question

我在多线程环境中遇到一种情况，没有逻辑问题，但我担心 java Set 的迭代器是否会在某些极端情况下中断。我不知道这是否会在一段时间后崩溃：

class Bar{
    AtomicLong a = 0;
    AtomicLong b = 0;

    public void addA(int value){
        a.addAndGet(value);
    }
    public void addB(int value){
        b.addAndGet(value);
    }
}

class Foo{
    Set<Bar> bars = new CopyOnWriteArraySet();

    public void addBarData(Bar bar){
        if (!bars.add(bar)){
            for (Bar b: bars){
                if (bar.equals(b)){
                    b.addA(bar.a);
                    b.addB(bar.b);
                }
            }
        }
    }
}

问题可能是如果两个线程在不久的时间调用 addBarData(barInstance)，一个正在进入 ifs 主体，并开始迭代 for-each，但另一个此时将另一个元素添加到集合中。 for循环（或迭代器）是否会因为元素数量的变化而崩溃？

Answer 1

不，for-each 循环只是一种编译器魔法，相当于使用Iterator 进行集合迭代和数组索引。它不会添加任何其他效果，例如线程安全。

Answer 2

您问题中的代码看起来是线程安全的，因为：

• Bar 类使用原子和线程安全的 AtomicLong。
• CopyOnWriteArraySet 也是线程安全的（它会在每次写入操作时复制整个数组）。

所以这一切都是线程安全的，你不会得到ConcurrentModificationException，而且这段代码的结果是可以预测的，因为你迭代的集合实际上是不可变的。