f-lab-edu / JimCarry

0 stars 1 forks source link

CustomException 관련 이슈 #31

Closed HongSeokWoo1991 closed 1 day ago

HongSeokWoo1991 commented 5 days ago

1.접근 제어자(private, protected, public)가 없는 이유는 무엇인가요? [JimCarry/src/main/java/com/study/jimcarry/exception/CustomException.java] private static final long serialVersionUID = 1L; int code;

  1. 200 부분 보다는 명확한 상수를 사용하도록 수정하여 의미를 더 명확하게 표현하면 좋겠습니다. public CustomException(String msg){ super(msg); this.code = 200;

3.this.code = 200;, this.code = code;, this.code = errCode.getCode();와 같은 코드가 반본적으로 사용되고 있습니다. 이를 하나의 생성자로 통합하는 형태를 고민해 보시겠어요? public CustomException(int code, String msg) { super(msg); this.code = code; }

 public CustomException(String msg) {

this(xxxx, msg); }

public CustomException(ErrorCode errCode) { this(errCode.getCode(), errCode.getMessage()); }

4.실제로 직렬화될 필요가 있는지 궁금합니다. 대부분의 경우 @ExceptionHandler로 예외를 처리하고 HTTP 응답을 반환하는 방식이 일반적인 방식으로 보이는데요. 의도를 문의 드립니다.