Prefer #isEmpty over #size/#length > 0

Author: iconara

jmespath-gson/src/main/java/io/burt/jmespath/gson/GsonRuntime.java

@@ -91,13 +91,13 @@ public boolean isTruthy(JsonElement value) {
       case BOOLEAN:
         return value.getAsBoolean();
       case STRING:
-        return value.getAsString().length() > 0;
+        return !value.getAsString().isEmpty();
       case NUMBER:
         return true;
       case ARRAY:
-        return value.getAsJsonArray().size() > 0;
+        return !value.getAsJsonArray().isEmpty();
       case OBJECT:
-        return value.getAsJsonObject().size() > 0;
+        return !value.getAsJsonObject().isEmpty();
     }
     throw new IllegalStateException(String.format("Unknown node type encountered: %s", value.getClass()));
   }

jmespath-jackson/src/main/java/io/burt/jmespath/jackson/JacksonRuntime.java

@@ -97,9 +97,9 @@ public boolean isTruthy(JsonNode value) {
       case ARRAY:
       case BINARY:
       case OBJECT:
-        return value.size() > 0;
+        return !value.isEmpty();
       case STRING:
-        return value.textValue().length() > 0;
+        return !value.textValue().isEmpty();
       case BOOLEAN:
         return value.booleanValue();
       case MISSING:

jmespath-jakarta-jsonp/src/main/java/io/burt/jmespath/jakarta/jsonp/JsonpRuntime.java

@@ -110,11 +110,11 @@ public boolean isTruthy(JsonValue value) {
       case TRUE:
         return true;
       case ARRAY:
-        return ((JsonArray) value).size() > 0;
+        return !((JsonArray) value).isEmpty();
       case OBJECT:
-        return ((JsonObject) value).size() > 0;
+        return !((JsonObject) value).isEmpty();
       case STRING:
-        return ((JsonString) value).getString().length() > 0;
+        return !((JsonString) value).getString().isEmpty();
       default:
         throw new IllegalStateException(String.format("Unknown node type encountered: %s", value.getValueType()));
     }