본문 바로가기
멋쟁이사자처럼 동아리/코드 리뷰

코드리뷰 Spring MVC

by PlusUltraCode 2024. 6. 22.

코드 리뷰 요약 및 권장사항

HomeController.java

리뷰어 의견:

  • hkjbrian: @GetMapping을 통해 기본 페이지를 설정해주신 것 같아요. 물론 가능한 방법이지만, 스프링은 정적(static) 경로에서 index를 찾고 그 후 템플릿(templates) 경로에서 index를 가장 먼저 찾게 되어있다고 배웠던 내용을 활용하는 것도 좋을 것 같아요!
  • PlusUltraCode: 네, 참고할게요. 감사합니다.

ReservationController.java

리뷰어 의견 1:

  • hkjbrian: Reservation 클래스를 활용하여 Post 기능을 구현하셨군요. dto를 활용하면 오류가 날 확률이 줄어들고 가독성이 더 좋아질 것 같아요. dto를 활용하여 RequestReservation과 같은 이름으로 만들고, id를 제외한 3개의 멤버변수만 갖게 설정하는 것이 좋습니다. 관련 자료를 첨부합니다.
  • PlusUltraCode: 저도 id 처리에 대해 고민하고 있었습니다. 자료까지 첨부해주셔서 너무 감사합니다!

리뷰어 의견 2:

  • hkjbrian: HttpHeaders 설정 부분에 대해 새로운 방법을 알게 되었네요. 감사합니다!
  • PlusUltraCode: 저도 도움이 되셨다니 기쁩니다.

Exception.java

리뷰어 의견:

  • hkjbrian: Exception 클래스를 따로 설정하여 예외 처리를 한 번에 해두신 점이 매우 좋은 것 같아요. 저도 코드 리팩터링할 때 적용해보도록 하겠습니다.
  • PlusUltraCode: 네, 감사합니다.

이제 프로젝트 코드가 전반적으로 더 일관성 있고 효율적으로 개선되었습니다. 리뷰어의 피드백을 적극 반영하여 더 나은 코드 품질을 유지해 나가도록 하겠습니다. 앞으로도 코드 컨벤션과 설계 원칙을 준수하여 협업하는 동료들과 함께 높은 품질의 소프트웨어를 개발하겠습니다.