[REFACTOR] 에러코드 개선 (#316)

* refactor: Apply 도메인 에러 코드를 문자열 형태로 수정

* refactor: Auth 도메인 에러 코드를 문자열 형태로 수정

* refactor: File 도메인 에러 코드를 문자열 형태로 수정

* style: 클래스 끝에 개행 추가

* refactor: Member 도메인 에러 코드를 문자열 형태로 수정

* refactor: Project 도메인 에러 코드를 문자열 형태로 수정

* refactor: Question 도메인 에러 코드를 문자열 형태로 수정

* refactor: Recruit 도메인 에러 코드를 문자열 형태로 수정

* refactor: Semester 도메인 에러 코드를 문자열 형태로 수정

* test: 테스트 코드에서 에러 코드를 문자열이 아닌 상수로 변경

* test: hasSize(0)에서 isEmpty로 개선

* refactor: Global 에러 코드를 문자열 형태로 수정

Author: 023-dev

src/main/java/org/ject/support/common/exception/GlobalErrorCode.java

@@ -15,29 +15,23 @@
 @Getter
 @AllArgsConstructor
 public enum GlobalErrorCode implements ErrorCode {
-    UNSUPPORTED_PARAMETER_TYPE(BAD_REQUEST, 1, "Unsupported parameter type"),
-    RESOURCE_NOT_FOUND(NOT_FOUND, 2, "Resource not found"),
-    TEMPLATE_NOT_FOUND(NOT_FOUND, 3, "Template file not found"),
-    JSON_MARSHALLING_FAILURE(INTERNAL_SERVER_ERROR, 4, "Json marshalling failure"),
-    EMPTY_ACCESS_TOKEN(UNAUTHORIZED, 5, "Empty access token"),
-    INVALID_ACCESS_TOKEN(UNAUTHORIZED, 6, "Invalid access token"),
-    INVALID_PERMISSION(FORBIDDEN, 7, "Invalid permission"),
-    AUTHENTICATION_REQUIRED(UNAUTHORIZED, 8, "Authentication is required"),
-    OVER_PERIOD(CONFLICT, 9, "모집 기간이 아닙니다."),
-    MISS_REQUIRED_REQUEST_PARAMETER(BAD_REQUEST, 10, "Missing required parameter"),
-    MISS_REQUEST_BODY(BAD_REQUEST, 11, "Missing request body"),
-    MISS_REQUIRED_JOB_FAMILY_PARAMETER(BAD_REQUEST, 12, "Missing required JobFamily parameter"),
-    AUTHENTICATION_PROCESSING_ERROR(INTERNAL_SERVER_ERROR, 13, "인증 처리 중 오류가 발생했습니다."),
-    REQUEST_METHOD_NOT_ALLOWED(METHOD_NOT_ALLOWED, 14, "Method not allowed"),
-    METHOD_VALIDATION_FAILED(BAD_REQUEST, 15, "Method validation failed");
+    UNSUPPORTED_PARAMETER_TYPE(BAD_REQUEST, "GLOBAL-1", "Unsupported parameter type"),
+    RESOURCE_NOT_FOUND(NOT_FOUND, "GLOBAL-2", "Resource not found"),
+    TEMPLATE_NOT_FOUND(NOT_FOUND, "GLOBAL-3", "Template file not found"),
+    JSON_MARSHALLING_FAILURE(INTERNAL_SERVER_ERROR, "GLOBAL-4", "Json marshalling failure"),
+    EMPTY_ACCESS_TOKEN(UNAUTHORIZED, "GLOBAL-5", "Empty access token"),
+    INVALID_ACCESS_TOKEN(UNAUTHORIZED, "GLOBAL-6", "Invalid access token"),
+    INVALID_PERMISSION(FORBIDDEN, "GLOBAL-7", "Invalid permission"),
+    AUTHENTICATION_REQUIRED(UNAUTHORIZED, "GLOBAL-8", "Authentication is required"),
+    OVER_PERIOD(CONFLICT, "GLOBAL-9", "모집 기간이 아닙니다."),
+    MISS_REQUIRED_REQUEST_PARAMETER(BAD_REQUEST, "GLOBAL-10", "Missing required parameter"),
+    MISS_REQUEST_BODY(BAD_REQUEST, "GLOBAL-11", "Missing request body"),
+    MISS_REQUIRED_JOB_FAMILY_PARAMETER(BAD_REQUEST, "GLOBAL-12", "Missing required JobFamily parameter"),
+    AUTHENTICATION_PROCESSING_ERROR(INTERNAL_SERVER_ERROR, "GLOBAL-13", "인증 처리 중 오류가 발생했습니다."),
+    REQUEST_METHOD_NOT_ALLOWED(METHOD_NOT_ALLOWED, "GLOBAL-14", "Method not allowed"),
+    METHOD_VALIDATION_FAILED(BAD_REQUEST, "GLOBAL-15", "Method validation failed");
 
     private final HttpStatus httpStatus;
     private final String code;
     private final String message;
-
-    GlobalErrorCode(HttpStatus httpStatus, int code, String message) {
-        this.httpStatus = httpStatus;
-        this.code = String.format("GLOBAL-%d", code);
-        this.message = message;
-    }
 }

src/main/java/org/ject/support/domain/apply/exception/ApplyErrorCode.java

@@ -11,20 +11,14 @@
 @Getter
 @AllArgsConstructor
 public enum ApplyErrorCode implements ErrorCode {
-    NOT_FOUND_APPLY(NOT_FOUND, 1, "지원 정보를 찾을 수 없습니다."),
-    ALREADY_SUBMITTED(CONFLICT, 2, "이미 지원서를 제출한 상태입니다."),
-    NOT_FOUND_TEMP_APPLICATION_FORM(NOT_FOUND, 3, "임시 저장한 지원서가 존재하지 않습니다."),
-    NOT_FOUND_SUBMITTED_APPLICATION_FORM(NOT_FOUND, 4, "제출된 지원서가 존재하지 않습니다."),
-    NOT_SUBMITTED(CONFLICT, 5, "제출 완료된 지원서가 아닙니다.")
+    NOT_FOUND_APPLY(NOT_FOUND, "APPLY-1", "지원 정보를 찾을 수 없습니다."),
+    ALREADY_SUBMITTED(CONFLICT, "APPLY-2", "이미 지원서를 제출한 상태입니다."),
+    NOT_FOUND_TEMP_APPLICATION_FORM(NOT_FOUND, "APPLY-3", "임시 저장한 지원서가 존재하지 않습니다."),
+    NOT_FOUND_SUBMITTED_APPLICATION_FORM(NOT_FOUND, "APPLY-4", "제출된 지원서가 존재하지 않습니다."),
+    NOT_SUBMITTED(CONFLICT, "APPLY-5", "제출 완료된 지원서가 아닙니다.")
     ;
 
     private final HttpStatus httpStatus;
     private final String code;
     private final String message;
-
-    ApplyErrorCode(HttpStatus httpStatus, int code, String message) {
-        this.httpStatus = httpStatus;
-        this.code = String.format("APPLY-%d", code); // TODO 별도 유틸 메서드로 분리 (this.getClass().getSimpleName().replace("ErrorCode", "").toUpperCase())
-        this.message = message;
-    }
-}
+}

src/main/java/org/ject/support/domain/auth/exception/AuthErrorCode.java

@@ -10,11 +10,11 @@
 @Getter
 @AllArgsConstructor
 public enum AuthErrorCode implements ErrorCode {
-    INVALID_AUTH_CODE(UNAUTHORIZED, "INVALID_AUTH_CODE", "인증 번호가 유효하지 않습니다."),
-    NOT_FOUND_AUTH_CODE(UNAUTHORIZED, "NOT_FOUND_AUTH_CODE", "인증 번호를 찾을 수 없습니다."),
-    INVALID_REFRESH_TOKEN(UNAUTHORIZED, "INVALID_REFRESH_TOKEN", "유효하지 않은 리프레시 토큰입니다."),
-    EXPIRED_REFRESH_TOKEN(UNAUTHORIZED, "EXPIRED_REFRESH_TOKEN", "만료된 리프레시 토큰입니다."),
-    INVALID_CREDENTIALS(UNAUTHORIZED, "INVALID_CREDENTIALS", "PIN 번호가 올바르지 않습니다.");
+    INVALID_AUTH_CODE(UNAUTHORIZED, "AUTH-1", "인증 번호가 유효하지 않습니다."),
+    NOT_FOUND_AUTH_CODE(UNAUTHORIZED, "AUTH-2", "인증 번호를 찾을 수 없습니다."),
+    INVALID_REFRESH_TOKEN(UNAUTHORIZED, "AUTH-3", "유효하지 않은 리프레시 토큰입니다."),
+    EXPIRED_REFRESH_TOKEN(UNAUTHORIZED, "AUTH-4", "만료된 리프레시 토큰입니다."),
+    INVALID_CREDENTIALS(UNAUTHORIZED, "AUTH-5", "PIN 번호가 올바르지 않습니다.");
 
     private final HttpStatus httpStatus;
     private final String code;

src/main/java/org/ject/support/domain/file/exception/FileErrorCode.java

@@ -11,10 +11,10 @@
 @Getter
 @AllArgsConstructor
 public enum FileErrorCode implements ErrorCode {
-    INVALID_EXTENSION(BAD_REQUEST, "INVALID_EXTENSION", "유효하지 않은 확장자입니다."),
-    EXCEEDED_PORTFOLIO_MAX_SIZE(PAYLOAD_TOO_LARGE, "EXCEEDED_PORTFOLIO_MAX_SIZE", "첨부 가능한 포트폴리오 최대 용량을 초과했습니다.");
+    INVALID_EXTENSION(BAD_REQUEST, "FILE-1", "유효하지 않은 확장자입니다."),
+    EXCEEDED_PORTFOLIO_MAX_SIZE(PAYLOAD_TOO_LARGE, "FILE-2", "첨부 가능한 포트폴리오 최대 용량을 초과했습니다.");
 
     private final HttpStatus httpStatus;
     private final String code;
     private final String message;
-}
+}

src/main/java/org/ject/support/domain/member/exception/MemberErrorCode.java

@@ -12,20 +12,14 @@
 @Getter
 @AllArgsConstructor
 public enum MemberErrorCode implements ErrorCode {
-    NOT_FOUND_MEMBER(NOT_FOUND, 1, "멤버를 찾을 수 없습니다."),
-    ALREADY_EXIST_MEMBER(CONFLICT, 2, "이미 가입되어 있는 회원입니다."),
-    NOT_FOUND_SEMESTER_OF_MEMBER(NOT_FOUND, 3, "회원의 기수를 찾을 수 없습니다."),
-    DUPLICATE_EMAIL(CONFLICT, 4, "이미 사용 중인 이메일입니다."),
-    EXCEEDED_INTERESTED_DOMAINS_MAX_SIZE(PAYLOAD_TOO_LARGE, 5, "관심 도메인 목록의 최대 크기를 초과했습니다.")
+    NOT_FOUND_MEMBER(NOT_FOUND, "MEMBER-1", "멤버를 찾을 수 없습니다."),
+    ALREADY_EXIST_MEMBER(CONFLICT, "MEMBER-2", "이미 가입되어 있는 회원입니다."),
+    NOT_FOUND_SEMESTER_OF_MEMBER(NOT_FOUND, "MEMBER-3", "회원의 기수를 찾을 수 없습니다."),
+    DUPLICATE_EMAIL(CONFLICT, "MEMBER-4", "이미 사용 중인 이메일입니다."),
+    EXCEEDED_INTERESTED_DOMAINS_MAX_SIZE(PAYLOAD_TOO_LARGE, "MEMBER-5", "관심 도메인 목록의 최대 크기를 초과했습니다.")
     ;
 
     private final HttpStatus httpStatus;
     private final String code;
     private final String message;
-
-    MemberErrorCode(HttpStatus httpStatus, int code, String message) {
-        this.httpStatus = httpStatus;
-        this.code = String.format("MEMBER-%d", code);
-        this.message = message;
-    }
 }

src/main/java/org/ject/support/domain/project/exception/ProjectErrorCode.java

@@ -10,7 +10,7 @@
 @Getter
 @AllArgsConstructor
 public enum ProjectErrorCode implements ErrorCode {
-    NOT_FOUND_PROJECT(NOT_FOUND, "PROJECT_NOT_FOUND", "프로젝트를 찾을 수 없습니다."),
+    NOT_FOUND_PROJECT(NOT_FOUND, "PROJECT-1", "프로젝트를 찾을 수 없습니다."),
     ;
 
     private final HttpStatus httpStatus;

src/main/java/org/ject/support/domain/recruit/exception/QuestionErrorCode.java

@@ -10,7 +10,7 @@
 @Getter
 @AllArgsConstructor
 public enum QuestionErrorCode implements ErrorCode {
-    NOT_FOUND_QUESTION(NOT_FOUND, "QUESTION_NOT_FOUND", "해당 질문을 찾을 수 없습니다."),
+    NOT_FOUND_QUESTION(NOT_FOUND, "QUESTION-1", "해당 질문을 찾을 수 없습니다."),
     ;
 
     private final HttpStatus httpStatus;

src/main/java/org/ject/support/domain/recruit/exception/RecruitErrorCode.java

@@ -11,9 +11,9 @@
 @Getter
 @AllArgsConstructor
 public enum RecruitErrorCode implements ErrorCode {
-    NOT_FOUND_RECRUIT(NOT_FOUND, "NOT_FOUND_RECRUIT", "모집 공고를 찾을 수 없습니다."),
-    DUPLICATED_JOB_FAMILY(CONFLICT, "DUPLICATED_JOB_FAMILY", "이미 모집중인 직군입니다."),
-    UPDATE_NOT_ALLOW_FOR_CLOSED(CONFLICT, "UPDATE_NOT_ALLOW_FOR_CLOSED", "마감된 모집 정보는 수정할 수 없습니다.");
+    NOT_FOUND_RECRUIT(NOT_FOUND, "RECRUIT-1", "모집 공고를 찾을 수 없습니다."),
+    DUPLICATED_JOB_FAMILY(CONFLICT, "RECRUIT-2", "이미 모집중인 직군입니다."),
+    UPDATE_NOT_ALLOW_FOR_CLOSED(CONFLICT, "RECRUIT-3", "마감된 모집 정보는 수정할 수 없습니다.");
 
     private final HttpStatus httpStatus;
     private final String code;

src/main/java/org/ject/support/domain/recruit/exception/SemesterErrorCode.java

@@ -10,8 +10,8 @@
 @Getter
 @AllArgsConstructor
 public enum SemesterErrorCode implements ErrorCode {
-    NOT_FOUND_RECRUITING_SEMESTER(NOT_FOUND, "NOT_FOUND_RECRUITING_SEMESTER", "현재 모집중인 기수가 존재하지 않습니다."),
-    NOT_FOUND_SEMESTER(NOT_FOUND, "NOT_FOUND_SEMESTER", "기수가 존재하지 않습니다."),
+    NOT_FOUND_RECRUITING_SEMESTER(NOT_FOUND, "SEMESTER-1", "현재 모집중인 기수가 존재하지 않습니다."),
+    NOT_FOUND_SEMESTER(NOT_FOUND, "SEMESTER-2", "기수가 존재하지 않습니다."),
     ;
 
     private final HttpStatus httpStatus;

src/test/java/org/ject/support/domain/file/controller/FileControllerTest.java

@@ -26,6 +26,7 @@
 import java.time.LocalDateTime;
 
 import static org.hamcrest.Matchers.containsString;
+import static org.ject.support.domain.file.exception.FileErrorCode.INVALID_EXTENSION;
 import static org.mockito.Mockito.when;
 import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.post;
 import static org.springframework.test.web.servlet.result.MockMvcResultHandlers.print;
@@ -137,7 +138,7 @@ void invalid_portfolio_content_type() throws Exception {
                                     }
                                 ]
                                 """))
-                .andExpect(content().string(containsString("INVALID_EXTENSION")))
+                .andExpect(content().string(containsString(INVALID_EXTENSION.getCode())))
                 .andDo(print());
     }
 
@@ -175,7 +176,7 @@ void exceeded_portfolio_max_size() throws Exception {
                                 """)
                 )
                 .andExpect(status().isPayloadTooLarge())
-                .andExpect(content().string(containsString(FileErrorCode.EXCEEDED_PORTFOLIO_MAX_SIZE.name())))
+                .andExpect(content().string(containsString(FileErrorCode.EXCEEDED_PORTFOLIO_MAX_SIZE.getCode())))
                 .andDo(print())
                 .andReturn();
     }