为什么LiveData observer被触发两次用于新附加的观察器

Question

我对livedata的理解是，它将触发观察者对当前数据的状态变化，而不是对一系列历史数据的状态变化。

目前，我有一个mainfragment，它执行room写操作，将非垃圾数据更改为垃圾数据。

我还提供了另一个trashfragment，它观察被丢弃的数据。

请考虑以下场景。

1. 当前有0个被丢弃的数据。
2. mainfragment是当前活动的片段。trashfragment尚未创建。
3. mainfragment添加了%1个废数据。
4. 现在，有%1个被丢弃的数据
5. 我们使用导航抽屉，将mainfragment替换为trashfragment。
6. trashfragment的观察者将首先接收onchanged，其中有0个被丢弃的数据
7. 同样，trashfragment的观察者将第二次接收onchanged，其中有1个被丢弃的数据

出乎我意料的是，第（6）项不应该发生。trashfragment应只接收最新的已废弃数据，即%1。

这是我的密码

public class TrashFragment extends Fragment {
    @Override
    public void onCreate(Bundle savedInstanceState) {
        noteViewModel = ViewModelProviders.of(getActivity()).get(NoteViewModel.class);
    }

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
        ...

        noteViewModel.getTrashedNotesLiveData().removeObservers(this);
        noteViewModel.getTrashedNotesLiveData().observe(this, notesObserver);

public class MainFragment extends Fragment {
    @Override
    public void onCreate(Bundle savedInstanceState) {
        noteViewModel = ViewModelProviders.of(getActivity()).get(NoteViewModel.class);
    }

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
        ...

        noteViewModel.getNotesLiveData().removeObservers(this);
        noteViewModel.getNotesLiveData().observe(this, notesObserver);

public class NoteViewModel extends ViewModel {
    private final LiveData<List<Note>> notesLiveData;
    private final LiveData<List<Note>> trashedNotesLiveData;

    public LiveData<List<Note>> getNotesLiveData() {
        return notesLiveData;
    }

    public LiveData<List<Note>> getTrashedNotesLiveData() {
        return trashedNotesLiveData;
    }

    public NoteViewModel() {
        notesLiveData = NoteplusRoomDatabase.instance().noteDao().getNotes();
        trashedNotesLiveData = NoteplusRoomDatabase.instance().noteDao().getTrashedNotes();
    }
}

public enum NoteRepository {
    INSTANCE;

    public LiveData<List<Note>> getTrashedNotes() {
        NoteDao noteDao = NoteplusRoomDatabase.instance().noteDao();
        return noteDao.getTrashedNotes();
    }

    public LiveData<List<Note>> getNotes() {
        NoteDao noteDao = NoteplusRoomDatabase.instance().noteDao();
        return noteDao.getNotes();
    }
}

@Dao
public abstract class NoteDao {
    @Transaction
    @Query("SELECT * FROM note where trashed = 0")
    public abstract LiveData<List<Note>> getNotes();

    @Transaction
    @Query("SELECT * FROM note where trashed = 1")
    public abstract LiveData<List<Note>> getTrashedNotes();

    @Insert(onConflict = OnConflictStrategy.REPLACE)
    public abstract long insert(Note note);
}

@Database(
        entities = {Note.class},
        version = 1
)
public abstract class NoteplusRoomDatabase extends RoomDatabase {
    private volatile static NoteplusRoomDatabase INSTANCE;

    private static final String NAME = "noteplus";

    public abstract NoteDao noteDao();

    public static NoteplusRoomDatabase instance() {
        if (INSTANCE == null) {
            synchronized (NoteplusRoomDatabase.class) {
                if (INSTANCE == null) {
                    INSTANCE = Room.databaseBuilder(
                            NoteplusApplication.instance(),
                            NoteplusRoomDatabase.class,
                            NAME
                    ).build();
                }
            }
        }

        return INSTANCE;
    }
}

知道如何防止对同一数据两次接收onchanged吗？

我创建了一个演示项目来演示这个问题。

如您所见，我在mainfragment中执行写操作（单击ADD TRASHED NOTE按钮）后，当我切换到trashfragment时，我希望trashfragment中的onchanged只会被调用一次。然而，它正在被调用两次。

演示项目可从https://github.com/yccheok/live-data-problem下载

Answer 1

我只在您的代码中引入了一个更改：

noteViewModel = ViewModelProviders.of(this).get(NoteViewModel.class);

而不是：

noteViewModel = ViewModelProviders.of(getActivity()).get(NoteViewModel.class);

在fragment的oncreate(Bundle)方法中。现在它工作得天衣无缝。

在您的版本中，您获得了两个片段共同的NoteViewModel引用（来自activity）。我想，viewmodel在前面的片段中注册了observer。因此，LiveData保留了对Observer的引用（在MainFragment和TrashFragment中），并调用了这两个值。

因此，我想结论可能是，您应该从viewmodelproviders中获取viewmodel，其来源是：

fragment中的
• fragment activity
中的 activity

顺便说一句。

NoteViewModel.GetTrashedNotesLiveData().RemoveObservers(this)；

在片段中不是必需的，但是我建议将其放在onstop中。

Answer 2

我分叉了你的项目并测试了一下。据我所知，你发现了一个严重的窃听器。

为了使复制和调查更容易，我对你的项目做了一点编辑。您可以在这里找到更新的项目：https://github.com/techyourchance/live-data-problem。我还开了一个拉请求回你的回购。

为了确保这一点不会被忽视，我还在谷歌的问题追踪器中打开了一个问题：

再现的步骤：

1. 确保在MainFragment中将REPRODUCE_BUG设置为true
2. 安装应用程序
3. 单击“添加废弃便笺”按钮
4. 切换到TrashFragment
5. 注意，只有一个通知表单LiveData具有正确的值
6. 切换到MainFragment
7. 单击“添加废弃便笺”按钮
8. 切换到TrashFragment
9. 注意LiveData有两个通知，第一个的值不正确

注意，如果您将REPRODUCE_BUG设置为false，那么bug将不会重现。它演示了对MainFragment中的LiveData的订阅改变了TrashFragment中的行为。

预期结果：在任何情况下只有一个具有正确值的通知。没有因为以前的订阅而改变行为。

更多信息：我看了一下源代码，它看起来像是由于LiveData激活和新的Observer订阅而触发的通知。可能与ComputableLiveData将onActive（）计算卸载到Executor的方式有关。