[BE][refactor] 에러 코드 컨벤션 수정

[minSsan]

1. 에러 코드 컨벤션 변경

지난 오프라인 회의 때 논의된 에러 코드 컨벤션을 아래와 같이 수정하는 것을 제안합니다.

맥락 (ex. "리소스를 찾을 수 없음") 을 기준으로 객체를 분류하고, status 값과 message를 내부 필드로 정의하여 캡슐화하기
👉🏻 status 값을 캡슐화함으로써, @Jin409 가 우려한 에러 코드의 status 값 변경에 대한 대응이 더 수월하지 않을까?

// ErrorCode.java

@Getter
public enum ErrorCode {

    NOT_FOUND_MEMBER(HttpStatus.NOT_FOUND, "존재하지 않는 회원입니다."),
    NOT_FOUND_CUP(HttpStatus.NOT_FOUND, "존재하지 않는 컵입니다."),

    INVALID_ENUM_VALUE(HttpStatus.BAD_REQUEST, ""),
    INVALID_CUP_AMOUNT(HttpStatus.BAD_REQUEST, ""),
    INVALID_CUP_NICKNAME(HttpStatus.BAD_REQUEST, ""),
    INVALID_CUP_COUNT(HttpStatus.BAD_REQUEST, ""),
    DUPLICATED_CUP_RANKS(HttpStatus.BAD_REQUEST, ""),
    INVALID_DATE_RANGE(HttpStatus.BAD_REQUEST, ""),
    ;
    
    private final HttpStatus httpStatus;
    private final String description;

    ErrorCode(HttpStatus httpStatus, String description) {
        this.httpStatus = httpStatus;
        this.description = description;
    }
}

// CommonException.java

@Getter
public class CommonException extends RuntimeException {

    private final ErrorCode errorCode;

    public CommonException(ErrorCode errorCode) {
        super(errorCode.getDescription());
        this.errorCode = errorCode;
    }
}

// GlobalExceptionHandler.java

@RestControllerAdvice
public class GlobalExceptionHandler{

    @ExceptionHandler(HttpMessageNotReadableException.class)
    public ErrorResponse<FailureBody> handleInvalidEnum(HttpMessageNotReadableException e) {
        return ErrorResponse.from(BadRequestErrorCode.INVALID_ENUM_VALUE);
    }

    @ExceptionHandler(CommonException.class)
    public ResponseEntity<FailureBody> handleCommonException(CommonException e) {
        ErrorCode errorCode = e.getErrorCode();
        return ResponseEntity.status(errorCode.getHttpStatus()).body(new FailureBody(errorCode.name()));
    }
}

2. ErrorResponse 객체 삭제 제안

관련 코멘트

코멘트 인용:

그리고 개인적으로 예외 응답 커스텀을 ResponseEntity를 상속해서 하는게 맞는지 살짝? 의문이 들어요

ResponseEntity는 헤더를 포함한 응답 정보를 담고 있는데, 헤더를 제외하고 body만 커스텀해도 충분하지 않을까 하는 생각이 들었습니다
👉🏻 커스텀 예외에서는 불필요한 인터페이스까지 노출되는 느낌이랄까..?