Firebase Firestore文档ID获取及RecyclerView心愿单删除失败排查

你的删除功能失效主要有三个核心问题，逐个解决就能恢复正常：

1. Document ID 完全不匹配 #

添加数据时，你用用户的uid作为Firestore文档ID：

firebaseFirestore.collection("wishlistDetails").document(uid).set(wishMap)

但删除时却用了列表项的getID()作为文档ID：

firebaseFirestore.collection("wishlistDetails").document(datalist.get(position).getID())

这两个操作指向Firestore里完全不同的文档，自然删不掉之前添加的数据。

2. 位置变量使用错误 #

你在方法开头已经通过holder.getBindingAdapterPosition()获取了当前列表项的正确位置id，但删除时却用了未定义的position变量，这会导致索引错误，甚至取到完全无关的数据ID。

3. 数据存储逻辑矛盾 #

如果你的设计是每个用户对应一个心愿清单文档（用uid作为文档ID），直接调用delete()会删除整个用户的心愿清单，而非单个项；如果是每个心愿项对应独立文档，那添加时就应该用项的ID作为文档ID，而非用户uid。

修复后的代码示例 #

情况1：每个心愿项是独立文档（推荐方案） #

这种结构更灵活，适合单独增删项：

holder.wishlist.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
    @Override
    public void onCheckedChanged(CompoundButton compoundButton, boolean isChecked) {
        int currentPosition = holder.getBindingAdapterPosition();
        // 避免无效位置
        if (currentPosition == RecyclerView.NO_POSITION) return;
        
        String itemId = datalist.get(currentPosition).getID();

        if (isChecked) {
            // 添加：用项ID作为文档ID，同时在wishMap里存入uid关联用户
            firebaseFirestore.collection("wishlistDetails")
                    .document(itemId)
                    .set(wishMap)
                    .addOnSuccessListener(unused -> 
                        Toast.makeText(context, "已添加到心愿单", Toast.LENGTH_SHORT).show()
                    )
                    .addOnFailureListener(e -> 
                        Toast.makeText(context, "添加失败：" + e.getMessage(), Toast.LENGTH_SHORT).show()
                    );
        } else {
            // 删除：用同一个项ID定位文档
            firebaseFirestore.collection("wishlistDetails")
                    .document(itemId)
                    .delete()
                    .addOnSuccessListener(unused -> 
                        Toast.makeText(context, "已从心愿单移除", Toast.LENGTH_SHORT).show()
                    )
                    .addOnFailureListener(e -> 
                        Toast.makeText(context, "移除失败：" + e.getMessage(), Toast.LENGTH_SHORT).show()
                    );
        }
    }
});

情况2：每个用户对应一个心愿清单文档 #

如果坚持用用户uid作为文档ID，需在文档里用数组存储多个心愿项，删除时更新文档移除对应项：

holder.wishlist.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
    @Override
    public void onCheckedChanged(CompoundButton compoundButton, boolean isChecked) {
        int currentPosition = holder.getBindingAdapterPosition();
        if (currentPosition == RecyclerView.NO_POSITION) return;
        
        String itemId = datalist.get(currentPosition).getID();

        if (isChecked) {
            // 添加：将项ID加入用户文档的wishlist数组
            firebaseFirestore.collection("wishlistDetails")
                    .document(uid)
                    .update("wishlist", FieldValue.arrayUnion(itemId))
                    .addOnSuccessListener(unused -> 
                        Toast.makeText(context, "已添加到心愿单", Toast.LENGTH_SHORT).show()
                    );
        } else {
            // 删除：从用户文档的wishlist数组移除项ID
            firebaseFirestore.collection("wishlistDetails")
                    .document(uid)
                    .update("wishlist", FieldValue.arrayRemove(itemId))
                    .addOnSuccessListener(unused -> 
                        Toast.makeText(context, "已从心愿单移除", Toast.LENGTH_SHORT).show()
                    )
                    .addOnFailureListener(e -> 
                        Toast.makeText(context, "移除失败：" + e.getMessage(), Toast.LENGTH_SHORT).show()
                    );
        }
    }
});

额外注意事项 #

• 处理RecyclerView复用问题：在onBindViewHolder里要根据Firestore的实际数据状态，正确设置心形按钮的checked状态，避免滚动后状态错乱。
• 权限检查：确保Firestore安全规则允许当前用户删除对应文档，比如验证用户uid和文档内的uid匹配。

内容的提问来源于stack exchange，提问作者Vivek M Fauzdar