我有一个数组列表，我想对它进行迭代。在迭代它时，我必须同时删除元素。显然，这会抛出一个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();
当前回答
两个选择:
创建一个您希望删除的值列表，在循环中添加到该列表中，然后在结束时调用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();
}
}
其他回答
两个选择:
创建一个您希望删除的值列表，在循环中添加到该列表中，然后在结束时调用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();
}
}
for (A a : new ArrayList<>(abc)) {
a.doSomething();
abc.remove(a);
}
以正常的方式执行循环，java.util.ConcurrentModificationException是一个与所访问的元素相关的错误。
所以尝试:
for(int i = 0; i < list.size(); i++){
lista.get(i).action();
}
“我应该先克隆列表吗?”
这将是最简单的解决方案，从克隆中删除，并在删除后将克隆复制回来。
以我的rummikub游戏为例:
SuppressWarnings("unchecked")
public void removeStones() {
ArrayList<Stone> clone = (ArrayList<Stone>) stones.clone();
// remove the stones moved to the table
for (Stone stone : stones) {
if (stone.isOnTable()) {
clone.remove(stone);
}
}
stones = (ArrayList<Stone>) clone.clone();
sortStones();
}
关于
import java.util.Collections;
List<A> abc = Collections.synchronizedList(new ArrayList<>());
