以正确的方式避免循环依赖-NestJS [英] Avoiding circular dependencies the right way - NestJS

问题描述

假设我有一个StudentService和一个LessonService，StudentService和LessonService，StudentService和LessonService分别用于将学生添加到课程中。在我的课程和学生解决程序中，我都希望能够更新此课程<；->学生关系。因此，在我的LessonResolver中，我有类似的内容：

  async assignStudentsToLesson(
    @Args('assignStudentsToLessonInput')
    assignStudentsToLesson: AssignStudentsToLessonInput,
  ) {
    const { lessonId, studentIds } = assignStudentsToLesson;
    await this.studentService.assignLessonToStudents(lessonId, studentIds); **** A.1 ****
    return this.lessonService.assignStudentsToLesson(lessonId, studentIds); **** A.2 ****
  }

基本上与我的StudentResolver

相反

上述A.1和A.2的区别在于，StudentService可以访问StudentRepository，而LessonService可以访问LessonRepository，我认为这符合坚实的关注点分离。

但是，StudentModule必须导入LessonModule和LessonModule必须导入StudentModule似乎是一种反模式。这是可以使用forwardRef方法修复的，但是在NestJS Documentation中提到，如果可能，应该避免此模式：

虽然应在可能的情况下避免循环依赖，但您 不能总是这样。(这是其中一个案例吗？)

这似乎应该是使用DI时的常见问题，但我正在努力获得一个明确的答案，以确定哪些选项可以消除这种情况，或者我是否偶然发现了一种无法避免的情况。

我的最终目标是能够编写以下两个GraphQL查询：

query {
  students {
    firstName
    lessons {
      name
    }
  }
}

query {
  lessons {
    name
    students {
      firstName
    }
  }
}

推荐答案

可能最简单的方法是完全删除依赖项，而引入依赖于其他两个模块的第三个模块。 在您的示例中，您可以将两个解析器合并为单个StudentLessonResolver，该解析器位于其自己的模块中，例如ResolverModule：

async assign({ lessonId, studentIds }: AssignStudentsToLessonInput) {
  await this.studentService.assignLessonToStudents(lessonId, studentIds);
  return this.lessonService.assignStudentsToLesson(lessonId, studentIds);
}

soStudentModule和LessonModule现在完全独立，而ResolverModule依赖于两者。不再循环：)

如果出于某种原因需要有两个解析器，并让它们相互更新，您可以使用事件或回调来发布更改。 然后，您将再次引入第三个模块，该模块侦听这些事件并更新另一个模块。

type AssignCallback = (assignStudentsToLesson: AssignStudentsToLessonInput) => Promise<void>;

class LessonResolver {  // and similar for StudentResolver
  private assignCallbacks: AssignCallback[] = [];

  // ... dependencies, constructor etc.

  onAssign(callback: AssignCallback) {
    assignCallbacks.push(callback);
  }

  async assignStudentsToLesson(
    @Args('assignStudentsToLessonInput')
    assignStudentsToLesson: AssignStudentsToLessonInput,
  ) {
    const { lessonId, studentIds } = assignStudentsToLesson;
    await this.lessonService.assignStudentsToLesson(lessonId, studentIds); **** A.2 ****
    for (const cb of assignCallbacks) {
      await cb(assignStudentsToLesson);
    }
  }
}

// In another module
this.lessonResolver.onAssign(({ lessonId, studentIds }) => {
  this.studentService.assignLessonToStudents(lessonId, studentIds);
});
this.studentResolver.onAssign(({ lessonId, studentIds }) => {
  this.lessonService.assignStudentsToLesson(lessonId, studentIds);
});

您再次打破了这个循环，因为StudentModule和LessonModule彼此不知道，而您注册的回调保证调用任何一个解析程序都会导致两个服务都得到更新。

如果您使用的是RxJS之类的反应库，则不应手动管理回调，而应使用解析程序发布到的Subject<AssignStudentsToLessonInput>和新引入的模块订阅的

。

更新

正如OP所建议的，还有其他替代方案，例如将两个存储库都注入到两个服务中。但是，如果每个模块同时包含存储库和服务，即，如果您从LessonModule导入LessonRepository和LessonService，这将不起作用，因为您仍然具有对模块级别的循环依赖。 但是如果学生和课程之间确实有紧密的联系，你也可以把两个模块合并成一个模块，这是没有问题的。

一个类似的选项是将第一个解决方案的单个解析器更改为直接使用存储库的服务。这是否是一个好的选择取决于管理商店的复杂程度。从长远来看，使用这项服务可能会更好。

我在单一解析器/服务解决方案中看到的一个优点是，它为将学生分配到课程提供了单一解决方案，而在事件解决方案中，StudentService.assignLessonToStudents和essonService.assignentsToLesson实际上做的是完全相同的事情，因此不清楚应该使用哪一个。

这篇关于以正确的方式避免循环依赖-NestJS的文章就介绍到这了，希望我们推荐的答案对大家有所帮助，也希望大家多多支持IT屋！