java - arraylist.forEach 中的引用成员被修改？

Question

我正在创建一个生存模拟器，并且我有一个 Entity 的 arrayList。我正在编写一个 checkForDead() 方法，如果成员死亡，它将删除该成员。现在，我有一个很长的 for 语句来执行此操作。但是，我想使用 arrayList.forEach() 以便使其更具可读性。如前所述，部分操作必须将其删除。如何引用 forEach() 方法中正在修改的成员？例如

a.forEach(a.remove(x));

其中 a 是列表，x 是正在修改的成员。我怎样才能得到x是什么？

checkForDead方法中的原始代码:

for (int x = 0; x < a.size(); x++) {

            if (a.get(x).calories <= 0) {
                Fates.addDeathRecord(a.get(x).name, x, "starved to death");
                a.remove(x);
            }

            else if (a.get(x).hydration <= 0) {
                Fates.addDeathRecord(a.get(x).name, x, "died of thirst");
                a.remove(x);
            }

            else if (a.get(x).heat <= 0) {
                Fates.addDeathRecord(a.get(x).name, x, "froze to death");
                a.remove(x);
            }

            else if (a.get(x).heat >= 14) {
                Fates.addDeathRecord(a.get(x).name, x, "overheated");
                a.remove(x);
            }

            else if (a.get(x).moral <= Chance.randomNumber(0, 2)) {
                Fates.addDeathRecord(a.get(x).name, x, "commited suicide");
                a.remove(x);
            }

        }

    }

Answer 1

forEach方法可能不适合在结构上修改正在迭代的集合。正如 javadoc 中所述:

The default implementation behaves as if:

 for (T t : this)
     action.accept(t);

根据您使用的 List 实现，通过添加或删除操作集合可能会导致 ConcurrentModificationException。这是使用传统Iterator 和 remove 的情况。可能仍然是最好的解决方案。

//keep track of index for death record
int x = 0;
for (Iterator<Entry> iter = a.iterator(); iter.hasNext(); ++x) {

        final Entry next = iter.next();
        if (next.calories <= 0) {
            Fates.addDeathRecord(next.name, x, "starved to death");
            iter.remove();
        }

        else if (next.hydration <= 0) {
            Fates.addDeathRecord(next.name, x, "died of thirst");
            iter.remove();
        }

        else if (next.heat <= 0) {
            Fates.addDeathRecord(next.name, x, "froze to death");
            iter.remove();
        }

        else if (next.heat >= 14) {
            Fates.addDeathRecord(next.name, x, "overheated");
            iter.remove();
        }

        else if (next.moral <= Chance.randomNumber(0, 2)) {
            Fates.addDeathRecord(next.name, x, "commited suicide");
            iter.remove();
        }

}