我有一个数组列表，我想对它进行迭代。在迭代它时，我必须同时删除元素。显然，这会抛出一个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中。
然后，迭代完成后，只需从第一个列表中删除第二个列表中的所有元素。
其他回答
“我应该先克隆列表吗?”
这将是最简单的解决方案，从克隆中删除，并在删除后将克隆复制回来。
以我的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();
}
这是一个例子，我使用一个不同的列表来添加对象删除，然后我使用流。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));
}
如果你的目标是从列表中删除所有元素，你可以遍历每一项，然后调用:
list.clear()
使用普通的For循环而不是For循环。例如，下面的代码删除了数组列表中的所有元素，而没有给出java.util.ConcurrentModificationException。您可以根据您的用例修改循环中的条件。
for(int i=0; i<abc.size(); i++) {
e.remove(i);
}
我知道我迟到了，但我回答这个问题，因为我认为这个解决方案简单而优雅:
List<String> listFixed = new ArrayList<String>();
List<String> dynamicList = new ArrayList<String>();
public void fillingList() {
listFixed.add("Andrea");
listFixed.add("Susana");
listFixed.add("Oscar");
listFixed.add("Valeria");
listFixed.add("Kathy");
listFixed.add("Laura");
listFixed.add("Ana");
listFixed.add("Becker");
listFixed.add("Abraham");
dynamicList.addAll(listFixed);
}
public void updatingListFixed() {
for (String newList : dynamicList) {
if (!listFixed.contains(newList)) {
listFixed.add(newList);
}
}
//this is for add elements if you want eraser also
String removeRegister="";
for (String fixedList : listFixed) {
if (!dynamicList.contains(fixedList)) {
removeResgister = fixedList;
}
}
fixedList.remove(removeRegister);
}
所有这些都是用于从一个列表更新到另一个列表，您可以从一个列表中进行所有更新 而在方法更新中，你既可以检查列表，也可以在列表之间删除或添加元素。 这意味着两个列表总是相同的大小
