当 RecyclerView 让 Google 工程师背了锅

nanchen2251
12,065 阅读4分钟
原文链接： mp.weixin.qq.com

在座（站）的各位，一定在 RecyclerView 的使用中遇到过这样的 bug 吧，且看日志。

java.lang.IndexOutOfBoundsException: Inconsistency detected. Invalid view
holder adapter positionViewHolder{2064e5c6 position=2 id=-1, oldPos=2, 
pLpos:-1 scrap [attachedScrap] tmpDetached no parent} at 
android.support.v7.widget.RecyclerView$Recycler.validateViewHolderForOffsetPosition(RecyclerView.java:4505) at 
android.support.v7.widget.RecyclerView$Recycler.getViewForPosition(Recycl
erView.java:4636) at 
android.support.v7.widget.RecyclerView$Recycler.getViewForPosition(Recycl
erView.java:4617) at 
android.support.v7.widget.LinearLayoutManager$LayoutState.next(LinearLayo
utManager.java:1994) at 
android.support.v7.widget.LinearLayoutManager.layoutChunk(LinearLayoutMa
nager.java:1390) at 
android.support.v7.widget.LinearLayoutManager.fill(LinearLayoutManager.java
:1353) at 
android.support.v7.widget.LinearLayoutManager.onLayoutChildren(LinearLayo
utManager.java:574) at 
android.support.v7.widget.RecyclerView.dispatchLayoutStep1(RecyclerView.ja
va:2979) at 
android.support.v7.widget.RecyclerView.dispatchLayout(RecyclerView.java:2904) at 
android.support.v7.widget.RecyclerView.onLayout(RecyclerView.java:3283) at 
android.view.View.layout(View.java:15912) at 
android.view.ViewGroup.layout(ViewGroup.java:5108) at 
android.widget.LinearLayout.setChildFrame(LinearLayout.java:1959) at 
android.widget.LinearLayout.layoutHorizontal(LinearLayout.java:1948) at 
android.widget.LinearLayout.onLayout(LinearLayout.java:1724) at 
android.view.View.layout(View.java:15912) at 
android.view.ViewGroup.layout(ViewGroup.java:5108) at 
android.widget.LinearLayout.setChildFrame(LinearLayout.java:1959) at 
android.widget.LinearLayout.layoutVertical(LinearLayout.java:1813) at 
android.widget.LinearLayout.onLayout(LinearLayout.java:1722) at 
android.view.View.layout(View.java:15912) at 
android.view.ViewGroup.layout(ViewGroup.java:5108) at 
android.widget.FrameLayout.layoutChildren(FrameLayout.java:633) at 
android.widget.FrameLayout.onLayout(FrameLayout.java:568) at 
android.view.View.layout(View.java:15912) at 
android.view.ViewGroup.layout(ViewGroup.java:5108) at 
android.widget.LinearLayout.setChildFrame(LinearLayout.java:1959) at 
android.widget.LinearLayout.layoutVertical(LinearLayout.java:1813) at 
android.widget.LinearLayout.onLayout(LinearLayout.java:1722) at 
android.view.View.layout(View.java:15912) at 
android.view.ViewGroup.layout(ViewGroup.java:5108) at 
android.widget.FrameLayout.layoutChildren(FrameLayout.java:633) at 
android.widget.FrameLayout.onLayout(FrameLayout.java:568) at 
android.view.View.layout(View.java:15912) at 
android.view.ViewGroup.layout(ViewGroup.java:5108) at 
android.widget.FrameLayout.layoutChildren(FrameLayout.java:633) at 
android.widget.FrameLayout.onLayout(FrameLayout.java:568) at 
android.view.View.layout(View.java:15912) at 
android.view.ViewGroup.layout(ViewGroup.java:5108) at 
android.widget.LinearLayout.setChildFrame(LinearLayout.java:1959) at 
android.widget.LinearLayout.layoutVertical(LinearLayout.java:1813) at 
android.widget.LinearLayout.onLayout(LinearLayout.java:1722) at 
android.view.View.layout(View.java:15912) at 
android.view.ViewGroup.layout(ViewGroup.java:5108) at 
android.widget.FrameLayout.layoutChildren(FrameLayout.java:633) at 
android.widget.FrameLayout.onLayout(FrameLayout.java:568) at 
android.view.View.layout(View.java:15912) at 
android.view.ViewGroup.layout(ViewGroup.java:5108) at 
android.widget.FrameLayout.layoutChildren(FrameLayout.java:633) at 
android.widget.FrameLayout.onLayout(FrameLayout.java:568) at 
android.view.View.layout(View.java:15912) at 
android.view.ViewGroup.layout(ViewGroup.java:5108) at 
android.widget.FrameLayout.layoutChildren(FrameLayout.java:633) at 
android.widget.FrameLayout.onLayout(FrameLayout.java:568) at 
android.view.View.layout(View.java:15912) at 
android.view.ViewGroup.layout(ViewGroup.java:5108) at 
android.widget.LinearLayout.setChildFrame(LinearLayout.java:1959) at 
android.widget.LinearLayout.layoutVertical(LinearLayout.java:1813) at 
android.widget.Line

数组越界？NoNoNo。整篇日志，均没有涉及到我们的代码，一时让人也摸不着头脑，经过一番查阅资料才发现，论坛上早已炸锅。该 bug 竟然是 Google 程序员的锅？

从 StackOverFlow 相关资料得知，该 Bug 主要是由于 Adapter 绑定的集合和 RecyclerView 的数据不一致而导致。

直接说解决思路

找到了症结，问题解决解决起来也是非常简单，且听我细细到来。

直接采用 notifyDataSetChanged 同步外部数据集和内部数据集。[ 不是很推荐 ]

use notifyDataSetChanged() will avoid this crash, but it will kill Animation and Performance.

该方法比较简单，但失去了动画效果，而且更新数据的性能较低。另外，如果对外部数据集做了两次以上的操作，却只调用 `notifyDataSetChanged` 同步一次，也极有可能报上述错误。

直接 Try 住这个 Bug【最简单粗暴】

直接复写 LinearLauoutManager。

package com.zxedu.ischool.common;import android.content.Context;import android.support.v7.widget.LinearLayoutManager;import android.support.v7.widget.RecyclerView;import android.util.AttributeSet;/**
* Author: nanchen
* Email: liushilin520@foxmail.com
* Date: 2017-05-19  15:56
*/public class WrapContentLinearLayoutManager extends LinearLayoutManager {  
    public WrapContentLinearLayoutManager(Context context) {      
        super(context);
    }   

    public WrapContentLinearLayoutManager(Context context, int orientation, boolean reverseLayout) {   
        super(context, orientation, reverseLayout);
    }   

    public WrapContentLinearLayoutManager(Context context, AttributeSet attrs, int defStyleAttr, int defStyleRes) {      
        super(context, attrs, defStyleAttr, defStyleRes);
    }  

    @Override
    public void onLayoutChildren(RecyclerView.Recycler recycler, RecyclerView.State state) {      
        try {           
            super.onLayoutChildren(recycler, state);
        } catch (IndexOutOfBoundsException e) {
           e.printStackTrace();
        }
    }
}

对，没错，直接更换 LayoutManaer 就 OK 了

//        mRecyclerView.setLayoutManager(new LinearLayoutManager(this));
         // 解决RecyclerView可能出现的holder数组越界Bug
        mRecyclerView.setLayoutManager(new WrapContentLinearLayoutManager(this, LinearLayoutManager.VERTICAL, false));

顺藤摸瓜，做最优雅的程序员

既然我们知道了崩溃发送原因，我们可以直接顺藤摸瓜。在进行数据移除和数据增加的时候，保证 RecyclerView 的 Adapter 中的数据集移除和添加等操作后的数据集保持一致。

RecyclerView 内部的数据集这里我们不妨叫「内部数据集」，而我们传递给 Adapter 的，姑且叫它 「外部数据集」。我们通过「外部数据集」更新「内部数据集」，一般会使用到以下的方法。

  • notifyItemRangeRemoved()

  • notifyItemRangeInserted()

  • notifyItemRangeChanged()

  • notifyDataSetChanged()

我们一般不采用 `notifyDataSetChanged()` 方法，因为它不但没有默认的动画效果，而且在更新数据的效率上会大打折扣，官方并不推荐。

Don’t call notifyDataSetChanged if you don’t have to.

我们一般在给 Adapter 设置数据时会这样做。

public void setData(List<Data> data){
    if(data != null){
        mData.clear();
        mData.addAll(data);
        notifyItemRangeChanged(0,mData.size());
    }
}

实际上这段代码，并无毛病，但假设 data 数据为接口返回的数据，刷新后返回的数据和现在 Adapter 中的数据数目不一致的时候，极有可能出现开题的 bug，直接崩溃。所以我们不妨可以这样修复。

public void setData(List<Data> data){    if(data != null){        int size = mData.size();
        mData.clear();
        notifyItemRangeRemoved(0,size);
        mData.addAll(data);
        notifyItemRangeInserted(0,mData.size());
    }
}

自此完毕。各位看官不妨一试。

我是南尘，只做比心的公众号，欢迎关注我。

推荐阅读：

给 Android 开发者的一点福利：免费模拟面试

「提离职」算正确的加薪姿势么？

我为什么想离职？

发年终奖的正确姿势

欢迎关注南尘的公众号：nanchen做不完的开源，写不完的矫情，只做比心的公众号，如果你喜欢，你可以选择分享给大家。如果你有好的文章，欢迎投稿，让我分享给大家。                               长按上方二维码关注                        做不完的开源，写不完的矫情                       一起来看 nanchen 的成长笔记

avatar
文章
阅读
粉丝
相关推荐
博客园又崩了，这个锅要不要阿里云背？
44k阅读
 · 
185点赞
2023年🧑‍💻工程师的 Mac 工具箱
68k阅读
 · 
832点赞
用一个RecyclerView实现抖音二级评论
4.6k阅读
 · 
114点赞
写给初中级 Vue 工程师的高级进阶指南
63k阅读
 · 
1.4k点赞
RecyclerView优化实战指南
2.9k阅读
 · 
30点赞
很佩服的一个Google大佬，离职了。。
71k阅读
 · 
586点赞
“当高启强遇到陈书婷”与TCP协议
308k阅读
 · 
207点赞
浅谈RecyclerView的性能优化
8.8k阅读
 · 
102点赞
API 请求慢？这次锅真不在后端
69k阅读
 · 
730点赞
作为一名前端工程师，我浪费了时间学习了这些技术
221k阅读
 · 
843点赞

友情链接：