删除List中元素这个场景很场景，很多人可能直接在循环中直接去删除元素，这样做对吗？我们就来聊聊。

for循环索引删除

删除长度为4的字符串元素。

    List<String> list = new ArrayList<String>();    list.add("AA");    list.add("BBB");    list.add("CCCC");    list.add("DDDD");    list.add("EEE");    for (int i = 0; i < list.size(); i++) {        if (list.get(i).length() == 4) {            list.remove(i);        }    }    System.out.println(list);}

实际上输出结果：

[AA, BBB, DDDD, EEE]

DDDD 竟然没有删掉！

原因是：删除某个元素后，list的大小size发生了变化，而list的索引也在变化，索引为i的元素删除后，后边元素的索引自动向前补位，即原来索引为i+1的元素，变为了索引为i的元素，但是下一次循环取的索引是i+1，此时你以为取到的是原来索引为i+1的元素，其实取到是原来索引为i+2的元素，所以会导致你在遍历的时候漏掉某些元素。

比如当你删除第1个元素后，继续根据索引访问第2个元素时，因为删除的关系后面的元素都往前移动了一位，所以实际访问的是第3个元素。不会报出异常，只会出现漏删的情况。

foreach循环删除元素

for (String s : list) {        if (s.length() == 4) {            list.remove(s);        }    }    System.out.println(list);

如果没有break，会报错：

java.util.ConcurrentModificationException at java.util.ArrayList$Itr.checkForComodification(ArrayList.java:911) at java.util.ArrayList$Itr.next(ArrayList.java:861) at com.demo.ApplicationTest.testDel(ApplicationTest.java:64) at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) at java.lang.reflect.Method.invoke(Method.java:498) at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50) at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12) at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47) at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)

报ConcurrentModificationException错误的原因：

看一下JDK源码中ArrayList的remove源码是怎么实现的：

public boolean remove(Object o) {        if (o == null) {            for (int index = 0; index < size; index++)                if (elementData[index] == null) {                    fastRemove(index);                    return true;                }        } else {            for (int index = 0; index < size; index++)                if (o.equals(elementData[index])) {                    fastRemove(index);                    return true;                }        }        return false;    }

一般情况下程序会最终调用fastRemove方法：

private void fastRemove(int index) {        modCount++;        int numMoved = size - index - 1;        if (numMoved > 0)            System.arraycopy(elementData, index+1, elementData, index,                             numMoved);        elementData[--size] = null; // clear to let GC do its work    }

在fastRemove方法中，可以看到第2行把modCount变量的值加一，但在ArrayList返回的迭代器会做迭代器内部的修改次数检查：

final void checkForComodification() {     if (modCount != expectedModCount)             throw new ConcurrentModificationException();     }

而foreach写法是对实际的Iterable、hasNext、next方法的简写，因为上面的remove(Object)方法修改了modCount的值，所以才会报出并发修改异常。

阿里开发手册也明确说明禁止使用foreach删除、增加List元素。

迭代器Iterator删除元素

    Iterator<String> iterator = list.iterator();    while(iterator.hasNext()){        if(iterator.next().length()==4){            iterator.remove();        }    }    System.out.println(list);

[AA, BBB, EEE]

这种方式可以正常的循环及删除。但要注意的是，使用iterator的remove方法，而不是List的remove方法,如果用list的remove方法同样会报上面提到的ConcurrentModificationException错误。

总结

无论什么场景，都不要对List使用for循环的同时，删除List集合元素，要使用迭代器删除元素。