SwiftUI环境下无法从Firebase/Firestore删除文档的问题排查

嘿，我一眼就看到你代码里的几个关键问题了，这就是为什么你删完之后视图刷新又会出现的原因，咱们一步步来修正：

核心问题梳理 #

• 没有传递要删除的文档ID：你的onDelete闭包根本没告诉ViewModel要删哪条数据，ViewModel和Repository也没有接收ID的逻辑
• 硬编码了文档ID：Repository里直接写了document("id")，这是在尝试删除一个ID就是字符串"id"的文档，而不是你列表中对应元素的真实ID

修正后的代码片段 #

1. ContentView - 传递要删除的元素ID #

把onDelete的闭包改成这样，获取选中元素的ID并传给ViewModel：

@ObservedObject var fileViewModel: FileViewModel
var body: some View {
    VStack {
        List{
            ForEach (fileViewModel.fileModel) { data in
                VStack {
                    Text(data.itemOne)
                    Text(data.itemTwo)
                }
            }.onDelete(perform: { indices in
                // 取第一个选中的元素（列表删除一般是单条）
                guard let index = indices.first else { return }
                let targetItem = fileViewModel.fileModel[index]
                self.fileViewModel.deleteItem(withId: targetItem.id)
            })
        }
    }
}

2. FileViewModel - 接收ID并转发给Repository #

修改deleteItem方法，增加ID参数，同时做非空判断：

func deleteItem(withId id: String?) {
    guard let documentId = id else {
        print("无法获取文档ID，删除失败")
        return
    }
    FileRepository.deleteItem(withId: documentId)
}

3. FileRepository - 使用真实ID删除文档 #

把硬编码的"id"改成传入的真实文档ID：

static func deleteItem(withId documentId: String) {
    db.collection("fileModel").document(documentId).delete() { err in
        if let err = err {
            print("删除文档出错: \(err)")
        } else {
            print("文档删除成功！")
        }
    }
}

额外注意事项 #

确保你的FileModel已经遵循了Identifiable协议，因为你用ForEach直接遍历数组，比如：

extension FileModel: Identifiable {}

另外，如果你是用Firestore的快照监听（比如addSnapshotListener）来更新fileViewModel.fileModel数组的话，删除文档后Firestore会自动通知数据源更新，列表就不会再显示已删除的元素了。

内容的提问来源于stack exchange，提问作者L. Puzer