在NestJS中能否跳过Service直接在Controller中处理数据库操作？

这种写法功能上完全能正常运行，但从NestJS的架构设计和长期维护角度来看，存在不少隐患，具体问题如下：

• 违反单一职责原则：Controller的核心职责是接收HTTP请求、处理参数校验、返回响应结果，不该承担数据层的操作逻辑。现在看似只有一行代码，但后续一旦需要添加业务逻辑（比如创建用户前校验唯一性、创建后触发通知日志、添加事务控制），Controller会快速膨胀成“大杂烩”，代码可读性和维护性急剧下降。
• 缺失可复用性：如果后续其他模块（比如定时任务、内部服务调用）也需要执行“创建用户”的逻辑，你只能重复编写this.userRepo.create(createData)这段代码，而抽离到Service后，所有需要创建用户的地方都可以直接复用Service方法，避免重复造轮子。
• 测试成本更高：单元测试时，Controller直接依赖Sequelize Model，你需要Mock整个Model的create方法；而通过Service做中间层，测试Controller只需要Mock Service的创建方法即可，测试逻辑更简洁，也能单独对Service的数据操作逻辑做针对性测试。
• 架构扩展性受限：NestJS的分层设计（Controller -> Service -> Repository）是为了让各层职责清晰，后续如果需要切换ORM（比如从Sequelize换成TypeORM），直接修改Service层的实现即可，Controller完全不用动；如果直接在Controller写死Sequelize方法，后续更换ORM时要改动大量Controller代码，成本极高。

优化后的示例代码 #

哪怕只有一行逻辑，也建议保持分层：

UserService

@Injectable()
export class UserService {
  constructor(
    @InjectModel(User)
    private readonly userRepo: User,
  ) {}

  async createUser(createData: CreateUserDto) {
    return this.userRepo.create(createData);
  }
}

UserController

export class UserController{
    constructor(
        private readonly userService: UserService,
    )

    async createUser(@Body createData: CreateUserDto){
        return this.userService.createUser(createData);
    }
}

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