Question

我有列表，每行都有复选框，我试着获取所有复选框位置然后循环它并用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)

Answer 1

您在迭代条目集时使用dish.remove(getItem(key2))从条目集中删除了一个项目，我本来希望抛出ConcurrentModificationException，但显然您有一些List没有告诉我们getItem可能也使用过，你也没有告诉我们，所以也许这就是为什么IndexOutOfBoundsException。

另外，为什么要在DishToDel上调用静态方法？这似乎应该是一个实例方法的调用。

通常你应该能够在一个基于迭代器的循环中循环和删除，但是你有太多复杂的事情要知道你正在尝试做什么。

分开创建任何

new makeDialog().makeDialog(mContext,mContext.getResources().getString(R.string.share_no_data));

尝试从方法removeAtAll创建并将结果分配给某个东西，或者将其传递到某个地方，这样它就不仅仅是一条无用的行。方法应该有一个目的，而不是多重目的，特别是不矛盾的目的。

一旦清理了代码，您就会发现诊断任何错误并为它们寻求帮助会更容易。如果您确实寻求帮助，请提供一个演示您问题的完整示例。不要压制所有相关信息。

并且不要使用原始类型！

recyclerview中的list.remove（）错误

1 个答案: