我有一个ArrayList需要遍历。在遍历时，我需要同时删除元素。显然这会抛出一个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();
}
}
来自ArrayList的JavaDocs
该类的iterator和listIterator方法返回的迭代器是快速失败的:如果在迭代器创建后的任何时间结构上修改列表，除了通过迭代器自己的remove或add方法之外的任何方式，迭代器将抛出ConcurrentModificationException。
for( int i = 0; i < lStringList.size(); i++ )
{
String lValue = lStringList.get( i );
if(lValue.equals("_Not_Required"))
{
lStringList.remove(lValue);
i--;
}
}
这同样有效。
在Java 8中，您可以使用Collection接口并通过调用removeIf方法来实现此操作：
yourList.removeIf((A a) -> a.value == 2);
更多信息可以在这里找到
你真的应该按照传统方式迭代数组。
每次从列表中删除一个元素后，后面的元素会向前移动。只要不改变除正在迭代的元素以外的元素，以下代码应该可以工作。
public class Test(){
private ArrayList<A> abc = new ArrayList<A>();
public void doStuff(){
for(int i = (abc.size() - 1); i >= 0; i--)
abc.get(i).doSomething();
}
public void removeA(A a){
abc.remove(a);
}
}
在遍历列表时，如果您想要删除元素是可能的。让我们看下面的例子，
ArrayList<String> names = new ArrayList<String>();
names.add("abc");
names.add("def");
names.add("ghi");
names.add("xyz");
我有一个数组列表，其中包含上述名称。我想从列表中删除“def”名称。
for(String name : names){
if(name.equals("def")){
names.remove("def");
}
}
以上代码会抛出ConcurrentModificationException异常，因为您在迭代时修改了列表。
因此，要按照以下方式从Arraylist中删除“def”名称：
Iterator<String> itr = names.iterator();
while(itr.hasNext()){
String name = itr.next();
if(name.equals("def")){
itr.remove();
}
}
按照正常的方式进行循环，java.util.ConcurrentModificationException 是与被访问元素相关的错误。
因此，请尝试以下方法：
for(int i = 0; i < list.size(); i++){
lista.get(i).action();
}
java.util.ConcurrentModificationException。有点棘手 :)
你真的不能称这为迭代列表的“正常方式”。 - Zsolt Sky这里有一个例子，我使用了另一个列表来添加要删除的对象，然后使用 stream.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));
}
您也可以使用CopyOnWriteArrayList而不是ArrayList。这是从JDK 1.5开始推荐的最新方法。
removeA方法修改为以下内容-public void removeA(A a,Iterator<A> iterator) {
iterator.remove(a);
}
doSomething() 应该能够将 iterator 传递给 remove 方法。这不是一个很好的主意。List 中。removeAll 删除第二个列表中的所有元素即可。