混合使用指针接收器与值接收器引发数据竞态问题求助

兄弟，我太懂你现在的迷惑了——明明给Cache加了读写互斥锁，测试跑起来没问题，但数据竞争检查次次失败，而且看起来竞争还不是出在data字段上，反而像是在整个cache变量本身，这到底咋回事？

我帮你扒了一遍代码，核心问题其实出在**Get方法的接收器类型**上——你用了值接收器，而不是指针接收器！

为啥会触发数据竞争？ #

咱们拆解下场景就能明白：

1. 你的cache是一个值类型变量，启动Start goroutine时，因为Start是指针接收器，Go会自动把cache转成指针传入，所以goroutine里的update方法修改的是原cache变量的data字段（这是对cache变量的写操作）。
2. 测试主线程调用cache.Get("P-0001")时，因为Get是值接收器，Go会完整复制一份cache结构体作为方法的接收器参数——这个复制操作需要读取cache的所有字段（包括data），但这个读操作完全没被互斥锁保护！
3. 这就触发了经典的数据竞争：goroutine在写cache的data字段，主线程在复制cache（读取所有字段），两个操作没有任何同步机制，race detector自然会报警。

你可能会问：我在Get里加了读锁啊？但那把锁保护的是复制后的副本里的data字段，根本管不到原cache变量的复制操作！

怎么解决这个问题？ #

最简单也最根本的修复，就是把Get方法改成指针接收器：

func (pch *Cache) Get(reference string) *Item {
    pch.mut.RLock()
    defer pch.mut.RUnlock()
    return pch.data[reference]
}

这样改了之后：

• 调用cache.Get()时，Go会自动传入原cache的指针，不会再复制整个结构体，也就没有了那个无保护的读操作。
• Get里的读锁会和update里的写锁正确同步：读的时候不能写，写的时候不能读，彻底避免数据竞争。

另外还有两个小细节要修正：

1. 你的update方法里有个笔误：pch.data = resp应该改成pch.data = data（前面你用data, err := pch.client.GetData(ctx)获取了数据，resp是未定义变量）。
2. 测试代码里的Cache初始化有拼写错误：cache: newMockClient(false)应该是client: newMockClient(false)，plants: make(map[string]*Item)应该是data: make(map[string]*Item)，这些笔误会导致测试跑不起来。

额外的建议 #

以后写Go的并发结构体时，最好统一用指针接收器处理所有涉及状态修改或并发访问的方法，这样能避免值接收器带来的结构体复制问题，也能让互斥锁的作用范围更清晰。

内容来源于stack exchange