Avoid wrapping single record in list (#382)

metafacture · Sep 3, 2021 · b4e056b · b4e056b
1 parent 7b47a1c
commit b4e056b
Showing 1 changed file with 18 additions and 12 deletions.
diff --git a/metafacture-json/src/main/java/org/metafacture/json/JsonDecoder.java b/metafacture-json/src/main/java/org/metafacture/json/JsonDecoder.java
@@ -132,18 +132,24 @@ public void resetRecordCount() {
     @Override
     public void process(final String json) {
         assert !isClosed();
-        final List<String> records = recordPath.isEmpty() ? Arrays.asList(json)
-                : matches(JsonPath.read(json, recordPath));
-        records.forEach(record -> {
-            createParser(record);
-            try {
-                decode();
-            } catch (final IOException e) {
-                throw new MetafactureException(e);
-            } finally {
-                closeParser();
-            }
-        });
+        if (recordPath.isEmpty()) {
+            processRecord(json);
+        } else {
+            matches(JsonPath.read(json, recordPath)).forEach(record -> {
+                processRecord(record);
+            });
+        }
+    }
+
+    private void processRecord(String record) {
+        createParser(record);
+        try {
+            decode();
+        } catch (final IOException e) {
+            throw new MetafactureException(e);
+        } finally {
+            closeParser();
+        }
     }
 
     private List<String> matches(Object obj) {