Question

此代码已简化并描述了我的问题。似乎atomic.StoreInt32不起作用，但我不确定原因。

package main
import (
    "fmt"
    "sync/atomic"
)

type slave struct {
    failed int32
}
func NewSlave() slave {
    return slave{0}
}
func (worker slave) Fail() {
    atomic.StoreInt32(&worker.failed, 1) // Here's the problem.
}
func (worker slave) IsFailed() bool {
    failed := atomic.LoadInt32(&worker.failed) == 1
    return failed
}
func (worker slave) FailureReset() {
    atomic.StoreInt32(&worker.failed, 0)
}

func main() {
    s := NewSlave()
    fmt.Println(s.IsFailed())
    s.Fail()
    fmt.Println(s.IsFailed())
    s.FailureReset()
    fmt.Println(s.IsFailed())
}

- ＆GT;输出：

false 0
false 0
false 0

- ＆gt;测试：

recolic@RECOLICPC ~/tmp> go version
go version go1.9.2 linux/amd64
recolic@RECOLICPC ~/tmp> uname -a
Linux RECOLICPC 4.13.12-1-ARCH #1 SMP PREEMPT Wed Nov 8 11:54:06 CET 2017 x86_64 GNU/Linux

我已阅读：usage golang atomic LoadInt32/StoreInt32 (64)

Answer 1

你需要让你的方法采用指针接收器。如上所述，函数在您的worker的副本上运行，因此用于调用该方法的worker不会更改。定义如下方法：

func (worker *slave) Fail() {
    // ...
}

同步/原子StoreInt32奇怪的行为

1 个答案: