Refactory.

Author: jonpereiradev

src/main/java/com/github/jonpereiradev/diffobjects/DiffObjects.java

@@ -25,14 +25,7 @@ public static <T> List<DiffResult<?>> diff(T beforeState, T afterState) {
         Objects.requireNonNull(beforeState, "Before state is required.");
         Objects.requireNonNull(afterState, "After state is required.");
 
-        List<DiffResult<?>> results = new LinkedList<>();
-        List<DiffMetadata> metadatas = DiffReflections.mapAnnotations(beforeState.getClass());
-
-        for (DiffMetadata metadata : metadatas) {
-            results.add(metadata.getStrategy().diff(beforeState, afterState, metadata));
-        }
-
-        return results;
+        return diff(beforeState, afterState, DiffReflections.mapAnnotations(beforeState.getClass()));
     }
 
     /**
@@ -46,17 +39,8 @@ public static <T> List<DiffResult<?>> diff(T beforeState, T afterState) {
     public static <T> boolean isEquals(T beforeState, T afterState) {
         Objects.requireNonNull(beforeState, "Before state is required.");
         Objects.requireNonNull(afterState, "After state is required.");
-        List<DiffMetadata> metadatas = DiffReflections.mapAnnotations(beforeState.getClass());
-
-        for (DiffMetadata metadata : metadatas) {
-            DiffResult<T> result = metadata.getStrategy().diff(beforeState, afterState, metadata);
 
-            if (!result.isEquals()) {
-                return false;
-            }
-        }
-
-        return true;
+        return isEquals(beforeState, afterState, DiffReflections.mapAnnotations(beforeState.getClass()));
     }
 
     /**

src/main/java/com/github/jonpereiradev/diffobjects/strategy/DiffReflections.java

@@ -4,13 +4,13 @@
 import com.github.jonpereiradev.diffobjects.annotation.DiffMapping;
 import com.github.jonpereiradev.diffobjects.annotation.DiffMappings;
 import com.github.jonpereiradev.diffobjects.builder.DiffBuilder;
+import com.github.jonpereiradev.diffobjects.builder.DiffConfigurationBuilder;
 import com.github.jonpereiradev.diffobjects.builder.DiffInstanceBuilder;
 import org.apache.commons.lang.StringUtils;
 import org.apache.commons.lang.reflect.MethodUtils;
 
 import java.lang.reflect.InvocationTargetException;
 import java.lang.reflect.Method;
-import java.util.List;
 import java.util.Map;
 import java.util.concurrent.ConcurrentHashMap;
 
@@ -21,15 +21,15 @@
  */
 public final class DiffReflections {
 
-    private static final Map<String, List<DiffMetadata>> CACHE_MAP = new ConcurrentHashMap<>();
+    private static final Map<String, DiffConfigurationBuilder> CACHE_MAP = new ConcurrentHashMap<>();
 
     /**
      * Map the methods of the object that has the annotations for diff and stores in cache.
      *
      * @param diffClass class that have the diff annotations.
      * @return the diff mappings of the class.
      */
-    public static List<DiffMetadata> mapAnnotations(Class<?> diffClass) {
+    public static DiffConfigurationBuilder mapAnnotations(Class<?> diffClass) {
         if (!CACHE_MAP.containsKey(diffClass.getName())) {
             DiffInstanceBuilder builder = DiffBuilder.map(diffClass);
 
@@ -44,7 +44,7 @@ public static List<DiffMetadata> mapAnnotations(Class<?> diffClass) {
                 }
             }
 
-            CACHE_MAP.put(diffClass.getName(), builder.configuration().build());
+            CACHE_MAP.put(diffClass.getName(), builder.configuration());
         }
 
         return CACHE_MAP.get(diffClass.getName());

src/test/java/com/github/jonpereiradev/diffobjects/strategy/BaseStrategyTest.java

@@ -3,7 +3,7 @@
 public abstract class BaseStrategyTest {
 
     protected DiffMetadata discoverByName(Class<?> classMap, String name) {
-        for (DiffMetadata metadata : DiffReflections.mapAnnotations(classMap)) {
+        for (DiffMetadata metadata : DiffReflections.mapAnnotations(classMap).build()) {
             if (metadata.getMethod().getName().equals(name)) {
                 return metadata;
             }