Java 中是否有公认的在迭代列表时删除列表元素的最佳实践？

Question

我发现在执行此操作时避免 ConcurrentModificationException 的最佳方法存在相互矛盾的建议：

    List<Apple> Apples = appleCart.getApples();
    for (Apple apple : Apples)
    {
        delete(apple);
    }

我倾向于使用 Iterator 代替 List code> 并调用其 remove 方法。

这在这里最有意义吗？

Answer 1

是的，使用迭代器。然后你可以使用它的删除方法。

  for (Iterator<Apple> appleIterator = Apples.iterator(); appleIterator.hasNext();) {
     Apple apple = appleIterator.next();
     if (apple.isTart()) {
        appleIterator.remove();
     }
  }
}

Answer 2

如果您收到 ConcurrentModificationException，则您可能有多个线程。

因此，完整的答案包括使用 Iterator.remove() 和同步对集合的访问。

例如（其中lock由可能修改列表的所有线程同步）：

synchronized ( lock ) {
   List<Apple> apples = appleCart.getApples();
   for ( Iterator<Apple> it = apples.iterator(); it.hasNext(); )
   {
      Apple a = it.next(); 
      if ( a.hasWorm() ) {
         it.remove();
      }
   }
}

Answer 3

List<Apple> apples = appleCart.getApples();
for (Iterator<Apple> appleIterator = apples.iterator(); appleIterator.hasNext();)
{
   Apple apple = appleIterator.next();
   if ( apple.isYucky() ) {
     appleIterator.remove();
   }
}

Answer 4

您可以保留要删除的项目列表，然后在循环后删除它们：

List<Apple> apples = appleCart.getApples();
List<Apple> badApples = new ArrayList<Apple>();
for (Apple apple : apples) {
    if (apple.isBad()) {
        badApples.add(apple);
    } else {

    eat(apple);
}

apples.removeAll(badApples);

Answer 5

从 Java 8 开始，您现在可以执行以下操作：
<代码>
apples.removeIf(苹果 -> apple.equals(this))