还在 for 循环中 remove 元素？必须劝退……

Posted 2022-03-24 Java技术栈

来源：juejin.cn/post/6844903906449358856

业务中有需要过滤的需求，踩了 foreach 的坑。本来是这样写的：

user.forEach(u -> 
   ageList.forEach(a -> 
        if (u.getId().equals(a)) 
            user.remove(u);
            
        );
    );

改进后是这样的：

Iterator<SocNearbyRespDto> ui = user.iterator();
    while (ui.hasNext())
        SocNearbyRespDto u = ui.next();
        ageList.forEach(a->
            if (!u.getId().equals(a))
                ui.remove();
            
        );
    

Java 中通常有三种循环

for (int i = 0; i < list.size(); i++) 
    System.out.print(list.get(i) + ",");


Iterator iterator = list.iterator();
while (iterator.hasNext()) 
    System.out.print(iterator.next() + ",");


for (Integer i : list) 
    System.out.print(i + ",");

list.foreach 跟代码中第三种增强型 for 循环一样，反编译的内容是

    Integer i;
    for(Iterator iterator = list.iterator(); iterator.hasNext(); System.out.println(i))
        i = (Integer)iterator.next();        
    

这样在 for 循环中调用会出现 ConcurrentModificationException 异常。

Iterator是工作在一个独立的线程中，并且拥有一个 mutex 锁。Iterator被创建之后会建立一个指向原来对象的单链索引表，当原来的对象数量发生变化时，这个索引表的内容不会同步改变，所以当索引指针往后移动的时候就找不到要迭代的对象，所以按照 fail-fast 原则 Iterator 会马上抛出java.util.ConcurrentModificationException异常。

所以改成用 Iterator.remove()就好了。

近期热文推荐：

1.1,000+ 道 Java面试题及答案整理(2022最新版)

2.劲爆！Java 协程要来了。。。

3.Spring Boot 2.x 教程，太全了！

4.别再写满屏的爆爆爆炸类了，试试装饰器模式，这才是优雅的方式！！

5.《Java开发手册（嵩山版）》最新发布，速速下载！

觉得不错，别忘了随手点赞+转发哦！