DolphaGo / TIL

TIL & issues
0 stars 1 forks source link

[SonarQube] New exception is thrown in catch block, original stack trace may be lost #98

Open DolphaGo opened 2 years ago

DolphaGo commented 2 years ago

Exception을 Rewrapping하여 throw할 때 소나큐브에 걸리는 코드 스멜이다.

로그로 기존의 exception을 출력하고 있었으나, 다시 throw를 할 때는 custom message로 넣을 때 소나큐브에서는 기존의 exception stack trace를 잃는다는 점에서 이러한 코드 스멜이 나타난다.

CustomException을 굳이 리턴하고 싶다면 소나큐브에서는 throwable 객체를 생성자로 추가하라는 것을 권장한다.

    public OrderCreateException(final String message, final Throwable cause) {
        super(message, cause);
    }