一次 GO 项目重构的疑问

type A struct {
   a ComponentA
   b ComponentB
   d ComponentD
}
func NewA(a ComponentA, b ComponentB, d ComponentD) {}
type B struct {
   b ComponentB
   c ComponentC
   d ComponentD
}
func NewB(b ComponentB, c ComponentC, d ComponentD) {}


// 大概有十几个这样的 struct
....

这十几个 struct 都有一些共同的变量（当然也不是完全一模一样）。现在是每个结构体的 New 方法传的参数都不一样。当然这十几个 struct 其实做的功能比较相似（每个 struct 其实是 k8s 里面 crd 的 controller ），代码结构也有一定的相似性。

现在重构后的代码

type Common struct {
   a ComponentA
   b ComponentB
   c ComponentC
   d ComponentD
   // 一共大概 20 几个
   ...
}

type A struct {
    *Common
}
type B struct {
    *Common
}

这样重构后，每个结构体的 New 方法的参数就很简单，只要传一个 Common 就好了。但是，这样每个结构体其实引入了一些自己不需要的东西，我自己感觉不是特别好。

各位大佬怎么看？

vczyh

2024-03-06 09:47:04 +08:00

struct 你要有实际意义，common 这种模糊的东西最好去掉，看似提出公共东西了，但是一点也不好维护，不同的东西就是应该分成多个 struct ，要是你觉得一样那就重新思考到底是不是不同的东西，如果是那就分开好了，成员变量相似又不是相同。

sampeng

2024-03-06 18:49:07 +08:00

你是反着来重构的，我上一版代码就是你想弄个 Common 的模式，我叫 GlobalService 。后来仔细琢磨了一下，这玩意就是写时一时爽，维护火葬场。一般项目维护时间是要大于写代码时间的。写代码稍微啰嗦点其实也没啥。所以用 wire 来解决写 New 的啰嗦问题。然后代码就清晰很多了。明确知道依赖是怎么进来的，怎么初始化的。

只有一些像 db 啊之类中间件的句柄我就直接整了个 GlobalComponent 。这种我个人仔细思考了是无所谓的。中间件这东西选好了基本就不会动，最多动选型。依赖的是接口。后续也好调整。

这是一个专为移动设备优化的页面（即为了让你能够在 Google 搜索结果里秒开这个页面），如果你希望参与 V2EX 社区的讨论，你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/1020966