피드백 - 정현우 튜터

[Nuung]

안녕하세요 서린님!! 매번 새로운 도전과제가 생겨서 쉽지않음에도 잘 해주시고 계셔서 좋습니다!! 🔥🔥🔥

전체 과제 체크 사항

• 아직 서버가 배포가 안되었네요 :) 고로,, 코드 기반 피드백 부터 드릴게유!
• swagger 를 적용하려는 모습 너무 좋습니다! 이 경우 배포 후, 해당 스웨거 링크까지 첨부해 두신다면, 정말 좋겠죠?! 쪼금 아쉽네요!!
• 보너스 기능 구현 못해도 좋으니,, 시도라도 해보면 더 좋을 것 같아요,, 너무 아쉽습니다.
• 그래도 필수 기능 자체는 핵심 구현을 잘 해주신것 같아요!
• 이제 CI/CD 달아 볼까요? 왜 테스트 코드는 하나도 없나요 ㅠㅠ 🥹🥹
• 세부 피드백을 한 번 봐주시고, 모델링 부터 다시 해볼까요?! 이번엔 마무리 해보자구요!

필수 기능 구현 리스트

• [x] 로그인 / 회원가입
• [x] 프로필 보기
• [x] 새 공연 등록
  • 조금 애매한 등록입니다 ㅎㅎㅎ!
• [x] 공연 목록 보기
• [x] 공연 검색하기
• [x] 공연 상세보기
• [x] 좌석을 지정하지 않고 공연 예매하기
• [x] 예매 확인하기

보너스 기능 구현 리스트

• [ ] 공연의 좌석 예매 정보 확인하기
• [ ] 좌석을 지정하여 예매하기
• [ ] 동시성 처리하기
• [ ] 예매 취소하기
• [ ] 테스트 코드 내용 채우기

---

세부 피드백

인가 인증

https://github.com/slianzg/nestproject-in/blob/f270d923657e7059056efe97c158f406179136b8/src/user/user.controller.ts#L28

https://github.com/slianzg/nestproject-in/blob/f270d923657e7059056efe97c158f406179136b8/src/auth/jwt.strategy.ts#L10-L30

• 쉽지 않았을텐데,, 아주 잘하셨습니다! jwt 를 그냥 응답으로 주는 대신 cookie set을 하는 것 좋습니다.
• 하나 고려해보면 좋은게 jwtFromRequest: ExtractJwt.fromExtractors([...]) 배열 형태로 들어가죠? 이 뜻은 여러 방법으로 "추출" 할 수 있는데, 즉, 지금 처럼 cookie 값에서만 auth 정보를 얻어올 수 있게 하면, header 에 { "authorization" : "Bearer ..." } 값이 있어도 인식을 하지 못합니다.
• 고로 fromAuthHeaderAsBearerToken 도 같이 사용하면 좋습니다.

user.service.ts

https://github.com/slianzg/nestproject-in/blob/f270d923657e7059056efe97c158f406179136b8/src/user/user.service.ts#L27

• user 라는 변수가 더 무의미 한 것 같고, 문맥상 더 의미가 옳지 않은 것 같아요.

https://github.com/slianzg/nestproject-in/blob/f270d923657e7059056efe97c158f406179136b8/src/user/user.service.ts#L41-L47

• 여기서 user 대신 createUserDto 가 훨씬 올바른 것 같아요. 왜냐면 user 만 사용하면 이미 이 유저는 만들어진 모델인가? 라는 착각을 만들어요. 지금 request 에서 아무 전/후 처리 없이 Dto 값만 쓰고 있는데, "사용자가 요청한 값" 이라는 점에서 createUserDto 가 나을 것 같고
• 이렇게 쓰고 싶으시면 차라리 구조 분해 할당 을 해보시는 것은 어떨까요?!

https://github.com/slianzg/nestproject-in/blob/f270d923657e7059056efe97c158f406179136b8/src/user/user.service.ts#L88-L92

• 이 라인때문에 point 의 서비스 로직까지 의존성을 가지는 것은 조금 별로인 것 같아요. 단순하게 find 하는 것인데, 차라리 repo 계층을 활용하는게 좋을 것 같아요!

show.controller.ts & show.service.ts

https://github.com/slianzg/nestproject-in/blob/f270d923657e7059056efe97c158f406179136b8/src/show/show.controller.ts#L26-L29

• Role 이 admin 인 사용자만 접근 가능하게 잘 만들어주셨습니다. src/auth/roles.guard.ts 역시 requiredRoles 좋았습니다.
• context.switchToHttp().getRequest(); 이게 납득이 안되실 수 있는데, 저희 특강때 nestjs 에서 Request 흐름을 한 번 보는게 좋을 것 같아요!

https://github.com/slianzg/nestproject-in/blob/f270d923657e7059056efe97c158f406179136b8/src/show/show.controller.ts#L37-L49

• 이 2가지 API는 합쳐질 수 있습니다. @Get('/') 으로요! 그리고 합쳐지는게 더 REST 아키텍쳐에 맞습니다.
• 합쳤으면 search 기능은 url query string 을 활용해야 합니다!
• https://docs.nestjs.com/controllers#request-object 에서 해답을 얻으실 수 있을거에요!

https://github.com/slianzg/nestproject-in/blob/f270d923657e7059056efe97c158f406179136b8/src/show/dto/create-show.dto.ts#L56-L59

• entity 는 array 가 아니어도 Dto 는 array가 될 수 있죠. DTO 는 Data Transfer Object 라서 계층간 데이터 공유에 활용이 되는 것일 뿐이고, 사용자의 request를 규격화 된 데이터 셋을 컨트롤러에서 처리하게 하죠!

https://github.com/slianzg/nestproject-in/blob/f270d923657e7059056efe97c158f406179136b8/src/show/show.service.ts#L17-L31

• 고로 create 할 때 단순하게 showDate 만 하면 안될 것 같아요. 혹시 "데이터 베이스 정규화" 에 대해서 아시나요?!
• 가장 정석적인 방법은 테이블 분리(정규화) 입니다. 배열 형태 저장은 제1정규형에 위반된다고 볼 수 있습니다.
• 하지만 이와 반대로 역 정규화 가 있죠! 괜히 조인해서 DB 성능 떨어뜨리지말고 그냥 배열 그대로 테이블에 집어넣어 단순하게 꺼내오도록 하는 겁니다!
• MySQL Ver. 5.7.8부터 아예 JSON 데이터를 저장할 수 있게 지원합니다!

reservation

https://github.com/slianzg/nestproject-in/blob/f270d923657e7059056efe97c158f406179136b8/src/reservation/reservation.service.ts#L18-L41

• 여기서 "동시에 누군가 예매를 하려고 한다면" 에 대한 "동시성" 처리를 하면 됩니다!
• 조회할때는 this.showService.findOne 가능했는데, 동시에 누군가 this.reservationRepository.save 를 처리해서 동시 예매가 되어버린다면?!
• 이때 상황을 "race condition" 이라고 합니다. 이를 동시 접근 때문에 발생하며, 동시성 처리를 해야하는 이유입니다.
• DBMS 의 "격리 수준(Lock)" 에 대해 알고 계시면 이해를 바탕으로 직접 해결 할 수 있으며, 더 쉽게는 "Transaction" 을 사용할 수 있습니다.
• 이미 prisma 에서 시도한 것들입니다!!!

[slianzg]

꼼꼼한 피드백 감사합니다!☺️

[Nuung]

[ 동시성 처리하기 ]

(1) 가장 쉽게 처리하는 방법은 query builder 를 호출하셔서 DBMS 에서 트랜잭션을 하는 것

(2) DBMS 의 Lock 을 활용하는 것

• https://orkhan.gitbook.io/typeorm/docs/select-query-builder#set-locking
• 이는 구현 난이도보다 이해의 난이도가 좀 더 있을 겁니다.

  (3) 3rd party (REDIS 등)를 활용해 외부의존성으로 동시성 해결하는 것