[멘토님] 미니 프로젝트 코드 리뷰 요청

[khsrla9806]

리뷰 범위가 너무 많아서 리뷰 받고 싶은 부분 추려 봅니다.

🙋🏻‍♂️ 코드리뷰 요청

강경민

@query에 작성한 쿼리 위주로 코드리뷰 받고 싶습니다. 🙆🏻‍♂️

• 개인 연차 취소 및 취소 신청 기능 구현
• 개인 당직 조회 기능 구현
• 개인 연차 조회 기능 구현
• 연차 당직 조회 기능 구현

김훈섭

Validation을 사용하는 모든 Controller에서 공통적으로 사용되는 코드라고 생각돼서 Utils 클래스로 분리를 해두었는데, 반복적이라도 그냥 각 Controller에서 각자 작성하는게 나았을지 궁금합니다. AOP로 해볼까 하다가 시간이 부족해서 그냥 넘겼던 부분인데, 궁금함이 남아서 리뷰 부탁 드립니다 🙆🏻‍♂️

• bindingResult 결과를 받아서 유효성 검사를 진행하는 ValidationUtils 추가
• ValidationUtils 사용하는 부분

@query를 사용해서 쿼리 결과를 Projection할 때, JPQL 안에서 new 연산자를 통해서 DTO를 생성하거나, 인터페이스 구현(interface + getter)을 통해서 받아오는 방법이 있다고 해서, 제가 느끼기에는 interface + getter를 사용해서 결과를 받아오는 것이 가독성이 좋아보여서 이렇게 처리했는데, 혹시 현업에서 이런 경우가 있다면 어떤 방법을 이용해서 Projection 결과를 DTO에 담게 되나요? QueryDSL을 쓰면 좀더 깔끔하게 사용할 수 있다고 하는데, 해당 내용을 학습하기 이전이어서 리뷰 부탁드립니다! 🙆🏻‍♂️

• AdminDto 생성 후 JPQL Projection 결과를 담을 interface 구현
• AdminMemberRepository 생성 후 JPQL을 사용하여 쿼리 메서드 구현