Go语言中第三方结构体转自定义结构体的惯用方法

你的mapResultsToUsers实现其实完全符合Go语言的惯用风格，不存在所谓的"代码异味"——Go本身就推崇显式、简洁的代码风格，手动映射不同结构体的字段是处理这类转换的标准做法，理由如下：

• 可读性极强：任何人看代码都能立刻明白ResultUser的哪些字段对应User的哪些字段，后续维护成本极低。
• 性能最优：没有反射、序列化/反序列化带来的额外开销，是效率最高的转换方式。

可选的小优化（非必须，但更规范） #

如果想让代码更整洁、可维护性更强，可以做两个小调整：

1. 给User添加构造函数

把字段映射逻辑封装到构造函数中，后续如果User的字段或构造规则变化，只需要修改这一处：

func NewUser(r ResultUser) User {
    return User{
        id:        r.uid,
        firstName: r.fname,
    }
}

2. 预先分配切片容量

因为你知道转换后的切片长度和输入的SearchResults长度一致，预先分配容量可以避免append操作时的内存扩容，提升性能：

func mapResultsToUsers(results SearchResults) []User {
    // 预先分配与输入长度一致的切片，避免扩容
    users := make([]User, len(results))
    for i, result := range results {
        users[i] = NewUser(result)
    }
    return users
}

不推荐的方案 #

对于你的场景，不建议使用反射或序列化（比如利用结构体标签通过JSON/XML中转）这类方式：

• 反射会带来明显的性能开销，且代码可读性大幅下降，只有在需要处理大量重复转换逻辑时才值得考虑。
• 序列化中转需要将结构体字段改为导出状态（首字母大写），会破坏你当前的封装逻辑，同时性能远低于手动映射。

最终优化后的完整代码 #

package main

import "fmt"

type (
    User struct {
        id        int32
        firstName string
    }

    ResultUser struct {
        uid   int32
        fname string
    }
    SearchResults []ResultUser
)

func main() {
    results := getSearchResults()
    users := mapResultsToUsers(results)
    fmt.Println("User struct:", users[0].id, users[0].firstName)
    fmt.Println("User struct:", users[1].id, users[1].firstName)
}

func getSearchResults() SearchResults {
    return append(SearchResults{}, ResultUser{1, "John"}, ResultUser{2, "Jane"})
}

func NewUser(r ResultUser) User {
    return User{
        id:        r.uid,
        firstName: r.fname,
    }
}

func mapResultsToUsers(results SearchResults) []User {
    users := make([]User, len(results))
    for i, r := range results {
        users[i] = NewUser(r)
    }
    return users
}

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