如何在java中将两个迭代器保留在map上,并在没有ConcurrentModificationException的情况下删除其间的键

时间:2013-03-11 07:58:57

标签: java iterator hashmap concurrentmodification

我必须处理Map <BitSet,List<List<Integer>> MyMap

if (key1 contains all of corresponding true bits of key2)
     Remove from key2 all those values which are common with key1)

在此过程中,如果列表中的元素数量低于THRESHOLD(用户定义的正整数),则会将其删除。此外,如果Map包含空列表,则删除相应的键。

我正在使用以下代码:

List<BitSet> keys = new ArrayList<>(MyMap.keySet());  
ListIterator it1=keys.listIterator();
while(it1.hasNext())  {
     BitSet key1=(BitSet)it1.next();
     ListIterator it2=keys.listIterator(it1.nextIndex());
     while(it2.hasNext()) {
         BitSet key2=(BitSet)it2.next();                 
         BitSet ankey=(BitSet)key1.clone();
         ankey.and(key2);    
         if(ankey.equals(key1)) {//key1 is subset and key2 is superset
               if(removePoints(key1,key2))  {
                     it1.remove();
                     break;
               }
         }
         else if(ankey.equals(key2))  {                           
              if(removePoints(key2,key1))  {
                    it2.remove();                         
              }
         }
     }
}

public static boolean removePoints(BitSet key1,BitSet key2)
 {
     List<List<Integer>> list1=MyMap.get(key1);         
     List<List<Integer>> list2=MyMap.get(key2);
     Boolean ret=false;         
     for(int i=0;i<list1.size();i++)  {                   
         List<Integer> sublist1=list1.get(i);            
         for(int j=0;j<list2.size();j++)  {            
             List<Integer> sublist2=list2.get(j);                 
             sublist1.removeAll(sublist2);
             if(sublist1.isEmpty())
                 break;
         }
         if(sublist1.size()<=THRESHOLD)
             list1.remove(sublist1);
         if( list1.isEmpty()) {             
             MyMap.remove(key1); 
             ret=true;                 
         }
     }
     return ret;
 }

但程序出错:

java.util.ConcurrentModificationException at java.util.ArrayList$Itr.checkForComodification
at java.util.ArrayList$Itr.next

另外,我不确定这是否是有效的代码方式?由于地图包含~2000个条目。请指教。

3 个答案:

答案 0 :(得分:1)

在创建ConcurrentModificationException后修改基础集合时,可能会发生Iterator,而且修改不是通过Iterator本身完成的。

在您编写的代码中,只有一个地方可以发生:it1it2之间的交互,它们是同一集合上的迭代器。如果您在一个电话上拨打remove,则另一个电话会在您下次拨打next时中断。

有多种方法可以解决这个问题,但有一种方法是将您从'key'集合中删除的内容与该集合的迭代分开,如下所示:

List<BitSet> allKeys = new ArrayList<>(MyMap.keySet());  
List<BitSet> removedKeys = new ArrayList<>();

for (ListIterator<BitSet> it1 = allKeys.listIterator(); it1.hasNext(); ) {
   BitSet key1 = it1.next();
   for (ListIterator<BitSet> it2 = allKeys.listIterator(it1.nextIndex()); it2.hasNext(); ) {
       BitSet key2 = it2.next();
       BitSet ankey=(BitSet)key1.clone();
       ankey.and(key2);    
       if(ankey.equals(key1)) {//key1 is subset and key2 is superset
           if(removePoints(key1,key2))  {
                 removedKeys.add(key1);
                 break;
           }
       }
       else if(ankey.equals(key2))  {                           
          if(removePoints(key2,key1))  {
                 removedKeys.add(key2);
                 break;
          }
       }
    }
}

allKeys.removeAll(removedKeys);

allKeys将处于您期望的状态。我假设以后你会想要调用MyMap.keySet().retainAll()或类似的。

答案 1 :(得分:0)

在看到ll1, ll2, list1, list2, it1, it2.等变量名后,我放弃了尝试阅读代码。 但是你可以查看这个link来解决类似的问题。

答案 2 :(得分:0)

你不能在Map的keySet上使用iterator.remove(),因为它只是地图内部结构的“视图”。

但是你可以在map的entrySet()上使用迭代器,其中每个元素都是Map.Entry的一个实例,包含你所有的map条目(键/值对)。您可以在此迭代器上调用iterator.remove(),这将有效地从地图中删除相应的键/值对。

Map<Integer, String> map = new HashMap<Integer, String>();
map.put(Integer.valueOf(0), "0");
map.put(Integer.valueOf(1), "1");
map.put(Integer.valueOf(2), "2");
map.put(Integer.valueOf(3), "3");
map.put(Integer.valueOf(4), "4");

System.out.println(map);

Iterator<Map.Entry<Integer, String>> entryIter = map.entrySet().iterator();
while (entryIter.hasNext()) {
   Map.Entry<Integer, String> entry = entryIter.next();
   if (entry.getKey().intValue() % 2 == 0)
       entryIter.remove();
}

System.out.println(map);

希望它有所帮助。

此致