Skip to content

Commit b2e032f

Browse files
committed
Simplify toJson() method.
1 parent 4d37f6c commit b2e032f

File tree

2 files changed

+50
-58
lines changed
  • lodash-plugin/src/main/java/com/github/underscore/lodash
  • string-plugin/src/main/java/com/github/underscore/string

2 files changed

+50
-58
lines changed

lodash-plugin/src/main/java/com/github/underscore/lodash/$.java

Lines changed: 25 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -950,15 +950,15 @@ public static void writeJson(byte[] array, StringBuilder builder) {
950950
} else if (array.length == 0) {
951951
builder.append("[]");
952952
} else {
953-
builder.append("[");
953+
builder.append('[');
954954
builder.append(String.valueOf(array[0]));
955955

956956
for (int i = 1; i < array.length; i++) {
957-
builder.append(",");
957+
builder.append(',');
958958
builder.append(String.valueOf(array[i]));
959959
}
960960

961-
builder.append("]");
961+
builder.append(']');
962962
}
963963
}
964964

@@ -968,15 +968,15 @@ public static void writeJson(short[] array, StringBuilder builder) {
968968
} else if (array.length == 0) {
969969
builder.append("[]");
970970
} else {
971-
builder.append("[");
971+
builder.append('[');
972972
builder.append(String.valueOf(array[0]));
973973

974974
for (int i = 1; i < array.length; i++) {
975-
builder.append(",");
975+
builder.append(',');
976976
builder.append(String.valueOf(array[i]));
977977
}
978978

979-
builder.append("]");
979+
builder.append(']');
980980
}
981981
}
982982

@@ -986,15 +986,15 @@ public static void writeJson(int[] array, StringBuilder builder) {
986986
} else if (array.length == 0) {
987987
builder.append("[]");
988988
} else {
989-
builder.append("[");
989+
builder.append('[');
990990
builder.append(String.valueOf(array[0]));
991991

992992
for (int i = 1; i < array.length; i++) {
993-
builder.append(",");
993+
builder.append(',');
994994
builder.append(String.valueOf(array[i]));
995995
}
996996

997-
builder.append("]");
997+
builder.append(']');
998998
}
999999
}
10001000

@@ -1004,15 +1004,15 @@ public static void writeJson(long[] array, StringBuilder builder) {
10041004
} else if (array.length == 0) {
10051005
builder.append("[]");
10061006
} else {
1007-
builder.append("[");
1007+
builder.append('[');
10081008
builder.append(String.valueOf(array[0]));
10091009

10101010
for (int i = 1; i < array.length; i++) {
1011-
builder.append(",");
1011+
builder.append(',');
10121012
builder.append(String.valueOf(array[i]));
10131013
}
10141014

1015-
builder.append("]");
1015+
builder.append(']');
10161016
}
10171017
}
10181018

@@ -1022,15 +1022,15 @@ public static void writeJson(float[] array, StringBuilder builder) {
10221022
} else if (array.length == 0) {
10231023
builder.append("[]");
10241024
} else {
1025-
builder.append("[");
1025+
builder.append('[');
10261026
builder.append(String.valueOf(array[0]));
10271027

10281028
for (int i = 1; i < array.length; i++) {
1029-
builder.append(",");
1029+
builder.append(',');
10301030
builder.append(String.valueOf(array[i]));
10311031
}
10321032

1033-
builder.append("]");
1033+
builder.append(']');
10341034
}
10351035
}
10361036

@@ -1040,15 +1040,15 @@ public static void writeJson(double[] array, StringBuilder builder) {
10401040
} else if (array.length == 0) {
10411041
builder.append("[]");
10421042
} else {
1043-
builder.append("[");
1043+
builder.append('[');
10441044
builder.append(String.valueOf(array[0]));
10451045

10461046
for (int i = 1; i < array.length; i++) {
1047-
builder.append(",");
1047+
builder.append(',');
10481048
builder.append(String.valueOf(array[i]));
10491049
}
10501050

1051-
builder.append("]");
1051+
builder.append(']');
10521052
}
10531053
}
10541054

@@ -1058,15 +1058,15 @@ public static void writeJson(boolean[] array, StringBuilder builder) {
10581058
} else if (array.length == 0) {
10591059
builder.append("[]");
10601060
} else {
1061-
builder.append("[");
1061+
builder.append('[');
10621062
builder.append(String.valueOf(array[0]));
10631063

10641064
for (int i = 1; i < array.length; i++) {
1065-
builder.append(",");
1065+
builder.append(',');
10661066
builder.append(String.valueOf(array[i]));
10671067
}
10681068

1069-
builder.append("]");
1069+
builder.append(']');
10701070
}
10711071
}
10721072

@@ -1094,15 +1094,15 @@ public static void writeJson(Object[] array, StringBuilder builder) {
10941094
} else if (array.length == 0) {
10951095
builder.append("[]");
10961096
} else {
1097-
builder.append("[");
1097+
builder.append('[');
10981098
JsonValue.writeJson(array[0], builder);
10991099

11001100
for (int i = 1; i < array.length; i++) {
1101-
builder.append(",");
1101+
builder.append(',');
11021102
JsonValue.writeJson(array[i], builder);
11031103
}
11041104

1105-
builder.append("]");
1105+
builder.append(']');
11061106
}
11071107
}
11081108
}
@@ -1388,7 +1388,7 @@ private String readName() {
13881388
if (current != '"') {
13891389
throw expected("name");
13901390
}
1391-
return readStringInternal();
1391+
return readString();
13921392
}
13931393

13941394
private String readNull() {
@@ -1423,10 +1423,6 @@ private void readRequiredChar(char ch) {
14231423
}
14241424

14251425
private String readString() {
1426-
return readStringInternal();
1427-
}
1428-
1429-
private String readStringInternal() {
14301426
read();
14311427
startCapture();
14321428
while (current != '"') {

string-plugin/src/main/java/com/github/underscore/string/$.java

Lines changed: 25 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -532,15 +532,15 @@ public static void writeJson(byte[] array, StringBuilder builder) {
532532
} else if (array.length == 0) {
533533
builder.append("[]");
534534
} else {
535-
builder.append("[");
535+
builder.append('[');
536536
builder.append(String.valueOf(array[0]));
537537

538538
for (int i = 1; i < array.length; i++) {
539-
builder.append(",");
539+
builder.append(',');
540540
builder.append(String.valueOf(array[i]));
541541
}
542542

543-
builder.append("]");
543+
builder.append(']');
544544
}
545545
}
546546

@@ -550,15 +550,15 @@ public static void writeJson(short[] array, StringBuilder builder) {
550550
} else if (array.length == 0) {
551551
builder.append("[]");
552552
} else {
553-
builder.append("[");
553+
builder.append('[');
554554
builder.append(String.valueOf(array[0]));
555555

556556
for (int i = 1; i < array.length; i++) {
557-
builder.append(",");
557+
builder.append(',');
558558
builder.append(String.valueOf(array[i]));
559559
}
560560

561-
builder.append("]");
561+
builder.append(']');
562562
}
563563
}
564564

@@ -568,15 +568,15 @@ public static void writeJson(int[] array, StringBuilder builder) {
568568
} else if (array.length == 0) {
569569
builder.append("[]");
570570
} else {
571-
builder.append("[");
571+
builder.append('[');
572572
builder.append(String.valueOf(array[0]));
573573

574574
for (int i = 1; i < array.length; i++) {
575-
builder.append(",");
575+
builder.append(',');
576576
builder.append(String.valueOf(array[i]));
577577
}
578578

579-
builder.append("]");
579+
builder.append(']');
580580
}
581581
}
582582

@@ -586,15 +586,15 @@ public static void writeJson(long[] array, StringBuilder builder) {
586586
} else if (array.length == 0) {
587587
builder.append("[]");
588588
} else {
589-
builder.append("[");
589+
builder.append('[');
590590
builder.append(String.valueOf(array[0]));
591591

592592
for (int i = 1; i < array.length; i++) {
593-
builder.append(",");
593+
builder.append(',');
594594
builder.append(String.valueOf(array[i]));
595595
}
596596

597-
builder.append("]");
597+
builder.append(']');
598598
}
599599
}
600600

@@ -604,15 +604,15 @@ public static void writeJson(float[] array, StringBuilder builder) {
604604
} else if (array.length == 0) {
605605
builder.append("[]");
606606
} else {
607-
builder.append("[");
607+
builder.append('[');
608608
builder.append(String.valueOf(array[0]));
609609

610610
for (int i = 1; i < array.length; i++) {
611-
builder.append(",");
611+
builder.append(',');
612612
builder.append(String.valueOf(array[i]));
613613
}
614614

615-
builder.append("]");
615+
builder.append(']');
616616
}
617617
}
618618

@@ -622,15 +622,15 @@ public static void writeJson(double[] array, StringBuilder builder) {
622622
} else if (array.length == 0) {
623623
builder.append("[]");
624624
} else {
625-
builder.append("[");
625+
builder.append('[');
626626
builder.append(String.valueOf(array[0]));
627627

628628
for (int i = 1; i < array.length; i++) {
629-
builder.append(",");
629+
builder.append(',');
630630
builder.append(String.valueOf(array[i]));
631631
}
632632

633-
builder.append("]");
633+
builder.append(']');
634634
}
635635
}
636636

@@ -640,15 +640,15 @@ public static void writeJson(boolean[] array, StringBuilder builder) {
640640
} else if (array.length == 0) {
641641
builder.append("[]");
642642
} else {
643-
builder.append("[");
643+
builder.append('[');
644644
builder.append(String.valueOf(array[0]));
645645

646646
for (int i = 1; i < array.length; i++) {
647-
builder.append(",");
647+
builder.append(',');
648648
builder.append(String.valueOf(array[i]));
649649
}
650650

651-
builder.append("]");
651+
builder.append(']');
652652
}
653653
}
654654

@@ -676,15 +676,15 @@ public static void writeJson(Object[] array, StringBuilder builder) {
676676
} else if (array.length == 0) {
677677
builder.append("[]");
678678
} else {
679-
builder.append("[");
679+
builder.append('[');
680680
JsonValue.writeJson(array[0], builder);
681681

682682
for (int i = 1; i < array.length; i++) {
683-
builder.append(",");
683+
builder.append(',');
684684
JsonValue.writeJson(array[i], builder);
685685
}
686686

687-
builder.append("]");
687+
builder.append(']');
688688
}
689689
}
690690
}
@@ -970,7 +970,7 @@ private String readName() {
970970
if (current != '"') {
971971
throw expected("name");
972972
}
973-
return readStringInternal();
973+
return readString();
974974
}
975975

976976
private String readNull() {
@@ -1005,10 +1005,6 @@ private void readRequiredChar(char ch) {
10051005
}
10061006

10071007
private String readString() {
1008-
return readStringInternal();
1009-
}
1010-
1011-
private String readStringInternal() {
10121008
read();
10131009
startCapture();
10141010
while (current != '"') {

0 commit comments

Comments
 (0)