我有列表,每行都有复选框,我尝试获取所有复选框位置,然后在其上循环并使用list.remove(indexOfpostion)删除每一行
但它抛出java.lang.IndexOutOfBoundsException
注意:我的列表名称是“ Dish”。
代码:
public void removeAtAll() {
Iterator it = selectedCheckBox.entrySet().iterator();
if(!it.hasNext()){
new makeDialog().makeDialog(mContext,mContext.getResources().getString(R.string.share_no_data));
}
else {
while (it.hasNext()) {
Map.Entry pair = (Map.Entry) it.next();
int key = (Integer) pair.getKey();
if ((Boolean) pair.getValue()) {
// it.remove();
DishToDel.add(dish.get(key).getDishID());
}
}
//
for (Object o : selectedCheckBox.entrySet()) {
Map.Entry pair2 = (Map.Entry) o;
int key2 = (Integer) pair2.getKey();
if ((Boolean) pair2.getValue()) {
dish.remove(getItem(key2)); // error here
notifyItemRemoved(key2);
notifyItemRangeChanged(key2, dish.size());
}
}
chooseAllradioButton.setSelected(false);
favhtask = new FavAsyncTaskDelete(mContext, DishToDel);
favhtask.execute();
}
// process message deleteing
}
错误日志
FATAL EXCEPTION: main
Process: com.TopLine.nokhbatalatbaq, PID: 32327
java.lang.IndexOutOfBoundsException: Index: 2, Size: 1
at java.util.ArrayList.get(ArrayList.java:411)
at com.TopLine.nokhbatalatbaq.adapter.FavAdapterDishes.removeAtAll(FavAdapterDishes.java:344)
at com.TopLine.nokhbatalatbaq.adapter.FavAdapterDishes$4.onClick(FavAdapterDishes.java:151)
at android.view.View.performClick(View.java:6205)
at android.view.View$PerformClick.run(View.java:23653)
at android.os.Handler.handleCallback(Handler.java:751)
at android.os.Handler.dispatchMessage(Handler.java:95)
at android.os.Looper.loop(Looper.java:154)
at android.app.ActivityThread.main(ActivityThread.java:6682)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1520)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1410)
最佳答案
在迭代条目集时,您已使用dish.remove(getItem(key2))
从条目集中删除了一个项目,而我本来希望该条目会抛出ConcurrentModificationException
,但是显然您有一些List
未能向我们展示可能被getItem
,您也无法向我们展示,所以也许这就是IndexOutOfBoundsException
的原因。
另外,为什么要在DishToDel
上调用静态方法?似乎应该对实例方法进行这种调用。
通常,您应该能够在单个基于迭代器的循环中进行循环和删除,但是您遇到了太多复杂的事情,无法确切地知道您要执行的操作。
单独创建
new makeDialog().makeDialog(mContext,mContext.getResources().getString(R.string.share_no_data));
正在尝试从方法
removeAtAll
创建并将结果分配给某些内容,或将其传递到某个地方,以便它不仅仅是一条无用的行。方法应该有一个单一目的,而不是多重目的,尤其是没有矛盾的目的。清理代码后,您会发现诊断任何错误并为他们寻求帮助变得容易得多。如果您确实寻求帮助,请给出一个完整的示例来演示您的问题。不要隐瞒所有相关信息。
并且不要使用原始类型!