记一次全民K歌的crash定位过程(2)

对于线索1，我们先假设是第一种情况，通过追踪FLAG_REMOVED设置的路径，发现只有当业务调用了Adapter的notifyXXXRemoved方法时，才会为ViewHolder添加FLAG_REMOVED标记。而线索二中的Footer实际上是一个容器，业务调用addFooterView添加进来的布局都会填入容器中，不管用户如何操作，对RecyclerView来说，Footer始终是有且只有一个，不存在删除Footer的情况。于是线索一纠正为：从mAttachedScrap中取到的ViewHolder类型与Adapter取到的不一致。

[原文来自：www.11jj.com]

mAttachedScrap中的ViewHolder是通过对比LayoutPosition查找到的，而Adapter.getItemType的结果则是分析数据集而来，两者的不一致说明了RecyclerView的状态与数据集产生了不同步的情况，往往出现在Adapter中的列表数据发生了变化而又没有调用notityXXX方法通知到RecyclerView的情况下。 [原文来自：www.11jj.com]

crash所在的列表并没有请求后台数据却产生了数据的变化，能产生这一现象的只有用户发布作品后，由客户端自己构造的假数据了。

因作品发布与K歌业务逻辑关联较大，参考意义不大，这里只做简要的文字说明：

用户发布作品后，会生成一条发布数据在动态中显示，这条数据是存在于单例中的，两个FeedSubFragment都能取到，发布完成并刷新列表才会把它从单例中清除。另外，用户在K歌内的一些互动操作会触发广播，比如在作品详情页评论了作品，那动态中这个作品的feed评论计数会实时更新，不需要等待列表的刷新操作，广播也都是有..的。

作品刚发布时，不可见的那个页面对此无感知，会出现RecyclerView是Refresh、Header、Footer、Empty、Load五个item的状态，而Adapter的数据集中在Header与Footer间多了一条假feed，虽然没有调用notifyXXX，但当有互动操作或跳其它Activity返回等其它原因触发layout时，也不会引起crash，如下：

①② 通过position可以从mAttachedScrap正确获取到原来的ViewHolder并直接复用

③ 通过position取到了Footer的ViewHolder，发现类型不同，把它从布局中remove并添加到缓存池RecycledViewPool，最后新创建一个假Feed的ViewHolder

④ 取到了Empty的ViewHolder，同样回收至RecycledViewPool，但因为上一步有把Footer的ViewHolder添加到了RecycledViewPool，处理完Empty后，会尝试从RecycledViewPool查找，而这里是通过viewType来查找的，所以可以找到上一步添加进来的ViewHolder，从而复用

⑤⑥ 同④

当假feed已经被layout出来，数据被删除却没有notify的情况下执行layout又会怎样呢？

①② 可直接复用

③ 取到了假feed的ViewHolder，回收至RecycledViewPool，然后重新创建了一个Footer的ViewHolder，这就导致了两个ViewHolder指向同一个View的出现，一个新创建的添加到RecyclerView中显示，并清除FLAG_TMP_DETACHED标记，另一个仍然存在于Scrap缓存中未被使用

④ 取到了Scrap缓存中Footer的ViewHolder，尝试回收至RecycledViewPool，却发现Footer已经不是FLAG_TMP_DETACHED的状态，因为上一步已经把它添加到RecyclerView中，清除了这一标记，于是抛出文章开头的IllegalArgumentException异常

可能有人会感兴趣增删数据并调用了notifyXXXRemoved的正常情况下，RecyclerView是如何在preLayout及postLayout阶段都能通过position获取到正确的ViewHolder的，可以自行了解下ViewHolder的mPreLayoutPosition跟mPosition的作用，这里不细说了

至此，原因也就比较清晰了：用户使用K歌停留在动态非好友页，退后台被系统杀掉重启时，没有考虑到Fragment恢复的情况，导致在正常的Fragment下多生成了一个不可见的Fragment，之后发布了作品并对其执行了会引起数据变化的互动操作，使其layout到布局中，刷新列表后不可见的RecyclerView列表状态与Adapter数据不同步，跳转到其它Activity再返回时，触发了RecyclerView的重新布局，检测到了状态不对并抛出了异常。

QQ音乐团队诚聘测试、研发。有意者请发送简历至tmezp@tencent.com，请注明来自公众号，我们将优先拜读。