原因

工作的原因是我调用了搭档的一个函数,这个函数回来了一个map[string]string结构体的变量optionMap(请忽略为什么要回来map结构体,后边有机会再讲),这个函数主要是查DB取获取当时体系的space_id和pkey,回来的内容基本上如下

 // 回来
 optionMap = map[string]string{
     "space_id":"xxx",
     "pkey": "xxxx",
 }

然后我修正了这个变量,添加了


optionMap["is_base"] = 1

然后我就return出我当时的函数了,然后在同一个恳求内,但是当我再一次恳求搭档的函数时,回来给我的却是

 optionMap = map[string]string{
     "space_id":"xxx",
     "pkey": "xxxx",
     "is_base": 1,
 }

what! 怎么后边再次恳求搭档的的函数总是会多一个参数呢!!!!

经过

看了搭档写的函数我才发现,原来他在内部使用了gin上下文去做了一个缓存,大约的代码意思减少重复space根底信息的查询,存入上下文中做缓存,提高代码功率,这儿我写了一个示例大家可以看下

// 搭档的代码
func BadReturnMap(ctx *gin.Context, key string) map[string]interface{} {
	m := make(map[string]interface{})
	// 查询缓存
	value, ok := ctx.Get(key)
	if ok {
		bm, ok := value.(map[string]interface{})
		if ok {
			return bm
		}
	}
	// io查询后存入变量
	m["a"] = 1
	// 保存缓存
	fmt.Println("set cache: ")
	fmt.Println(m) // map[a:1]
	ctx.Set(key, m)
	return m
}
// 我的使用
func TestBadReturnMap() {
	fmt.Println("bad return map start")
	ctx := &gin.Context{}
	key := "cached:map_key"
	mapOpt := BadReturnMap(ctx, key)
	fmt.Printf("%pn", mapOpt) // 0xc0003a6750 指向地址
	mapOpt["b"] = 1
	value, ok := ctx.Get(key)
	fmt.Printf("%pn", mapOpt) // 0xc0003a6750 指向地址
	if ok {
		fmt.Println("get cache: ")
		fmt.Println(value.(map[string]interface{})) // map[a:1 def:1]
	} else {
		fmt.Println("unknown")
	}
	fmt.Println("bad return map end")
}

打印结果是

bad return map start
set cache: 
map[a:1]
0xc0003a6750
0xc0003a6750
get cache: 
map[a:1 b:1]
bad return map end

解释

Go语言中,map是引证类型,当将一个map赋值给另一个变量时,实际上是将它们指向同一个底层的map目标。因而,当你修正其间一个变量的map时,另一个变量也会受到影响。

当你将函数内m变量赋值给外部函数内的变量时,它们实际上指向同一个map目标。所以当你在外部函数内修正mapOpt的值时,原始的缓存也会被修正。

如图所示

好坑啊,调用了搭档写的根底代码,bug藏得还挺深!!

如何修正

当然修正方法有很多种,我这儿列举了一种便是序列化存储到缓存然后反序列化取,假如你有更好的方法可下方留言


func ReturnMap(ctx *gin.Context, key string) map[string]interface{} {
	m := make(map[string]interface{})
	value, ok := ctx.Get(key)
	if ok {
		bytes := value.([]byte)
		err := json.Unmarshal(bytes, &m)
		if err != nil {
			panic(err)
		}
		return m
	}
	// io查询后存入变量
	m["a"] = 1
	jsonBytes, err := json.Marshal(m)
	if err != nil {
		panic(err)
	}
	fmt.Println("set cache: ")
	fmt.Println(m)
	ctx.Set(key, jsonBytes)
	return m
}
func TestReturnMap() {
	fmt.Println("return map start")
	ctx := &gin.Context{}
	key := "cached:map_key"
	mapOpt := ReturnMap(ctx, key)
	fmt.Printf("%pn", mapOpt)
	mapOpt["b"] = 1
	fmt.Printf("%pn", mapOpt)
	value, ok := ctx.Get(key)
	if ok {
		m := make(map[string]interface{})
		bytes := value.([]byte)
		err := json.Unmarshal(bytes, &m)
		if err != nil {
			panic(err)
		}
		fmt.Println("get cache: ")
		fmt.Println(m)
	} else {
		fmt.Println("unknown")
	}
	fmt.Println("return map end")
}

打印的结果为:

return map start
set cache: 
map[a:1]
0xc0003a6870
0xc0003a6870
get cache: 
map[a:1]
return map end

知识点