Refactor/1365 커스텀 예외 수정

[This2sho]

#️⃣연관된 이슈

#1365

📝작업 내용

BadRequestException

변경 전

@Getter
public class BadRequestException extends RuntimeException {

    private BadRequestCode codeAndMessage = BadRequestCode.findByClass(this.getClass());

    private String message;
    private int code;

    public BadRequestException() {
        this.message = codeAndMessage.getMessage();
        this.code = codeAndMessage.getCode();
    }
}

변경 후

@Getter
public class BadRequestException extends RuntimeException {

    private String message;
    private int code;

    public BadRequestException(BadRequestCode badRequestCode) {
        this.message = badRequestCode.getMessage();
        this.code = badRequestCode.getCode();
    }
}

• 생성자에서 BadRequestCode를 받게해서 무조건 예외 등록을 하도록 수정

BadRequestCode

변경 전

@AllArgsConstructor
@Getter
public enum BadRequestCode {
    NOT_FOUND_ERROR_CODE(0000, "해당 에러의 에러코드를 찾을 수 없습니다.", NotFoundErrorCodeException.class),

    ...

    CURRICULUM_NOT_FOUND_EXCEPTION(8007, "해당하는 커리큘럼을 찾을 수 없습니다", CurriculumNotFoundException.class);

    private int code;
    private String message;
    private Class<? extends BadRequestException> type;

    public static BadRequestCode findByClass(Class<?> type) {
        return Arrays.stream(BadRequestCode.values())
            .filter(code -> code.type.equals(type))
            .findAny()
            .orElseThrow(NotFoundErrorCodeException::new);
    }

}

public class NotFoundErrorCodeException extends BadRequestException {

}

public class CurriculumNotFoundException extends BadRequestException {

}

…

변경 후

@AllArgsConstructor
@Getter
public enum BadRequestCode {
    NOT_FOUND_ERROR_CODE(0000, "해당 에러의 에러코드를 찾을 수 없습니다."),
		
		... 

    ROADMAP_KEYWORD_SEQUENCE_EXCEPTION(8007, "키워드 시퀀스가 유효하지 않습니다.");

    private int code;
    private String message;
}

• 복잡한 Custom 예외 다 삭제 ⇒ BadRequestCode만 관리하도록 변경 (CustomException 만들 필요 없음)

실제 코드 작성에서 바뀐 부분

변경 전

public MemberTags(List<MemberTag> values) {
        final int originalSize = values.size();
        final long count = values.stream().distinct().count();
        if(originalSize != count) throw new DuplicateMemberTagException();
        this.values = values;
    }

변경 후

public MemberTags(List<MemberTag> values) {
        final int originalSize = values.size();
        final long count = values.stream().distinct().count();
        if (originalSize != count) {
            throw new BadRequestException(DUPLICATE_MEMBER_TAG);
        }

        this.values = values;
    }

• 기존에는 CustomException을 알고 있었다면, 바뀐 후에는 BadRequestCode를 알고 있도록 변경

---

[sonarcloud]

SonarCloud Quality Gate failed.   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
14 Code Smells

65.9% Coverage
1.0% Duplication

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

[splitCoding]

호이 쓰러지는거 아니죠?

[This2sho]

호이 쓰러지는거 아니죠?

뒷목잡고 쓰러질뻔했어여

[splitCoding]

호이 쓰러지는거 아니죠?

뒷목잡고 쓰러질뻔했어여

수고햇어유우ㅋㅋ

[java-saeng]

호이 고생하셨습니다,,

그런데 제가 했던 것들이 왜 호이 커밋에 포함되어있는거죠,,? ㅋㅋㅋㅋ

괜찮을 줄 알았는데,, 아직 main 에 들어가야하지 말 것이 들어가있습니다,,

이 커밋 에서

이 파일입니다,,