V4 的 LazyInject遇到问题

[Wanton-y]

Describe the problem（描述问题）

升级了最新的V4版本（^4.2.1），项目中有循环依赖，使用了LazyInject，报错 ：
TypeError: Class constructor Aservice cannot be invoked without 'new'
at formatObjectIdentifier (D:\code\server\node_modules@midwayjs\core\dist\context\managedResolverFactory.js:26:16)
at ManagedResolverFactory.createInstance (D:\code\server\node_modules@midwayjs\core\dist\context\managedResolverFactory.js:233:31)
at ManagedResolverFactory.createInstance (D:\code\server\node_modules@midwayjs\core\dist\context\managedResolverFactory.js:247:34)
at ManagedResolverFactory.createInstance (D:\code\server\node_modules@midwayjs\core\dist\context\managedResolverFactory.js:247:34)
at ManagedResolverFactory.createAsync (D:\code\server\node_modules@midwayjs\core\dist\context\managedResolverFactory.js:86:31)
at MidwayRequestContainer.getAsync (D:\code\server\node_modules@midwayjs\core\dist\context\requestContainer.js:36:14)
at D:\code\server\node_modules@midwayjs\core\dist\common\webGenerator.js:42:61
at processTicksAndRejections (node:internal/process/task_queues:95:5)
at D:\code\server\src\middleware\auth.middleware.ts:92:9

大致代码如下
@provide()
export class Bservice {
@LazyInject(（） => Aservice )
private a: Aservice
}

@provide()
export class Aservice {
@Inject()
private b: Bservice
}

---

但是代码中这样写不报错：

@autoload()
@singleton()
export class Cservice {
@Inject()
d: Dservice
}

@provide()
@singleton()
export class Dservice {
@LazyInject(() => Cservice)
c: Cservice
}

求解疑

Midway Versions（Midway 版本）

---

Current version @midwayjs/decorator(4.2.1) and @midwayjs/core(4.2.1) not found in @midwayjs/version, please check it.

---

[Wanton-y]

如果LazyInject添加到另一个，就会报循环依赖错误

@provide()
export class Bservice {
@Inject()
private a: Aservice
}

@provide()
export class Aservice {
@LazyInject(（） => Bservice)
private b: Bservice
}

报错：
D:\code\server\node_modules@midwayjs\core\dist\context\managedResolverFactory.js:173
throw new error_1.MidwayCommonError(Circular dependency detected: ${this.translateIdentifiers(cycle).join(' -> ')});

MidwayCommonError: Circular dependency detected: SocketController -> Aservice -> Bservice -> Aservice
at ManagedResolverFactory.createInstance (D:\code\server\node_modules@midwayjs\core\dist\context\managedResolverFactory.js:173:23)
at ManagedResolverFactory.createInstance (D:\code\server\node_modules@midwayjs\core\dist\context\managedResolverFactory.js:247:34)
at ManagedResolverFactory.createInstance (D:\code\server\node_modules@midwayjs\core\dist\context\managedResolverFactory.js:247:34)
at ManagedResolverFactory.createInstance (D:\code\server\node_modules@midwayjs\core\dist\context\managedResolverFactory.js:247:34)
at ManagedResolverFactory.createAsync (D:\code\server\node_modules@midwayjs\core\dist\context\managedResolverFactory.js:86:31)
at MidwayRequestContainer.getAsync (D:\code\server\node_modules@midwayjs\core\dist\context\requestContainer.js:36:14)
at WebSocketServer. (D:\code\server\node_modules@midwayjs\ws\dist\framework.js:190:68)
at processTicksAndRejections (node:internal/process/task_queues:95:5)

[czy88840616]

这里要注意 @LazyInject 不是把这个依赖完全跳过初始化，而是在当前创建链已经出现目标对象时，才会用 proxy 延迟取值来打断循环。

所以 A -> B -> A 这种场景，LazyInject 要放在实际闭环的那条边上。比如当前入口是 SocketController -> Aservice -> Bservice -> Aservice，闭环发生在 Bservice -> Aservice，因此至少需要：

@Provide()
export class Aservice {
  @Inject()
  private b: Bservice;
}

@Provide()
export class Bservice {
  @LazyInject(() => Aservice)
  private a: Aservice;
}

如果 Aservice 和 Bservice 都可能作为入口被单独创建，建议两边都用 @LazyInject(() => Xxx)，或者更推荐把公共逻辑拆出去，避免两个 service 直接互相依赖。

另外 TypeError: Class constructor Aservice cannot be invoked without "new" 这类错误，通常要检查是不是写成了 @LazyInject(Aservice) 或编译后等价于直接传 class。这里需要传函数：@LazyInject(() => Aservice)，不要直接把 class 传进去。

[Wanton-y]

感谢解答！

我的场景是：Aservice和Bservice都会作为入口被创建，Bservice是一个聚合校验，会注入很多service（包括Aservice），Aservice要用Bservice里一些校验方法

我尝试了双边 @LazyInject(() => Xxx)，
和您的代码示例

`@Provide()
export class Aservice {
@Inject()
private b: Bservice;
}

@provide()
export class Bservice {
@LazyInject(() => Aservice)
private a: Aservice;
}`

都会报TypeError: Class constructor Aservice cannot be invoked without "new" 或 TypeError: Class constructor Bservice cannot be invoked without "new"（不同入口进入报的错）
我确认是@LazyInject(() => Xxx) 这样写的
我目前的解决方案是：先拆分公共逻辑，解除循环依赖。

[czy88840616]

拆公共逻辑这个处理是更推荐的，两个 service 互相依赖后面维护成本会比较高，聚合校验这类逻辑建议下沉成独立的 validator/service/function，再由 A、B 分别依赖它。

补充一下这个报错本身：Class constructor Xxx cannot be invoked without "new" 从栈上看不是循环依赖报错，而是 LazyInject 解析到的 identifier 仍然是 class 构造函数，随后被当成 () => Xxx 这类 thunk 调用了。

可以重点确认两点：

1. v4 里不要再从 @midwayjs/decorator 导入装饰器，统一从 @midwayjs/core 导入 Provide / Inject / LazyInject，并尽量移除 @midwayjs/decorator 依赖。你版本检测里出现 @midwayjs/decorator(4.2.1) 也比较可疑。
2. 看一下编译后的代码或者最小复现，确认 metadata 里保存的确实是 () => Aservice，而不是 Aservice 本身；如果最后还是复现，可以发一个最小 repo，我再按 v4 的容器逻辑看是不是有边界 case。