ViewPager 异常状态之无法切换、循环切换

网上关于 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。继而循环了。

所以出现该问题的路径可能有一下两种情况