Firebase Database子数组操作异常：重复添加Room数据丢失问题

问题重现 #

现有以下Android Java代码，用于将自定义Room对象添加到Firebase数据库的rooms数组中：

public void addRoom(Room room) {
    this.addRoomToList(room);
    DatabaseReference dbRef = FirebaseDatabase.getInstance().getReference("/users/" + FirebaseAuth.getInstance().getUid() + "/rooms");

    dbRef.addValueEventListener(new ValueEventListener() {
        @Override
        public void onDataChange(@NonNull DataSnapshot dataSnapshot) {
            ArrayList<HashMap<String, Object>> data = (ArrayList<HashMap<String, Object>>)dataSnapshot.getValue();

            if (data == null){
                data = new ArrayList<>();
            }

            data.add(room.getHashMapFromRoom());
            // Save the updated subarray to Firebase
            dbRef.setValue(data);

            dbRef.removeEventListener(this);
        }

        @Override
        public void onCancelled(@NonNull DatabaseError databaseError) {
            Log.w(TAG, "onCancelled", databaseError.toException());
            dbRef.removeEventListener(this);
        }
    });
}

首次调用可正常上传数据，但第二次调用时，dataSnapshot.getValue()返回null，导致创建新数组覆盖原有数据，丢失之前添加的Room对象。已确认数据库读写权限正常，父层级可读取，但rooms节点未显示数组结构。

问题原因 #

Firebase Realtime Database对数组的存储和解析有特殊处理：

• 第一次用setValue(ArrayList)写入时，Firebase会以连续整数索引的对象结构存储（类似{"0": {...}, "1": {...}}），此时可被解析为ArrayList。
• 后续操作或并发场景下，存储结构可能变化，Firebase会将其解析为HashMap<String, Object>而非ArrayList，强制类型转换会失败得到null，进而创建新数组覆盖原有数据。
• 使用addValueEventListener后立即移除监听，可能存在时序问题，导致读取到的不是最新数据。

解决方案 #

方案1：使用Firebase的push()直接添加元素（推荐） #

无需读取整个数组，直接通过push()生成唯一键并添加元素，避免类型转换问题和并发覆盖风险：

public void addRoom(Room room) {
    this.addRoomToList(room);
    DatabaseReference dbRef = FirebaseDatabase.getInstance()
            .getReference("/users/" + FirebaseAuth.getInstance().getUid() + "/rooms");
    
    // 生成唯一子节点并写入数据
    dbRef.push().setValue(room.getHashMapFromRoom())
            .addOnSuccessListener(aVoid -> {
                // 写入成功逻辑
            })
            .addOnFailureListener(e -> {
                Log.w(TAG, "addRoom failed", e);
            });
}

这种方式的优势：

• 无需读取整个数组，减少网络请求和资源消耗
• 自动生成唯一键，避免并发写入时的冲突
• 不存在类型转换问题，数据结构更稳定

方案2：兼容ArrayList和HashMap的读取逻辑 #

如果必须保持数组结构，需要在读取时判断数据类型，将HashMap转换为ArrayList：

public void addRoom(Room room) {
    this.addRoomToList(room);
    DatabaseReference dbRef = FirebaseDatabase.getInstance()
            .getReference("/users/" + FirebaseAuth.getInstance().getUid() + "/rooms");

    dbRef.addListenerForSingleValueEvent(new ValueEventListener() {
        @Override
        public void onDataChange(@NonNull DataSnapshot dataSnapshot) {
            ArrayList<HashMap<String, Object>> data = new ArrayList<>();
            
            // 处理不同类型的返回值
            if (dataSnapshot.getValue() instanceof ArrayList) {
                data = (ArrayList<HashMap<String, Object>>) dataSnapshot.getValue();
            } else if (dataSnapshot.getValue() instanceof HashMap) {
                HashMap<String, Object> map = (HashMap<String, Object>) dataSnapshot.getValue();
                // 将HashMap的value按索引顺序加入ArrayList
                for (Object value : map.values()) {
                    if (value instanceof HashMap) {
                        data.add((HashMap<String, Object>) value);
                    }
                }
            }

            data.add(room.getHashMapFromRoom());
            dbRef.setValue(data);
        }

        @Override
        public void onCancelled(@NonNull DatabaseError databaseError) {
            Log.w(TAG, "onCancelled", databaseError.toException());
        }
    });
}

注意：这里改用addListenerForSingleValueEvent，它只会触发一次读取，比addValueEventListener更适合这种一次性读取更新的场景，避免不必要的监听和时序问题。

额外建议 #

• 尽量避免在Firebase中使用数组结构，推荐使用push()生成的键值对结构，更符合NoSQL数据库的设计理念，也能避免数组解析的各种问题。
• 操作数据库时优先使用addOnSuccessListener和addOnFailureListener来处理结果，便于捕获异常和处理成功逻辑。

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