Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[6주차] 기본/도전 과제 제출 #5

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

[6주차] 기본/도전 과제 제출 #5

wants to merge 3 commits into from

Conversation

m1njae
Copy link
Member

@m1njae m1njae commented Nov 24, 2022

SERVER PR


🐕 과제 구현 명세

  • [기본] 오늘 세미나에서 나온 코드들을 직접 타이핑하며 익혀보기, 기존 CRUD API들 세미나에서 배운 것들로 리팩토링
    dto를 활용해서 코드 리팩토링 진행하였습니다. 세미나에서 배운 auth를 추가하여 route에 넣어주었습니다.
    유저 정보 수정 API 경우 Prisma + spread 연산자를 사용해서 여러가지 수정 사항을 변경할 수 있도록 구현했습니다.
  • [도전] 세미나에서 배운 내용을 기반으로 넷플릭스 API 모두 리팩토링
    마찬가지로 dto를 활용하여 코드 리팩토링 진행하였고, auth를 route에 넣어주었습니다. 클라분들에게 데이터를 이쁘게 드리기 위해서 데이터 가공도 진행해보았습니다. 콘텐츠 정보 조회는 switch문을 활용하여 query가 있을경우, 에피소드 조회가 가능하도록 구현했으며, query가 없을 경우 전체 에피소드 조회가 가능하도록 구현했습니다.

🐥 이런 점이 새로웠어요 / 어려웠어요

  • 지난기수 미들웨어가 어려웠어서 그냥 흘러가듯 지나쳤는데 팟장님이 차근차근 잘 설명해주셔서 코드 한 줄 한 줄 음미해보았습니다. 어느정도 개념이 정립된거 같아요!
  • dto 알듯말듯한 친구인거 같아요! 조금 더 공부해보도록 하겠습니다.
  • 사실 아직까지는 예외처리가 미숙해서 다시 한 번 플로우를 살펴보고 고민해보아야 될 것 같습니다!

Comment on lines +101 to +105
if (!userUpdateDto) {
return res
.status(statusCode.BAD_REQUEST)
.send(fail(statusCode.BAD_REQUEST, message.UPDATE_USER_FAIL));
}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이 조건문이 userUpdateDto가 null이면 이라는 뜻이 맞을까요??
그럼 userUpdateDto의 변수 하나가 null인건 체크가 안되는 게 맞겠죠? 헷갈려서 물어봅니다!

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

네! 그런 의도로 작성하긴 하였습니다:)

id: userId,
},
data: {
...userUpdateDto,

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

혹시 ... 이건 어떤 의미인가요? 연산자로 검색해봤는데 안나와서 여쭤봅니다!

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spread 연산자라고 합니다! prisma data에서 spread 연산자를 사용한다면 변경된 내용 모두를 보여준다고 팟장님께서 알려주셔서 적용해보았습니다😊

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

오 그렇군요! 바로 찾아보겠습니다!

Comment on lines +14 to +20
include: {
episode: {
where: {
episodeNumber: episode,
},
},
},

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

include 배워갑니다!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants