RecyclerView IndexOutOfBoundsException

Posted 2023-02-14

【中文标题】RecyclerView IndexOutOfBoundsException

【英文标题】：

【发布时间】：2015-06-16 10:35:47

【问题描述】：

当我使用循环删除RecyclerView 中的某些项目时，为什么会执行异常？ 我在适配器中使用了Collentions.synchronizedMap，'deleteItem 方法'也使用了同步（片段中的方法）。

public void elementController(JsonObject jsonObject , String type)   
    if ( jsonObject == null || type == null )    
        return;
    

    int position =0 , resultPosition =0;
    if ( type.equals("update") || type.equals("delete"))    

        String id = jsonObject.get(ELEMENT_ID).getAsString();
        Map<String , Element> map = gridFragment.getMap();

        synchronized (map)   
            for (String s : map.keySet()) 
                if (s.equals(id)) 
                    resultPosition = position;
                 else 
                    position++;
                
            
        
    

    if(position-1 > gridFragment.getmAdapter().getData().size() || position <0)   
        return;
    
    switch (type)   
        case "add":
            if (gridFragment.addElement(MyJsonParser.ElementParse(jsonObject),0))
                LogUtils.logDebug(TAG,"add end");
            
            break;
        case "update":
            if(gridFragment.updateElement( updateParser(jsonObject),resultPosition))
                LogUtils.logDebug(TAG,"update end");
            
            break;
        case "delete":
            if(gridFragment.deleteElement(jsonObject.get(ELEMENT_ID).getAsString(),resultPosition))
                LogUtils.logDebug(TAG,"delete end");
            
            break;
    

---

public boolean deleteElement(final String id , final int position)
    new Thread(new Runnable() 
        @Override
        public void run() 
            getActivity().runOnUiThread(new Runnable()
                @Override
                public void run() 
                    synchronized (map) 
                        map.remove(id);
                        mAdapter.setData(map);
                        mAdapter.notifyItemRemoved(position);
                    
                
            );
        
    ).start();

    return true;
 

我的错误日志：

java.lang.IndexOutOfBoundsException: Inconsistency detected. Invalid item position 0(offset:0).state:4
        at android.support.v7.widget.RecyclerView$Recycler.getViewForPosition(RecyclerView.java:3382)
        at android.support.v7.widget.RecyclerView$Recycler.getViewForPosition(RecyclerView.java:3340)
        at android.support.v7.widget.LinearLayoutManager$LayoutState.next(LinearLayoutManager.java:1810)
        at android.support.v7.widget.GridLayoutManager.layoutChunk(GridLayoutManager.java:356)
        at android.support.v7.widget.LinearLayoutManager.fill(LinearLayoutManager.java:1269)
        at android.support.v7.widget.LinearLayoutManager.onLayoutChildren(LinearLayoutManager.java:523)
        at android.support.v7.widget.GridLayoutManager.onLayoutChildren(GridLayoutManager.java:151)
        at android.support.v7.widget.RecyclerView.dispatchLayout(RecyclerView.java:1942)
        at android.support.v7.widget.RecyclerView.resumeRequestLayout(RecyclerView.java:1171)
        at android.support.v7.widget.RecyclerView$1.run(RecyclerView.java:167)
        at android.view.Choreographer$CallbackRecord.run(Choreographer.java:761)
        at android.view.Choreographer.doCallbacks(Choreographer.java:574)
        at android.view.Choreographer.doFrame(Choreographer.java:543)
        at android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:747)
        at android.os.Handler.handleCallback(Handler.java:733)
        at android.os.Handler.dispatchMessage(Handler.java:95)
        at android.os.Looper.loop(Looper.java:212)
        at android.app.ActivityThread.main(ActivityThread.java:5137)
        at java.lang.reflect.Method.invokeNative(Native Method)
        at java.lang.reflect.Method.invoke(Method.java:515)
        at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:902)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:718)
        at dalvik.system.NativeStart.main(Native Method)

找不到设备

【问题讨论】：

第一个代码没有意义，你循环并且只从循环中获得一个位置。无中断条件

您在移除项目后设置 mAdapter 数据，然后尝试移除指定位置，在这种情况下该位置不存在。

服务调用的所有第一个代码（我使用带有服务器的套接字网络）。我从服务器获取 jsonarray 并通过 for 循环将 jsonobjects 发送到 jsonarray 中的第一个代码。所以这是第一个循环。我猜第二个 jsonobject 在结束第一个代码之前由 params 发送到第一个代码。

我添加了我所有的第一个方法。

【参考方案1】：

对于动画不消失，使用：

holder.deleteItem.setOnClickListener(new View.OnClickListener() 
            @Override
            public void onClick(View v) 
                itemList.remove(position);
                notifyItemRemoved(position);
                notifyItemRangeChanged(position, itemList.size());
            
        );

【讨论】：

notifyItemRangeChanged(position, itemList.size());这对我很有帮助。谢谢

谢谢。 IMO notifyItemRemoved 应该自动执行此操作，但是哦。

【参考方案2】：

尚未阅读您的代码，因为我不熟悉您使用的类，但我知道解决此问题的方法。当删除的行元素不是最后一个时会出现问题，因为dataList 索引（用于存储数据的ArrayList）与方法onBindViewHolder 中的参数final int position 之间的索引差异。让我以图形方式解释这个问题：

假设我们有一个三行的 RecyclerView，我们删除第三行，在dataList 索引 2，（注意删除的不是最后一个元素）删除后，dataList 最后一个元素的索引减少3 from 2 但是，final int position 仍将其位置检索为 3，这会导致问题。因此，删除后，您不能使用final int position 作为要从dataList 中删除的元素的索引。

要解决此问题，请引入 int shift=0 并在 while 循环中，try 删除 (position-shift) 处的第三项，如果失败（在我的情况下，它将）增加移位并尝试再次删除它直到没有异常发生。

 public void onBindViewHolder(RecyclerView.ViewHolder holder, final int position) 
    
    ...
    
        holder.removeButton.setOnClickListener(new View.OnClickListener() //button used to remove rows
                    @Override
                    public void onClick(View view) 
                        if (position == dataList.size() - 1)  // if last element is deleted, no need to shift
                            dataList.remove(position);
                            notifyItemRemoved(position);
                         else  // if the element deleted is not the last one
                            int shift=1; // not zero, shift=0 is the case where position == dataList.size() - 1, which is already checked above
                            while (true) 
                                try 
                                    dataList.remove(position-shift);
                                    notifyItemRemoved(position);
                                    break;
                                 catch (IndexOutOfBoundsException e)  // if fails, increment the shift and try again
                                    shift++;
                                
                            
                        
                    
                );
   ...
    

【讨论】：

运行良好一段时间，直到我得到一个异常循环。

【参考方案3】：

是的，这是因为并发访问。我遇到过。使用Iterator 或一个不错的技巧很容易修复：您开始从最后一个项目中删除。然而，Iterator 的 Z-A 订单更好。

最近我改进并创建了一个使用此解决方案的 FlexibleAdapter。 请同时查看描述和完整的工作示例：https://github.com/davideas/FlexibleAdapter

【参考方案4】：

它对我有用，像这样：

fun removeData(position: Int) 
    this.notifyItemRemoved(position)
    this.notifyItemRangeChanged(position, imgList.size)
    this.imgList.removeAt(position)

【参考方案5】：

 mAdapter.notifyItemRemoved(position); 

这只会通知正在监听布局变化的观察者。

请尝试调用：

mAdpater.nofityDataChanged()

【讨论】：

除非更改整个列表内容，否则不要调用 nofityDataChanged()，否则所有动画都会被杀死，因为 Item 被重新绑定。请查看我的答案和我的所有 notify() 方法的 FlexibleAdapter

【参考方案6】：

可以这样解决：

dataList.remove(item)
notifyItemRemoved(adapterPosition)

【讨论】：

欢迎来到 SO。尽管我们感谢您的回答，但如果它在其他答案之上提供额外的价值会更好。在这种情况下，您的答案不会提供额外的价值，因为 Nastromo 已经发布了该解决方案。如果之前的答案对您有帮助，那么您应该在获得足够声誉后投票。

【参考方案7】：

其实，比你想象的要简单，把map.notifyItemRemoved(position)从这里移开就行了

...
public void run() 
                    synchronized (map) 
                        map.remove(id);
                        mAdapter.setData(map);
                        mAdapter.notifyItemRemoved(position);
                    
                

到这里

public void run() 
                    synchronized (map) 
                        mAdapter.notifyItemRemoved(position);
                        map.remove(id);
                        mAdapter.setData(map);