Android Fragment切换时LiveData观察者重复引发RecyclerView数据重复问题排查

首先，咱们来拆解你遇到的问题：

为什么会出现数据重复？ #

核心原因大概率是两个点：

1. Fragment实例重复创建：你用replace方法切换Fragment时，如果每次点击菜单都new一个新的Fragment A实例，那么每次新实例都会给ViewModel的LiveData注册一个新的观察者。虽然旧Fragment实例的观察者会在onDestroyView时被LiveData自动移除（因为你用了getViewLifecycleOwner()），但新实例注册观察者时，LiveData会立刻把缓存的最新数据推送给它。
2. 数据源未清空：如果你的观察者回调里是直接把新数据追加到RecyclerView的列表中（比如list.addAll(details)），而不是先清空旧数据再添加，那每次收到LiveData的缓存数据，就会导致列表重复累加。

为什么手动移除观察者无效？ #

你用getViewLifecycleOwner()作为观察者宿主时，这个LifecycleOwner的生命周期和Fragment的View绑定。当Fragment执行onDestroyView时，这个LifecycleOwner的状态会变为DESTROYED，LiveData会自动移除对应的观察者——也就是说，你手动调用removeObservers或removeObserver其实是多此一举，而且针对的是当前Fragment实例的观察者，旧实例的观察者已经被系统处理了。

如果是你误用了this（Fragment本身）作为LifecycleOwner，那Fragment后台时可能还处于活跃状态，观察者不会被自动移除，但你代码里用的是getViewLifecycleOwner()，所以这个情况不成立。另外，如果你的mObserver是每次在onViewCreated里新建的匿名类实例，那调用removeObserver(mObserver)时，移除的是当前这个新实例，旧的匿名观察者根本找不到，也会导致移除无效，但看你代码里用的是成员变量mObserver，这个可能性也比较小。

解决办法 #

办法一：复用Fragment实例，避免重复创建 #

不要每次切换都new新的Fragment，而是在Activity中保留Fragment的实例，用show/hide代替replace，这样Fragment实例不会被销毁，观察者只会注册一次：

// 在MainActivity中定义成员变量
private Fragment currentFragment;
private Fragment fragmentA;
private Fragment fragmentB;

// 切换到Fragment A的方法
private void switchToFragmentA() {
    if (fragmentA == null) {
        fragmentA = new FragmentA();
    }
    switchFragment(fragmentA);
}

// 切换到Fragment B的方法
private void switchToFragmentB() {
    if (fragmentB == null) {
        fragmentB = new FragmentB();
    }
    switchFragment(fragmentB);
}

// 通用切换逻辑
private void switchFragment(Fragment targetFragment) {
    FragmentTransaction transaction = getSupportFragmentManager().beginTransaction();
    if (currentFragment != null) {
        transaction.hide(currentFragment);
    }
    if (!targetFragment.isAdded()) {
        transaction.add(R.id.fragment_container, targetFragment);
    } else {
        transaction.show(targetFragment);
    }
    transaction.commit();
    currentFragment = targetFragment;
}

办法二：在观察者回调中先清空数据源 #

不管是否重复注册观察者，确保每次收到数据时先清空旧列表，再添加新数据，从根源上避免重复：

// 你的观察者回调修改成这样
private Observer<List<Detail>> mObserver = new Observer<List<Detail>>() {
    @Override
    public void onChanged(List<Detail> details) {
        // 先清空旧数据
        mRecyclerViewDataList.clear();
        // 再添加新数据
        mRecyclerViewDataList.addAll(details);
        // 通知适配器更新
        mAdapter.notifyDataSetChanged();
    }
};

办法三：如果坚持用replace，确保Fragment实例复用 #

如果不想改replace的逻辑，那在Activity中保留Fragment A的实例，不要每次都新建：

private Fragment fragmentA;

private boolean loadFragment(Fragment fragment) { 
    if (fragment != null) { 
        // 处理Fragment A的复用
        if (fragment instanceof FragmentA) {
            if (fragmentA == null) {
                fragmentA = (FragmentA) fragment;
            } else {
                fragment = fragmentA;
            }
        }
        getSupportFragmentManager() 
            .beginTransaction() 
            .replace(R.id.fragment_container, fragment) 
            .commit(); 
        return true; 
    } 
    return false; 
}

这样处理后，不管是观察者重复注册的问题，还是数据重复的问题都能解决啦。

内容的提问来源于stack exchange，提问作者DevAntho