bug描述
初次使用 ViewPage2 + Fragment,遇到一个bug:
- FragmentStateAdapter 中的 原始数据
fragmentListA
清空 - 然后FragmentStateAdapter 中设置新数据
fragmentListB
- 期望ViewPage2 中展示
fragmentListB
中的数据, - 结果 ViewPage2 第一条 fragment 展示的 是
fragmentListA
中的第一条。
问题原因
FragmentStateAdapter 数据设置检查没有问题,但是createFragment
方法没有调用那就很明显问题出在缓存上,某个地方缓存了上一个fragmentList中的第一条数据并使用了,于是check了FragmentStateAdapter 的源码,果然又发现:
final LongSparseArray<Fragment> mFragments = new LongSparseArray<>();
public final void onBindViewHolder(final @NonNull FragmentViewHolder holder, int position) {
// ...
ensureFragment(int position);
// ...
}
// FragmentStateAdapter 创建 Fragment 的方法
private void ensureFragment(int position) {
long itemId = getItemId(position);
if (!mFragments.containsKey(itemId)) {
Fragment newFragment = createFragment(position);
newFragment.setInitialSavedState(mSavedStates.get(itemId));
mFragments.put(itemId, newFragment);
}
}
public long getItemId(int position) {
return position;
}
问题很明显了:
- FragmentStateAdapter 通过
getItemId
方法获取获得的id作为key,来缓存fragment - 而默认的
getItemId
方法返回的id是position - 收到新的 fragmentlist 之后,在
onBindViewHolder
的时候发现position是0 的数据在缓存中,于是就没有走createFragment
方法。
那么新问题来了,为什么后面从position == 1 之后的数据就正常了呢?createFragment 地方是唯一的,那肯定是某个地方将缓存数据删除了,再看看源码吧,删除缓存的地方也是再唯一方法removeFragment中,有三个地方调用它:
-
gcFragments
:遍历缓存,将不在列表中的缓存(使用containsItem
方法判断), 和fragment失效的缓存删除 -
onBindViewHolder
: 绑定时检查,如果ViewHolder中的item 的id 与 mItemIdToViewHolder 缓存的id不相同, 则删除该缓存 -
onViewRecycled
: 触发回收时删除FragmentViewHolder 所绑定的数据,并删除其缓存
gcFragments
, onBindViewHolder
都没有处理,那就肯定是onViewRecycled
中干的了, 才疏学浅,不再继续深究了。
public boolean containsItem(long itemId) {
return itemId >= 0 && itemId < getItemCount();
}
修复方案
FragmentStateAdapter 中 重写 getItemId
和 containsItem
方法。
建议
使用 ViewPage2 +FragmentStateAdapter + Fragment 的时候, 如果 fragment数据 是可变的
,那么就一定要重写 getItemId
和 containsItem
方法,以免出现因为缓存造成页面信息异常。因为默认 getItemId
取的是当前元素的position, 在 fragmentlist是可变的情况下不具有唯一性,会导致加载出过期页面。