optimise type conversion

Author: sidhant92

src/main/java/com/github/sidhant92/boolparser/datatype/AbstractDataType.java

@@ -19,6 +19,9 @@ public AbstractDataType(final Class<T> clazz) {
 
     public boolean defaultIsValid(final Object value, final ObjectMapper objectMapper) {
         try {
+            if (clazz.isInstance(value)) {
+                return true;
+            }
             return objectMapper.convertValue(value, clazz) != null;
         } catch (final Exception ex) {
             log.error("Unable to convert value = {} to type = {}", value, clazz);
@@ -28,7 +31,10 @@ public boolean defaultIsValid(final Object value, final ObjectMapper objectMappe
 
     public Optional<T> defaultGetValue(final Object value, final ObjectMapper objectMapper) {
         try {
-            return (Optional<T>) Optional.of(objectMapper.convertValue(value, clazz));
+            if (clazz.isInstance(value)) {
+                return Optional.of(clazz.cast(value));
+            }
+            return Optional.of(objectMapper.convertValue(value, clazz));
         } catch (final Exception ex) {
             log.error("Unable to convert value = {} to type = {}", value, clazz);
         }

src/main/java/com/github/sidhant92/boolparser/util/ValueUtils.java

@@ -1,6 +1,7 @@
 package com.github.sidhant92.boolparser.util;
 
 import java.util.Optional;
+import org.apache.maven.artifact.versioning.ComparableVersion;
 import com.github.sidhant92.boolparser.constant.DataType;
 
 public class ValueUtils {
@@ -14,6 +15,8 @@ public static Object convertValue(final String value, final DataType dataType) {
                 return Double.parseDouble(value);
             case BOOLEAN:
                 return Boolean.parseBoolean(value);
+            case APP_VERSION:
+                new ComparableVersion(value);
             default:
                 if (value.startsWith("'") && value.endsWith("'")) {
                     return value.substring(1, value.length() -1);