如果我的猜测是正确的话,我的结构有问题,指针也可能有问题。
这个结构有一些字段和一个包含 slice 的字段:
type Bot struct {
// ...
connlist []Connection
}
这个连接
看起来像这样:
type Connection struct {
conn net.Conn
messages int32
channels []string
joins int32
connactive bool
}
我的问题是将 connactive
的值更改为 true
。
Bot
有一个监听连接的方法:
func (bot *Bot) ListenToConnection(connection Connection) {
reader := bufio.NewReader(connection.conn)
tp := textproto.NewReader(reader)
for {
line, err := tp.ReadLine()
if err != nil {
log.Printf("Error reading from chat connection: %s", err)
break // break loop on errors
}
if strings.Contains(line, "tmi.twitch.tv 001") {
connection.activateConn()
}
if strings.Contains(line, "PING ") {
fmt.Fprintf(connection.conn, "PONG tmi.twitch.tv\r\n")
}
fmt.Fprintf(bot.inconn, line+"\r\n")
}
}
connection.activeConn()
是无法正常工作的部分,该方法如下所示:
func (connection *Connection) activateConn() {
connection.connactive = true
}
这实际上被执行了,所以这不是连接没有得到响应之类的问题。
但是如果我稍后尝试在 Bot
的方法中循环它,connactive
由于某种原因总是 false
(这是默认)。
for i := 0; i < len(bot.connlist); i++ {
log.Println(bot.connlist[i].connactive)
}
我认为我正在使用原始连接的副本左右,而不是具有 connactive = true
的更改连接。
有什么想法吗?感谢您的帮助。
最佳答案
你的 ListenToConnection()
方法有一个参数:connection Connection
.
当您调用此 ListenToConnection()
时方法(你没有发布这段代码),你传递了一个值 Connection
. Go 中的所有内容都是按值传递的,因此将对传递的值进行复制。里面ListenToConnection()
您使用此副本进行操作。你称它为activateConn()
方法,但该方法(具有指针接收器)将接收此副本(局部变量)的地址。
解决方法很简单,更改ListenToConnection()
的参数即可成为一个指针:
func (bot *Bot) ListenToConnection(connection *Connection) {
// ...
}
用 Bot.connlist
中的值调用它:
bot.ListenToConnection(&bot.connlist[0])
A for
用 conlist
的每个元素循环调用它:
for i := range bot.connlist {
bot.ListenToConnection(&bot.conlist[i])
}
注意!我特意使用了 for ... range
它只使用索引而不使用值。使用 for ... range
使用索引和值,或仅使用值,您会观察到相同的问题( connactive
将保留 false
):
for _, v := range bot.connlist {
bot.ListenToConnection(&v) // BAD! v is also a copy
}
因为 v
也只是一个副本,将其地址传递给bot.ListenToConnection()
, 那只会指向副本而不是 connlist
中的元素 slice 。
关于pointers - Golang 在另一个结构的方法中更改一个结构的值,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/36517373/