我有一个数组列表,我想对它进行迭代。在迭代它时,我必须同时删除元素。显然,这会抛出一个java.util.ConcurrentModificationException。

处理这个问题的最佳实践是什么?我应该先克隆列表吗?

我删除的元素不是循环本身,而是代码的另一部分。

我的代码是这样的:

public class Test() {
    private ArrayList<A> abc = new ArrayList<A>();

    public void doStuff() {
        for (A a : abc) 
        a.doSomething();
    }

    public void removeA(A a) {
        abc.remove(a);
    }
}

a.doSomething可能调用Test.removeA();


当前回答

使用迭代器而不是数组列表

有一个集合被转换为类型匹配的迭代器

然后移动到下一个元素并删除

Iterator<Insured> itr = insuredSet.iterator();
while (itr.hasNext()) { 
    itr.next();
    itr.remove();
}

移动到下一个元素在这里很重要,因为它应该使用索引来删除元素。

其他回答

两个选择:

创建一个您希望删除的值列表,在循环中添加到该列表中,然后在结束时调用originalList.removeAll(valuesToRemove) 对迭代器本身使用remove()方法。注意,这意味着您不能使用增强的for循环。

作为第二个选项的示例,从列表中删除长度大于5的任何字符串:

List<String> list = new ArrayList<String>();
...
for (Iterator<String> iterator = list.iterator(); iterator.hasNext(); ) {
    String value = iterator.next();
    if (value.length() > 5) {
        iterator.remove();
    }
}

以正常的方式执行循环,java.util.ConcurrentModificationException是一个与所访问的元素相关的错误。

所以尝试:

for(int i = 0; i < list.size(); i++){
    lista.get(i).action();
}

有时候守旧是最好的。只需要一个简单的for循环,但要确保从列表的末尾开始,否则当你删除项目时,你会与索引不同步。

List<String> list = new ArrayList<>();
for (int i = list.size() - 1; i >= 0; i--) {
  if ("removeMe".equals(list.get(i))) {
    list.remove(i);
  }
}

你试图在高级的“for循环”中从列表中删除值,这是不可能的,即使你应用了任何技巧(你在你的代码中做了)。 更好的方法是编写迭代器级别的代码。

我想知道为什么人们没有建议传统的for循环方法。

for( int i = 0; i < lStringList.size(); i++ )
{
    String lValue = lStringList.get( i );
    if(lValue.equals("_Not_Required"))
    {
         lStringList.remove(lValue);
         i--; 
    }  
}

这也同样有效。

你也可以使用CopyOnWriteArrayList来代替ArrayList。这是JDK 1.5以后推荐的最新方法。