我有一个数组列表，我想对它进行迭代。在迭代它时，我必须同时删除元素。显然，这会抛出一个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();
当前回答
一种选择是将removeA方法修改为this -
public void removeA(A a,Iterator<A> iterator) {
iterator.remove(a);
}
但这意味着doSomething()应该能够将迭代器传递给remove方法。这不是个好主意。
你能分两步做到吗? 在第一个循环中，当遍历列表时，不是删除所选元素，而是将它们标记为要删除。为此，您可以简单地将这些元素(浅复制)复制到另一个List中。
然后，迭代完成后，只需从第一个列表中删除第二个列表中的所有元素。
其他回答
从数组列表的JavaDocs
该类的迭代器和listIterator返回的迭代器 方法是快速失败的:如果列表在任何地方被结构修改 方法创建迭代器后的任何时间 迭代器自己的remove或add方法时，迭代器将抛出 并发修改异常
如果你的目标是从列表中删除所有元素，你可以遍历每一项，然后调用:
list.clear()
“我应该先克隆列表吗?”
这将是最简单的解决方案，从克隆中删除，并在删除后将克隆复制回来。
以我的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();
}
在Java 8中，你可以通过调用removeIf方法来使用Collection Interface:
yourList.removeIf((A a) -> a.value == 2);
更多信息可以在这里找到
这是一个例子，我使用一个不同的列表来添加对象删除，然后我使用流。Foreach从原始列表中删除元素:
private ObservableList<CustomerTableEntry> customersTableViewItems = FXCollections.observableArrayList();
...
private void removeOutdatedRowsElementsFromCustomerView()
{
ObjectProperty<TimeStamp> currentTimestamp = new SimpleObjectProperty<>(TimeStamp.getCurrentTime());
long diff;
long diffSeconds;
List<Object> objectsToRemove = new ArrayList<>();
for(CustomerTableEntry item: customersTableViewItems) {
diff = currentTimestamp.getValue().getTime() - item.timestamp.getValue().getTime();
diffSeconds = diff / 1000 % 60;
if(diffSeconds > 10) {
// Element has been idle for too long, meaning no communication, hence remove it
System.out.printf("- Idle element [%s] - will be removed\n", item.getUserName());
objectsToRemove.add(item);
}
}
objectsToRemove.stream().forEach(o -> customersTableViewItems.remove(o));
}
