请教 golang 依赖注入的实际问题

参数类型为啥定义在 task ？不是应该定义在 handler 吗？

"因为 sshclient 需要的 ip 是在 task 中的其他函数获取的"
能否将这个依赖剥离出来, 作为 sshclient 的依赖呢? 如果 task 也需要这个, 让 task 也依赖就好了.

以及"循环导入" 的问题, 我之前接触过 nestjs, 也好奇过循环依赖如何解决
https://docs.nestjs.com/fundamentals/circular-dependency
或许可以另一种思路参考

「这个 sshclient 实例化过程只能在 task 中动态生成, 因为 sshclient 需要的 ip 是在 task 中的其他函数获取的.」

没看懂这前后两句话的因果

你建一个 sshclient 的实例，然后在 task 的时候调不就行了。把 ip 当作参数传进去
譬如

sc := NewSshClient()

func task(client *SshClient) {
ip := GetIp()
client.connect(ip)
}

task(&sc)

@NessajCN 因为注入嘛, 我在注入的时候(也就是初始化的时候) 是不知道 ip 的.

我现在的代码和你的例子差不多, 只不过不是用的 connect 而是注入了一个无参的函数, 这个函数返回一个工厂函数, 这个工厂函数返回 sshclient.

如果 sshclient 的 ip 是静态的类似 db.DB, 直接定义在 handler 和 task 的公共底层. 注入即可, 就没有这么多麻烦事了.

task 没有 sshclient 的依赖，依赖是在 handler 中，task 只需要提供参数给 handler 就行了，task 依赖一个工厂函数或者工厂接口就行。

@chaleaochexist 你直接发代码吧，我怀疑你提了个 xy 问题，
也许你的原始需求有更直接简单的解决法而不用注入来注入去

要不要在这里叠加 factory 看你的倾向和你使用的 di 框架的能力啊。有的时候工厂是个很简单的解决方案，特别是你想更精确地控制生命周期时。

我虽然不是很确信你的业务，不过如果按照我对你业务的猜测，这里使用 factory pattern 是正确的。

我感觉直接 interface ，然后套娃引用就行了

https://go.dev/play/p/sa9s8QpKE29

我也有这种嵌套的依赖，最后是用 interface 解决的

op 最好能给出现在的伪代码或者哪个循环引用的伪代码

lz 不如画图说明下设想的流程和调用逻辑，这里 handler 和 task 名称和职责感觉不清楚

假设是 handler 生成 task ，task 调用 sshclient 干活

1. handler 没有直接管 sshclient
2. 没有看懂

type CommandRunner interface {
exec(cmd string) (ret string, err error)
}

@everhythm
@darksword21
@Sendya
@iseki
@NessajCN
@sthwrong
@sunny352787
@Dorathea
@NessajCN

谢谢你们的回复, 我写了一个 伪代码, 希望可以把问题解释清楚.

https://github.com/chaleaoch/golang_demo.git

一共是三个版本 v1, v2, v3 我把问题都放到了注释中, 可以搜索关键字 "问题" . 各个版本之间的主要差别, 可以搜索关键字 "修改"

如果我得代码有其他的地方的写法上的问题, 随意批评指导, 谢谢你们. 谢谢!!

@chaleaochexist 你完全搞错 interface 的用法了，
go 的 interface 跟 jvav 的不是一回事，
你定义了一大堆完全永不上的 interface

go 的 interface 可以类比 python 的 protocol 或 rust 的 trait,
是为了方便你写「非特定类型参数」的函数，或者早期的泛型来用的。
具体的用法是你定义一个 struct 和 interface ， 并给 struct 实现 interface 里的函数，
之后你定义的参数为该 interface 的函数就可以直接传这个 struct 的实例了

我的建议还是如 3# 时候讲的，你初始化就初始化 client 然后传引用进 task 函数就好，初始化时候也根本不需要知道 ip

type SSHClient struct {}
func (s *SSHClient) ExecuteCommand(user string, pass string, host string, cmd string) {
// blablabla
}

// 初始化 sshclient 放到 main 函数里， 然后传给下面的 task

func (q *DemoTask) Run(sshClient *SSHClient) {
mIps, _ := q.mIPRepo.GetByType("exampleType") // 这一条也可以大幅简化
for _, mIp := range mIps {
out, _ := sshClient.ExecuteCommand(mIp.Username, mIp.Password, mIp.Ip+":"+mIp.Port, "exampleCommand")
fmt.Println(out)
}
}

@NessajCN 像你这么写, 最大的问题是如何 mock?

无论是 GetByType 是查询数据库的, 如果不做成接口, 如何 mock
ssh 也是, 在单元测试的时候, 我不希望真的去 ssh 执行一条命令.

@chaleaochexist 就像我上一个回复内容提到的，你先要弄明白 go 的 interface 究竟是啥。
先把 jvav 思维彻底舍弃才好跟你讲下一步

无论是 --> ~~无论是~~

@NessajCN 好吧...其实我根本就不会 java...
golang 如果不通过 interface 没法 mock.

或者 sshClient 这个接口 可以有多个实现. 我希望注入而不是将 NewsshClient1 改成 NewsshClient2

@chaleaochexist 都说了接口不是这么用.....
sshClient 显然是个实例，咋会是接口呢。
go 的 interface 是在定义函数时候作为「类泛型」传参用的，你的 sshClient 必然是要初始化然后调方法的，
概念风马牛不相及呀