循环和地图参考答案

【问题标题】：Loops and map references循环和地图参考
【发布时间】：2018-05-19 04:43:09
【问题描述】：

我的代码循环遍历 Task 的数组并创建一个带有字符串键和 Task 值的 Map：

package main

import (
    "fmt"
)

type Task struct {
    Name    string
    Project string
}

func main() {
    taskMap := map[string]*Task{}
    taskList := []Task{
        {
            Name:    "name1",
            Project: "project1",
        },
        {
            Name:    "name2",
            Project: "project2",
        },
        {
            Name:    "name3",
            Project: "project3",
        },
    }
    for _, task := range taskList {
        taskMap[task.Name] = &task
    }

    // Print results
    for k, v := range taskMap {
        fmt.Println(k, v)
    }
}

这段代码的输出如下：

name1 &{name3 project3}
name2 &{name3 project3}
name3 &{name3 project3}

这不是我所期望的，因为它每次都打印相同的内容。我对这里发生的事情有一点线索，因为如果我用以下内容替换循环，它会按预期工作：

for _, task := range taskList {
        taskMap[task.Name] = &Task{
            Name:        task.Name,
            Project: task.Project,
        }
}

因此，不知何故，对上次访问任务的引用最终存储在 taskMap 的每个键中，显然。

有没有办法在不手动复制每个字段的情况下做到这一点？

【问题讨论】：

变量task 存储了一个在每次迭代中更改的本地副本，但task 的地址（存储在地图中）保持不变。因此，当您打印地图内容时，所有地图值都指向最后一个列表项的副本。

标签： loops pointers go

【解决方案1】：

所有键的值都相同，因为您在 for 循环中引用了相同的 task 变量。使用以下 sn-p 获取对切片中项目的引用。

for idx, task := range taskList {
    taskMap[task.Name] = &taskList[idx]
}

顺便说一句（浅）复制一个变量，您可以将它分配给一个新变量，例如

for _, task := range taskList {
    x := task
    taskMap[task.Name] = &x
}

将在每次迭代中创建一个新的x 变量，并在映射中使用指向该变量的指针。

【讨论】：

我认为正确的解决方案是for i := range taskList { taskMap[taskName] = &taskList[i] }。我的意思是，如果想要在每次迭代中获取指向切片元素的指针，那么获取元素本身是没有意义的——它的索引就足够了，而且它可以很容易地用于获取指向其对应元素的指针。
除了在您的示例中 taskName 不存在；您需要元素来获取名称。发布的解决方案对我来说似乎是正确的。
“让它存在”并不难。不，真的。
哦，您可以只使用range 运算符的两个参数形式，而不必使用中间变量或附加切片标识符。说真的，为什么要让这比需要的更难。两个参数的形式在这里非常合适。

【解决方案2】：

您存储的是task 的地址而不是值。

您可以改为将taskList 切换为指针切片：taskList := []*Task{ 并使用taskMap[task.Name] = task。

【讨论】：

其实我比我更喜欢Arman 的回答，它向您展示了如何在不改变现有数据结构的情况下做到这一点。我的在这种情况下是可以接受的，但你并不总是能控制进来的东西。