Split JSON_TABLE rendering into focused clause helpers

Signed-off-by: minleejae <mmj9808@gmail.com>

Author: minleejae

src/main/java/net/sf/jsqlparser/expression/JsonTableFunction.java

@@ -699,6 +699,27 @@ public StringBuilder appendTo(StringBuilder builder,
                 builder.append(" FOR ORDINALITY");
                 return builder;
             }
+            appendTypeOptions(builder);
+            appendPathOptions(builder, expressionPrinter);
+            if (onEmptyBehavior != null && !onEmptyAfterOnError) {
+                builder.append(" ");
+                onEmptyBehavior.appendTo(builder, expressionPrinter);
+                builder.append(" ON EMPTY");
+            }
+            if (onErrorBehavior != null) {
+                builder.append(" ");
+                onErrorBehavior.appendTo(builder, expressionPrinter);
+                builder.append(" ON ERROR");
+            }
+            if (onEmptyBehavior != null && onEmptyAfterOnError) {
+                builder.append(" ");
+                onEmptyBehavior.appendTo(builder, expressionPrinter);
+                builder.append(" ON EMPTY");
+            }
+            return builder;
+        }
+
+        private void appendTypeOptions(StringBuilder builder) {
             if (dataType != null) {
                 builder.append(" ").append(dataType);
             }
@@ -719,6 +740,10 @@ public StringBuilder appendTo(StringBuilder builder,
             if (exists) {
                 builder.append(" EXISTS");
             }
+        }
+
+        private void appendPathOptions(StringBuilder builder,
+                Consumer<Expression> expressionPrinter) {
             if (wrapperClause != null && wrapperClause.isBeforePathExpression()) {
                 builder.append(" ").append(wrapperClause);
             }
@@ -732,22 +757,6 @@ public StringBuilder appendTo(StringBuilder builder,
             if (quotesClause != null) {
                 builder.append(" ").append(quotesClause);
             }
-            if (onEmptyBehavior != null && !onEmptyAfterOnError) {
-                builder.append(" ");
-                onEmptyBehavior.appendTo(builder, expressionPrinter);
-                builder.append(" ON EMPTY");
-            }
-            if (onErrorBehavior != null) {
-                builder.append(" ");
-                onErrorBehavior.appendTo(builder, expressionPrinter);
-                builder.append(" ON ERROR");
-            }
-            if (onEmptyBehavior != null && onEmptyAfterOnError) {
-                builder.append(" ");
-                onEmptyBehavior.appendTo(builder, expressionPrinter);
-                builder.append(" ON EMPTY");
-            }
-            return builder;
         }
     }
 
@@ -925,6 +934,36 @@ public String toString() {
 
     public StringBuilder appendTo(StringBuilder builder, Consumer<Expression> expressionPrinter) {
         builder.append("JSON_TABLE(");
+        appendInputAndPath(builder, expressionPrinter);
+        appendPassingClauses(builder, expressionPrinter);
+        if (onErrorClause != null && onErrorClause.isBeforeColumns()) {
+            builder.append(" ").append(onErrorClause);
+        }
+        if (parsingTypeClause != null) {
+            builder.append(" ").append(parsingTypeClause);
+        }
+        if (onEmptyClause != null) {
+            builder.append(" ").append(onEmptyClause);
+        }
+        builder.append(" ");
+        if (columnsClause == null) {
+            builder.append("null");
+        } else {
+            columnsClause.appendTo(builder, expressionPrinter);
+        }
+        if (planClause != null) {
+            builder.append(" ");
+            planClause.appendTo(builder, expressionPrinter);
+        }
+        if (onErrorClause != null && !onErrorClause.isBeforeColumns()) {
+            builder.append(" ").append(onErrorClause);
+        }
+        builder.append(")");
+        return builder;
+    }
+
+    private void appendInputAndPath(StringBuilder builder,
+            Consumer<Expression> expressionPrinter) {
         printExpression(builder, jsonInputExpression, expressionPrinter);
         if (formatJson) {
             builder.append(" FORMAT JSON");
@@ -936,6 +975,10 @@ public StringBuilder appendTo(StringBuilder builder, Consumer<Expression> expres
         if (pathName != null) {
             builder.append(" AS ").append(pathName);
         }
+    }
+
+    private void appendPassingClauses(StringBuilder builder,
+            Consumer<Expression> expressionPrinter) {
         if (!passingClauses.isEmpty()) {
             builder.append(" PASSING ");
             boolean first = true;
@@ -951,30 +994,6 @@ public StringBuilder appendTo(StringBuilder builder, Consumer<Expression> expres
                 first = false;
             }
         }
-        if (onErrorClause != null && onErrorClause.isBeforeColumns()) {
-            builder.append(" ").append(onErrorClause);
-        }
-        if (parsingTypeClause != null) {
-            builder.append(" ").append(parsingTypeClause);
-        }
-        if (onEmptyClause != null) {
-            builder.append(" ").append(onEmptyClause);
-        }
-        builder.append(" ");
-        if (columnsClause == null) {
-            builder.append("null");
-        } else {
-            columnsClause.appendTo(builder, expressionPrinter);
-        }
-        if (planClause != null) {
-            builder.append(" ");
-            planClause.appendTo(builder, expressionPrinter);
-        }
-        if (onErrorClause != null && !onErrorClause.isBeforeColumns()) {
-            builder.append(" ").append(onErrorClause);
-        }
-        builder.append(")");
-        return builder;
     }
 
     private static void printExpression(StringBuilder builder, Expression expression,