Android Firebase根据酒店ID跨节点取数据适配加载异常求助

首先得明确你遇到的核心问题：Firebase的数据库操作是异步执行的。你在循环里给每个酒店ID添加的addValueEventListener，它们的onDataChange方法会在后台线程完成数据获取后才触发，但外层代码（设置Adapter）会在循环结束后立刻执行——这时候hotels列表还没有被填充任何数据，所以界面自然显示为空。

下面给你两个可行的解决方案，你可以根据自己的需求选择：

方案一：跟踪数据加载进度，完成后再更新Adapter #

这个方法适合不能修改数据库结构的场景，核心思路是统计已经加载完成的酒店数量，当所有酒店都加载完成（包括加载失败的情况），再创建并设置Adapter：

// 获取酒店ID列表
DatabaseReference hotelIdRef = FirebaseDatabase.getInstance()
        .getReference()
        .child(getResources().getString(R.string.all_countries))
        .child(country)
        .child(city)
        .child("Hotel");

hotelIdRef.addListenerForSingleValueEvent(new ValueEventListener() {
    @Override
    public void onDataChange(@NonNull DataSnapshot dataSnapshot) {
        hotels = new ArrayList<>();
        hotelIDs = new ArrayList<>();
        
        // 先收集所有酒店ID
        for (DataSnapshot idSnapshot : dataSnapshot.getChildren()) {
            String id = idSnapshot.getKey();
            hotelIDs.add(id);
        }

        int totalHotels = hotelIDs.size();
        final int[] loadedCount = {0}; // 用数组实现内部类修改外部变量

        // 逐个加载酒店详情
        for (String hotelId : hotelIDs) {
            DatabaseReference hotelDetailRef = FirebaseDatabase.getInstance()
                    .getReference()
                    .child("Hotel")
                    .child(hotelId);

            // 用addListenerForSingleValueEvent替代addValueEventListener，避免重复监听
            hotelDetailRef.addListenerForSingleValueEvent(new ValueEventListener() {
                @Override
                public void onDataChange(@NonNull DataSnapshot dataSnapshot) {
                    Hotel hotel = dataSnapshot.getValue(Hotel.class);
                    if (hotel != null) { // 避免空数据加入列表
                        hotels.add(hotel);
                    }
                    loadedCount[0]++;
                    checkAndUpdateAdapter();
                }

                @Override
                public void onCancelled(@NonNull DatabaseError databaseError) {
                    Log.e("FirebaseError", "加载酒店详情失败: " + databaseError.getMessage());
                    loadedCount[0]++;
                    checkAndUpdateAdapter();
                }

                private void checkAndUpdateAdapter() {
                    // 当所有酒店都处理完成时更新Adapter
                    if (loadedCount[0] == totalHotels) {
                        hotelsAdapter = new HotelsAdapterUser(hotels, UserViewHotels.this, "Hotel");
                        hotelsRV.setAdapter(hotelsAdapter);
                        hotelsAdapter.notifyDataSetChanged(); // 确保列表刷新
                    }
                }
            });
        }
    }

    @Override
    public void onCancelled(@NonNull DatabaseError databaseError) {
        Log.e("iamhere", "获取酒店ID失败: " + databaseError.getMessage());
    }
});

关键优化点： #

• 使用addListenerForSingleValueEvent替代addValueEventListener：如果你只需要一次性加载数据（不需要实时同步更新），这个方法更合适，避免不必要的重复回调。
• 增加空数据判断：防止dataSnapshot.getValue(Hotel.class)返回null导致列表出现异常。
• 错误处理：即使某个酒店加载失败，也要递增计数，避免程序卡住。

方案二：优化数据库结构，避免嵌套查询 #

从性能和代码简洁性来看，这是更优的方案。如果你的业务允许修改数据库结构，建议把酒店的完整信息直接存储在城市节点下，而不是只存ID：

原结构： #

all_countries
  └── country
      └── city
          └── Hotel
              ├── hotelId1: true
              └── hotelId2: true
Hotel
  ├── hotelId1
  │   ├── name: "XX酒店"
  │   └── address: "XX地址"
  └── hotelId2
      ├── name: "YY酒店"
      └── address: "YY地址"

优化后结构： #

all_countries
  └── country
      └── city
          └── Hotel
              ├── hotelId1
              │   ├── name: "XX酒店"
              │   └── address: "XX地址"
              └── hotelId2
                  ├── name: "YY酒店"
                  └── address: "YY地址"

这样你只需要一次查询就能拿到所有酒店的完整信息，代码会简化很多：

DatabaseReference hotelRef = FirebaseDatabase.getInstance()
        .getReference()
        .child(getResources().getString(R.string.all_countries))
        .child(country)
        .child(city)
        .child("Hotel");

hotelRef.addValueEventListener(new ValueEventListener() {
    @Override
    public void onDataChange(@NonNull DataSnapshot dataSnapshot) {
        hotels = new ArrayList<>();
        for (DataSnapshot hotelSnapshot : dataSnapshot.getChildren()) {
            Hotel hotel = hotelSnapshot.getValue(Hotel.class);
            if (hotel != null) {
                hotels.add(hotel);
            }
        }
        hotelsAdapter = new HotelsAdapterUser(hotels, UserViewHotels.this, "Hotel");
        hotelsRV.setAdapter(hotelsAdapter);
    }

    @Override
    public void onCancelled(@NonNull DatabaseError databaseError) {
        Log.e("iamhere", "获取酒店数据失败: " + databaseError.getMessage());
    }
});

这种方式不仅避免了异步嵌套的问题，还减少了数据库查询次数，提升了加载效率。

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