Skip to content

Conversation

@dongchan0105
Copy link

✅ 워크북 체크리스트

  • 모든 핵심 키워드 정리를 마쳤나요?
  • 핵심 키워드에 대해 완벽히 이해하셨나요?
  • 이론 학습 이후 직접 실습을 해보는 시간을 가졌나요?
  • 미션을 수행하셨나요?
  • 미션을 기록하셨나요?

✅ 컨벤션 체크리스트

  • 디렉토리 구조 컨벤션을 잘 지켰나요?
  • pr 제목을 컨벤션에 맞게 작성하였나요?
  • pr에 해당되는 이슈를 연결하였나요?
  • 적절한 라벨을 설정하였나요?
  • 스터디원들에게 code review를 요청하기 위해 reviewer를 등록하였나요?
  • 닉네임/main 브랜치의 최신 상태를 반영하고 있는지 확인했나요?

📌 주안점

Copy link
Member

@jjin70 jjin70 left a comment

Choose a reason for hiding this comment

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

커밋을 세세하게 나눠서 해서 보기 좋은 것 같습니다~! 예외 처리도 더 추가한 것두 좋습니다
controller단에서 ResponseEntity를 사용하셨는데 응답 형식 통일이 되었기 때문에 상태 코드 지정을 위한 게 아니라면 생략해도 괜찮을 것 같다는 점만 참고해주시면 될 것 같습니다

  • MethodArgumentNotValidException
  • ConstraintViolationException
  • MethodArgumentTypeMismatchException
  • HttpMessageNotReadableException 등등 ..
    추가적으로 위와 같은 예외도 처리하면 좋을 것 같아서 살짝.. 알려드립니다

응답 통일을 하면 프론트엔드와 소통도 편해지고, 예외 처리를 해둬야 혹시 문제가 생겼을 때 사용자에게 더 적절한 상황 전달을 할 수 있다고 생각해서 프로젝트 초반에 세팅하고 시작하는 걸 추천드립니다. 7주차도 수고하셨고, 이제 정말 얼마 안 남았는데 끝까지 파이팅입니다!!

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.

3 participants