Можно ли сделать наследование для DTO в Nest Js? - PullRequest
Новая
       31

Можно ли сделать наследование для DTO в Nest Js?

0 голосов
/ 14 февраля 2020

У меня есть гнездо Js Получить контроллер, который ищет список сотрудников. 

@Controller('employee')
export class EmployeeController {
    private logger = new Logger('EmployeeController');
    constructor(private employeeService:EmployeeService){}

    @Get()
    @UsePipes(new ValidationPipe({ transform: true }))
    getEmployees(
        @Query() filter:GetEmployeesFilterDto
    ):Promise<Employee[]>{
        console.log(JSON.stringify(filter), filter.batch);
        return this.employeeService.getEmployees(filter);
    }
}

Для поддержки разбиения на страницы метод принимает фильтр DTO для принятия таких параметров, как размер пакета и страница число. Видя, как это будут общие параметры для других классов DTO, я подумал о создании базового фильтра DTO: 

export class BaseQueryFilterDto{

    @IsOptional()
    @IsInt()
    @UsePipes(ParseIntPipe)
    batch:number;

    @IsOptional()
    @IsInt()
    @UsePipes(ParseIntPipe)
    skip:number;

}

, а затем унаследовал этот DTO для GetEmployeesFilterDto: 

export class GetEmployeesFilterDto extends BaseQueryFilterDto{
    @IsOptional()
    @IsNotEmpty()
    search:string;

    @IsOptional()
    primarySupervisorId:string;

}

Running следующий запрос http://localhost: 3000 / employee? batch = 30 выдаст следующее исключение: 

[Nest] 27920   - 02/14/2020, 6:29:54 PM   [ExceptionsHandler] Cannot assign to read only property 'batch' of object '#<GetEmployeesFilterDto>' +606974ms
TypeError: Cannot assign to read only property 'batch' of object '#<GetEmployeesFilterDto>'
    at _loop_1 (C:\ng-proj\nestjs-tutorial\node_modules\class-transformer\TransformOperationExecutor.js:242:47)
    at TransformOperationExecutor.transform (C:\ng-proj\nestjs-tutorial\node_modules\class-transformer\TransformOperationExecutor.js:260:17)
    at ClassTransformer.plainToClass (C:\ng-proj\nestjs-tutorial\node_modules\class-transformer\ClassTransformer.js:17:25)    at Object.plainToClass (C:\ng-proj\nestjs-tutorial\node_modules\class-transformer\index.js:20:29)
    at ValidationPipe.transform (C:\ng-proj\nestjs-tutorial\node_modules\@nestjs\common\pipes\validation.pipe.js:55:41)
    at transforms.reduce (C:\ng-proj\nestjs-tutorial\node_modules\@nestjs\core\pipes\pipes-consumer.js:15:33)
    at process._tickCallback (internal/process/next_tick.js:68:7)

Однако, когда я перемещаю все свойства из суперкласса в GetEmployeesFilterDto, как итак: 

export class GetEmployeesFilterDto extends BaseQueryFilterDto{
    @IsOptional()
    @IsNotEmpty()
    search:string;

    @IsOptional()
    primarySupervisorId:string;

    //Moved from BaseQueryFilterDto
    @IsOptional()
    @IsInt()
    @UsePipes(ParseIntPipe)
    batch:number;

    @IsOptional()
    @IsInt()
    @UsePipes(ParseIntPipe)
    skip:number;

}

Запрос успешно выполнен. Я что-то упускаю здесь или не могу реализовать наследование контроллеров DTO в Nest Js?

1 Ответ

1 голос
/ 17 февраля 2020

Спасибо @JayMcDoniel за указание, что я не должен использовать @UsePipes() для свойств.

Вот решение, которое мне подходит. 

export class BaseQueryFilterDto{

    @IsOptional()
    @Type(() => Number)
    batch:number;

    @IsOptional()
    @Type(() => Number)
    skip:number;

}
...