好坑啊，调用了同事写的基础代码，bug藏得还挺深！！

起因

事情的起因是我调用了同事的一个函数，这个函数返回了一个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("%p\n", mapOpt) // 0xc0003a6750 指向地址
	mapOpt["b"] = 1
	value, ok := ctx.Get(key)
	fmt.Printf("%p\n", 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的值时，原始的缓存也会被修改。

如图所示

如何修改

当然修改方式有很多种，我这里列举了一种就是序列化存储到缓存然后反序列化取，如果你有更好的方式可下方留言

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("%p\n", mapOpt)

	mapOpt["b"] = 1
	fmt.Printf("%p\n", 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

知识点