Java删除List和Set集合中元素

今天在做项目时，需要删除List和Set中的某些元素，当时使用边遍历，边删除的方法，却报了以下异常：

ConcurrentModificationException

为了以后不忘记，使用烂笔头把它记录如下：

错误代码的写法，也就是报出上面异常的写法：

Set<CheckWork> set = this.getUserDao().getAll(qf).get(0).getActionCheckWorks();
for(CheckWork checkWork : set){
    if(checkWork.getState()==1){
        set.remove(checkWork);
    }
}

 注意：使用上面的写法就会报上面的ConcurrenModificationException异常，原因是，集合不可以一边遍历一边删除。

正确的写法如下：

1. 遍历删除List

List<CheckWork> list = this.getUserDao().getAll();
Iterator<CheckWork> chk_it = list.iterator();
while(chk_it.hasNext()){
    CheckWork checkWork = chk_it.next();
    if(checkWork.getPlanState()==1){
        chk_it.remove();
    }
}

 2. 遍历删除Set

Set<CheckWork> set =  this.getUserDao().getAll().get(0).getActionCheckWorks();
		Iterator<CheckWork> it = set.iterator();
		while(it.hasNext()){
			CheckWork checkWork = it.next();
			if(checkWork.getState()==1){
				it.remove();
			}
		}

 

评论

5 楼 www314599782 2015-10-09  

CopyOnWriteArraySet
CopyOnWriteArrayList 可以

HashSet  用Iterator 也一样不行 估计楼主 集合中只有一个值  ，我多加了几个就不行了

4 楼 www314599782 2015-10-09  

你这种方法还是一样的不行

3 楼 dsxwjhf 2015-04-20  

blackstreet 写道

CopyOnWriteArraySet
CopyOnWriteArrayList 用这个就行了，搞这么烦

CopyOnWriteArrayList 里面有个轻量级锁，性能会差一些

2 楼 YorkTsai 2014-02-13  

blackstreet 写道

CopyOnWriteArraySet
CopyOnWriteArrayList 用这个就行了，搞这么烦

我更喜欢朴素的解决办法

1 楼 blackstreet 2013-06-18  

CopyOnWriteArraySet
CopyOnWriteArrayList 用这个就行了，搞这么烦