refactor: CommentService

[sudo-Terry]

• 비즈니스 로직의 값을 검증하는 역할은 EntityValidator에게 일임한다
  • 값에 문제가 있으면 GlobalExceptionHandler까지 예외를 던져서 클라이언트에 오류 코드를 내려줌
• 역할을 분리함에 따라 테스트도 별도로 분리

[mjkhub]

보통 Validator라고 하면 엔티티의 상태를 검증하는 역할을 하게 될것 같은데, DB에서 엔티티를 직접 가져오는 것까지 담당한다면 "한 클래스가 많은 책임을 담당하는게 아닌가?" 라는 생각이 들었습니다.

감정밴드의 유효성을 판단 로직을 별도의 클래스에서 처리하는 것은 매우 좋다고 생각합니다!

[sudo-Terry]

@mjkhub
동의합니다~ 쿼리자체는 서비스에서 하고, Validator에서는 받아온 값을 서비스 로직에 따라 검증하는 것만 하는게 좋아보여요. 테스트코드까지 구성을 하고 다시 리뷰요청 드릴게요~!