Go语言中为map赋值指针遇问题，求排查原因及解决方法

嘿，我看你遇到了Go里给map赋值指针的问题，先别急着怀疑是语言bug——这其实是Go遍历slice时一个非常容易踩的常见陷阱，很多刚接触Go的开发者都会在这里栽跟头！

问题根源：循环变量的复用 #

当你用range遍历slice的时候，循环里的变量（比如你代码中的col）是同一个内存地址的变量，每次循环只是把slice当前元素的值复制到这个变量里。如果你直接把&col存入map，那所有的指针都会指向这个循环变量的地址。当循环结束时，这个变量保存的是slice最后一个元素的值，所以你最终看到map里所有指针都指向了最后那个元素，这就导致了不符合预期的结果。

举个简化的错误示例：

type Collection struct {
    ID string
}

func main() {
    collections := []Collection{{ID: "1"}, {ID: "2"}}
    colMap := make(map[string]*Collection)

    // 错误写法：&col始终指向同一个循环变量
    for _, col := range collections {
        colMap[col.ID] = &col
    }

    // 输出会发现两个条目都指向ID为"2"的元素
    for k, v := range colMap {
        fmt.Printf("Key: %s, Value ID: %s\n", k, v.ID)
    }
}

两种修正方案 #

方案1：创建临时变量

在循环内部创建一个临时变量，把当前元素的值赋值给它，这样每次循环都会有一个新的内存地址，指针也就指向了各自的元素：

for _, col := range collections {
    tempCol := col // 临时变量，每次循环都分配新内存
    colMap[tempCol.ID] = &tempCol
}

方案2：通过索引访问元素

直接通过slice的索引获取元素的指针，这样就能拿到slice中对应位置元素的真实地址：

for i := range collections {
    col := &collections[i] // 直接取slice对应索引元素的指针
    colMap[col.ID] = col
}

总结 #

这不是Go的bug，而是语言为了性能优化设计的特性——循环变量复用减少了内存分配的开销。只要记住这个特性，在需要存储循环元素指针的时候，通过上面两种方式规避，就能得到你预期的结果啦！

内容的提问来源于stack exchange，提问作者jordan2175