阿里巴巴Java规约学习-foreach
阿里巴巴Java规约学习-foreach
7. 【强制】不要在 foreach 循环里进行元素的 remove / add 操作。 remove 元素请使用 Iterator
方式，如果并发操作，需要对 Iterator 对象加锁。
反例：
List<String> a = new ArrayList<String>();
a.add("1");
a.add("2");
for (String temp : a) {
if("1".equals(temp)){
a.remove(temp);
}
}
说明：以上代码的执行结果肯定会出乎大家的意料，那么试一下把“1”换成“2”，会是同样的
结果吗？
正例：
Iterator<String> it = a.iterator();
while(it.hasNext()){
String temp = it.next();
if(删除元素的条件){
it.remove();
}
}
先直接看测试结果:
public class Test {
public static void main(String[] args) {
List<String> a = new ArrayList<String>();
a.add("1");
a.add("2");
// for (String temp : a) {
// if ("1".equals(temp)) {
// a.remove(temp);//正常
// }
// }
// for (String temp : a) {// java.util.ConcurrentModificationException at java.util.ArrayList$Itr.checkForComodification(ArrayList.java:859) at java.util.ArrayList$Itr.next(ArrayList.java:831)
// if ("2".equals(temp)) {
// a.remove(temp);
// }
// }
Iterator<String> it = a.iterator();
while (it.hasNext()) {
String temp = it.next();
if ("1".equals(temp)) {//不论是1,2都正常
it.remove();
}
}
System.out.println(a.toString());
}
}
为什么用foreach的时候，1没问题，2出错了呢？
先从报错看异常是怎么出来的。
由报错可知，是next方法中调用checkForComodification出错。
所以是modeCount !=expectedModCount导致的。
foreach是通过Iterable接口在序列中进行移动。
foreach处打断点，进入可以知道：
赋值给temp的时候先new Itr()(仅第一次)，Itr对象再hasNext()判断，接着再调用next()赋值。
expectedModCount是初始化(此时生成一个Iterator对象)的时候被赋值的。而modCount又是ArrayList的变量。
每次调用list的add/remove的时候，modCount++，而这时它并没有传递给expectedModCount。
当第一次remove掉"2"后，modCount改变 ，for(String temp:a)的时候，会调用ArrayList$Itr.next()方法，next方法会检查modCount是否等于expectedModCount，所以用foreach的时候remove 2出错了。
foreach的时候remove 1没出错呢？
稍微改下代码:
List<String> a = new ArrayList<String>();
a.add("1");
a.add("2");
int n=0;
for (String temp : a) {//正常,但是n为1，说明只有一次
n++;//
System.out.println(n);//只输出1次:1
if ("1".equals(temp)) {
a.remove(temp);
}
}
输出结果:
1
[2]
可见,foreach 只执行了一次循环。
是什么导致本该两次的循环只进入了一次？
之前说到foreach赋值的时候，会先先用hasNext判断。
cursor！=size才进入循环体。
这里的**cursor指的是当前元素的index**。循环体进入一次后，remove掉"1"，此时size由2变成了1，cursor也由于next后变为1。再次 for (String temp : a)的时候先调用hasNext因为其相等，所以不再进入循环体。
remove时–size:
next后,cursor+1:
hasNext比较
为什么直接用Iterator的remove是正常的？
a.iterator()直接return new Itr()，Itr 实现了Iterator。
我们看看Itr的remove方法：
这里明显看出当它remove后， 马上把modCount的值赋值到了expectedModCount上，所以不会出现list.remove那样报ConcurrentModificationException 。
** 当然如果并发操作，需要对 Iterator 对象加锁。**
感谢阿里巴巴的Java规约~
