refactor: improve code readability for ConstructorUtils, FieldsExtraU…

…tils, MethodEnhancementsUtils, MiscellaneousUtils
alxkm · Sep 9, 2024 · 3b5ec0e · 3b5ec0e
1 parent 1f37443
commit 3b5ec0e
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 9 deletions.
diff --git a/src/main/java/org/reflector/ConstructorUtils.java b/src/main/java/org/reflector/ConstructorUtils.java
@@ -8,6 +8,8 @@
 
 public final class ConstructorUtils {
 
+    private static final Logger LOGGER = LoggerFactory.getLogger(ConstructorUtils.class);
+
     private ConstructorUtils() {}
 
     /**
@@ -38,9 +40,6 @@ public static int getConstructorModifiers(Constructor<?> constructor) {
         return constructor.getModifiers();
     }
 
-
-    private static final Logger LOGGER = LoggerFactory.getLogger(ConstructorUtils.class);
-
     /**
      * Retrieves all public constructors of the specified class.
      *

diff --git a/src/main/java/org/reflector/FieldsExtraUtils.java b/src/main/java/org/reflector/FieldsExtraUtils.java
@@ -66,7 +66,7 @@ public static List<Field> getAllAnnotatedFields(final Class<?> type, final Class
         List<Field> annotatedFields = new ArrayList<>();
         for (Field field : FieldUtils.getAllFields(type)) {
             if (field.isAnnotationPresent(annotation)) {
-                field.setAccessible(true); // Ensure the field is accessible
+                field.setAccessible(true);
                 annotatedFields.add(field);
             }
         }

diff --git a/src/main/java/org/reflector/MethodEnhancementsUtils.java b/src/main/java/org/reflector/MethodEnhancementsUtils.java
@@ -18,7 +18,7 @@ private MethodEnhancementsUtils() {}
      * @return a list of methods annotated with the specified annotation
      * @throws NullPointerException if the clazz or annotationClass is null
      */
-    public static List<Method> getAnnotatedMethods(Class<?> clazz, Class<? extends Annotation> annotationClass) {
+    public static List<Method> getAnnotatedMethods(final Class<?> clazz, Class<? extends Annotation> annotationClass) {
         if (clazz == null || annotationClass == null) {
             throw new NullPointerException("Class and annotation class cannot be null");
         }
@@ -40,7 +40,7 @@ public static List<Method> getAnnotatedMethods(Class<?> clazz, Class<? extends A
      * @return a list of constructors annotated with the specified annotation
      * @throws NullPointerException if the clazz or annotationClass is null
      */
-    public static List<Constructor<?>> getAnnotatedConstructors(Class<?> clazz, Class<? extends Annotation> annotationClass) {
+    public static List<Constructor<?>> getAnnotatedConstructors(final Class<?> clazz, Class<? extends Annotation> annotationClass) {
         if (clazz == null || annotationClass == null) {
             throw new NullPointerException("Class and annotation class cannot be null");
         }

diff --git a/src/main/java/org/reflector/MiscellaneousUtils.java b/src/main/java/org/reflector/MiscellaneousUtils.java
@@ -14,9 +14,9 @@ private MiscellaneousUtils() {}
      * @return a new instance of the specified class
      * @throws NullPointerException if the class is null
      * @throws InstantiationException if the class represents an abstract class, an interface, an array class, a primitive type, or void;
-     *                                or if the class has no nullary constructor
-     * @throws IllegalAccessException if the class or its nullary constructor is not accessible
-     * @throws InvocationTargetException if the nullary constructor throws an exception
+     *                                or if the class has no nullable constructor
+     * @throws IllegalAccessException if the class or its nullable constructor is not accessible
+     * @throws InvocationTargetException if the nullable constructor throws an exception
      */
     public static <T> T newInstance(Class<T> clazz) throws InstantiationException, IllegalAccessException, InvocationTargetException, NoSuchMethodException {
         if (clazz == null) {