Question

为什么要杀掉一个简单的“ClientList.Remove（entry）”来自其他客户端的所有连接？

我有一个非常简单的Go TCP服务器，它可以进行连接处理和登录处理。之后，如果创建一个客户端并使用TCP客户端启动GO例程。

newClient：=＆amp; Client {“”，“”，login.LoginToken，conn} go ClientReader（newClient） ClientList.PushBack（* newClient）

Go例程读取所有传入数据。当Connection有超时或网络更改时（客户端获取新IP）它从客户列表中删除了客户端。

但是当它从列表中删除客户端....所有其他客户端连接都已死？在循环中找到合适的客户端并将其删除。

看看removeloop：

例程：

func ClientReader(client *Client) {
    buffer := make([]byte, 2048)
    for {
        bytesRead, error := client.Conn.Read(buffer)
    if error != nil {
        Log(error)
        break
    }

    var m Message
    err := json.Unmarshal([]byte(buffer[0:bytesRead]), &m)
    if err != nil {
        Log(err)
    } else {

        switch m.Cmd {
        case "Message":

        case "Ping":
            Log("Ping from: ", client.Name, " on ", client.Conn.RemoteAddr())
            client.Conn.SetDeadline(time.Now().Add(25 * time.Second))
            pong := []byte(`{"PONG":"..."}` + "\r\n")
            client.Conn.Write(pong)
            Log("PONG: " + time.Now().Format(time.RFC850))
            Log("User Online: " + strconv.Itoa(ClientList.Len()))
            Log("Goroutines: " + strconv.Itoa(runtime.NumGoroutine()))

        default:
            Log("Not supported Command: ", m.Cmd)
            clienterror := []byte(`{"Err":"Command not supported"}` + "\r\n")
            client.Conn.Write(clienterror)
        }
        for i := 0; i < 2048; i++ {
            buffer[i] = 0x00
        }
    }

}

RemoveLoop:
    for entry := ClientList.Front(); entry != nil; entry = entry.Next() {
        listclient := entry.Value.(Client)
        if client.Conn.RemoteAddr() == listclient.Conn.RemoteAddr() {
        ClientList.Remove(entry)
        Log("## SEARCH: ", client.Name, client.Conn.RemoteAddr())
        Log("## FOUND: ", listclient.Name,listclient.Conn.RemoteAddr())
    Log("## REMOVED: ", entry.Value)
    break RemoveLoop
    }
}
Log("Exit Client Reader Routine ", client.Name, " on ", client.Conn.RemoteAddr())

}

Answer 1

目前您的代码：

不是线程安全的
使用缓冲区而不是buffer object（更容易出错）

list

看起来正在使用（链接）map。

请修复这些内容并从那里重新开始调试。

P.S。您可能还想将ClientReader更改为Client的方法而不是函数？

看看这个版本（http://play.golang.org/p/MDZlFSStiN）：

func ClientReader(client *Client) {
buffer := new(bytes.Buffer)
for {
    _, err := buffer.ReadFrom(client.Conn)
    if err != nil {
        log.Println(err)
        break
    }
    var m Message
    if err = json.Unmarshal(buffer.Bytes(), &m); err != nil {
        log.Println(err)
    } else {
        switch m.Cmd {
        case "Message":
        case "Ping":
            log.Printf("Ping from: %s on %s\n", client.Name, client.Conn.RemoteAddr())
            client.Conn.SetDeadline(time.Now().Add(25 * time.Second))
            client.Conn.Write([]byte("{\"PONG\":\"...\"}\r\n"))
            log.Printf("PONG: %s\n", time.Now().Format(time.RFC850))
            clientListLock.RLock()
            log.Printf("User Online: %s\n", strconv.Itoa(len(clientList)))
            clientListLock.RUnlock()
            log.Printf("Goroutines: %s\n", strconv.Itoa(runtime.NumGoroutine()))
        default:
            log.Printf("Not supported Command: %s\n", m.Cmd)
            client.Conn.Write([]byte("{\"Err\":\"Command not supported\"}\r\n"))
        }
        buffer.Truncate(0)
    }
}
clientListLock.Lock()
delete(clientList, client.Conn.RemoteAddr().String())
clientListLock.Unlock()
log.Printf("Exit Client Reader Routine %s on %s\n", client.Name, client.Conn.RemoteAddr())
}

Answer 2

感谢您的快速回答:) 我已经提出你的建议（线程安全的clientList），但我的问题仍然存在。

经过测试和重写后，我在服务器日志中看到了一个非常疯狂的事情。

在ServerStart之后它有4个Goroutines ....没关系。（Golang实习生常规？）

客户端1已登录...并且Goroutine 5已启动....正常（第一个ClientReader）客户端2登录...并且Goroutine 6已启动....正常（第二个ClientReader）

客户端2是MobilePhone，i在移动和WLAN之间随机切换。

服务器检测到电话的IP更改...没问题。 MobilePhone使用新IP地址登录，服务器在ConnectionTimout ...之后终止旧连接。

而且，从MobilePhone的2-n ping后，服务器将Goroutines从6增加到7，电话丢失了连接？

Goroutine不是Clientreader（它记录每个START和EXIT） Clientreader是我代码中唯一的Goroutine。

GOLANG在做什么，为什么有一个Goroutine 7号？

从列表中删除客户端会终止其他连接

2 个答案: