【极客日常】Golang一个的slice数据替换的bug排查

上周某天下班前，接到同事转来一个bug要排查，症状是代码重构之后某些业务效果不符合预期，由于代码重构人是笔者，于是blame到笔者这边。经过10min左右的排查和尝试后，解决了这个问题：既往逻辑没有改动，重构时候出笔误了。

简单来讲，重构之前的代码大概是这个样子：

func beforeRefactor() {
    var configListExpr = fetchConfigListExpr()
    
    newConfigListExpr := replaceConfigVariables(configListExpr)

    var configList []Config
    if err := json.Unmarshal([]byte(newConfigListExpr), &configList); err != nil {
        panic("unmarshal configList err: " + err.Error())
    }
    return configList
}

而改造业务逻辑时，因为类似于replaceVariables这类对configList批量处理的流程更多，所以定义了很多函数节点去对configList做批量的替换以及内部属性的修改。因此重构时候，就把replaceConfigVariables做了更细粒度的转换，然后同时把整个configList替换逻辑也顺带修改了下。大概改成了这样：

func afterRefactor() {
    var configList []Config = fetchConfigList()

    for _, config := range configList {
        urlList := config.URLList
        newURLList = replaceURLListVariables(urlList)
        config.URLList = newURLList
    }

    return configList
}

这段代码是存在问题的。由于这次重构提的代码很多，cr时候没有发现；然后一开始排查以为是重构引起的业务逻辑变更，所以忽略了Golang本身机制导致的问题。这里的问题是，遍历slice的时候，要想改动原来slice里成员的属性，不应该用config这个复制品，而得用configList取下标这种方式来取到原始的成员实例，或者干脆把configList整个全新替换，这样才可以达到效果。