函数按值返回锁

Question

我有以下结构

    type Groups struct {

        sync.Mutex

        Names []string

    }

和以下功能

    func NewGroups(names ...string) (Groups, error) {

        // ...

        return groups, nil

    }

当我检查语义错误时go vet，我收到以下警告：

NewGroups 按值返回 Lock：组

像go vet喊叫一样，不好。这段代码会带来什么问题？我怎样才能解决这个问题？

Answer 1

您需要将 sync.Mutex 作为指针嵌入：type Groups struct {    *sync.Mutex    Names []strng}解决您对问题的评论：在文章http://blog.golang.org/go-maps-in-action通知中，Gerrand 没有从函数返回结构，而是立即使用它，这就是为什么他不是t 使用指针。在您的情况下，您将返回它，因此您需要一个指针，以免复制 Mutex。更新：正如@JimB 指出的那样，嵌入指向 的指针可能并不谨慎sync.Mutex，最好返回指向外部结构的指针并继续将 嵌入sync.Mutex作为值。考虑一下您在特定情况下要完成的工作。

Answer 2

*Groups而是返回一个指针。嵌入互斥指针也可以，但有两个缺点，需要您格外小心：结构的零值将有一个 nil 互斥体，因此您必须每次显式初始化它func main() {    a, _ := NewGroups()    a.Lock() // panic: nil pointer dereference}func NewGroups(names ...string) (Groups, error) {    return Groups{/* whoops, mutex zero val is nil */ Names: names}, nil}分配一个结构值，或将其作为函数 arg 传递，会创建一个副本，因此您还可以复制互斥指针，然后锁定所有副本。（在某些特定情况下，这可能是一个合法的用例，但大多数时候它可能不是您想要的。）func main() {       a, _ := NewGroups()    a.Lock()    lockShared(a)    fmt.Println("done")}func NewGroups(names ...string) (Groups, error) {    return Groups{Mutex: &sync.Mutex{}, Names: names}, nil}func lockShared(g Groups) {    g.Lock() // whoops, deadlock! the mutex pointer is the same}保留原始结构并返回指针。您不必显式地初始化嵌入式互斥锁，而且很直观，互斥锁不与您的结构副本共享。func NewGroups(names ...string) (*Groups, error) {    // ...    return &Groups{}, nil}游乐场（有失败的例子）：https : //play.golang.org/p/CcdZYcrN4lm