[refactor] 코드 스타일 통일

[june-777]

💡 다음 이슈를 해결했어요.

Issue Link - #63

💡 이슈를 처리하면서 추가된 코드가 있어요.

1. 컨트롤러 계층 - xxxRequest, 서비스 계층 - xxxCommand

• 서비스 계층은 xxxCommand 파라미터 하나만 받도록 수정했습니다.
• 그에 따라, 기존에 없었던 Vendor 조회 로직, 검증 로직등이 추가되었습니다.
• request, command DTO를 분리함에 따라 메뉴 등록 DTO인 MenuLineItem을 외부 record로 분리했습니다.

2. 컨트롤러 메서드 반환타입 - xxxResponse

3. 커스텀 예외 - BadRequestException, NotFoundException 등을 상속하도록 수정

4. 서비스 계층 반환 타입 - void 에서 Long으로 수정

5. 도메인 컨트롤러에서 예외 핸들링

💡 이런 고민을 했어요.

각 도메인 컨트롤러에서 핸들링을 세부적으로 할 필요가 있을까?

• 커스텀 예외에 대한 회의 떄, 상속 계층으로 예외를 핸들링하기 용이한 장점이 있는데, 현재는 그러한 장점을 활용하지 못하고 있다.
• 세부적으로 예외를 핸들링할 필요가 있을까? 예외를 캐치해서 수행하는 작업이 모두 동일하다. 그냥 BadRequestException, NotFoundException 등등 상위 예외만 핸들링하면 되지 않을까
• 그럼 결국 GlobalExceptionHandler에서 공통으로 핸들링하는게 낫지 않을까

✅ 셀프 체크리스트

• 내 코드를 스스로 검토했습니다.
• 필요한 테스트를 추가했습니다.
• 모든 테스트를 통과합니다.
• 브랜치 전략에 맞는 브랜치에 PR을 올리고 있습니다.
• 커밋 메세지를 컨벤션에 맞추었습니다.
• wiki를 수정했습니다.

[kimhyun5u]

저도 도메인 예외 처리가 모두 같은 작업을 처리해서 예외 처리에 대해서 고민이 생기긴 하네요!

도메인 별로 분류하면 나중에 멀티모달 등으로의 확장에 용이하긴 하지만 아직은 큰 효용이 없는 것 같긴해요 😅

[Hyeon-Uk]

좋습니다! 고생하셨습니다!!!!!