inflearn logo
강의

강의

N
챌린지

챌린지

멘토링

멘토링

N
클립

클립

로드맵

로드맵

지식공유

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

문제와 풀이2

Enum 과 null object pattern

234

rjf1138

작성한 질문수 12

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 를 어떻게 구현하는지 궁금합니다.

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

java 객체지향 null enum

답변 1

1

David

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

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

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

잘 구현하신 것 같습니다.

감사합니다.

0

rjf1138

넵 답변 감사합니다!

문제와 풀이

0

23

1

섹션 2 퀴즈 9번

0

31

2

강의노트 내용이 이해가 되지 않아서 문의드립니다.

0

47

1

중급1편, 래퍼클래스 문제와풀이2 로또제너레이터 질문

0

67

1

setVlaue질문

0

92

3

안녕하세요 코딩테스트와 병행하고 싶어서 질문드립니다.

0

93

1

findValue의 파라메터까지 래퍼클래스 배열인 이유가 있을까요?

0

74

1

아래 글처럼 toString() 강의부터 화면이 안보여요

0

99

2

TemporalAdjusters의 lastDayOfNextMonth(), lastDayOfNextYear() 메서드 지원 안함

0

84

2

오타 제보도 되나요?

0

90

2

지역 클래스 예제3 지역변수 질문

0

69

2

래퍼클래스 박싱 출력

0

60

1

자바 equals() 메소드에 대한 물리적 동일성, 논리적 동등성 질문

0

105

3

강의록 오타 제보

0

69

2

자바의 기본 클래스들에 관하여

0

70

1

실무에서 중첩(내부) 클래스 사용하나요?

0

79

1

지역 클래스 - 지역 변수 캡처3

0

55

1

12:30 의 newClassGrade 질문이 있습니다.

0

57

1

날짜와 시간 문제풀이가 막히는데

0

100

2

내부 클래스의 바깥 클래스 참조

0

68

1

스트링 풀, new String()

0

105

3

정적 중첩 클래스 사용 이유

0

97

1

status.getCode() 대신 status.code 사용

1

81

1

메서드의 지역변수명과 내부클래스의 멤버메서드에서의 지역변수명의 중복관련

0

75

1