Java：在这种特殊情况下如何处理 ConcurrentModificationException？

Question

在我发布这个问题之前，我阅读了here 的帖子。但是对于这个特殊的问题，我真的不知道我该如何应用同样的理念？

在以下问题中，我的变量graph 具有以下结构：HashMap<Integer, ArrayList<Integer>>。 而u 和v 的类型为int。基本上，我想删除键值为i 的arraylists 中的值v，其中i 出现在v 的arraylist 中。

所以，如果整数i出现在v的arraylist中，我们在图中得到i的arraylist并从中删除值v。

这听起来很复杂。但是我现在被这个 ConcurrentModificationException 困了一段时间。

public int random_contraction_algo(){
    while(graph.size() > 2){
        int u = select_remaining_edges_at_random(new ArrayList<Integer> (graph.keySet()));
        int v = select_remaining_edges_at_random(graph.get(u));      
        merge(u,v);        
    }
    return -1;
}

// select a pair of vertices from an edge
public int select_remaining_edges_at_random(ArrayList<Integer> vertices){
    int index = (int)(Math.random() * (vertices.size()));
    return vertices.get(index);
}

public void merge(int u, int v){

    graph.get(u).addAll(graph.get(v));

    // remove self-loops
    graph.get(u).removeAll(Collections.singleton(u));
    graph.get(u).removeAll(Collections.singleton(v));

    // make sure all the edges of v are connected to u instead v
    for(Iterator<Integer> iterator = graph.get(v).iterator(); iterator.hasNext();){
        Integer i = iterator.next();
        graph.get(i).remove((Integer) v);
        graph.get(i).add(u);

    }
    // remove key
      graph.remove(v);
}

更新：非常感谢您的回答。但是，我意识到我忘了告诉你们我的代码中有一个外循环。

我尝试实施您的解决方案，但 ConcurrentModificationException 仍然发生。

这些是错误消息：

Exception in thread "main" java.util.ConcurrentModificationException
    at java.util.ArrayList$Itr.checkForComodification(ArrayList.java:886)
    at java.util.ArrayList$Itr.next(ArrayList.java:836)
    at HashMapOfArrayList.merge(ArraylistOfArrayList.java:96)
    at HashMapOfArrayList.random_contraction_algo(ArraylistOfArrayList.java:69)
    at RunAlgo.main(ArraylistOfArrayList.java:111)

Answer 1

问题在于，当v == i 时，您正在修改与迭代相同的列表。

一个简单的解决方案是防范这种情况：

for(Iterator<Integer> it = graph.get(v).iterator(); it.hasNext();) {
    Integer i = it.next();
    if (i == v) {
        it.remove();
    } else {
        graph.get(i).remove((Integer) v);
    }
}

Answer 2

从循环前的列表中显式删除v：graph.get(v).remove(v)

List<Integer> list = graph.get(v);
list.remove(v);
for(Integer i: list){
   graph.get(i).remove(v)
}

Answer 3

我可能误解了你。但是如果你提

所以，如果整数 i 出现在 v 的 arraylist 中，我们就得到图中 i 的 arraylist 并从中移除值 v。

这是你想做的吗？

HashMap<Integer, ArrayList<Integer>> graph = new HashMap<Integer, ArrayList<Integer>>();
for (int i = 0; i < 100; i++) {
    graph.put(i, new ArrayList<Integer>());
}

int i = 0; // change value to whatever
int v = 0; // change value to whatever
if (graph.get(v).contains(i)) {
    graph.get(i).remove(new Integer(v));
}

如果不是，您可以尝试准确地解释您需要什么。

编辑：

我认为这应该可行：

HashMap<Integer, ArrayList<Integer>> graph = new HashMap<Integer, ArrayList<Integer>>();
for (int i = 0; i < 100; i++) {
    graph.put(i, new ArrayList<Integer>());
}

int v = 0; // change value to whatever
int u = 0;
ArrayList<Integer> list = new ArrayList<Integer>(graph.get(v));
for(Integer i : list){
    if(graph.get(i).contains(v)){
        graph.get(i).remove(new Integer(v));
        graph.get(i).add(u);
    }
}

这会创建一个有问题的ArrayList 的副本，因此我们永远不会在地图本身上进行迭代，从而使对地图的编辑成为可能。

告诉我它是否有效。