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

来源：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开发手册（嵩山版）》最新发布，速速下载！

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