我有一个数组列表，我想对它进行迭代。在迭代它时，我必须同时删除元素。显然，这会抛出一个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();
当前回答
从数组列表的JavaDocs
该类的迭代器和listIterator返回的迭代器 方法是快速失败的:如果列表在任何地方被结构修改 方法创建迭代器后的任何时间 迭代器自己的remove或add方法时，迭代器将抛出 并发修改异常
其他回答
如果你的目标是从列表中删除所有元素，你可以遍历每一项，然后调用:
list.clear()
做一些简单的事情，像这样:
for (Object object: (ArrayList<String>) list.clone()) {
list.remove(object);
}
一种选择是将removeA方法修改为this -
public void removeA(A a,Iterator<A> iterator) {
iterator.remove(a);
}
但这意味着doSomething()应该能够将迭代器传递给remove方法。这不是个好主意。
你能分两步做到吗? 在第一个循环中，当遍历列表时，不是删除所选元素，而是将它们标记为要删除。为此，您可以简单地将这些元素(浅复制)复制到另一个List中。
然后，迭代完成后，只需从第一个列表中删除第二个列表中的所有元素。
这是一个例子，我使用一个不同的列表来添加对象删除，然后我使用流。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));
}
关于
import java.util.Collections;
List<A> abc = Collections.synchronizedList(new ArrayList<>());
