尝试从ArrayList中删除项时出现ConcurrentModificationException[重复]

Question

实际上，在按下一个按钮后，我有一个方法可以检查itemStorico ArrayList中是否存在从itemModel添加的新项，然后如果该项存在于名为itemModel的新列表中，我将从itemStorico中删除旧项并添加新项，否则只添加新项而不删除旧项。

但我得到以下错误：

2018-12-11 16:34:04.067 29033-29033/it.gabtamagnini.realco E/AndroidRuntime: FATAL EXCEPTION: main
    Process: it.gabtamagnini.realco, PID: 29033
    java.util.ConcurrentModificationException
        at java.util.ArrayList$Itr.next(ArrayList.java:831)
        at it.gabtamagnini.realco.OrdiniActivity.saveStorico(OrdiniActivity.java:1396)
        at it.gabtamagnini.realco.OrdiniActivity$29.onClick(OrdiniActivity.java:1368)
        at android.view.View.performClick(View.java:5637)
        at android.view.View$PerformClick.run(View.java:22433)
        at android.os.Handler.handleCallback(Handler.java:751)
        at android.os.Handler.dispatchMessage(Handler.java:95)
        at android.os.Looper.loop(Looper.java:154)
        at android.app.ActivityThread.main(ActivityThread.java:6130)
        at java.lang.reflect.Method.invoke(Native Method)
        at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:886)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:776)

下面是我的方法中的代码：

 for(ItemModel itemModels : itemModel){
            boolean exist = false;
            for(ItemModel itemModel2 : itemStorico){
                if(itemModels.getCodiceArticolo().contains(itemModel2.getCodiceArticolo())) {
                    itemStorico.remove(itemModel2);
                    itemStorico.add(itemModels);
                    exist = true;
                }
            }
            if(!exist) {
                itemStorico.add(itemModels);
            }
        }

Answer 1

不能从“for-each”循环(for(item:collection)结构）内部调用remove()。这样做将引发ConcurrentModificationException。

如果要在循环时循环并移除项，可以使用传统的“for”循环：

for (int i = 0; i < itemStorico.size(); ++i) {
    ItemModel itemModel2 = itemStorico.get(i);
    if (...) {
        itemStorico.remove(i);
        ...
    }
}

但是，这会引入一个微妙的bug。假设您正在循环[a,b,c,D]。如果您位于索引1，并从集合中删除b，则所有内容都将“滑过”，您将得到[a,c,d]。当您下一次查看索引2时，您将跳过C。

您可以通过在删除时手动减少索引来避免该错误，但这有点粗略。

...
    itemStorico.remove(i);
    --i;
...

您还可以通过使用集合的iterator()来完全避免这个问题。

for (Iterator<ItemModel> iterator = itemStorico.iterator(); iterator.hasNext(); ) {
    ItemModel itemModel2 = iterator.next();
    if (...) {
        iterator.remove();
        ...
    }
}