for _, v := range s语句将s的元素赋值给局部变量v。值被复制。没有引用语义学
go vet命令警告您互斥体字段已被复制。互斥体一旦使用就不应该被复制。
这不是incrementResource中唯一的问题。函数修改的是局部变量中的client的副本，而不是切片中的client。因为局部变量在从函数返回时被丢弃，所以函数incrementResource没有效果。运行程序https://go.dev/play/p/kL9GZSL6d2j以查看问题的演示。
通过指针访问slice元素修复了incrementResource中的bug。

func (ac *ActiveClients) addToResource(clientId int) {
    for i := range ac.clients {
        existingClient := &ac.clients[i] // existingClient is ptr to slice element
        if existingClient.id == clientId {
            existingClient.Lock()
            defer existingClient.Unlock()
            existingClient.resource.increment()
            fmt.Println("data in addToResource: ", existingClient.resource.data)
        }
    }
}

字符串
下面是修复的程序：https://go.dev/play/p/wMSUOjoTauB的
上述更改修复了问题中的问题，但这不是应用程序的唯一问题。方法ActiveClients.add中对append的调用在增长切片时复制Client的值。这种复制会在切片元素上造成数据竞争，并且违反了互斥体一旦使用就不应复制的规则。
要修复所有问题，请使用*Client切片，而不是Client。当我们在做这件事的时候，利用append对nil片的处理。

type ActiveClients struct {
    clients []*Client
    mu      sync.RWMutex
}

func (ac *ActiveClients) add(client *Client) {
    ac.mu.Lock()
    defer ac.mu.Unlock()
    ac.clients = append(ac.clients, client)
}

型
以下是最后的程序：https://go.dev/play/p/miNK90ZDNCu的

在写问题的时候找到了答案。解决方案是使用一片指针而不是一片结构体-使用clients []*Client而不是clients []Client：

type ActiveClients struct {
    clients []*Client
    mu      sync.RWMutex
}

func (ac *ActiveClients) add(client *Client) {
    ac.mu.Lock()
    defer ac.mu.Unlock()
    if ac.clients == nil {
        ac.clients = make([]*Client, 0)
    }
    ac.clients = append(ac.clients, client)
}

func (ac *ActiveClients) incrementResource(clientId int) {
    for _, existingClient := range ac.clients {
        if existingClient.id == clientId {
            existingClient.mu.Lock()
            defer existingClient.mu.Unlock()
            existingClient.resource.increment()
        }
    }
}

字符串
完整的工作示例是here。对于有经验的Go开发人员来说，这可能是显而易见的，但我无法找到这个特定案例的答案，所以希望这个条目能帮助其他学习者。（解决方案与代码中错误的位置是分开的，因此如果没有足够的内存模型经验，乍一看并不明显。