Merge pull request #31 from Project-Catcher/refactor-cw-baseresponse

Refactor: CommonResponse, BaseResponseStatus를 Core-Service와 호환 맞추는 작업
Project-Catcher · Nov 25, 2023 · 5ed9e4b · 5ed9e4b
2 parents f2b1c6d + 0660c97
commit 5ed9e4b
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 16 deletions.
diff --git a/src/main/java/com/catcher/common/exception/BaseResponseStatus.java b/src/main/java/com/catcher/common/exception/BaseResponseStatus.java
@@ -4,44 +4,40 @@
 
 @Getter
 public enum BaseResponseStatus {
-    /**
-     * 1000 : 요청 성공
-     */
-    SUCCESS(true, 1000, "요청에 성공하였습니다."),
+
+    SUCCESS(200, "요청에 성공하였습니다."),
 
     /**
      * 2000 : Request 오류
      */
     // Common
-    NO_ADDRESS_RESULT_FOR_QUERY(false, 2000, "해당하는 요청에 대응되는 주소 검색 결과가 없습니다."),
-    NO_LOCATION_RESULT(false, 2001, "해당하는 요청에 대응되는 법정동 코드가 없습니다"),
+    NO_ADDRESS_RESULT_FOR_QUERY(2000, "해당하는 요청에 대응되는 주소 검색 결과가 없습니다."),
+    NO_LOCATION_RESULT(2001, "해당하는 요청에 대응되는 법정동 코드가 없습니다"),
 
     /**
      * 3000 : Response 오류
      */
     // Common
-    RESPONSE_ERROR(false, 3000, "값을 불러오는데 실패하였습니다."),
+    RESPONSE_ERROR(3000, "값을 불러오는데 실패하였습니다."),
 
     /**
      * 4000 : Database, Server 오류
      */
-    DATABASE_ERROR(false, 4000, "데이터베이스 연결에 실패하였습니다."),
-    REDIS_ERROR(false, 4002, "redis 연결에 실패하였습니다."),
+    DATABASE_ERROR(4000, "데이터베이스 연결에 실패하였습니다."),
+    REDIS_ERROR(4002, "redis 연결에 실패하였습니다."),
 
     /**
      * 5000: AWS Error
      */
-    S3UPLOAD_ERROR(false, 5000, "파일 업로드를 실패하였습니다."),
-    KMS_ERROR(false, 5001, "암호화 및 복호화 과정에서 실패하였습니다."),
-    AWS_IO_ERROR(false, 5002, "파일의 정보를 가져오는 데 실패했습니다.")
+    S3UPLOAD_ERROR(5000, "파일 업로드를 실패하였습니다."),
+    KMS_ERROR(5001, "암호화 및 복호화 과정에서 실패하였습니다."),
+    AWS_IO_ERROR(5002, "파일의 정보를 가져오는 데 실패했습니다.")
     ;
 
-    private final boolean isSuccess;
     private final int code;
     private final String message;
 
-    BaseResponseStatus(boolean isSuccess, int code, String message) { //BaseResponseStatus 에서 각 해당하는 코드를 생성자로 맵핑
-        this.isSuccess = isSuccess;
+    BaseResponseStatus(int code, String message) { //BaseResponseStatus 에서 각 해당하는 코드를 생성자로 맵핑
         this.code = code;
         this.message = message;
     }

diff --git a/src/main/java/com/catcher/common/response/CommonResponse.java b/src/main/java/com/catcher/common/response/CommonResponse.java
@@ -20,11 +20,23 @@ public CommonResponse(int code, boolean success, T result) {
         this.result = result;
     }
 
+    public static <T> CommonResponse<T> success() {
+        return new CommonResponse<>(200, true, null);
+    }
+
+    public static <T> CommonResponse<T> success(T result) {
+        return new CommonResponse<>(200, true, result);
+    }
+
     public static <T> CommonResponse<T> success(int code, T result) {
         return new CommonResponse<>(code, true, result);
     }
 
+    public static <T> CommonResponse<T> fail(int code, T result) {
+        return new CommonResponse<>(code, false, result);
+    }
+
     public static CommonResponse<String> fail(BaseException e) {
-        return new CommonResponse<>(e.getStatus().getCode(), e.getStatus().isSuccess(), e.getStatus().getMessage());
+        return new CommonResponse<>(e.getStatus().getCode(), false, e.getStatus().getMessage());
     }
 }