Go语言：调用函数无法更新嵌套结构体切片的问题

问题场景 #

我遇到了比《Go - append to slice in struct》更复杂的情况：包含切片的嵌套结构体，将该结构体切片传入函数后无法完成更新。

代码示例 #

package main

import (
    "fmt"
)

type MyBoxItem struct {
    Name string
}

type MyBox struct {
    BoxName string
    Items   []MyBoxItem
}

type MyBoxes []MyBox

func (box *MyBoxes) AddItem(items []MyBoxItem, boxName string) {
    for _, b := range *box {
        fmt.Println("Processing: ", b.BoxName)
        if b.BoxName == boxName {
            for _, item := range items {
                fmt.Println("Processing item: ", item.Name)
                b.Items = append(b.Items, item)
            }
        }
    }
}

func main() {

    item1 := MyBoxItem{Name: "Test Item 1"}
    item2 := MyBoxItem{Name: "Test Item 2"}
    item3 := MyBoxItem{Name: "Test Item 3"}
    item4 := MyBoxItem{Name: "Test Item 4"}

    boxItems1 := []MyBoxItem{}
    box1 := MyBox{BoxName: "firstBox", Items: boxItems1}
    boxItems2 := []MyBoxItem{}
    box2 := MyBox{BoxName: "secondBox", Items: boxItems2}

    boxSet := MyBoxes{}
    boxSet = append(boxSet, box1)
    boxSet = append(boxSet, box2)

    fmt.Println(boxSet)

    boxItems1 = append(boxItems1, item1)
    boxItems1 = append(boxItems1, item2)
    boxItems2 = append(boxItems2, item3)
    boxItems2 = append(boxItems2, item4)
    boxSet.AddItem(boxItems1, "firstBox")
    boxSet.AddItem(boxItems2, "secondBox")

    fmt.Println(boxSet)

}

运行输出 #

[{firstBox []} {secondBox []}]
Processing:  firstBox
Processing item:  Test Item 1
Processing item:  Test Item 2
Processing:  secondBox
Processing:  firstBox
Processing:  secondBox
Processing item:  Test Item 3
Processing item:  Test Item 4
[{firstBox []} {secondBox []}]

可见结构体切片并未被更新，需要解决这个问题。

原因分析 #

问题出在AddItem方法的遍历逻辑：

• 使用for _, b := range *box遍历切片时，b是切片中MyBox结构体的副本，而非原结构体的引用。
• 对b.Items执行append操作，仅修改了副本的切片字段，原切片中的MyBox结构体完全不受影响。

解决方法 #

遍历过程中通过索引访问原切片的元素，直接修改原结构体即可：

修改后的AddItem方法：

func (box *MyBoxes) AddItem(items []MyBoxItem, boxName string) {
    for i := range *box {
        fmt.Println("Processing: ", (*box)[i].BoxName)
        if (*box)[i].BoxName == boxName {
            // 直接用items...追加整个切片，无需循环遍历单个元素
            (*box)[i].Items = append((*box)[i].Items, items...)
        }
    }
}

修改后的运行结果 #

[{firstBox []} {secondBox []}]
Processing:  firstBox
Processing:  secondBox
Processing:  firstBox
Processing:  secondBox
[{firstBox [{Test Item 1} {Test Item 2}]} {secondBox [{Test Item 3} {Test Item 4}]}]

此时结构体切片的Items字段已被正确更新。

另一种思路是将MyBoxes定义为[]*MyBox（指针切片），遍历时操作原结构体的指针也能生效，但上述方法更贴合当前代码结构。

内容的提问来源于stack exchange，提问作者learning-man