extract canConvertToInteger logic

Author: harrel56

src/main/java/dev/harrel/jsonschema/providers/AbstractJsonNode.java

@@ -3,6 +3,7 @@
 import dev.harrel.jsonschema.JsonNode;
 import dev.harrel.jsonschema.SimpleType;
 
+import java.math.BigDecimal;
 import java.util.List;
 import java.util.Map;
 import java.util.Objects;
@@ -54,4 +55,8 @@ public final Map<String, JsonNode> asObject() {
     abstract List<JsonNode> createArray();
     abstract Map<String, JsonNode> createObject();
     abstract SimpleType computeNodeType(T node);
+
+    static boolean canConvertToInteger(BigDecimal bigDecimal) {
+        return bigDecimal.scale() <= 0 || bigDecimal.stripTrailingZeros().scale() <= 0;
+    }
 }

src/main/java/dev/harrel/jsonschema/providers/GsonNode.java

@@ -12,6 +12,8 @@
 import java.util.*;
 
 public final class GsonNode extends AbstractJsonNode<JsonElement> {
+    private BigDecimal asNumber;
+
     private GsonNode(JsonElement node, String jsonPointer) {
         super(Objects.requireNonNull(node), jsonPointer);
     }
@@ -32,12 +34,12 @@ public String asString() {
 
     @Override
     public BigInteger asInteger() {
-        return node.getAsBigDecimal().toBigInteger();
+        return asNumber.toBigInteger();
     }
 
     @Override
     public BigDecimal asNumber() {
-        return node.getAsBigDecimal();
+        return asNumber;
     }
 
     @Override
@@ -84,8 +86,8 @@ SimpleType computeNodeType(JsonElement node) {
             } else if (jsonPrimitive.isString()) {
                 return SimpleType.STRING;
             } else {
-                BigDecimal bigDecimal = jsonPrimitive.getAsBigDecimal();
-                if (bigDecimal.scale() <= 0 || bigDecimal.stripTrailingZeros().scale() <= 0) {
+                asNumber = jsonPrimitive.getAsBigDecimal();
+                if (canConvertToInteger(asNumber)) {
                     return SimpleType.INTEGER;
                 } else {
                     return SimpleType.NUMBER;

src/main/java/dev/harrel/jsonschema/providers/JakartaJsonNode.java

@@ -76,8 +76,7 @@ SimpleType computeNodeType(JsonValue node) {
             case FALSE:
                 return SimpleType.BOOLEAN;
             case NUMBER:
-                JsonNumber jsonNumber = (JsonNumber) node;
-                if (jsonNumber.isIntegral() || jsonNumber.bigDecimalValue().stripTrailingZeros().scale() <= 0) {
+                if (canConvertToInteger(((JsonNumber) node).bigDecimalValue())) {
                     return SimpleType.INTEGER;
                 } else {
                     return SimpleType.NUMBER;

src/main/java/dev/harrel/jsonschema/providers/KotlinxJsonNode.java

@@ -78,7 +78,7 @@ SimpleType computeNodeType(JsonElement node) {
                 return SimpleType.BOOLEAN;
             }
             asNumber = new BigDecimal(content);
-            if (asNumber.scale() <= 0 || asNumber.stripTrailingZeros().scale() <= 0) {
+            if (canConvertToInteger(asNumber)) {
                 return SimpleType.INTEGER;
             } else {
                 return SimpleType.NUMBER;

src/main/java/dev/harrel/jsonschema/providers/SnakeYamlNode.java

@@ -91,7 +91,7 @@ SimpleType computeNodeType(Node node) {
                 return SimpleType.STRING;
             }
             asNumber = floatToBigDecimal(node);
-            if (asNumber.scale() <= 0 || asNumber.stripTrailingZeros().scale() <= 0) {
+            if (canConvertToInteger(asNumber)) {
                 return SimpleType.INTEGER;
             } else {
                 return SimpleType.NUMBER;