网上关于 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 中插入或者 替换了之前已经存在的视频
解决思路
- 禁止 mAdapter 中出现重复的数据,禁止 index 返回出现问题
- 修改 getItemPosition 的逻辑,反向遍历,优先拿后者
- 用户反向滑动的时候,可能会出现问题,又业务逻辑来判断是否采取这种方式解决。
无法切换
其实原理一样,不想写了。