在工作中写了如下一段代码:
for _, tag := range tags {
args := append(fgtArgs, "--tag", tag)
cmd := utils.Command{
...
Arguments: args,
}
cmds = append(cmds, cmd)
}
虽然 tags 中的各个 tag 不相同,但最后,所有的 cmd 中都使用了最后一个 tag。
经跟踪发现,在循环体中 args 是变化的,但在变化之后就覆盖了以前的 args,它们之间并没有保持独立。即 args 看似是局部变量,实际上跟全局变量类似。经测试,发现原因出在 fgtArgs 是由 append 生成的。
看如下测试代码:
a := append([]string{"abc", "abc", "abc", "abc", "abc", "abc", "abc", "abc", "abc", "abc", "abc", "abc", "abc", "abc", "abc", "abc"}, "000")
// a := []string{"abc", "abc", "abc", "abc", "abc", "abc", "abc", "abc", "abc", "abc", "abc", "abc", "abc", "abc", "abc", "abc", "000"}
// a := append([]string{"abc"}, "000")
b := append(a, "123")
c := append(a, "456")
fmt.Println(a)
fmt.Println(b)
fmt.Println(c)
当采用 append 初始化 a 时(第1行),最后运行结果中 b 和 c 的值是一样的,最后都是“456”,当采用数组初始化的方式初始化 a 时(第2行),最后的运行结果中 b 和 c 的值是不同的。
但是,若用少量元素初始化 a,即使使用 append(第3行),b 和 c 的值也是不同的。
这应该是跟 append 为数组开辟空间及go数组的实现方式有关,没有深入研究。
最终的解决办法就是每次创建新的拷贝,而不是在原来的 fgtArgs 上生成 args,代码如下:
for _, tag := range tags {
args := append(fgtArgs[:0:0], fgtArgs...)
args = append(fgtArgs, "--tag", tag)
cmd := utils.Command{
...
Arguments: args,
}
cmds = append(cmds, cmd)
}