From f953de3cd6591bcf63d28814f267fc79538247ae Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E6=A2=81=E7=90=A6=E6=B6=9B?= <liangqitao@syxysoft.com>
Date: Mon, 6 Mar 2023 13:32:54 +0800
Subject: [PATCH] =?UTF-8?q?=E3=80=90fix=20sonar=E3=80=91ReflectHelper?=
 =?UTF-8?q?=E6=96=87=E4=BB=B6?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../com/jero/common/util/ReflectHelper.java   | 23 ++++++++-----------
 1 file changed, 10 insertions(+), 13 deletions(-)

diff --git a/jero-boot/jero-boot-base/jero-boot-base-core/src/main/java/com/jero/common/util/ReflectHelper.java b/jero-boot/jero-boot-base/jero-boot-base-core/src/main/java/com/jero/common/util/ReflectHelper.java
index 268a75db..1764ecac 100644
--- a/jero-boot/jero-boot-base/jero-boot-base-core/src/main/java/com/jero/common/util/ReflectHelper.java
+++ b/jero-boot/jero-boot-base/jero-boot-base-core/src/main/java/com/jero/common/util/ReflectHelper.java
@@ -1,6 +1,7 @@
 package com.jero.common.util;
 
 import lombok.extern.slf4j.Slf4j;
+import org.springframework.util.ReflectionUtils;
 
 import java.lang.reflect.Field;
 import java.lang.reflect.Method;
@@ -25,11 +26,11 @@ public class ReflectHelper {
     /**
      * 存放get方法
      */
-    private Hashtable<String, Method> getMethods = null;
+    private HashMap<String, Method> getMethods = null;
     /**
      * 存放set方法
      */
-    private Hashtable<String, Method> setMethods = null;
+    private HashMap<String, Method> setMethods = null;
 
     /**
      * 定义构造方法 -- 一般来说是个pojo
@@ -45,8 +46,8 @@ public class ReflectHelper {
      * @desc 初始化
      */
     public void initMethods() {
-        getMethods = new Hashtable<String, Method>();
-        setMethods = new Hashtable<String, Method>();
+        getMethods = new HashMap<String, Method>();
+        setMethods = new HashMap<String, Method>();
         cls = obj.getClass();
         Method[] methods = cls.getMethods();
         // 定义正则表达式，从方法中过滤出getter / setter 函数.
@@ -66,8 +67,6 @@ public class ReflectHelper {
             } else if (Pattern.matches(ss, methodName)) {
                 param = setM.matcher(methodName).replaceAll(rapl).toLowerCase();
                 setMethods.put(param, m);
-            } else {
-                // logger.info(methodName + " 不是getter,setter方法！");
             }
         }
     }
@@ -198,10 +197,10 @@ public class ReflectHelper {
     public static Object getFieldVal(String fieldName, Object o) {
         try {
             // 暴力反射获取属性
-            Field filed = o.getClass().getDeclaredField(fieldName);
+            Field field = o.getClass().getDeclaredField(fieldName);
             // 设置反射时取消Java的访问检查，暴力访问
-            filed.setAccessible(true);
-            Object val = filed.get(o);
+            ReflectionUtils.makeAccessible(field);
+            Object val = field.get(o);
             return val;
         } catch (Exception e) {
             e.printStackTrace();
@@ -216,7 +215,6 @@ public class ReflectHelper {
         Field[] fields = o.getClass().getDeclaredFields();
         String[] fieldNames = new String[fields.length];
         for (int i = 0; i < fields.length; i++) {
-            //log.info(fields[i].getType());
             fieldNames[i] = fields[i].getName();
         }
         return fieldNames;
@@ -227,8 +225,7 @@ public class ReflectHelper {
      */
     public static List<Map> getFiledsInfo(Object o) {
         Field[] fields = o.getClass().getDeclaredFields();
-        String[] fieldNames = new String[fields.length];
-        List<Map> list = new ArrayList<Map>();
+        List<Map> list = new ArrayList<>();
         Map<String, Object> infoMap = null;
         for (int i = 0; i < fields.length; i++) {
             infoMap = new HashMap<String, Object>();
@@ -252,4 +249,4 @@ public class ReflectHelper {
         return value;
     }
 
-}
\ No newline at end of file
+}