Swift聊天应用UITableView用户在线状态显示异常问题排查

1. Cell复用+异步请求竞态冲突 #

UITableView会复用滚出屏幕的cell，你在configureHomeFeedCell里发起的在线状态查询是异步操作。当cell被复用时，之前的请求可能还未完成，等请求返回后会覆盖当前cell已配置好的状态，导致显示错误。

2. 监听未清理，重复绑定触发混乱 #

原代码中每个cell都会给用户在线状态添加snapshotListener，但cell被复用时没有移除旧的监听。这会导致同一个cell绑定多个用户的状态监听，任何一个用户的状态变化都会修改这个cell的在线标识，出现串号问题。

3. 数据查询路径与存储路径不匹配 #

原checkUserOnlineStatus查询的是用户文档下的IS_ONLINE子集合，但更新在线状态的逻辑是直接把USER_IS_ONLINE字段存在用户文档本身。查询路径和数据存储路径不一致，导致初始状态获取错误。

你给出的更新方案已经解决了大部分核心问题，关键修正点如下：

1. 业务逻辑剥离，统一状态管理 #

把在线状态检测移到Service类，避免cell中混杂业务逻辑，同时保证查询逻辑和更新逻辑的一致性。

2. 修正查询路径，匹配存储逻辑 #

现在直接监听用户文档本身，和isUserOnline的更新逻辑对应，能正确获取USER_IS_ONLINE字段的值。

补充优化建议（规避潜在问题） #

当前方案虽能解决显示错误，但还需处理监听清理问题：

• 在Cell的prepareForReuse方法中移除旧监听，避免内存泄漏和重复触发：

class HomeFeedCell: UITableViewCell {
    private var listenerRegistration: ListenerRegistration?

    func configureHomeFeedCell(member: Member) {
        // 先移除旧监听
        listenerRegistration?.remove()
        
        profileImage.loadImage(with: member.imageURL)
        profileName.text = "\(member.name), \(member.age)"
        
        listenerRegistration = Service.checkUserOnlineStatus(with: member.documentId) { isOnline in
            self.onlineViewStatus.backgroundColor = isOnline ? .green : .red
        }
    }
    
    override func prepareForReuse() {
        super.prepareForReuse()
        listenerRegistration?.remove()
        listenerRegistration = nil
        // 重置在线状态视图，避免复用残留
        onlineViewStatus.backgroundColor = .red
    }
}

• 修改Service的checkUserOnlineStatus，返回监听注册对象用于后续移除：

static func checkUserOnlineStatus(with userId: String, completion: @escaping(Bool) -> Void) -> ListenerRegistration {
    let docRef = USERS_COLLECTION.document(userId)
    
    // 先获取初始状态
    docRef.getDocument { (document, error) in
        if let document = document, document.exists {
            let isOnline = document.get(USER_IS_ONLINE) as? Bool ?? false
            completion(isOnline)
        }
    }
    
    // 返回监听注册对象
    return docRef.addSnapshotListener { (document, error) in
        if let document = document, document.exists {
            let isOnline = document.get(USER_IS_ONLINE) as? Bool ?? false
            completion(isOnline)
        }
    }
}

其他细节优化 #

原isUserOnline方法里的getDocument判断多余，直接调用updateData即可（不存在的文档会自动创建）：

static func isUserOnline(bool: Bool) {
    guard let currentUID = Auth.auth().currentUser?.uid else { return }
    let dictionary = [
        USER_IS_ONLINE: bool,
        USER_LASTEST_ONLINE: Date().timeIntervalSince1970
    ]
    USERS_COLLECTION.document(currentUID).updateData(dictionary) { error in
        if let error = error {
            print("更新在线状态失败：\(error.localizedDescription)")
        }
    }
}

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