Skip to content

Commit 384df2e

Browse files
author
Henady Zakalusky
committed
fix for default value with array for generate all setters with default value
- optimization
1 parent e5a630f commit 384df2e

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

src/main/java/com/bruce/intellijplugin/generatesetter/actions/GenerateAllSetterBase.java

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -540,11 +540,14 @@ else if (psiClassOfParameter!=null && psiClassOfParameter.isEnum()) {
540540
}
541541
else {
542542
String realName = paramInfo.getParams().get(0).getRealName();
543+
builder.append("new " + realName);
543544

544545
if (paramInfo.isArray()) {
545-
builder.append("new " + realName + StringUtil.repeat("[0]", paramInfo.getArrayDimensions()));
546+
for (int i = 0; i < paramInfo.getArrayDimensions(); i++) {
547+
builder.append("[0]");
548+
}
546549
} else {
547-
builder.append("new " + realName + "()");
550+
builder.append("()");
548551
}
549552
}
550553
if (!javaSimpleTypes.contains(realPackage)) {
@@ -566,7 +569,9 @@ private static void appendCollectNotEmpty(StringBuilder builder,
566569
Set<String> newImportList) {
567570
builder.append("new ").append(defaultImpl);
568571
if (paramInfo.isArray()) {
569-
builder.append(StringUtil.repeat("[0]", paramInfo.getArrayDimensions()));
572+
for (int i = 0; i < paramInfo.getArrayDimensions(); i++) {
573+
builder.append("[0]");
574+
}
570575
} else {
571576
builder.append("<");
572577
for (int i = 0; i < paramInfo.getParams().size(); i++) {

0 commit comments

Comments
 (0)