【问题标题】:@UsePipes(ValidationPipe) not working with generics (abstract controller)@UsePipes(ValidationPipe) 不适用于泛型(抽象控制器)
【发布时间】:2019-04-23 20:08:39
【问题描述】:

我正在使用 Nest.js 和 MySQL 构建 API。由于敏捷性和 DRY 原则,我正在创建一个 OOP 结构,它为给定实体(来自 TypeORM)设置所有基本 CRUD 端点。主要目标是避免为不同的实体编写相同的通用方法。

为了实现这一点,我使用了一种带有 TypeScript 泛型 的策略。我仍然需要为每个实体创建所有公共文件(.controller.ts.service.ts.module.ts.entity.ts),但我不必编写它的方法。相反,我只是扩展了两个类:RestControllerRestService。这些类已经实现了常用方法,但我必须传递一些 T 类型 作为参数,以便 TypeORM 可以将正确的存储库注入到服务中。

问题: @UsePipes 装饰器在父类(RestController)中使用时没有被调用,但是当我覆盖 de RestController em> 在子类中创建方法 (SubcategoriesController)。

rest.controller.ts:

import { Get, Post, Body, Param, Put, Delete, UsePipes, ValidationPipe } from '@nestjs/common';
import { RestService } from './rest.service';
import { ObjectLiteral } from 'typeorm';

export abstract class RestController<T, C = T, U = T> {
  constructor(protected service: RestService<T, C, U>) {}

  @Get()
  async index(): Promise<T[]> {
    return this.service.getAll();
  }

  @Post('create')
  @UsePipes(ValidationPipe) //HERE!
  async create(@Body() data: C): Promise<T> {
    return this.service.create(data as C);
  }
}

rest.service.ts:

import { Repository, UpdateResult, DeleteResult, Entity, DeepPartial } from 'typeorm';
import { InjectRepository } from '@nestjs/typeorm';

export interface RestClass<T, C = T, U = T> {
  // Properties
  repository: Repository<T>;

  // Default Methods
  getAll(): Promise<T[]>;
  create(model: T | C | U): Promise<T>;
}

export class RestService<T, C = T, U = T> implements RestClass<T, C, U> {
  constructor(
    public repository: Repository<T>,
  ) {}

  getAll = async () => {
    return await this.repository.find({relations:: this.repository.metadata.ownRelations.map(r => r.propertyName)});
  }

  create = async (model: C) => {
    return await this.repository.save(model as C);
  }
}

这是我如何设置真实实体端点,扩展上述类:

subcategories.controller.ts:

import { Controller, Get, Post, UsePipes, ValidationPipe, Body } from '@nestjs/common';
import { SubcategoriesService } from './subcategories.service';
import { Subcategory } from './subcategory.entity';
import { RestController } from '../rest.controller';
import { CreateSubcategoryDTO } from './dto/createSubcategory.dto';

//NOTE THE TYPE PARAMS IN <>
@Controller('subcategories')
export class SubcategoriesController extends RestController<Subcategory, CreateSubcategoryDTO> {
  constructor(public service: SubcategoriesService) {
    super(service);
  }    
}

subcategories.service.ts

import { Injectable } from '@nestjs/common';
import { InjectRepository } from '@nestjs/typeorm';
import { Subcategory } from './subcategory.entity';
import { Repository } from 'typeorm';
import { RestService } from '../rest.service';
import { CreateSubcategoryDTO } from './dto/createSubcategory.dto';

//NOTE THE TYPE PARAMS IN <>
@Injectable()
export class SubcategoriesService extends RestService<Subcategory, CreateSubcategoryDTO> {

  constructor(
    @InjectRepository(Subcategory) repository: Repository<Subcategory>,
  ) {
    super(repository);
  }
}

createSubcategory.dto.ts

import { IsString, Length, IsInt } from 'class-validator';

export class CreateSubcategoryDTO {

  @IsString()
  @Length(5, 60)
  name: string;

  @IsString()
  @Length(0, 140)
  summary: string;

  @Length(0, 140)
  icon: string;

  @IsInt()
  category: number;
}

可以看到父类接受3种类型参数:

  • T:实体
  • C:CreateDTO,可选
  • U :UpdateDTO,可选

上面的代码完美地创建了端点,但是 它没有验证 /create 中的有效负载,正如 ValidationPipe 所期望的那样。

如果我覆盖 SubcategoriesController 中的create 方法,并在那里添加 UsePipes,它可以工作

我认为这可能是涉及 Nests 生命周期的错误,它可能不支持在抽象类中使用 Pipes。

有人有想法吗?

附注。没有编译错误、lint 警告或运行时异常。

【问题讨论】:

  • 我也是这种情况,请问你是怎么解决这个问题的?

标签: node.js typescript oop generics nestjs


【解决方案1】:

对此的一种解决方案是为您的控制器创建一个工厂函数,该函数将接受您的主体参数类作为参数,然后将其传递给自定义的 ValidationPipe 扩展,如下所示:

@Injectable()
export class AbstractValidationPipe extends ValidationPipe {
  constructor(
    options: ValidationPipeOptions,
    private readonly targetTypes: {
      body?: Type<any>;
      query?: Type<any>;
      param?: Type<any>;
      custom?: Type<any>;
    },
  ) {
    super(options);
  }

  async transform(value: any, metadata: ArgumentMetadata) {
    const targetType = this.targetTypes[metadata.type];
    if (!targetType) {
      return super.transform(value, metadata);
    }
    return super.transform(value, { ...metadata, metatype: targetType });
  }
}

export interface IController<T> {
  hello(body: T);
}

export function Factory<T>(bodyDto: ClassType<T>): ClassType<IController<T>> {
  @Controller()
  class ControllerHost<T> implements IController<T> {
    @Post()
    @UsePipes(new AbstractValidationPipe({whitelist: true, transform: true}, {body: bodyDto}))
    hello(@Body() body: T) {
      return "hello"
    }
  }
  return ControllerHost;
}

export class MyDto {
  @Expose()
  @IsDefined()
  @IsString()
  hello: string;
}

export class AppController extends Factory<MyDto>(MyDto) {}

没有关于反射可用泛型的信息,因此标准 ValidationPipe 没有从metadata.metatype 获得任何有意义的信息。我通过为它提供可选类型参数来解决这个问题,它可以用来覆盖metadata.metatype 的内容。它有一个很好的特性,它也适用于普通用例(没有泛型)。如果您也想覆盖 queryparam,只需通过 targetTypes 参数提供适当的值。

【讨论】:

    【解决方案2】:

    【讨论】:

      猜你喜欢
      • 1970-01-01
      • 2021-06-05
      • 2012-10-31
      • 1970-01-01
      • 2012-08-27
      • 2023-03-18
      • 1970-01-01
      • 1970-01-01
      相关资源
      最近更新 更多