RecyclerView不停调用onCreateViewHolder问题

  最近在项目中看到同事使用RecyclerView时搞得很复杂,在View创建后再给RecyclerView的LayoutParams高度赋值,还需要在View创建好后通过Post延迟执行赋值,长这样:

    protected void initEventAndData() {
        //这里做一下Pos,这样会在保证在View 测量事件之后,可以得到View的高度
        UiUtil.post(new Runnable() {
            @Override
            public void run() {
                try {
                    initLogView();
                }catch (Exception e){

                }
            }
        },500);
    }
    
    private void initLogView() {
        //改变 mRecycleViewLog的高度
        ViewGroup.LayoutParams layoutParams = mRecycleViewLog.getLayoutParams();
        layoutParams.height = (int) (mRecycleViewLog.getY() + mRecycleViewLog.getHeight() - ll_log_start.getY() - ll_log_start.getHeight());
        mRecycleViewLog.setLayoutParams(layoutParams);
        
        logLayoutManager = new LinearLayoutManager(mActivity, LinearLayoutManager.VERTICAL, false);
        faultStateAdapter = new FaultStateAdapter(realErrorStateBeans);
        mRecycleViewLog.setLayoutManager(logLayoutManager);
        mRecycleViewLog.setAdapter(faultStateAdapter);
    }

问其为什么,被告知如果不设置高度RecyclerView就不会复用ViewHolder。这,,,不科学,理论上match_parent不就好了吗。

  于是在Adapter的onCreateViewHolder中打印日志,因为本身列表1秒添加两个数据并notify一次,因此看到日志中设置高度时1s创建一个ViewHolder,而不设置高度时1s创建2个以上。为了排除notify的影响把每秒添加数据和notify去掉,换成初始化时添加固定的数量50个,页面只能显示9个。这时查看日志就会看到,添加高度设置的在初始时执行了10次onCreateViewHolder,未设置高度则会执行50次整,看来不设置高度是会为每个item都创建一个ViewHolder。

接下来就是看为什么会创建这么多ViewHolder,打开RececlerView源代码,发现Adapter的createViewHolder只有tryGetViewHolderForPositionByDeadline(int position, boolean dryRun, long deadlineNs)一个地方调用,这里其实就是复用缓存的大本营,代码较长,这里就不贴出来了,可以按照X级缓存来理解。根据对比断点调试,发现设置高度的RecyclerView调用position只是0-9,而不设置高度的则会执行0-49,基本上可以判定不设置高度的RecyclerView本身match_parent高度计算出问题了,它可能认为50个都能显示出来,继续往上跟踪, tryGetViewHolderForPositionByDeadline是被getViewForPosition调用。

        @NonNull
        public View getViewForPosition(int position) {
            return getViewForPosition(position, false);
        }

        View getViewForPosition(int position, boolean dryRun) {
            return tryGetViewHolderForPositionByDeadline(position, dryRun, FOREVER_NS).itemView;
        }

而getViewForPosition则是被LinearLayoutManager的next调用(我这里使用的RececlerView是线性布局)。

        View next(RecyclerView.Recycler recycler) {
            if (mScrapList != null) {
                return nextViewFromScrapList();
            }
            final View view = recycler.getViewForPosition(mCurrentPosition);
            mCurrentPosition += mItemDirection;
            return view;
        }

next是被layoutChunk调用。

    void layoutChunk(RecyclerView.Recycler recycler, RecyclerView.State state,
            LayoutState layoutState, LayoutChunkResult result) {
        View view = layoutState.next(recycler);
        if (view == null) {
            if (DEBUG && layoutState.mScrapList == null) {
                throw new RuntimeException("received null view when unexpected");
            }
            // if we are laying out views in scrap, this may return null which means there is
            // no more items to layout.
            result.mFinished = true;
            return;
        }
       ...........此处省略大段代码
    }

layoutChunk是被fill调用。

    int fill(RecyclerView.Recycler recycler, LayoutState layoutState,
            RecyclerView.State state, boolean stopOnFocusable) {
        // max offset we should set is mFastScroll + available
        final int start = layoutState.mAvailable;
        if (layoutState.mScrollingOffset != LayoutState.SCROLLING_OFFSET_NaN) {
            // TODO ugly bug fix. should not happen
            if (layoutState.mAvailable < 0) {
                layoutState.mScrollingOffset += layoutState.mAvailable;
            }
            recycleByLayoutState(recycler, layoutState);
        }
        int remainingSpace = layoutState.mAvailable + layoutState.mExtra;
        LayoutChunkResult layoutChunkResult = mLayoutChunkResult;
        while ((layoutState.mInfinite || remainingSpace > 0) && layoutState.hasMore(state)) {
            layoutChunkResult.resetInternal();
            if (RecyclerView.VERBOSE_TRACING) {
                TraceCompat.beginSection("LLM LayoutChunk");
            }
            layoutChunk(recycler, state, layoutState, layoutChunkResult);
            if (RecyclerView.VERBOSE_TRACING) {
                TraceCompat.endSection();
            }
               ...........此处省略大段代码
        }
       ...........此处省略大段代码
    }

基本可以看出是while这里不断调用造成的数据有多少个就创建多少个ViewHolder。接下来我们着重分析while的条件。先看layoutState.hasMore(state)。

        boolean hasMore(RecyclerView.State state) {
            return mCurrentPosition >= 0 && mCurrentPosition < state.getItemCount();
        }

逻辑很简单,就是在Adapter中列表的数量范围内,问题不是它造成的。接下来看remainingSpace ,调试可以看到发现在创建几个ViewHolder后remainingSpace就变成0了,所以它不是真凶。


图片.png

那么很明显问题就是layoutState.mInfinite造成的,因为它一直是true,因此列表里有多少数据就会创建多少个ViewHolder。mInfinite是resolveIsInfinite()给赋的值,resolveIsInfinite代码如下:

    boolean resolveIsInfinite() {
        return mOrientationHelper.getMode() == View.MeasureSpec.UNSPECIFIED
                && mOrientationHelper.getEnd() == 0;
    }

因为我们用的是竖向列表,所以这里的mOrientationHelper是OrientationHelper的createVerticalHelper创建的,省略代码如下:

    public static OrientationHelper createVerticalHelper(LayoutManager layoutManager) {
        return new OrientationHelper(layoutManager) {
            public int getEnd() {
                return this.mLayoutManager.getHeight();
            }
            public int getMode() {
                return this.mLayoutManager.getHeightMode();
            }
           ...........此处省略大段代码
        };
    }

调试可看到,最后一次给mInfinite赋值,两个都是0,对比没问题的列表时getMode()也是0,因此我们重点看一下为什么getEnd()即mLayoutManager.getHeight()高度是0。

图片.png

调试可以看到,最后一次onMeasure传入LayoutManager.setMeasureSpecs的heightSpec为0,从而导致的mHeight和mHeightMode都是0。
图片.png

可能让人疑惑的是为什么屏幕显示正确,Android Studio的Layout Inspector分析的高度也是正确的,这是因为进行了多次测量后,measureSpecModeExactly变量值为true,便不会走LayoutManager.setMeasureSpecs流程。
  我们继续看为什么onMeasure传入的heightSpec为0,在布局中RecycleView的父布局为RelativeLayout,recyclerview的heightSpec由父布局measure时传入。

!!!TODO:因为布局调用次数太多,源码又不太匹配,暂时不再深究了。后续有时间再来 -_- ~~~

最后解决方法:RecycleView的直接或者间接RelativeLayout父布局均不能带weight,可把RelativeLayout切换为LinearLayout。

©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 204,293评论 6 478
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 85,604评论 2 381
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 150,958评论 0 337
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 54,729评论 1 277
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 63,719评论 5 366
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 48,630评论 1 281
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 38,000评论 3 397
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 36,665评论 0 258
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 40,909评论 1 299
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 35,646评论 2 321
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 37,726评论 1 330
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 33,400评论 4 321
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 38,986评论 3 307
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 29,959评论 0 19
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 31,197评论 1 260
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 44,996评论 2 349
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 42,481评论 2 342

推荐阅读更多精彩内容