작성
·
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 를 어떻게 구현하는지 궁금합니다.
긴 글 읽어주셔서 감사합니다.
넵 답변 감사합니다!