hezhizheng / go-gin-chat

Golang(Gin)+websocket 的多人聊天室 | go 聊天室 | golang 聊天室
http://117.50.186.143:6005/
MIT License
381 stars 81 forks source link

go_ws/server.go #9

Open anxiaowen opened 10 months ago

anxiaowen commented 10 months ago

read方法里面修改全局变量clientMsg,难道不会有并发问题?

hezhizheng commented 10 months ago

read方法里面修改全局变量clientMsg,难道不会有并发问题?

尝试修复了

anxiaowen commented 9 months ago

你这个明显不对,消息不能用全局变量存,不然并发上去肯定会出错.

anxiaowen commented 9 months ago

你不应该使用全局的变量,应该使用局部变量,然后通过方法传递.或者使用结构体方法的方式

hezhizheng commented 9 months ago

你不应该使用全局的变量,应该使用局部变量,然后通过方法传递.或者使用结构体方法的方式

ok 谢谢指出 有空我再验证一下

anxiaowen commented 9 months ago

server.go 不应该 使用select {} 来挂起,这样会内存泄露,我不知你为什么要注释掉


        select {
        case <-done:
            return
        }