Java Lambda表达式forEach遍历列表删元素无法跳过已删项如何解决

问题根因 #

你遇到的现象是Java集合forEach方法的设计特性决定的：

• 你使用的CopyOnWriteArrayList的迭代器是快照迭代器，调用forEach时会先复制当前列表的元素数组生成快照，后续对原列表的所有修改都不会影响这个快照，所以会固定遍历原始列表的所有元素。
• 哪怕你使用普通的ArrayList，在forEach遍历过程中修改列表，还会直接抛出ConcurrentModificationException异常，根本无法运行。

可行解决方案 #

方案1：最小改动，遍历前校验元素是否存在 #

只需要在lambda开头新增判断，当前元素已经被移除就直接跳过处理，不需要修改其他逻辑：

stringsToRemove.forEach(device -> {
    // 元素已被移除，直接跳过
    if (!stringsToRemove.contains(device)) {
        return;
    }
    // 原业务逻辑，直接用removeAll替代内层forEach更简洁
    stringsToRemove.removeAll(deleteDeviceAndReferences());
    System.out.println("Processing"+stringsToRemove);
});

方案2：使用显式迭代器遍历（兼容普通ArrayList） #

如果后续你可能替换为非CopyOnWrite的列表实现，推荐用迭代器写法避免并发修改异常：

Iterator<String> iterator = stringsToRemove.iterator();
while (iterator.hasNext()) {
    String device = iterator.next();
    // 元素已被移除，跳过
    if (!stringsToRemove.contains(device)) {
        continue;
    }
    stringsToRemove.removeAll(deleteDeviceAndReferences());
    System.out.println("Processing"+stringsToRemove);
}

方案3：改用while循环（最符合你业务逻辑的写法） #

你的需求是只处理当前列表中剩余的元素，while循环完全规避了快照迭代的问题，逻辑更清晰也不会漏处理元素：

// 只要列表不为空就持续处理
while (!stringsToRemove.isEmpty()) {
    // 按需取当前要处理的元素，这里示例取第一个
    String currentDevice = stringsToRemove.get(0);
    stringsToRemove.removeAll(deleteDeviceAndReferences());
    System.out.println("Processing"+stringsToRemove);
}

补充说明 #

你目前用的传统for循环存在潜在bug：每次循环i固定+1，但removeAll会让列表长度动态变小，会出现部分元素没被遍历到就退出循环的情况，推荐优先改用上面的while循环方案。

内容的提问来源于stack exchange，提问作者RagaSGNur