Java:为什么ConcurrentModificationException与同步列表一起出现?

时间:2014-05-02 11:45:12

标签: java arraylist iterator concurrentmodification

使用此代码:

public class SynchroApp {

    public static void main(String[] args) {

        final List<String> unsyList = new ArrayList<>();
        final List<String> syList = Collections.synchronizedList(unsyList);

        TimerTask changeList = new TimerTask() {
            boolean addElem = false;

            @Override
            public void run() {
                // add / remove elements to keep size between 2 and 9
                if (syList.size() < 2)
                    addElem = true;
                else if (syList.size() > 8)
                    addElem = false;
                if (addElem)
                    syList.add(String.valueOf(System.currentTimeMillis()));
                else
                    syList.remove(0);
            }
        };

        TimerTask reverseList = new TimerTask() {
            @Override
            public void run() {
                try {
                    for (String s : syList)
                        s = new StringBuffer(s).reverse().toString();
                } catch (Exception e) {
                    System.out.println("Exception: " + e);
                }
            }
        };

        new Timer().scheduleAtFixedRate(changeList, 0L, 30L);
        new Timer().scheduleAtFixedRate(reverseList, 0L, 20L);
    }
}

为什么我仍会在ConcurrentModificationException上收到一些Iterator.next

编辑:reverseList中列表元素的更新不起作用(如评论中所述)。此代码应按预期工作:

for (int i = 0; i < syList.size(); i++)
    syList.set(i, new StringBuffer(syList.get(i)).reverse().toString());

3 个答案:

答案 0 :(得分:3)

因为您在迭代时修改列表。

请注意,同步列表仅使其每个方法及其迭代器的方法同步。迭代同步列表仍然是非原子操作,涉及对同步方法的多次调用。如果你想让整个迭代成为原子,你必须明确地同步它,使用列表本身作为锁:

synchronized (syList) {
    for (String s : syList) {
        s = new StringBuffer(s).reverse().toString();
    }
}

答案 1 :(得分:1)

即使是大多数同步的集合也不喜欢修改和迭代。来自Collections.synchronizedList的API描述:

  

用户必须手动同步返回的内容   迭代时列出:

     

List list = Collections.synchronizedList(new ArrayList()); ... synchronized (list) { Iterator i = list.iterator(); // Must be in synchronized block while (i.hasNext()) foo(i.next()); }

另外:您可以使用java.concurrent中的集合。他们通常有更精确的同步方法。

答案 2 :(得分:0)

iterator和modifier都必须使用synchronizedList()才能在对象上进行正确的锁定/同步。