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

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

业务背景

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

核心业务流程:

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

并发 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 的严重隐患。

相关推荐
basketball6168 小时前
C++ 高级编程:2. 基本线程池实现
java·开发语言·c++
chao1898448 小时前
SGM(Semi-Global Matching)立体匹配算法 — C++ 实现
开发语言·c++·算法
WiChP9 小时前
【V0.1B11】从零开始的2D游戏引擎开发之路
开发语言·游戏引擎
10岁的博客9 小时前
IOI 2018 高速公路收费(Highway)题解:二分与树的巧妙结合
开发语言·c++
自动跟随9 小时前
UWB自动跟随技术全栈解析:从定位算法到“位控一体化“
java·网络·人工智能
长和信泰光伏储能9 小时前
远离电网的底气:离网光伏系统核心原理与搭建要点
网络
不知名的老吴9 小时前
C++运算符重载的常见注意点
开发语言·c++
弹简特9 小时前
【Java项目-轻聊】07-实现主页面模块
java·开发语言
天天进步20159 小时前
Tunnelto 源码解析 #8:多路复用机制:StreamId、ActiveStreams 与并发请求生命周期
网络