2021-08-12T08:06:41.081Z

0

아하 제가 궁금했던 것은 필드를 부분적으로 가져온 것도 Cat타입에 속하는 것인지였습니다! 답변 감사드립니다

근데 제가 임의적으로 type CatMyDto {email:string, name: string} 이런식으로 설정한 뒤, @CurrentUser() cat: CatMyDto 이렇게 임의적으로 설정해줘도 되는 건가요?

예를 들어서 큰 규모의 어플리케이션에서, 컨트롤러에서 각자 다른 Cat의 PickType을 리턴하는 서비스 여러개를 호출해서 비즈니스 로직의 변수로 사용된다고 가정했을 때, 실제 데이터 필드가 다른데도 불구하고 전부 Cat으로 타이핑 하면, 실제로 컴파일 에러가 발생해야 할 로직에서(ex 예를들어 스키마에 '수량'을 추가하고, A 변수는 수량을 포함한 Cat타입, B 변수는 수량을 select로 뺀  Cat타입일 때 A와 B의 수량 비교 로직 등) 컴파일 에러가 발생하지않고 런타임에서 터지는 경우가 있을 것 같아서 다시 질문드립니다!

1

안녕하세요!

아 무슨 말씀인지 이해했습니다.

사실 @CurrentUser() cat은 값은 데코레이터에 의해 첨가가 되어 request.user 값이 부여가 되지만 타이핑은 어떤 타입이든지 타이핑을 할 수 있습니다. 따라서 string이라고 타이핑해도 임의로 부여가 됩니다. (의도치 않은 타이핑인 것이지요.)

따라서 데코레이팅된 값과 동일한 타입(request.user의 타입)으로 타이핑을 해주어야 합니다.

저희가 의도한 request.user는 select로 인해 password 필드가 undefined인 Cat 타입입니다. 따라서 @CurrentUser() cat : Cat으로 타이핑을 해도 됩니다.  이때 cat.password는 undefined 값입니다.

하지만 이 경우 말씀해주신 것처럼 의도치 않게 개발자가 cat.password = "1205" 으로 접근할 수 있습니다. (컴파일 에러가 발생하지 않지만 안전하지는 않습니다. 런타임 에러가 발생할 확율이 높습니다.)

따라서 필요에 따라 더 강력하게 타이핑을 하는 것이 좋습니다. ( 더 권장드립니다.)

1. 먼저 CatCurrentDto를 설계합니다. (여기서 OmitType은 PickType과 반대되는 빼는 것입니다.)

2. "cats.repository.ts"에서 광범위하게 잡은 Cat 타이핑 대신 CatCurrentDto를 사용합니다.

3. "user.decorator.ts"에서 request.user의 타입을 CatCurrentDto으로 캐스팅합니다. 

일반적으로 request.user 는 any 타입입니다. express에서는 처음에는 어떤 user 타입인지 모르기 때문이죠.

(이 방법 외에 아예 처음부터 node_modules의 express의 User 타입 자체를 정의할 수도 있습니다.)

4. "cats.controller.ts"에서 @CurrentUser() cat : CatCurrentDto로 타이핑합니다.

추가적으로 이해안되는 것이 있으시면 말씀해주세요! 🙂

감사합니다 :)