2026-05-22T12:11:33.219Z

0

넵 코드는 약간 수정을하였습니다.

@Slf4j
@RestControllerAdvice
public class GlobalExceptionHandler {

    @ExceptionHandler(ApiException.class)
    public ProblemDetail handleApiException(ApiException e) {
        log.error("API Exception message={}, className={}", e.getErrorMessage(), e.getClass().getName());
        return getProblemDetail(e.getHttpStatus(), e.getMessage(), e.getErrorType());
    }

    @ExceptionHandler(Exception.class)
    public ProblemDetail handleException(Exception e) {
        log.error("Exception message={}, className={}", e.getMessage(), e.getClass().getName());
        return getProblemDetail(
                HttpStatus.INTERNAL_SERVER_ERROR,
                ErrorType.UNKNOWN.getDescription(),
                ErrorType.UNKNOWN);
    }

    @ExceptionHandler(NoResourceFoundException.class)
    public ProblemDetail handleNoResourceFoundException(NoResourceFoundException e) {
        log.error("NoResourceFoundException message={}, className={}", e.getMessage(), e.getClass().getName());
        return getProblemDetail(
                HttpStatus.NOT_FOUND,
                ErrorType.NO_RESOURCE.getDescription(),
                ErrorType.NO_RESOURCE);
    }

    @ExceptionHandler(BindException.class)
    public ProblemDetail handleBindException(BindException e) {
        log.error("Bind message={}, className={}", e.getMessage(), e.getClass().getName());
        return getProblemDetail(HttpStatus.BAD_REQUEST, createMessage(e), ErrorType.INVALID_PARAMETER);
    }

    private String createMessage(BindException e) {
        if (e.getFieldError() != null && e.getFieldError().getDefaultMessage() != null) {
            return e.getFieldError().getDefaultMessage();
        }
        return e.getFieldErrors().stream()
                .map(FieldError::getField)
                .collect(Collectors.joining(", ")) + " 값들이 정확하지 않습니다.";
    }

    private ProblemDetail getProblemDetail(HttpStatus httpStatus, String message, ErrorType errorType) {
        ProblemDetail problemDetail = ProblemDetail.forStatusAndDetail(httpStatus, message);

        problemDetail.setProperty("exception", errorType);
        problemDetail.setProperty("timestamp", LocalDateTime.now());

        return problemDetail;
    }
}

만약 요청예시가

curl --location 'localhost:8080/v1/books?query=HTTP&page=-1&size=-1

이런 식으로 나가서, page 및 size 의 유효성 검증이 위반되었을때

현재는

{
    "detail": "페이지 번호는 1이상이어야합니다.",
    "instance": "/v1/books",
    "status": 400,
    "title": "Bad Request",
    "exception": "INVALID_PARAMETER",
    "timestamp": "2026-05-27T19:15:01.156153"
}

이런식으로 내려주고 있습니다.

메시지를 생성해주는

if (e.getFieldError() != null && e.getFieldError().getDefaultMessage() != null) {
    return e.getFieldError().getDefaultMessage();
}

해당 부분에서의 조건이 항상 참이기 떄문에 그런것이라고 생각하는데,

return e.getFieldErrors().stream()
        .map(FieldError::getField)
        .collect(Collectors.joining(", ")) + " 값들이 정확하지 않습니다.";

해당 코드는 default 메시지가 null 인 케이스만을 대비하기 위한 의도로 작성된 것인지

아니면 유효성검증을 여러개 통과하지 못하였을때,

{
    "detail": "size, page 값들이 정확하지 않습니다.",
    "instance": "/v1/books",
    "status": 400,
    "title": "Bad Request",
    "exception": "INVALID_PARAMETER",
    "timestamp": "2026-05-27T19:18:27.164512"
}

다음과 같이 내려오는 것을 의도하고 작성하신 코드인지 궁금합니다.

0

질문 잘 짚어주신거같습니다! 분석하신 내용이 맞습니다. 정리하면

default message가 null인케이스만을 방어하기위한 fallback이라고 봐주시면 될것같고, 여러필드를 각각 에러처리를 한다고하면 항상 stream을 돌리는게 맞을거같네요.

필드각각을 처리한다면 아래처럼 수정을 해야될거같습니다.

private String createMessage(BindException e) {
    return e.getFieldErrors().stream()
            .map(fe -> {
// 로직 필요에 맞게 수정
                String msg = fe.getDefaultMessage();
                return msg != null ? msg : fe.getField() + " 값이 정확하지 않습니다.";
            })
            .collect(Collectors.joining(", "));
}

혹은 둘중 하나라도 틀리면 그냥 공통된 메세지로 처리해도 될거같구요.

코드의 의도가 제대로 드러나지 않은 부분을 잘 짚어주신거같습니다!

2026-05-21T05:56:16.223Z