Java调用getData()返回MutableLiveData为null如何解决

你遇到的null返回问题核心是两个点：

1. 异步时序错误：Firebase Firestore的addSnapshotListener是非阻塞异步接口，注册监听后方法会立刻往下执行返回空的MutableLiveData实例，此时网络请求还没完成、数据填充逻辑还在回调队列里没执行，此时如果直接调用LiveData.getValue()必然拿到null。
2. 代码存在隐藏的数据重复bug：你在方法内/类成员位置复用同一个ArrayList实例存储评论，后续快照监听多次触发时会不断往旧列表里追加数据，导致评论重复。

1. 修正getData方法逻辑 #

不要复用持久化的评论列表，每次监听到快照更新时重新构建数据列表，再通过postValue推送给LiveData，修正后代码如下：

public MutableLiveData<List<Comment>> getData(BaiHat song){
    MutableLiveData<List<Comment>> commentLiveData = new MutableLiveData<>();
    db.collection(KEY_COLLECTION_COMMENT)
            .whereEqualTo(KEY_SONG, song.getIdBaiHat())
            .addSnapshotListener((value, error) -> {
                // 出错时主动推送空列表，避免观察者永久等待
                if (error != null) {
                    commentLiveData.postValue(Collections.emptyList());
                    return;
                }
                // 每次回调都新建列表，避免数据重复
                List<Comment> comments = new ArrayList<>();
                if (value != null) {
                    // 首次加载全量数据直接遍历所有文档，不要只取增量变更，避免漏数据
                    for (DocumentSnapshot doc : value.getDocuments()) {
                        Comment comment = new Comment();
                        comment.setComment(doc.getString(KEY_COMMENT));
                        comment.setEmail(doc.getString(KEY_EMAIL));
                        comments.add(comment);
                    }
                }
                // 数据填充完成后再推送，观察者此时才能拿到有效数据
                commentLiveData.postValue(comments);
            });

    return commentLiveData;
}

2. 修正调用端写法 #

LiveData本身是观察者模式的实现，绝对不要在拿到LiveData实例后立刻调用getValue()取数据，必须注册观察者，等数据推送过来后再处理：

错误写法（100%拿null）：

MutableLiveData<List<Comment>> liveData = getData(song);
List<Comment> commentList = liveData.getValue(); // 执行时回调还未触发，值为null

正确写法：

getData(song).observe(this, comments -> {
    // 此处代码会在Firestore返回数据、postValue执行后自动触发
    // 拿到的comments就是有效评论列表，直接刷新UI即可
    commentAdapter.submitList(comments);
});

• 如果你不需要实时监听评论变更，只需要拉取一次数据，可以把addSnapshotListener换成get()方法，异步回调的处理逻辑完全一致，还能避免忘记移除监听导致的内存泄漏。
• 如果使用addSnapshotListener，必须在页面onDestroy时调用ListenerRegistration的remove方法移除监听，避免内存泄漏。
• 不要试图用Thread.sleep、CountDownLatch之类的方式阻塞主线程等待异步返回，会触发ANR导致应用崩溃。

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