Firebase Realtime Database群聊UITableView新消息错误展示问题排查

核心问题根源 #

• UITableViewCell重用未清理旧监听：你通过dequeueReusableCell获取的是可复用的cell，之前给旧cell绑定的Firebase持久化监听没有移除，旧监听触发时会把其他群组的消息更新到当前正在展示的cell上，是消息串位的核心原因。
• Firebase监听未做群组校验：你调用的observe是长期存活的监听，回调直接持有cell引用，没有校验当前cell绑定的群组和监听的群组是否一致，异步回调触发时cell很可能已经被复用去展示其他群组，自然会出现内容乱跳。
• 你点进对应群组再返回会触发viewWillAppear重新拉取群组列表并刷新全量cell，所以内容会临时恢复正常，只要有新消息触发旧监听就会再次复现问题。

修复步骤 #

1. 扩展cell存储关联属性 #

给GroupsTableViewCell新增属性，用来存储当前绑定的群组ID、Firebase监听句柄，方便复用的时候清理旧资源：

class GroupsTableViewCell: UITableViewCell {
    // 保留你原有其他属性
    var currentGroupID: String?
    var lastMsgObserverHandle: UInt?
    var unreadCountObserverHandle: UInt?
}

2. 重写cell复用前的清理方法 #

重写prepareForReuse方法，在cell被复用前移除旧监听、重置UI状态，避免旧内容残留：

override func prepareForReuse() {
    super.prepareForReuse()
    // 移除旧的Firebase监听
    if let lastHandle = lastMsgObserverHandle {
        rootRef2.removeObserver(withHandle: lastHandle)
    }
    if let unreadHandle = unreadCountObserverHandle {
        rootRef2.removeObserver(withHandle: unreadHandle)
    }
    lastMsgObserverHandle = nil
    unreadCountObserverHandle = nil
    currentGroupID = nil
    // 重置UI避免旧内容闪现
    lbl_lastmsg.text = nil
    img_small.image = nil
    img_small_width.constant = 0
    view_count.isHidden = true
    lbl_count.text = nil
}

3. 修改监听方法加群组校验 #

修改GetLastMsg和GetUnReadMsgCount，增加群组ID校验，只有当前cell绑定的群组ID和监听的群组ID一致时才更新UI，同时把监听句柄存到cell中：

func GetLastMsg(groupID: String, cell: GroupsTableViewCell) {
    cell.currentGroupID = groupID
    let userRef2 = rootRef2.child("message").child(groupID).queryLimited(toLast: 1)
    // 存储监听句柄方便后续清理
    let observerHandle = userRef2.observe(.value, with: { snapshot in
        // 首次校验：cell当前绑定的群组是否和监听群组一致
        guard cell.currentGroupID == groupID else { return }
        guard let valueDict = snapshot.value as? [String:AnyObject] else {
            print("Error")
            return
        }
        // 你原有解析message的逻辑保持不变
        // ... 省略原有解析代码 ...
        DispatchQueue.main.async {
            // 二次校验：异步回调执行时再校验一次，避免中间cell被复用
            guard cell.currentGroupID == groupID else { return }
            cell.lbl_lastmsg.text = msg_last
            // 图片、宽度约束等其他UI更新也挪到这里执行
        }
    })
    cell.lastMsgObserverHandle = observerHandle
}

同理修改GetUnReadMsgCount：

func GetUnReadMsgCount(groupID : String, cell : GroupsTableViewCell)  {
    guard cell.currentGroupID == groupID else { return }
    let ref = rootRef2.child("group_message_unread").child(current_user!).child(groupID)
    let observerHandle = ref.observe(.childAdded, with: { (snapshot) -> Void in
        guard cell.currentGroupID == groupID else { return }
        DispatchQueue.main.async {
            guard cell.currentGroupID == groupID else { return }
            cell.view_count.isHidden = snapshot.childrenCount <= 0
            cell.lbl_count.text = snapshot.childrenCount > 0 ? String(snapshot.childrenCount) : nil
        }
    })
    cell.unreadCountObserverHandle = observerHandle
}

可选优化建议 #

不要在cellForRow层面绑定监听，建议把每个群组的最新消息、未读数存在groupList对应的Groupslist模型中，在控制器层面统一监听所有群组的更新，数据变化后只刷新对应indexPath的cell即可，减少不必要的监听开销。

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