解决RecyclerView执行动画后item边距错误的问题

cover

原文链接:kyleduo.com

RecyclerView和StaggeredGradLayoutManager可以方便的实现瀑布流效果,但是在使用中会遇到一个问题:如果使用ItemDecoration设置offset,在触发动画之后,如果一个item从一个span移动到另一个span(比如插入一条数据),会出现位置偏移的情况。

wrong_position

上图中红色虚线的位置是Item原本应该在的位置。(忽略下面的文字)

出现这个偏移的原因,是RecyclerView会缓存child的offset,当span发生变化时,又没有更新,所以使用了动画之前的offset。

我们可以通过设置offset的左右值相等,来规避这个问题。但是规避终归并没有解决这个问题。这篇文章就来寻找一种方便的解决方法。

保存和更新

offset存储在RecyclerView.LayoutParams.mDecorInsets这个属性中,后文也是用mDecorInsets来表示。

和它关联的,还有mInsetsDirty这个变量,用来标记是否需要更新mDecorInsets

mDecorInsets在RecyclerView.getItemDecorInsetsForChild方法中被更新赋值,这个方法在measureChildmeasureChildWithMargins中调用。RecyclerView.getItemDecorInsetsForChild方法会判断mInsetsDirty标记位是否为true,如果为false将直接使用缓存的值而不重新计算。

布局流程

RecyclerView的布局流程分为三步完成,对应三个方法:

  1. dispatchLayoutStep1
  2. dispatchLayoutStep2
  3. dispatchLayoutStep3

动画统一在dispatchLayoutStep3中执行,动画的起始值和结束值分别在dispatchLayoutStep1dispatchLayoutStep3中储存。而
dispatchLayoutStep2是真正对子View进行布局的地方,也就是这个方法确定了动画的结束值。

dispatchLayoutStep2中调用LayoutManager.onLayoutChildren方法进行真正的布局,StaggeredGridLayoutManager.onLayoutChildren最终调用fill方法。

fill方法中更新span,并调用measureChildWithDecorationsAndMargin方法进行测量,而这个方法,就最终调用前文说的RecyclerView.getItemDecorInsetsForChild方法获取mDecorInsets

定位问题

你可能注意到了,既然是先设置span,后获取mInsetsDirty,那怎么会出现偏差呢?

问题出在dispatchLayoutStep1中,这个方法包含这样的逻辑:

private void dispatchLayoutStep1() {
  // ......
  if (mState.mRunPredictiveAnimations) {
    // ......
    // temporarily disable flag because we are asking for previous layout
    mLayout.onLayoutChildren(mRecycler, mState);
    // ......
  }
  // ......
}

processAdapterUpdatesAndSetAnimationFlags也就是说当满足mRunPredictiveAnimations时,会调用一次onLayoutChildren方法,儿dispatchLayoutStep2中,还会无条件的再调用一次这个方法,这就导致onLayoutChildren在一次布局流程中被调用两次,而第二次调用时,更新了span,但是使用了缓存的mDecorInsets,也就造成了偏差。

dispatchLayoutStep1会调用processAdapterUpdatesAndSetAnimationFlags方法,这个方法最终会调用RecyclerView.ViewHolder.offsetPosition这个方法,这个方法中会将mInsetsDirty设置为true。所以在dispatchLayoutStep1中第一次调用onLayoutChildren时,就会更新mDecorInsets的值,而第二次进来的时候,就不会了。

通过调试,分别在设置span时和getItemDecorInsetsForChild中打印log,然后在index==1的位置插入一条数据,可以得到如下结果:

log

1247341404是第一条数据,因为没有变化,所以第一次layout也没有更新mDecorInsets

1247341072是新添加的数据,因为第一次layout使用的是旧数据,所以并不包含这一条,而在第二次layout时获取到mDecorInsets

观察1247341250这条数据,第一次layout时span为1,第二次时变为0,但是没更新mDecorInsets,所以出现了偏差。

解决问题

既然知道了问题所在,下面我们就解决它。既然第二次onLayoutChildren时使用了缓存才导致mDecorInsets没有更新,那么我们就强制设置mInsetsDirty为true,这样在fill方法中就会更新了。很遗憾RecyclerView并没有提供public方法在做这件事,那就只有使用反射了。

StaggeredGridLayoutManager lm = new StaggeredGridLayoutManager(2, StaggeredGridLayoutManager.VERTICAL) {

  private Method markItemDecorInsetsDirty = null;
  private boolean reflectError = false;

  @Override
  public void onLayoutChildren(RecyclerView.Recycler recycler, RecyclerView.State state) {
    if (markItemDecorInsetsDirty == null && !reflectError) {
      try {
        markItemDecorInsetsDirty = RecyclerView.class.getDeclaredMethod("markItemDecorInsetsDirty");
        markItemDecorInsetsDirty.setAccessible(true);
      } catch (NoSuchMethodException e) {
        e.printStackTrace();
        reflectError = true;
      }
    }
    // 更新条件
    if (markItemDecorInsetsDirty != null && state.willRunSimpleAnimations()) {
      // noinspection TryWithIdenticalCatches
      try {
          markItemDecorInsetsDirty.invoke(mRecyclerView);
      } catch (IllegalAccessException e) {
          e.printStackTrace();
      } catch (InvocationTargetException e) {
          e.printStackTrace();
      }
    }
      super.onLayoutChildren(recycler, state);
  }

  // 7.28 更新
  @Override
  public void requestSimpleAnimationsInNextLayout() {
    super.requestSimpleAnimationsInNextLayout();
    if (markItemDecorInsetsDirty != null) {
      // noinspection TryWithIdenticalCatches
      try {
          markItemDecorInsetsDirty.invoke(mRecyclerView);
      } catch (IllegalAccessException e) {
          e.printStackTrace();
      } catch (InvocationTargetException e) {
          e.printStackTrace();
      }
    }
  }
};

我们反射调用markItemDecorInsetsDirty这个方法标记dirty,即可强制更新。但是我们并不想每次layout的时候都强制更新,这样效率太低了。所以还要判断state.willRunSimpleAnimations(),这个标记表示将执行ItemAnimator动画,也就是只有当触发动画时,才强制更新。这样就解决问题了。

并不是只有修改数据时才会出现这个问题,当使用GAP_HANDLING_MOVE_ITEMS_BETWEEN_SPANS这个策略时,填充GAP时也会触发动画,所以也会有这个问题。使用上面的标记位即可涵盖这两种情况,并不会有多余调用。

7月28日更新

StaggeredGridLayoutManager在检查GAP时,有两种情况,一种是执行动画移动child,另一种是需要重新layout。上面的逻辑只包含第一种情况,第二种情况需要再添加一些代码。已经更新到上面的代码块儿里了。

因为如果checkForGaps方法返回true,调用的onLayoutChildren为三个参数的私有方法,而不是我们覆写的方法,所以不能在通过覆写判断标记为实现。但是checkForGaps返回true之前,一定会调用requestSimpleAnimationsInNextLayout方法(只在这个地方调用,所以也不会有额外性能损耗),而且这个方法是public的,所以我们可以覆写这个方法,在里面标记mInsetsDirty为true。

总结

我们在阅读源码,尤其是像RecyclerView这种逻辑复杂的类时,因为代码量很大,逻辑分支多,不要逐行阅读,否则一不小心就会“迷失”。正确的做法是在阅读前先进行思考,拆成多个子功能逐个分析,比如Measure/Layout,Touch事件,Animation等等,每个子功能找到一个入口,按照调用关系去分析,比如Touch事件和布局相关功能可以在onXxx回调方法中开始。

我在解决这个问题时,就是从Animation这个点切入,因为动画是notifyXxx方法触发的,所以自然就找到了入口,然后按照调用路径去寻找最终影响动画的各个因素。

当然阅读源码时不能光看代码,合理的断点和调试也是必不可少的。毕竟最初编写代码的人,也一定不是一气呵成写完,也是按照功能点逐步填充这个类的。不要把自己当成阅读者,而要延着代码的逻辑思考问题。

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

推荐阅读更多精彩内容