这一天,本程序猿在项目中遇到一个需求,简化版本如下:
[1, 1, 1, 2, 2, 1]
将这个列表中所有的2改为1。
哟嚯,这么简单的需求不是秒秒钟就搞定?于是撸起袖子刷刷地写:
func main() {
demo := []int{1, 1, 1, 2, 2, 1, 1}
for _, v := range demo {
if v == 2 {
v = 1
}
}
}
so easy。
愉快地点了运行键,得到运行结果:
哦豁,原切片根本没有变化。
仔细检查后,本猿终于发现问题出在range上面,_, v := range demo这一句代码的意思是声明一个v变量,将demo里面相应的元素复制到v中,因此v是demo元素的一个副本,更改它并不会对demo有影响。
要想完成这个需求,那就需要换一种写法:
func main() {
demo := []int{1, 1, 1, 2, 2, 1, 1}
for i := 0; i < len(demo); i++ {
if demo[i] == 2 {
demo[i] = 1
}
}
fmt.Println(demo)
}
本猿在项目中需要完成的需求远比这个例子复杂,排查时间花了一个多小时,仅仅是忽略了range遍历的时候是拷贝一份副本。。。