Skip to content

Commit 22796a4

Browse files
committed
refactor: split DROP target and parameter rendering
1 parent 00aed16 commit 22796a4

1 file changed

Lines changed: 12 additions & 4 deletions

File tree

  • src/main/java/net/sf/jsqlparser/statement/drop

src/main/java/net/sf/jsqlparser/statement/drop/Drop.java

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -202,6 +202,15 @@ public StringBuilder appendTo(StringBuilder builder, Consumer<Table> tablePrinte
202202
builder.append("MATERIALIZED ");
203203
}
204204
builder.append(type).append(ifExists ? " IF EXISTS " : " ");
205+
appendNames(builder, tablePrinter);
206+
if ("FUNCTION".equals(type)) {
207+
builder.append(formatFuncParams(getParamsByType("FUNCTION")));
208+
}
209+
appendParameters(builder, tablePrinter);
210+
return builder;
211+
}
212+
213+
private void appendNames(StringBuilder builder, Consumer<Table> tablePrinter) {
205214
for (int i = 0; i < names.size(); i++) {
206215
if (i > 0) {
207216
builder.append(", ");
@@ -212,9 +221,9 @@ public StringBuilder appendTo(StringBuilder builder, Consumer<Table> tablePrinte
212221
builder.append(names.get(i));
213222
}
214223
}
215-
if ("FUNCTION".equals(type)) {
216-
builder.append(formatFuncParams(getParamsByType("FUNCTION")));
217-
}
224+
}
225+
226+
private void appendParameters(StringBuilder builder, Consumer<Table> tablePrinter) {
218227
int size = parameters == null ? 0 : parameters.size();
219228
for (int i = 0; i <= size; i++) {
220229
if (table != null && i == tablePosition) {
@@ -225,7 +234,6 @@ public StringBuilder appendTo(StringBuilder builder, Consumer<Table> tablePrinte
225234
builder.append(' ').append(parameters.get(i));
226235
}
227236
}
228-
return builder;
229237
}
230238

231239
public List<String> getParamsByType(String type) {

0 commit comments

Comments
 (0)