연차/당직, 관리자 리뷰드립니다! 🙇

[seonghye0n]

안녕하세요!🙇
제가 미니프로젝트에서 연차/당직, 관리자 부분을 맡았어서 부족하지만 제가 아는 지식 선에서 최대한 리뷰 작성을 해봤습니다!

예외처리

1. 현재 OrderStatus의 예외처리와 OrderType의 예외처리가 서로 달라서 맞춰주시면 좋을 것 같습니다!
2. 연차 신청 시, annualCount를 업데이트 하는 부분이 있는데 현재 조건을 봤을 때, 사용자가 잔여 연차가 1일 일 경우에 2일~3일 연차 사용 시 가능해보일 것 같습니다!
   -> 내년 연차를 당겨쓰는 것을 의도하신거라면 무시하셔도 될 것 같습니다..!
3. 연차 신청 시, 시작일 > 종료일인 경우에 대한 예외처리가 없는 것 같습니다!
4. 연차/당직 삭제 메서드에서 다른 사용자의 연차/당직을 삭제할 수 있게 되어있는 것 같습니다!
5. 현재 로직이 사용자가 연차 신청, 삭제할 경우에 연차 수 업데이트가 진행되는 것 같은데 관리자가 반려해도 연차 수가 그대로 인 것 같습니다!

---

개선점

1. OrderService 클래스에 @transactional 어노테이션이 붙어있는데, 그렇다면 find메서드에는 readOnly = true 옵션을 주면 어떨까 싶습니다!
2. 서비스 단에서 처리하는 예외들에 메세지가 겹치는 것들이 있는데 오타 방지 등을 위해 enum 클래스로 정의하여 사용하면 좋지 않을까 싶습니다!
3. 결재 메서드에서 두 개의 테이블을 업데이트하는데 트랜잭션 처리가 안되어 있는 것 같습니다!

도움이 되시는 내용들이 있으면 좋겠네요! 다들 화이팅입니다!💪

[ho30archive]

신경써서 리뷰 남겨주셔서 감사합니다!🙇‍♂️