我需要从一个java的集合中,根据另一个集合的内容,删除第一个集合中不特定的元素。这看上去非常简单,但却遇到了问题。这是“Java中如何删除一个集合中的多个元素”的问题。

  这是我要写的方法的头部

  private void screenBlackNameList(List<SharedBoardSmsWrapper> source, List<BlackNameListModel> blackNameList)

  事情是这样子的。source集合中保存了一些显示用的数据元素。blackNameList集合中保存的是黑名单列表。我们需要根据黑名单表,把source集合中黑名单用户的数据剔除掉。

  这个问题的解决看上去非常简单。

  我首先使用for each 语句进行删除。

    for(SharedBoardSmsWrapper tmpSharedBoardSmsWrapper:source){ 
             
            for(BlackNameListModel tmpBlackNameListModel:blackNameList){ 
                if(tmpSharedBoardSmsWrapper.getSource().equals(tmpBlackNameListModel.getSource())){ 
                   source.remove(tmpSharedBoardSmsWrapper); 
                   break; 
                } 
                 
            } 
        }

for(SharedBoardSmsWrapper tmpSharedBoardSmsWrapper:source){                  for(BlackNameListModel tmpBlackNameListModel:blackNameList){             if(tmpSharedBoardSmsWrapper.getSource().equals(tmpBlackNameListModel.getSource())){                source.remove(tmpSharedBoardSmsWrapper);                break;             }                      }     }

  非常简单的问题!我暗笑,

  测试…

  令我意外的是,这段代码居然抛出了异常

  java.util.ConcurrentModificationException。

  查看JDK6手册

  public class ConcurrentModificationException

  extends RuntimeException

  当方法检测到对象的并发修改,但不允许这种修改时,抛出此异常。

  例如,某个线程在 Collection 上进行迭代时,通常不允许另一个线性修改该 Collection。通常在这些情况下,迭代的结果是不确定的。如果检测到这种行为,一些迭代器实现(包括 JRE 提供的所有通用 collection 实现)可能选择抛出此异常。执行该操作的迭代器称为快速失败 迭代器,因为迭代器很快完全失败,而不会冒着在将来某个时间任意发生不确定行为的风险。

  注意,此异常不会始终指出对象已经由不同 线程并发修改。如果单线程发出违反对象协定的方法调用序列,则该对象可能抛出此异常。例如,如果线程使用快速失败迭代器在 collection 上迭代时直接修改该 collection,则迭代器将抛出此异常。

  注意,迭代器的快速失败行为无法得到保证,因为一般来说,不可能对是否出现不同步并发修改做出任何硬性保证。快速失败操作会尽大努力抛出 ConcurrentModificationException。因此,为提高此类操作的正确性而编写一个依赖于此异常的程序是错误的做法,正确做法 是:ConcurrentModificationException 应该仅用于检测 bug。

  Java中的For each实际上使用的是iterator进行处理的。而iterator是不允许集合在iterator使用期间删除的。而我在for each时,从集合中删除了一个元素,这导致了iterator抛出了ConcurrentModificationException。

  看来只有老老实实使用传统的for循环了!

    for(int i=0;i<source.size();i++){ 
            SharedBoardSmsWrapper tmpSharedBoardSmsWrapper=source.get(i); 
            for(int j=0;j<blackNameList.size();j++){ 
                BlackNameListModel tmpBlackNameListModel=blackNameList.get(j); 
                if(tmpSharedBoardSmsWrapper.getSource().equals(tmpBlackNameListModel.getSource())){ 
                   source.remove(tmpSharedBoardSmsWrapper); 
                   break; 
                } 
            } 
        }

for(int i=0;i<source.size();i++){         SharedBoardSmsWrapper tmpSharedBoardSmsWrapper=source.get(i);         for(int j=0;j<blackNameList.size();j++){             BlackNameListModel tmpBlackNameListModel=blackNameList.get(j);             if(tmpSharedBoardSmsWrapper.getSource().equals(tmpBlackNameListModel.getSource())){                source.remove(tmpSharedBoardSmsWrapper);                break;             }         }     }

  这下应该没问题了吧!信心满满地按下测试…

  晕!怎么回事,数据怎么过滤得不对?