我有列表，每一行都有复选框，我尝试获取所有复选框位置，然后循环遍历它并使用 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创建并将结果分配给某些东西，或者将其传递到某个地方，这样它就不仅仅是一个无用的行。方法应该有一个单一的目的，而不是多个，尤其是不矛盾的目的。
清理完代码后，您会发现诊断任何错误并为其寻求帮助变得更加容易。如果您确实寻求帮助，请提供一个完整的示例来说明您的问题。不要隐藏所有相关信息。
并且不要使用原始类型!
关于java - recyclerview 中的 list.remove() 错误，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/43559922/