为何不推荐在 for 循环中 remove 元素？

时间 2019-11-07

标签为何推荐循环 remove 元素繁體版

原文原文链接

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

user.forEach(u -> {
                ageList.forEach(a -> {
                    if (u.getId().equals(a)) {
                        user.remove(u);
                    }
                });
            });
        }
复制代码

改进后是这样的。ui

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

Java 中一般有三种循环。spa

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异常。code

因此改为用 Iterator.remove() 就行了。对象