인프런 커뮤니티 질문&답변

rjf1138님의 프로필 이미지
rjf1138

작성한 질문수

김영한의 실전 자바 - 중급 1편

문제와 풀이2

Enum 과 null object pattern

작성

·

127

·

수정됨

1

public enum HttpStatus {
    NONE(-1, "정의되지 않은 상태코드입니다."),
    OK(200, "OK"),
    BAD_REQUEST(400, "Bad Request"),
    NOT_FOUND(404, "Not Found"),
    INTERNAL_SERVER_ERROR(500, "Internal Server Error");

    ...
    public static HttpStatus findByCode(int code) {
        for (HttpStatus status : HttpStatus.values()) {
            if (status.getCode() == code) {
                return status;
            }
        }
        return HttpStatus.NONE;
    }

    public boolean isSuccess() {
        return code >= 200 && code < 300;
    }
}
public class HttpStatusMain {

    public static void main(String[] args) {
        Scanner scanner = new Scanner(System.in);
        System.out.print("HTTP CODE: ");
        int httpCodeInput = scanner.nextInt();
        HttpStatus status = HttpStatus.findByCode(httpCodeInput);


        System.out.println(status.getCode() + " " + status.getMessage());
        System.out.println("isSuccess = " + status.isSuccess());
    }

}


안녕하세요 김영한 강사님. 먼저 항상 양질의 강의를 제공해주셔서 감사하다는 말씀을 전해드리고 싶습니다.

HttpStatusMain 에서 null 체크하는 일이 발생하여 이를 제거해보면 어떻게 해야할까 에 대한 고민이 들었습니다.
그래서, Enum 에 null object pattern 을 적용해보면 어떨까 하여 HttpStatus.NONE(-1, "정의되지 않은 상태코드입니다.") 와 같이 정의해봤습니다.

이렇게 적용했을 때,
1. 다른 사이드 이펙트가 있을까요?
1-1. findByCode() 에서 return HttpStatus.NONE 이 아닌 throw new IllegalArgumentException("해당하는 상태코드가 없습니다."); 와 같이 RuntimeException 을 날리는 것이 더 나은 방법일까요?
2. 현업에서는 enum 타입의 null check, null object pattern 를 어떻게 구현하는지 궁금합니다.

긴 글 읽어주셔서 감사합니다.

답변 1

1

안녕하세요. rjf1138님, 공식 서포터즈 David입니다.

특별히 예상되는 사이드 이펙트는 잘 모르겠습니다.

케이스마다 조금 다를 것 같은데 예외를 발생시키고 발생된 예외를 잡아서 어떤 처리(예: 사용자 오류 응답 등)를 해야 한다면 예외를 발생시킬 것 같습니다. 다른 케이스는 NONE이라는 상태를 구분하여 가져야 한다면 예외보다는 NONE으로 반환하는 것도 고려할 것 같습니다.

잘 구현하신 것 같습니다.

감사합니다.

rjf1138님의 프로필 이미지
rjf1138
질문자

넵 답변 감사합니다!

rjf1138님의 프로필 이미지
rjf1138

작성한 질문수

질문하기