Go:一个可能导致锁失效的坑

先看代码：

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函数的时候会拷贝一份，就不存和主线程抢占的问题了。

至此，问题解决！

 

　　

转载于:https://www.cnblogs.com/Miracle-boy/p/10058826.html