optimize base controller

Author: easepan

src/main/java/com/zhazhapan/util/web/BaseController.java

@@ -8,6 +8,7 @@
 
 import javax.servlet.http.HttpServletRequest;
 import java.lang.reflect.Field;
+import java.util.List;
 
 /**
  * @author pantao
@@ -21,8 +22,6 @@ public class BaseController {
 
     private boolean checkSensitiveData = false;
 
-    private String sensitiveDataTip = "******";
-
     public BaseController() {}
 
     public BaseController(boolean checkSensitiveData) {
@@ -69,25 +68,40 @@ protected ResultObject parseResult(String okMsg, String errMsg, boolean isOk) {
         return isOk ? new ResultObject(okMsg) : CheckResult.getErrorResult(errMsg);
     }
 
+    protected <T> ResultObject<List<T>> parseResult(String errMsg, List<T> list) {
+        if (Checker.isEmpty(list)) {
+            return CheckResult.getErrorResult(errMsg);
+        }
+        if (checkSensitiveData) {
+            list.forEach(this::setSensitiveData);
+        }
+        return new ResultObject<>(list);
+    }
+
     protected <T> ResultObject<T> parseResult(String okMsg, String errMsg, T t) {
         if (Checker.isNull(t)) {
             return CheckResult.getErrorResult(errMsg);
         }
         if (checkSensitiveData) {
-            Field[] fields = t.getClass().getDeclaredFields();
-            try {
-                for (Field field : fields) {
-                    SensitiveData sensitiveData = field.getAnnotation(SensitiveData.class);
-                    if (Checker.isNotNull(sensitiveData) && field.getType() == String.class) {
-                        field.setAccessible(true);
-                        field.set(t, sensitiveDataTip);
-                    }
+            setSensitiveData(t);
+        }
+        return new ResultObject<>(okMsg, t);
+    }
+
+    private <T> void setSensitiveData(T t) {
+        Field[] fields = t.getClass().getDeclaredFields();
+        String sensitiveDataTip = "******";
+        try {
+            for (Field field : fields) {
+                SensitiveData sensitiveData = field.getAnnotation(SensitiveData.class);
+                if (Checker.isNotNull(sensitiveData) && field.getType() == String.class) {
+                    field.setAccessible(true);
+                    field.set(t, sensitiveDataTip);
                 }
-            } catch (IllegalAccessException e) {
-                LoggerUtils.error("set sensitive data error: {}", e.getMessage());
             }
+        } catch (IllegalAccessException e) {
+            LoggerUtils.error("set sensitive data error: {}", e.getMessage());
         }
-        return new ResultObject<>(okMsg, t);
     }
 
     protected <T> ResultObject<T> parseResult(String errMsg, T t) {