iOS合并多Firebase快照到TableView的排序与行点击异常问题

问题场景 #

需要根据用户订阅的群组生成动态feed流，通过TableView展示，后台发起两个Firebase数据请求将结果渲染到同个TableView，当前实现存在两个问题：

• 数据排序错误：预期按时间戳userTime倒序排列为4->3->2->1，实际为随机顺序
• 点击评论行时，didSelectRowAt代理方法响应逻辑完全异常

feed流展示效果示意图

问题根因 #

• 嵌套请求逻辑错误：两个Firebase请求采用嵌套回调写法，每次第一个请求触发都会重复注册第二个请求的监听，易出现数据重复、回调多次触发问题，数据源会乱序追加
• 数据插入逻辑错误：遍历单条数据时就调用insert(at:0)插入，且在循环内反复调用reloadData()，没有在两组数据全部返回后做统一排序，插入顺序完全跟随网络返回顺序，导致排序随机
• 全局变量污染：循环内反复给全局变量ViewComments.commentIDNew赋值，最终该变量只会保留最后一条数据的id，点击行时读取这个全局值自然逻辑异常；另外类名importComment小写开头不符合Swift编码规范。

修复实现 #

1. 并行请求+统一合并排序 #

使用DispatchGroup并行发起两个群组的数据请求，等所有请求返回后统一做数据合并、排序，再刷新UI，避免嵌套回调问题：

// 数据源数组，统一管理所有评论数据
var commentList: [CommentModel] = []

func fetchCommentFeed() {
    // 拉取前先清空旧数据，避免重复展示
    commentList.removeAll()
    let group = DispatchGroup()
    // 所有要拉取的群组节点路径
    let targetPlaceRefs = [
        Database.database().reference().child("placeID/h77e24d95a5479ed7588"),
        Database.database().reference().child("placeID/vh-b83b6e4475e04e3fbaa647d23b")
    ]
    
    for ref in targetPlaceRefs {
        group.enter()
        // 如果需要实时监听可以换成observe，单次拉取用observeSingleEvent即可
        ref.queryOrdered(byChild: "userTime").queryLimited(toLast: 10).observeSingleEvent(of: .value) { snapshot in
            defer { group.leave() }
            guard let items = snapshot.children.allObjects as? [DataSnapshot] else { return }
            for item in items {
                guard let dataDict = item.value as? [String: Any] else { continue }
                // 安全解析数据，避免强制解包崩溃
                let comment = CommentModel(
                    userName: dataDict["userName"] as? String,
                    userGroup: dataDict["userGroup"] as? String,
                    userComment: dataDict["userComment"] as? String,
                    userTime: dataDict["userTime"] as? Int,
                    userLikes: dataDict["userLikes"] as? Int,
                    commentId: dataDict["commentId"] as? String
                )
                self.commentList.append(comment)
            }
        }
    }
    
    // 所有请求返回后，在主线程排序、刷新UI
    group.notify(queue: .main) {
        // 按userTime字段倒序，时间越大排序越靠前
        self.commentList.sort { $0.userTime ?? 0 > $1.userTime ?? 0 }
        self.tableView.reloadData()
    }
}

2. 规范数据模型 #

将原小写开头的类名改为Swift规范的大驼峰命名，避免语义混淆：

class CommentModel {
    var userName: String?
    var userID: String?
    var userGroup: String?
    var userComment: String?
    var userTime: Int?
    var userLikes: Int?
    var commentId: String?

    init(
        userName: String?, userGroup: String?, userComment: String?, userTime: Int?,
        userLikes: Int?, commentId: String?
    ) {
        self.userName = userName
        self.userGroup = userGroup
        self.userComment = userComment
        self.userTime = userTime
        self.userLikes = userLikes
        self.commentId = commentId
    }
}

3. 修复行点击逻辑 #

禁止使用全局变量存储当前点击的评论id，直接从数据源中读取对应indexPath的模型数据，从根源避免取值错误：

func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
    tableView.deselectRow(at: indexPath, animated: true)
    // 直接获取当前点击行对应的评论模型
    let selectedComment = commentList[indexPath.row]
    guard let targetCommentId = selectedComment.commentId else { return }
    // 后续跳转、评论交互逻辑直接使用targetCommentId即可
}

额外注意事项 #

• 如果需要实时监听节点数据变化，不要嵌套注册observe，要给每个节点单独绑定监听，任意节点返回新数据时重新执行合并、排序、刷新逻辑即可，避免重复注册监听导致内存泄漏、数据重复
• 所有UI刷新操作必须放在主线程执行，否则会出现UI刷新延迟、卡顿甚至崩溃
• 网络数据解析尽量避免强制解包（as!），防止后端数据结构变动时触发崩溃

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