Revert "refactor: refactor the Result Class (apache#7176)"

This reverts commit 01877a9.

Author: xjlgod

Diff for: changes/en-us/2.x.md

@@ -72,7 +72,7 @@ Add changes here for all PR submitted to the 2.x branch.
 
 ### refactor:
 
-- [[#7176](https://github.com/apache/incubator-seata/pull/7176)] refactor the Result class
+- [[#PR_NO](https://github.com/apache/incubator-seata/pull/PR_NO)] refactor XXX
 
 Thanks to these contributors for their code commits. Please report an unintended omission.
 

Diff for: changes/zh-cn/2.x.md

@@ -72,7 +72,7 @@
 
 ### refactor:
 
-- [[#7176](https://github.com/apache/incubator-seata/pull/7176)] 重构 Result 类
+- [[#PR_NO](https://github.com/apache/incubator-seata/pull/PR_NO)] 重构XXX
 
 非常感谢以下 contributors 的代码贡献。若有无意遗漏，请报告。
 

Diff for: common/src/main/java/org/apache/seata/common/result/Code.java

@@ -25,21 +25,21 @@ public enum Code {
     /**
      * server error
      */
-    INTERNAL_SERVER_ERROR("500", "Server error"),
+    ERROR("500", "Server error"),
     /**
      * the custom error
      */
-    UNAUTHORIZED("401", "Login failed");
+    LOGIN_FAILED("401", "Login failed");
 
     /**
      * The Code.
      */
-    public final String code;
+    public String code;
 
     /**
      * The Msg.
      */
-    public final String msg;
+    public String msg;
 
     private Code(String code, String msg) {
         this.code = code;
@@ -55,6 +55,15 @@ public String getCode() {
         return this.code;
     }
 
+    /**
+     * Sets code.
+     *
+     * @param code the code
+     */
+    public void setCode(String code) {
+        this.code = code;
+    }
+
     /**
      * Gets msg.
      *
@@ -64,6 +73,15 @@ public String getMsg() {
         return msg;
     }
 
+    /**
+     * Sets msg.
+     *
+     * @param msg the msg
+     */
+    public void setMsg(String msg) {
+        this.msg = msg;
+    }
+
     /**
      * Gets error msg.
      *

Diff for: common/src/main/java/org/apache/seata/common/result/PageResult.java

@@ -16,14 +16,14 @@
  */
 package org.apache.seata.common.result;
 
-import static org.apache.seata.common.result.Code.SUCCESS;
-
+import java.io.Serializable;
 import java.util.List;
 
 /**
  * The page result
+ *
  */
-public class PageResult<T> extends Result {
+public class PageResult<T> extends Result<T> implements Serializable {
     private static final long serialVersionUID = 7761262662429121287L;
 
     /**
@@ -55,7 +55,7 @@ public PageResult(String code, String message) {
     }
 
     public PageResult(List<T> data, Integer total, Integer pages, Integer pageNum, Integer pageSize) {
-        super(SUCCESS.code, SUCCESS.msg);
+        super(SUCCESS_CODE, SUCCESS_MSG);
         this.total = total;
         this.pages = pages;
         this.pageNum = pageNum;
@@ -71,19 +71,19 @@ public static <T> PageResult<T> build(List<T> list, Integer pageNum, Integer pag
         }
         final int offset = pageSize * (pageNum - 1);
         return PageResult.success(
-            list.subList(
-                Math.min(offset, list.size()),
-                Math.min(offset + pageSize, list.size())
-            ),
-            list.size(),
-            pages,
-            pageNum,
-            pageSize
+                list.subList(
+                        Math.min(offset, list.size()),
+                        Math.min(offset + pageSize, list.size())
+                ),
+                list.size(),
+                pages,
+                pageNum,
+                pageSize
         );
     }
 
     public PageResult(List<T> data, Integer total, Integer pageNum, Integer pageSize) {
-        super(SUCCESS.code, SUCCESS.msg);
+        super(SUCCESS_CODE, SUCCESS_MSG);
         this.total = total;
         this.pageNum = pageNum;
         this.pageSize = pageSize;
@@ -101,14 +101,12 @@ public static <T> PageResult<T> failure(String code, String msg) {
     }
 
     public static <T> PageResult<T> success() {
-        return new PageResult<>(SUCCESS.code, SUCCESS.msg);
+        return new PageResult<>(SUCCESS_CODE, SUCCESS_MSG);
     }
 
-    public static <T> PageResult<T> success(List<T> data, Integer total, Integer pages, Integer pageNum,
-        Integer pageSize) {
+    public static <T> PageResult<T> success(List<T> data, Integer total, Integer pages, Integer pageNum, Integer pageSize) {
         return new PageResult<>(data, total, pages, pageNum, pageSize);
     }
-
     public static <T> PageResult<T> success(List<T> data, Integer total, Integer pageNum, Integer pageSize) {
         return new PageResult<>(data, total, pageNum, pageSize);
     }

Diff for: common/src/main/java/org/apache/seata/common/result/Result.java

@@ -16,21 +16,23 @@
  */
 package org.apache.seata.common.result;
 
-import static org.apache.seata.common.result.Code.SUCCESS;
-
 import java.io.Serializable;
 
 /**
  * The basic result
  */
-public class Result implements Serializable {
+public class Result<T>  implements Serializable {
     private static final long serialVersionUID = 7761261124298767L;
 
-    private final String code;
-    private final String message;
+    public static final String SUCCESS_CODE = "200";
+    public static final String SUCCESS_MSG = "success";
+    public static final String FAIL_CODE = "500";
+
+
+    private String code = SUCCESS_CODE;
+    private String message = SUCCESS_MSG;
 
     public Result() {
-        this(null, null);
     }
 
     public Result(String code, String message) {
@@ -39,14 +41,23 @@ public Result(String code, String message) {
     }
 
     public boolean isSuccess() {
-        return this.code.equals(SUCCESS.code);
+        return SUCCESS_CODE.equals(this.code);
     }
 
     public String getCode() {
         return code;
     }
 
+    public void setCode(String code) {
+        this.code = code;
+    }
+
     public String getMessage() {
         return message;
     }
+
+    public void setMessage(String message) {
+        this.message = message;
+    }
+
 }

Diff for: common/src/main/java/org/apache/seata/common/result/SingleResult.java

@@ -16,15 +16,18 @@
  */
 package org.apache.seata.common.result;
 
-import static org.apache.seata.common.result.Code.SUCCESS;
-import static org.apache.seata.common.result.Code.INTERNAL_SERVER_ERROR;
+import java.io.Serializable;
+
 
 /**
  * The single result
  */
-public class SingleResult<T> extends Result {
+public class SingleResult<T> extends Result<T>  implements Serializable {
     private static final long serialVersionUID = 77612626624298767L;
 
+    /**
+     * the data
+     */
     private T data;
 
     public SingleResult(String code, String message) {
@@ -41,34 +44,27 @@ public static <T> SingleResult<T> failure(String code, String msg) {
     }
 
     public static <T> SingleResult<T> failure(Code errorCode) {
-        return failure(errorCode.code, errorCode.msg);
+        return new SingleResult(errorCode.getCode(), errorCode.getMsg());
     }
 
     public static <T> SingleResult<T> failure(String msg) {
-        return failure(INTERNAL_SERVER_ERROR.code, msg);
-    }
-
-    public static <T> SingleResult<T> success(String msg, T data) {
-        return new SingleResult<>(SUCCESS.code, msg, data);
+        return new SingleResult<>(FAIL_CODE, msg);
     }
 
-    public static SingleResult<Void> success(String msg) {
-        return success(msg, null);
+    public static <T> SingleResult<T> success(T data) {
+        return new SingleResult<>(SUCCESS_CODE, SUCCESS_MSG,data);
     }
 
     public static <T> SingleResult<T> success() {
-        return success(SUCCESS.msg, null);
-    }
-
-    public static <T> SingleResult<T> successWithData(T data) {
-        return success(SUCCESS.msg, data);
+        return new SingleResult<>(SUCCESS_CODE, SUCCESS_MSG, null);
     }
 
     public T getData() {
         return data;
     }
 
-    public void setData(final T data) {
+    public void setData(T data) {
         this.data = data;
     }
+
 }

Diff for: common/src/test/java/org/apache/seata/common/code/CodeTest.java

@@ -28,15 +28,24 @@ public class CodeTest {
     public void testGetErrorMsgWithValidCodeReturnsExpectedMsg() {
         // Test case for SUCCESS
         assertEquals("ok", Code.SUCCESS.getMsg());
-        // Test case for INTERNAL_SERVER_ERROR
-        assertEquals("Server error", Code.INTERNAL_SERVER_ERROR.getMsg());
-        // Test case for UNAUTHORIZED
-        assertEquals("Login failed", Code.UNAUTHORIZED.getMsg());
+        // Test case for ERROR
+        assertEquals("Server error", Code.ERROR.getMsg());
+        // Test case for LOGIN_FAILED
+        assertEquals("Login failed", Code.LOGIN_FAILED.getMsg());
     }
 
     @Test
     public void testGetErrorMsgWithInvalidCodeReturnsNull() {
         // Test case for non-existing code
         assertNull(Code.getErrorMsg("404"));
     }
+
+    @Test
+    public void testSetCodeAndMsgUpdatesValuesCorrectly() {
+        // Test case to check if setCode and setMsg are working as expected
+        Code.SUCCESS.setCode("201");
+        Code.SUCCESS.setMsg("Created");
+        assertEquals("201", Code.SUCCESS.getCode());
+        assertEquals("Created", Code.SUCCESS.getMsg());
+    }
 }

Diff for: common/src/test/java/org/apache/seata/common/result/PageResultTest.java

@@ -23,7 +23,6 @@
 import org.mockito.InjectMocks;
 import org.mockito.MockitoAnnotations;
 
-import static org.apache.seata.common.result.Code.*;
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertNull;
 
@@ -42,7 +41,7 @@ void buildPageSizeDivisibleByListSize() {
         for (long i = 0; i < 100; i++) {
             list.add(i);
         }
-        PageResult<Long> pageResult = PageResult.build(list, 1, 10);
+        PageResult pageResult = PageResult.build(list, 1, 10);
         assertEquals(10, pageResult.getPages());
         assertEquals(10, pageResult.getData().size());
     }
@@ -53,7 +52,7 @@ void buildPageSizeNotDivisibleByListSize() {
         for (long i = 0; i < 9; i++) {
             list.add(i);
         }
-        PageResult<Long> pageResult = PageResult.build(list, 1, 10);
+        PageResult pageResult = PageResult.build(list, 1, 10);
         assertEquals(1, pageResult.getPages());
         assertEquals(9, pageResult.getData().size());
     }
@@ -64,24 +63,24 @@ void buildPageNumGreaterThanTotalPages() {
         for (long i = 0; i < 5; i++) {
             list.add(i);
         }
-        PageResult<Long> pageResult = PageResult.build(list, 10, 2);
+        PageResult pageResult = PageResult.build(list, 10, 2);
         assertEquals(10, pageResult.getPageNum().intValue());
         assertEquals(3, pageResult.getPages().intValue());
         assertEquals(0, pageResult.getData().size());
     }
 
     @Test
     void failureInvalidParams() {
-        PageResult<Void> pageResult = PageResult.failure("400", "error");
+        PageResult pageResult = PageResult.failure("400", "error");
         assertEquals("400", pageResult.getCode());
         assertEquals("error", pageResult.getMessage());
     }
 
     @Test
     void successNoData() {
-        PageResult<Void> pageResult = PageResult.success();
-        assertEquals(SUCCESS.code, pageResult.getCode());
-        assertEquals(SUCCESS.msg, pageResult.getMessage());
+        PageResult pageResult = PageResult.success();
+        assertEquals(PageResult.SUCCESS_CODE, pageResult.getCode());
+        assertEquals(PageResult.SUCCESS_MSG, pageResult.getMessage());
         assertNull(pageResult.getData());
     }
 
@@ -91,9 +90,9 @@ void successWithData() {
         for (long i = 0; i < 5; i++) {
             list.add(i);
         }
-        PageResult<Long> pageResult = PageResult.success(list, 5, 1, 5);
-        assertEquals(SUCCESS.code, pageResult.getCode());
-        assertEquals(SUCCESS.msg, pageResult.getMessage());
+        PageResult pageResult = PageResult.success(list, 5, 1, 5);
+        assertEquals(PageResult.SUCCESS_CODE, pageResult.getCode());
+        assertEquals(PageResult.SUCCESS_MSG, pageResult.getMessage());
         assertEquals(5, pageResult.getTotal().intValue());
         assertEquals(1, pageResult.getPageNum().intValue());
         assertEquals(5, pageResult.getPageSize().intValue());