Firebase数据库for循环内查询返回空列表问题求助

你猜得一点没错！问题根源就是Firebase的异步特性——你的GetInfoUsers()方法还没等Firebase把所有用户数据拉取完成，就早早返回了result，这时候list还没被填充，自然是空的。

Firebase的addListenerForSingleValueEvent是异步执行的，主线程不会停下来等它完成数据读取，而是直接往下走执行return result，而内部的回调逻辑还在后台慢慢跑，所以返回的结果肯定是空的。

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

Java里不能直接让异步操作同步返回结果（会阻塞主线程，Android甚至会触发ANR），所以最好的方式是用回调来处理数据加载完成后的逻辑。

第一步：定义回调接口 #

先创建一个接口，用来通知我们数据加载完成或者失败：

public interface OnUsersLoadedListener {
    void onUsersLoaded(List<InfoUser> userList);
    void onLoadFailed(DatabaseError error);
}

第二步：重构你的数据获取方法 #

把原来的GetInfoUsers()改成接收回调参数，等所有用户数据都加载完成后再触发回调：

private void GetInfoUsers(OnUsersLoadedListener listener) {
    DatabaseReference mDatabase = FirebaseDatabase.getInstance().getReference().child("Users");
    DatabaseReference postsRef = FirebaseDatabase.getInstance().getReference().child("Posts");
    List<InfoUser> userList = new ArrayList<>();
    // 用原子整数统计待加载的用户数量，避免提前触发回调
    AtomicInteger pendingLoads = new AtomicInteger(0);

    postsRef.addListenerForSingleValueEvent(new ValueEventListener() {
        @Override
        public void onDataChange(DataSnapshot postsSnapshot) {
            if (!postsSnapshot.exists()) {
                // 如果没有帖子，直接返回空列表
                listener.onUsersLoaded(userList);
                return;
            }

            // 初始化待加载的用户数量
            pendingLoads.set((int) postsSnapshot.getChildrenCount());

            // 遍历所有帖子，获取对应用户信息
            for (DataSnapshot postSnapshot : postsSnapshot.getChildren()) {
                String userId = postSnapshot.child("user_id").getValue(String.class);
                if (userId == null) {
                    // 如果帖子里没有user_id，计数减一
                    pendingLoads.decrementAndGet();
                    continue;
                }

                mDatabase.child(userId).addListenerForSingleValueEvent(new ValueEventListener() {
                    @Override
                    public void onDataChange(DataSnapshot userSnapshot) {
                        InfoUser user = userSnapshot.getValue(InfoUser.class);
                        if (user != null) {
                            userList.add(user);
                        }
                        // 每加载完一个用户，计数减一；当计数为0时，说明所有用户都加载完成
                        if (pendingLoads.decrementAndGet() == 0) {
                            listener.onUsersLoaded(userList);
                        }
                    }

                    @Override
                    public void onCancelled(DatabaseError databaseError) {
                        listener.onLoadFailed(databaseError);
                    }
                });
            }
        }

        @Override
        public void onCancelled(DatabaseError databaseError) {
            listener.onLoadFailed(databaseError);
        }
    });
}

第三步：调用重构后的方法 #

现在调用这个方法时，通过回调来处理最终的用户列表：

GetInfoUsers(new OnUsersLoadedListener() {
    @Override
    public void onUsersLoaded(List<InfoUser> loadedUsers) {
        // 这里拿到填充好的用户列表，做你需要的操作，比如更新UI或者封装到PostListResult里
        PostListResult result = new PostListResult();
        result.setUsers(loadedUsers);
        // 继续你的后续业务逻辑...
    }

    @Override
    public void onLoadFailed(DatabaseError error) {
        LogUtil.logError(TAG, "加载用户数据失败", new Exception(error.getMessage()));
    }
});

额外优化小建议 #

• 尽量避免嵌套的异步调用，你可以考虑调整数据结构（比如把用户名直接存在Posts节点里，这样就不用跨节点查询了，当然这要根据你的业务需求来决定）。
• 使用DataSnapshot的getChildren()遍历方式，比把数据转成Map更安全，也更符合Firebase的最佳实践。

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