我自己的代码中的 ConcurrentModificationException

Question

---

我认为我的解决方案被错误地误认为是另一个问题的解决方案。它也已被编辑，因此看起来解决方案是重复项所指向的解决方案，但这显然没有解决我的问题，只是避免了引发的异常（问题的表面）。由于我重新打开问题的努力无济于事，因此我重新打开了一个here。

---

我收到了ConcurrentModificationException，但我无法解决这个问题。所以我从列表中删除了一系列元素，我得到了一个错误。

这是错误：

04-Nov-2016 15:49:50.488 SEVERE [http-nio-8080-exec-199] org.apache.catalina.core.StandardWrapperValve.invoke Servlet.service() for servlet [Faces Servlet] in context with path [/...] threw exception [java.util.ConcurrentModificationException] with root cause
 java.util.ConcurrentModificationException
    at java.util.ArrayList$Itr.checkForComodification(ArrayList.java:886)
    at java.util.ArrayList$Itr.next(ArrayList.java:836)
    at com.ex.PersonController.removeGroup(...)
    at ....

Java 代码：

public void removeGroup(int age) {
    if (person != null) {
        List<Person> friends = person.getFriends();
        List<Person> removeFriends = new ArrayList<>();
        if (friends != null) {
            for (Person p : friends) {
                if (p.getAge() == age) {
                    removeFriends.add(p);
                } 
            }
            friends.removeAll(removeFriends);
        }
    }

---

编辑： 请注意，this 显然没有解决我的问题。虽然它避免了引发错误，但是 JSF 显示的结果不正确。是否有可能 JSF 在我的列表未完全更新时尝试刷新？

---

EDIT2： 我不认为这是重复的。我在一些测试中遇到的错误在this 或this 中消失了（请注意，即使使用 .add() 方法，由于某种原因我也遇到了同样的错误）。但是 JSF 仍然没有打印出我所期望的。我再次编写了 BalusC 从我的问题中删除的代码，并提供了更多信息：

xhtml Java 代码：

<c:forEach items="#{personController.groupedPersons}" var="personG">
    <h:commandButton action="#{personController.removeGroup(personG.key)}" type="submit" value="Remove" >
        <f:ajax render="@form" />
    </h:commandButton>
    #{personG.key} - #{personG.value.size()}
</c:forEach>

所以我第一次执行页面时，personController.groupedPersons 返回正确的人员列表（HashMap<Integer, List<Person>>）并打印出来就好了。在这一点上，我有 2 组：一组 3 个相同年龄的人和另一个不同年龄的人。当我点击 remove 3 个同龄人的组时，我会跟踪代码并使用迭代器删除所有必要的人，而不会引发ConcurrentModificationException。 person.getFriends(); 的返回列表 size = 1，这是正确的。然后 ajax 代码呈现表单。再次调用personController.groupedPersons 并按预期返回 1 个人。我已经验证了，这是我真正期望的回归人。但是 JSF 确实打印了错误的 #{personG.key}（我删除的那个）和 null #{personG.value.size()}。

我知道这可能很难理解，但你能想到任何解释吗？

---

编辑 3： 这更有趣......如果我删除有 1 人的组，它会被删除，然后 JSF 会正确打印有 3 人的组。如果我删除包含 3 个人的组，他们将被删除并且 JSF 打印（正如我在 EDIT2 中所说）我刚刚删除的组的 key 并且 size() 为空。在我修改列表的同时，JSF 刷新页面之间是否存在并发问题？ （这最初是我担心的，ConcurrentModificationException 来自 XHTML 和我的托管 bean 之间的并发问题，而不仅仅是托管 Bean 的代码中）。这可以解释为什么我得到了ConcurrentModificationException，即使我在列表中添加而不是从列表中删除。

Answer 1

在 Java 中，不允许遍历列表 (friends) 并在循环中从中删除元素 (friends.remove(p))。这会抛出一个ConcurrentModificationException。

您可以改用Iterator

Iterator<Person> iterator = friends.iterator();
while (iterator.hasNext()) {
    if (iterator.next().getAge() == age) {
        iterator.remove();
    }
}

编辑：

您使用removeFriends.add(p); 的方法应该有效。你不会得到一个并发修改异常。但是，如果您的基础列表不支持removeAll，您最终可能会得到UnsupportedOperationException。例如Arrays.asList(...) 就是这种情况。如果你包装你的friends 列表是new ArrayList(friends); 你应该很高兴：

public void removeGroup(int age) {
  if (person != null) {
    List<Person> friends = new ArrayList<>(person.getFriends());
    List<Person> removeFriends = new ArrayList<>();
    if (friends != null) {
        for (Person p : friends) {
            if (p.getAge() == age) {
                removeFriends.add(p);
            } 
        }
        friends.removeAll(removeFriends);
    }
}

之后别忘了设置好友列表：

person.setFriends(friends);