Skip to content

Commit 358e462

Browse files
author
nt
committed
Refactor for remove the loop and the StringBuilder
1 parent d1dbf84 commit 358e462

File tree

1 file changed

+5
-8
lines changed
  • src/main/java/org/mybatis/dynamic/sql/select/function

1 file changed

+5
-8
lines changed

src/main/java/org/mybatis/dynamic/sql/select/function/Add.java

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,8 @@
1919
import java.util.List;
2020
import java.util.Objects;
2121
import java.util.Optional;
22+
import java.util.StringJoiner;
23+
import java.util.stream.Collectors;
2224

2325
import org.mybatis.dynamic.sql.BindableColumn;
2426
import org.mybatis.dynamic.sql.render.TableAliasCalculator;
@@ -39,14 +41,9 @@ public Optional<String> alias() {
3941

4042
@Override
4143
public String renderWithTableAlias(TableAliasCalculator tableAliasCalculator) {
42-
StringBuilder builder = new StringBuilder();
43-
for (BindableColumn<T> column: columns) {
44-
if (builder.length() > 0) {
45-
builder.append(" + ");
46-
}
47-
builder.append(column.renderWithTableAlias(tableAliasCalculator));
48-
}
49-
return builder.toString();
44+
return columns.stream()
45+
.map(column -> column.renderWithTableAlias(tableAliasCalculator))
46+
.collect(Collectors.joining(" + "));
5047
}
5148

5249
@Override

0 commit comments

Comments
 (0)