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

至此,问题解决!

 

  

posted @ 2018-12-03 15:37  __Miracle  阅读(485)  评论(0编辑  收藏  举报