getContactsFromFirebase()返回空列表但日志有值问题求助

你碰到的这个问题，核心原因是Firebase Realtime Database的读写操作都是异步执行的。你的getContactsFromFirebase()方法会在数据库请求完成、onDataChange回调执行之前，就先返回了mContactsFromFirebase——这时候列表还没被填充任何数据，自然是空的。而你在日志里看到的列表内容，是数据库请求完成后才在回调里打印的，这时候方法早就已经返回了。

简单验证执行顺序 #

你可以给方法加几个日志，就能清楚看到执行顺序：

public List<String> getContactsFromFirebase(){
    Log.i("Test", "方法启动，准备发起数据库请求");
    FirebaseDatabase.getInstance().getReference().child("Users")
            .addListenerForSingleValueEvent(new ValueEventListener() {
                @Override
                public void onDataChange(DataSnapshot dataSnapshot) {
                    Log.i("Test", "数据库请求完成，开始填充列表");
                    for (DataSnapshot snapshot : dataSnapshot.getChildren()) {
                        Users user = snapshot.getValue(Users.class);
                        assert user != null;
                        String contact_found = user.getPhone_number();
                        mContactsFromFirebase.add(contact_found);
                        Log.i("Test", mContactsFromFirebase.toString());
                    }
                }

                @Override
                public void onCancelled(DatabaseError databaseError) {
                }
            });
    Log.i("Test", "方法即将返回列表，当前列表内容：" + mContactsFromFirebase.toString());
    return mContactsFromFirebase;
}

运行后日志顺序会是这样：

1. 方法启动，准备发起数据库请求
2. 方法即将返回列表，当前列表内容：[]
3. 数据库请求完成，开始填充列表
4. [xxx, xxx, ...]

这就实锤了：方法返回时，数据还没加载完成。

解决方案：用回调接口处理异步结果 #

Java里没法直接从异步操作返回结果，我们得用回调来传递加载好的数据。

第一步：定义回调接口 #

public interface OnContactsLoadedListener {
    void onContactsLoaded(List<String> contacts);
    void onLoadFailed(DatabaseError error);
}

第二步：修改方法，接受回调参数 #

public void getContactsFromFirebase(OnContactsLoadedListener listener){
    FirebaseDatabase.getInstance().getReference().child("Users")
            .addListenerForSingleValueEvent(new ValueEventListener() {
                @Override
                public void onDataChange(DataSnapshot dataSnapshot) {
                    List<String> contacts = new ArrayList<>();
                    for (DataSnapshot snapshot : dataSnapshot.getChildren()) {
                        Users user = snapshot.getValue(Users.class);
                        // 建议用if代替assert，默认Android会禁用断言，避免空指针崩溃
                        if (user != null) {
                            String contact_found = user.getPhone_number();
                            contacts.add(contact_found);
                        }
                    }
                    // 数据加载完成后，通过回调把结果传出去
                    listener.onContactsLoaded(contacts);
                }

                @Override
                public void onCancelled(DatabaseError databaseError) {
                    // 加载失败时也通过回调通知
                    listener.onLoadFailed(databaseError);
                }
            });
}

第三步：调用方法时使用回调 #

getContactsFromFirebase(new OnContactsLoadedListener() {
    @Override
    public void onContactsLoaded(List<String> contacts) {
        // 这里拿到的就是填充好的联系人列表，在这里处理你的业务逻辑
        Log.i("Test", "加载完成的联系人：" + contacts.toString());
        // 比如更新UI、保存到全局变量等
    }

    @Override
    public void onLoadFailed(DatabaseError error) {
        // 处理加载失败的情况，比如给用户提示
        Log.e("Test", "加载联系人失败：" + error.getMessage());
    }
});

额外优化建议 #

• 别用assert做非空判断：默认情况下Android会关闭断言功能，这会导致user != null的检查失效，换成if (user != null)更安全。
• 用LiveData（Jetpack组件）更优雅：如果你的项目用了Jetpack，可以用LiveData来观察数据变化，这是更现代化的异步处理方式：

  public LiveData<List<String>> getContactsFromFirebase(){
      MutableLiveData<List<String>> contactsLiveData = new MutableLiveData<>();
      FirebaseDatabase.getInstance().getReference().child("Users")
              .addListenerForSingleValueEvent(new ValueEventListener() {
                  @Override
                  public void onDataChange(DataSnapshot dataSnapshot) {
                      List<String> contacts = new ArrayList<>();
                      for (DataSnapshot snapshot : dataSnapshot.getChildren()) {
                          Users user = snapshot.getValue(Users.class);
                          if (user != null) {
                              contacts.add(user.getPhone_number());
                          }
                      }
                      contactsLiveData.setValue(contacts);
                  }
  
                  @Override
                  public void onCancelled(DatabaseError databaseError) {
                      // 这里可以处理错误，比如发送错误通知
                  }
              });
      return contactsLiveData;
  }
  

  然后在Activity/Fragment里观察LiveData：

  getContactsFromFirebase().observe(this, contacts -> {
      // 数据更新时会自动回调这里
      if (contacts != null) {
          Log.i("Test", "联系人列表：" + contacts.toString());
          // 在这里更新UI或者处理业务
      }
  });
  

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