使用 sync.Once 解决 Go 并发场景下的重复下线广播问题

使用 sync.Once 解决 Go 并发场景下的重复下线广播问题

业务背景

我们正在开发一个简单的基于 TCP 的聊天室服务端。在这个系统中,每个连接上来的客户端会被抽象为一个 User 对象。为了实现并发处理,服务端会为每一个 User 分配专门的 goroutine(协程)用来阻塞读取客户端发送的消息,并将消息广播给全局维护的其他在线用户。

核心业务流程:

  • 上线 :将 User 加入服务端的全局 OnlineMap 中,并广播"✅已上线!"给所有人。
  • 消息处理 :后台 ManagerMessage 协程循环调用 user.Conn.Read() 接收消息,如果是指令则处理,否则广播。
  • 下线 :当连接意外断开或用户主动输入 exit 指令时,将 User 从 OnlineMap 中移除,断开底层连接并广播"❌已下线!"。

并发 Bug 显现

在测试 exit 命令退出功能时,我们发现了一个奇怪的现象。其他在线用户会收到两遍完全相同的下线提醒:

复制代码
 [127.0.0.1:62215]127.0.0.1:62215:❌已下线!
 [127.0.0.1:62215]127.0.0.1:62215:❌已下线!

原因排查与相关代码

导致这个 Bug 的根本原因是用户注销与清理的逻辑(Logout)被并发触发了两次。看下我们的核心消息处理代码:

复制代码
 func (s *Server) ManagerMessage(user *User) {
     buf := make([]byte, 4096)
     for {
         n, err := user.Conn.Read(buf)
         if n == 0 || err != nil {
             // 【触发点 2】: 兜底清理。由于读到 EOF 或断开报错,触发注销
             user.Logout()
             return
         }
 ​
         rawMsg := string(buf[:n])
         if rawMsg == "exit" {
             // 【触发点 1】: 主动退出。当接收到了退出指令时,触发注销
             user.Logout()
             // (协程后续会因 Socket 断开而在上面 Read() 处终止)
         } else {
              // 处理正常消息...
         }
     }
 }

原先的注销逻辑极其简单:

复制代码
 func (u *User) Logout() {
     u.Offline() // 从在线列表中移除,并广播"❌已下线!"
     u.Close()   // 释放 Socket 套接字等资源
 }

Bug 复现路径:

  1. 用户输入 exit。
  2. 协程命中 rawMsg == "exit",开始执行 【触发点 1】 的主动 Logout(),广播第一次"已下线",并调用了底层的 Close() 关闭连接。
  3. 随后协程进入下一次 for 循环,调用 user.Conn.Read() 等待下一条消息。
  4. 因为底层 Socket 刚才就被关闭了,Read() 瞬间返回挂起并返回错误(err != nil)。
  5. 协程命中上方错误兜底逻辑,执行了 【触发点 2】 的被动 Logout(),广播了第二次"已下线"!

解决方案:引入 sync.Once

在服务端并发编程中,针对一个对象的资源释放和状态变更(特别是向外的下线广播),必须保证操作的幂等性------即无论清理方法被调用多少次,核心下线逻辑只能执行一次。

Go 标准库提供的 sync.Once 机制完美契合这一场景。我们在 User 结构体中增加一个专门用于控制注销逻辑的自带锁:

复制代码
 import (
     "net"
     "sync"
 )
 ​
 type User struct {
     Name       string
     Conn       net.Conn
     // 其他业务字段...
 ​
     // 加入 sync.Once 锁
     logoutOnce sync.Once
 }
 ​
 // 改造后的 Logout
 func (u *User) Logout() {
     // Do 内部传入的闭包函数,在对象生命周期内绝对只会被执行一次
     u.logoutOnce.Do(func() {
         u.Offline()
         u.Close()
     })
 }

总结

引入 sync.Once.Do 之后,无论是先解析到了 exit 从而主动触发退出,还是因网络异常直接引发 Read() 报错进行兜底,只有"最先到达"的那次 Logout() 会真实调用下线广播。后续引发的重复清理调用都会被 sync.Once 机制拦截并忽略。

这种做法十分优雅地解耦了 "不确定的多个业务触发条件" 和 "唯一的资源清理终态" ,它不仅修复了重复消息的体验 Bug,还彻底消除了对底层通道重复 close() 可能引发 Panic 的严重隐患。

相关推荐
奈葵6 分钟前
Python
开发语言·python
物联网软硬件开发-轨物科技13 分钟前
【轨物洞见】什么是产品碳足迹(PCF)?ISO 14067 计算方法与应用场景深度解读
大数据·网络·人工智能
Nebula_g20 分钟前
JavaSE项目实践:红包雨-线程池/线程
java·开发语言
外收内放21 分钟前
Python基础语法练习题(56优化版补充)
开发语言·python
小羊没烦恼!21 分钟前
jQuery1.5的改进细节
java·服务器·开发语言·前端·c#
程序员老陆1 小时前
C++ 未定义行为全景:对象模型、优化边界与工程防御
开发语言·c++·程序设计
嵌入式er1 小时前
影石系统组 一二面OC面经
java·开发语言
徐小黑ACG1 小时前
Golang 基础06 接口interface
开发语言·后端·golang
半杯咖啡半行码1 小时前
Qt 开发基础:版本演进、常用工具与多平台安装
开发语言·qt
下页、再停留2 小时前
【C#桌面客户端系列学习-8】批量导出数据为csv格式
开发语言·c#·visual studio