解决在for循环中remove list报错越界的问题
trebleZ 人气:0最近在搞一个购物车的功能，里面有一个批量删除的操作，采用的是ExpandableListView以及BaseExpandableListAdapter。视乎跟本篇无关紧要，主要是为了记录一个java基础。迭代器iterator的使用
一、错误代码（主要就是购物车的批量删除）
/**
* 删除选中的
*/
public void delSelect() {
int groupSize;
if (mGropBeens != null) {
groupSize = mGropBeens.size();
} else {
return;
}
for (int i = 0; i < groupSize; i++) {
int childSize = mGropBeens.get(i).getChildBean().size();
for (int j = 0; j < childSize; j++) {
if (mGropBeens.get(i).getChildBean().get(j).isChecked()) {
DataSupport.deleteAll(ShopcartBean.class, "product_id=?", mGropBeens.get(i).getChildBean().get(j).getProduct_id());
mGropBeens.get(i).getChildBean().remove(j);
}
}
}
notifyDataSetChanged();
}
分析一、其实就是一个循环遍历list进行remove的操作，后来我分析了一下，错误的很明显，就是你remove了这个list以后，list的长度就改变了，然后你继续遍历，就会报错。感觉躲不了啊。错误有了，我觉得无法下手，后面既然remove不了，我就先删除本地数据库的方式，然后遍历对data进行赋值。。。躲一下
/**
* 删除选中的
*/
public void delSelect() {
int groupSize;
if (mGropBeens != null) {
groupSize = mGropBeens.size();
} else {
return;
}
for (int i = 0; i < groupSize; i++) {
int childSize = mGropBeens.get(i).getChildBean().size();
for (int j = 0; j < childSize; j++) {
if (mGropBeens.get(i).getChildBean().get(j).isChecked()) {
DataSupport.deleteAll(ShopcartBean.class, "product_id=?", mGropBeens.get(i).getChildBean().get(j).getProduct_id());
// mGropBeens.get(i).getChildBean().remove(j);
}
}
}
//刷新数据源
for (int i = 0; i < mGropBeens.size(); i++) {
mGropBeens.get(i).getChildBean().clear();
List<ShopcartBean> shopcartBeanlists = DataSupport.where("top_category_id=?", mGropBeens.get(i).getGroupId()).find(ShopcartBean.class);
mGropBeens.get(i).setChildBean(shopcartBeanlists);
}
notifyDataSetChanged();
}
分析二、写了这样以后还是感觉很不爽啊。明明我都循环了一遍知道要删除这个对象了，还要等遍历完，仅仅改变它的适配器的data。感觉不爽，随意的百度了下，发现有专门的解决方案，就是迭代器iterator
二、争取的打开方式
/**
* 删除选中的
*/
public void delSelect() {
int groupSize;
if (mGropBeens != null) {
groupSize = mGropBeens.size();
} else {
return;
}
for (int i = 0; i < groupSize; i++) {
Iterator<ShopcartBean> iterator = mGropBeens.get(i).getChildBean().iterator();
while (iterator.hasNext()) {
ShopcartBean shopcartBean = iterator.next();
if (shopcartBean.isChecked()) {
DataSupport.deleteAll(ShopcartBean.class, "product_id=?", shopcartBean.getProduct_id());
iterator.remove();
}
}
}
notifyDataSetChanged();
}
分析三、发现这个玩意感觉还是很惊喜的，同时也感叹自己基础薄弱了。
一般循环for和foreach都需要先知道集合的类型，甚至是集合内元素的类型，即需要访问内部的成员；iterator是一个接口类型，他不关心集合或者数组的类型，而且他还能随时修改和删除集合的元素。他不包含任何有关他所遍历的序列的类型信息，能够将遍历序列的操作与序列底层的 结构分离。
基本使用模式就是：
List<object> arr=xxx;//把你的list赋值过来
Iterator it = arr.iterator();
while(it.hasNext()){
object o =it.next();//当前遍历对象
iterator.remove();//删除或修改等等
}
三、ok,先做记录，后面继续深入。have a nice day.这就是在一个list中删除多个元素的正确解法。
补充知识：详解ArrayList在遍历时remove元素所发生的并发修改异常的原因及解决方法
本文将以“在遍历中删除”为着手点，在其基础上进行源码分析及相关问题解决。modCount的含义、迭代器所包含的方法、为什么会发生并发修改异常都将会在这篇文章中进行说明。
引入
这是一个并发修改异常的示例，它使用了迭代器iterator来获取元素，同时使用ArrayList自身的remove方法移除元素（使用增强for循环去遍历获取元素亦会如此，增强for循环底层用的也是迭代器，enhanced for loop is nothing but a syntactic sugar over Iterator in Java）
public static void main(String[] args) {
//请动手实践运行一下
List<String> list = new ArrayList<String>();
list.add("a");
list.add("b");
list.add("c");
list.add("d");
list.add("e");
Iterator<String> iterator = list.iterator();
while (iterator.hasNext()) {
String str = iterator.next();
if (str.equals("a")) {
list.remove(str);
} else {
System.out.println(str);
}
}
}
原因分析
ArrayList内部实现了迭代器Itr，如图所示
通过迭代器获取元素时（iterator.next()）会进行checkForComodification，源码如下
public E next() {
checkForComodification();/***看这行***/
int i = cursor;
if (i >= size)
throw new NoSuchElementException();
Object[] elementData = ArrayList.this.elementData;
if (i >= elementData.length)
throw new ConcurrentModificationException();
cursor = i + 1;//cursor向后挪一位
return (E) elementData[lastRet = i];//lastRet为当前取出的元素所在索引，后面会用到
}
final void checkForComodification() {/***再看这里***/
if (modCount != expectedModCount)
throw new ConcurrentModificationException();
}
modCount即此列表已被结构修改的次数。 结构修改是改变列表大小的那些修改（如增删，注意列表大小是size而不是capacity），或以其他方式扰乱它，使得正在进行的迭代可能产生不正确的结果的那些修改。
而expectedModCount会在迭代器被创建出来时初始化为modCount，源码如下
private class Itr implements Iterator<E> {
int cursor; // index of next element to return
int lastRet = -1; // index of last element returned; -1 if no such
int expectedModCount = modCount;
//instance methods...
}
是不是发现什么端倪了呢？当调用remove时（进而调用fastRemove）即被视为结构修改，因此modCount的值是会发生变化的，这样当程序再次通过iterator.next()获取元素时，通过checkForComodification方法发现modCount变化了，而expectedModCount 依然是初始化时的值，因此抛出ConcurrentModificationException。
让我们来确认一下我们的想法，remove方法及fastRemove方法的源码如下
public boolean remove(Object o) {
if (o == null) {
for (int index = 0; index < size; index++)
if (elementData[index] == null) {
fastRemove(index);/***看这行调用了fastRemove***/
return true;
}
} else {
for (int index = 0; index < size; index++)
if (o.equals(elementData[index])) {
fastRemove(index);
return true;
}
}
return false;
}
private void fastRemove(int index) {
modCount++;/***再看这行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
}
这样我们就对ArrayList在遍历时remove元素所发生的并发修改异常有了一个明确的了解。
迭代器Itr的补充说明：size是数组大小，cursor是下一元素的索引（虽然是下一元素的索引，但数组开始索引是从0开始的，所以cursor默认初始化为0）数组的最大索引一定是小于size的（size-1）索引=size还要取元素的话将会越界。
在说明“如何在不发生异常的情况下删除数据”前，先说一下根据上述示例可能会产生的其他问题（如不感兴趣也可跳过0.0）。
删除不规范时所产生的其他问题
问题1.虽然remove的不规范，但是程序依然能够运行，虽然不符合预期，但是没有发生并发修改异常
如果我们删除的不是a，而是d的话（最大为e），将会输出a b c而不会发生并发修改异常，代码如下
while (iterator.hasNext()) {
String str = (String) iterator.next();
if (str.equals("d")) {//将原先的a改为d
list.remove(str);
} else {
System.out.println(str);
}
}
原因分析：因为删除d时cursor由3变为4（从0起算），size由5变为4。因此hasNext返回true，并且循环结束，因此不会输出e（循环结束也意味着不会通过next进行checkForComodification，所以不会引发异常）
问题2.看似不会发生并发修改异常，可实际却发生了0.0
如果我们将要删除的元素改为e，那么当删除e时cursor由4变为5，size由5变为4，5明明大于4了应该不会有下一元素了，不会进入循环通过next取元素了，可当这么想着的时候，异常却发生了，代码如下：
while (iterator.hasNext()) {
String str = (String) iterator.next();
if (str.equals("e")) {
list.remove(str);
} else {
System.out.println(str);
}
}
原因分析：这是由于iterator.hasNext()的原理导致，点击hasNext()查看源码可发现，hasNext并不是由cursor < size来实现的而是通过cursor != size来实现的，这样程序将再次进入循环取元素进而发生并发修改异常
public boolean hasNext() {
return cursor != size;
}
如何在不报错的情况下将元素删除？
1.通过iterator获取元素的同时使用iterator的remove方法移除元素，代码如下
while (iterator.hasNext()) {
String str = (String) iterator.next();
if (str.equals("a")) {
iterator.remove();
} else {
System.out.println(str);
}
}
通过Itr的remove源码可以发现（如下），它在每次删除的同时还会更新expectedModCount为当前自增后的modCount，使得下次通过iterator.next()取元素时经得住checkForComodification校验（试想一下如果没有checkForComodification的话，程序将继续循环下去，cursor本指向的是当前元素索引的下一位，但remove后数据将整体向前窜一位，从而导致cursor指向的索引位置对应的数据发生了偏差，上述问题2的情况时若没有进行checkForComodification则还会发生NoSuchElementException异常，详见上述next源码）。
lastRet的值为最新一次通过next获取元素时，那个元素所对应的索引，这里通过将cursor = lastRet，从而把cursor的索引向前移动了一位，继而避免了取数据时的偏差（cursor 与 lastRet的关系详见上面的next源码）
在这里lastRet 会归为-1（它所对应的元素已经被删除了），这也是为什么不能连续调用两次迭代器的remove方法的原因，若执意如此，该方法将会抛出IllegalStateException的异常
public void remove() {
if (lastRet < 0)
throw new IllegalStateException();
checkForComodification();
try {
ArrayList.this.remove(lastRet);
cursor = lastRet;
lastRet = -1;
expectedModCount = modCount;
} catch (IndexOutOfBoundsException ex) {
throw new ConcurrentModificationException();
}
}
2.通过list自身的get方法获取元素的同时通过自身的remove方法移除元素，代码如下
for (int i = 0;i<list.size();i++){
String s = list.get(i);
if ("a".equals(s)){
list.remove(i);//进而调用fastRemove
i--;//相当于cursor = lastRet;将返回的下一元素的索引 = 返回的最新元素的索引（当前元素索引）
}else {
System.out.println(s);
}
}
该种情况下不会将expectedModCount修正为最新的modCount，同时也不会进行checkForComodification的检查，若此时删除并不修正当前索引的话，将会造成上述的数据偏差（遍历条件中的list.size()保存为固定值或连续调用list.remove(i)次数过多还可以发生索引越界异常0.0）
注意，不能保证迭代器的快速失败行为，因为通常来说，在存在不同步的并发修改的情况下，不可能做出任何严格的保证。快速失败的迭代器会尽最大努力抛出ConcurrentModificationException。因此，编写依赖于此异常的程序的正确性是错误的：迭代器的快速失败行为应仅用于检测错误。
add请参照remove的方式去查阅ArrayList中的内部类ListItr
以上这篇解决在for循环中remove list报错越界的问题就是小编分享给大家的全部内容了，希望能给大家一个参考，也希望大家多多支持。
加载全部内容