如何解决Go语言中更新map内slice元素值不生效的问题

问题根因 #

你遇到的问题是两层值拷贝导致的修改没有生效：

1. slice取值拷贝：Go中从map读取值类型的slice时，返回的是原slice的副本。你当前的rulesByCountry的value是[]RuleDefinition值类型，GetRulesForCountry返回的是这个slice的拷贝，修改副本不会影响map中存储的原slice。
2. 循环元素拷贝：UpdateName方法中使用for _, rule := range *r遍历slice时，rule是每个RuleDefinition元素的临时值拷贝，修改该临时变量的Enabled字段不会对原slice元素产生任何影响，这也是你仅能在函数内部看到修改生效的原因。

修复方案 #

按以下步骤修改代码即可让修改同步到全局map：

步骤1：调整全局map存储类型 #

将map的value改为存储RuleChain指针，避免返回slice时产生值拷贝：

// 原定义
// var rulesByCountry map[string][]RuleDefinition = map[string][]RuleDefinition{
// 修改后
var rulesByCountry map[string]*RuleChain = map[string]*RuleChain{
    "DE": {
        {
            Name: RuleSeparateStreetNameFromHome,
            Enabled: false,
        },
    },
}

步骤2：修改GetRulesForCountry返回值类型 #

对应调整返回值为RuleChain指针，直接返回map中存储的指针：

// 原定义
// func GetRulesForCountry(country string) RuleChain {
// 修改后
func GetRulesForCountry(country string) *RuleChain {
    if rules, ok := rulesByCountry[country]; ok {
        return rules
    }
    return nil
}

步骤3：修改UpdateName遍历逻辑 #

通过索引访问slice元素，直接操作原元素的指针，避免值拷贝：

func (r *RuleChain) UpdateName(name string) {
    // 改用索引遍历，不取拷贝的元素值
    for i := range *r {
        rule := &(*r)[i]
        if rule.Name == name {
            rule.Enabled = true
            fmt.Print(rule)
        }
    }
}

修改完成后运行代码，全局rulesByCountry中对应元素的Enabled字段就会被正确更新为true。

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