网上关于 ViewPager 的用法、源码解析已经讲的很多了。但生产环境中,我们可能会遇到各种奇怪的问题。这篇文章将会聊聊自己遇到的比较奇怪的异常情况,并讲述分析思路与源码解析。
viewpager 异常
从视频中可以看到,当切换到 4 的时候,继续向右切换却变成了1。用户就会感觉“鬼打墙了”永远在这几个数据里面循环起来。
从复现的路径上可以看出,当切换到 4 的时候,下方的Navigation切换到了 0,但 viewPager 本身没有什么变化。
进一步的思考,排查setCurrentItemInternal的调用位置
setAdapter
dataSetChanged
onResotreInstanceState
onTouchEvent
endFakeDrag
1 是初始化 adapter 的时候,调用的,很明显不是这里的问题;3 是恢复的时候。5 很明显也不是;出问题的地方只有 2 或者 4。4 是拖动的时候触发的,这里看起来是切换过去以后才出问题。先不查它,后面再说。
那么 dataSetChanged 的嫌疑最大。
- class VerticalViewPager {
- void dataSetChanged() {
-
- boolean isUpdating = false;
- for (int i = 0; i < mItems.size(); i++) {
- final ItemInfo ii = mItems.get(i);
- final int newPos = mAdapter.getItemPosition(ii.object);
- if (ii.position != newPos) {
- if (ii.position == mCurItem) {
- // Our current item changed position. Follow it.
- newCurrItem = newPos;
- }
-
- ii.position = newPos;
- needPopulate = true;
- ...
- }
-
-
- if (needPopulate) {
- // Reset our known page widths; populate will recompute them.
- ...
- if (mSuspendOnePopulate) {
- // do nothing
- } else {
- setCurrentItemInternal(newCurrItem, false, true);
- }
- requestLayout();
- }
- }
- }
-
- class xMAdapter {
- override fun getItemPosition(any: Any): Int {
- items.forEachIndexed { index, view ->
- if (view == (`object` as? View)?.tag) {
- return index
- }
- }
- return POSITION_NONE
- }
- }
上述代码中,删除了无关的代码。可以发现,这里有一个非常可能导致问题的地方,就是 final int newPos = mAdapter.getItemPosition(ii.object);,可以看到,源码中使用 tag 去mAdapter 中去寻找 position。这就导致了一个问题,当列表中存在多个相同 tag, 且下标不一样的时候,会存在 position 查找错误的可能。举例说明。
当数据是(【】表示 items 中的数据)
0 1 2 3 4 5 6 【7 8 9】 的时候。当前视频是 8
当从 8 -> 9以后。
0 1 2 3 4 5 6 7 【8 9 10】 此时新来了一批数据,触发了 loadMoreResult,接着触发 dataSetChanged。
从代码中可以看到,此时会遍历items,然后从 mAdapter 种获取 newPos 的位置。问题来了,此时下标 1、9 的tag是一个,这时候,遍历 mAdapter 会先返回前者,也就是返回了 1。明明在 9 位置,却返回了 1。接着会执行
setCurrentItemInternal->populate->addNewItem。从这里就全错了,数据变成了
【0 1 2】 3 4 5 6 8 9 10。继而循环了。
所以出现该问题的路径可能有一下两种情况
服务端同一刷下发了两个相同的视频
有人在 mAdapter 中插入或者 替换了之前已经存在的视频
其实原理一样,不想写了。