[🚀 사이클2 - 미션 (예약 변경/취소와 에러 처리)] 캐모(김제신) 미션 제출합니다.

[jason0904]

인삿말

안녕하세요 아마찌!
이번 사이클에서 요구했던 내용 중 여러 개가 저번 미션, 사이클에서 이미 구현했던 내용들이라, 빠르게 구현해봤습니다..!
이번 미션도 잘 부탁드립니다!

체크 리스트

• 미션의 필수 요구사항을 모두 구현했나요?
• Gradle test를 실행했을 때, 모든 테스트가 정상적으로 통과했나요?
• 애플리케이션이 정상적으로 실행되나요?

베이스 코드 선택 체크

• 이전 미션의 내 코드에서 시작
• 이전 미션의 페어의 코드에서 시작

어떤 부분에 집중하여 리뷰해야 할까요?

예외 코드에 대하여..

현재 저의 ErrorResponse에는 에러코드가 아닌 ErrorMessage 만을 던집니다.

package roomescape.exception;

import java.time.LocalDateTime;

public record ErrorResponse(String message, LocalDateTime timestamp) {
    public ErrorResponse(String message) {
        this(message, LocalDateTime.now());
    }
}

이렇게 구현을 하긴 했지만, 에러 응답 방식에 대해 조금 더 고민해 보았는데요. 클라이언트 개발자가 디버깅을 위해 확인해야 하는 에러 메시지와, 실제 사용자에게 노출되어야 하는 친절한 안내 메시지는 성격이 다를 수 있다고 생각합니다.
또한 향후 다국어 지원 등 시스템이 확장될 상황을 고려한다면, 프론트엔드에서 에러를 명확히 식별하고 유연하게 대처할 수 있는 기준이 필요할 것 같습니다.
이런 관점에서 텍스트 메시지와 더불어 고유한 에러 코드를 도입하는 것이 훨씬 효과적인 방법으로 느껴지는데, 이 방향에 대한 아마찌의 생각은 어떠신지 궁금합니다!

커스텀 Exception 구분에 대하여..

현재는 예외를 도메인별로 나누지 않고, 발생하는 상황을 기준으로 묶어서 처리하고 있는데요.
이렇게 하다 보니 예외의 성격이 비즈니스의 의미를 담기보다는, HTTP 상태 코드라는 웹 계층의 개념에 너무 강하게 의존하고 있는 것은 아닌가 하는 우려가 들었습니다. 특히 NotFoundException, UnauthorizedActionException같은 부분들을 보니 이름도 Http 상태코드에 의존되어있는 것 같더라고요..

도메인 계층의 순수성 관점에서 볼 때, 현재의 예외 네이밍과 분류 방식에 대해 아마찌의 의견이 궁금합니다!

테스트에 대하여..

지난 사이클에서는 Repository, Service, Controller를 각각 Mock을 활용해 엄격한 단위 테스트로 분리했었습니다. 하지만 계층이 나뉘어 있더라도 결국 하나의 절차적인 비즈니스 흐름을 갖기 때문에, Mock을 남용하기보다는 비즈니스 로직을 검증하는 통합 테스트로 묶어내는 것이 더 실용적이라고 판단했습니다.

이에 따라 현재는 Controller 파라미터/DTO 검증 + Service 통합 테스트 + E2E 테스트 구조로 개편했는데, 결과적으로 중복되거나 의미 없는 테스트가 확연히 줄어든 것을 느꼈습니다.
다만, Layered Architecture의 원칙을 생각하면 여전히 계층별로 철저히 분리해서 테스트하는 것이 정석일까 하는 일말의 고민도 남아있는데요. 이 부분에 관한 아마찌의 생각이 궁금합니다..!

DTO의 디렉터리 위치에 대하여

저번 사이클 마지막 커멘트로 DTO의 위치에 관련되서 이야기를 해주셨는데요..
생각을 계속 해봤는데, 처음에는 Controller 패키지 내부에 DTO 패키지를 넣을 경우, 패키지 Depth가 불필요하게 깊어지고 기존에 의존성 방향을 명확히 하려던 패키지 분리의 의미가 다소 희석될 수 있다고 생각하여 일단 현재 구조를 유지했습니다.
하지만 며칠 더 코드를 들여다보니, Controller와 라이프사이클을 완전히 함께하는 DTO들을 하나의 디렉터리로 묶어두는 것이 파일 관리나 전체적인 구조 면에서 훨씬 깔끔할 수 있겠다라는 생각이 들기도 하네요..
결국 패키지 Depth 증가와 파일 관리의 응집도 향상 사이에서 팽팽하게 고민이 되는데요, 현업에서는 이 트레이드오프를 어떻게 다루는 편인지 아마찌의 생각이 궁금합니다!