先看代码:
package main import( "sync" ) var hclock sync.RWMutex func main() { a := make(map[int]string) a[0] = "z" a[1] = "a" a[2] = "b" a[3] = "c" for i:=0; i< 2; i++{ // go write(a,i) go func(b map[int]string){ hclock.Lock() b[i] = "aaa" hclock.Unlock() }(a) } }
如果这样子是会报错的
报错如下:
说在抢占数据
这就很奇怪了 明明加上了的锁的?
经过百般折磨,终于弄清了问题所在
!!!划重点!
并不是锁失效,出现这个的原因是,for里面的i变量每次循环都是同一个,所以其实并不是两个routine在抢资源,而是因为routine里面对i进行了操作
然后主线程又要继续循环,是主线程和子线程在抢i
为了验证这个想法,我把代码改成如下
package main import( "sync" ) var hclock sync.RWMutex // func write(b map[int]string,i int){ // hclock.Lock() // b[i] = "aaaaaa" // hclock.Unlock() // } func main() { a := make(map[int]string) a[0] = "z" a[1] = "a" a[2] = "b" a[3] = "c" for i:=0; i< 1; i++{ // go write(a,i) go func(b map[int]string){ hclock.Lock() b[i] = "aaa" hclock.Unlock() }(a) } }
只起一个,结果还是报错
再来一个更有说服力的版本
package main import( "sync" ) var hclock sync.RWMutex func write(b map[int]string,i int){ hclock.Lock() b[i] = "aaaaaa" hclock.Unlock() } func main() { a := make(map[int]string) a[0] = "z" a[1] = "a" a[2] = "b" a[3] = "c" for i:=0; i< 1; i++{ go write(a,i) } }
这样子就不报错了,另起一个函数,因为i是值类型,所以传递到write函数的时候会拷贝一份,就不存和主线程抢占的问题了。
至此,问题解决!