arraylist，在for循环中删除一个变量

Question

我没有经常使用ArrayList，并且我的印象是，要从中提取变量，我需要一个增强的 for 循环。

我正在尝试取出变量并将它们的 x 值与另一个 x 和 y 值与另一个 y 值进行比较，如果它们匹配则删除该变量。

到目前为止，我对这个方法的代码是；

public int detect(int x, int y){
    int count=0;
    for (EnemyShip tempEnemy:EList){
        if(x==tempEnemy.x && y==tempEnemy.y){
            EList.remove(tempEnemy);
            count++;
        }
    }
    return count;
}

我知道问题出在EList.remove(tempEnemy); 上，并且如果它是正常的 for 循环，我知道如何完成此任务。

但是这种增强的 for 循环（我的讲师称之为）让我感到困惑。

所以我想我的问题是如何从 Arraylist 中删除同时匹配 x 和 y 的变量？

Answer 1

使用 Jdk 8，您可以使用 removeIf 方法轻松安全地删除元素。

  EList.removeIf(e-> x==e.x && y== e.y);

有关removeIf方法的详细信息，请阅读此docs。

Answer 2

当您迭代列表时（for-each 循环在内部创建迭代器）同时您正在更改不支持并导致 ConcurrentModificationException 的 List 的结构。

为此使用Iterator<E> api -

for(Iterator iterator = EList.iterator(); iterator.hasNext();) {
    tempEnemy = iterator.next();
    if(x==tempEnemy.x && y==tempEnemy.y){
         iterator.remove();
    }
}

注意：建议使用type-safeList<E>和Iterator<E>。

Answer 3

因为坐标上只有一艘船，所以不需要计数器。

改用这个：

01 boolean strike;
02 do {
03     strike = false;
04     for (EnemyShip tempEnemy: EList) {
05         if (x==tempEnemy.x && y==tempEnemy.y) {
06             strike = true;
07             EList.remove(tempEnemy);
08             break; // We need to break here, because the line07 maybe made
09             // the list empty and cause a ConcurrentModificationException.
10             // Also: tempEnemy is not longer part of the list "EList" so we have a invalid state.
11         }
12     } 
13 } while (strike);