回答編集履歴

負数最大値に対応(暫定)

2019/11/15 03:04

投稿

スコア13355

answer CHANGED Viewed

@@ -5,12 +5,13 @@
   private static String commaSeparatedString(int i) {
     StringBuilder sb = new StringBuilder();
     //負数は正数化
+    long l = i;
-    if(i < 0) {
+    if(l < 0) {
       sb.append('-');
-      i *= -1;
+      l *= -1;
     }
     //文字列化
-    String s = ""+i;
+    String s = ""+l;
     //最初の1～SEPARATE_WIDTH文字を処理
     int e = s.length() % SEPARATE_WIDTH;
     if(e == 0) e = SEPARATE_WIDTH;

"3" を定数化

2019/11/15 03:04

投稿

スコア13355

answer CHANGED Viewed

@@ -1,6 +1,7 @@
 String を返す形にしています.
 ```java
+  static final int SEPARATE_WIDTH = 3;
   private static String commaSeparatedString(int i) {
     StringBuilder sb = new StringBuilder();
     //負数は正数化
@@ -10,13 +11,13 @@
     }
     //文字列化
     String s = ""+i;
-    //最初の1～3文字を処理
+    //最初の1～SEPARATE_WIDTH文字を処理
-    int e = s.length() % 3;
+    int e = s.length() % SEPARATE_WIDTH;
-    if(e == 0) e = 3;
+    if(e == 0) e = SEPARATE_WIDTH;
     sb.append(s.substring(0, e));
-    //残りを3文字ずつ処理
+    //残りをSEPARATE_WIDTH文字ずつ処理
-    for(int j=e; j<s.length(); j+=3) {
+    for(int j=e; j<s.length(); j+=SEPARATE_WIDTH) {
-      sb.append(',').append(s.substring(j, j+3));
+      sb.append(',').append(s.substring(j, j+SEPARATE_WIDTH));
     }
     return sb.toString();

s を更新せず index を使うように修正

2019/11/14 16:43

投稿

スコア13355

answer CHANGED Viewed

@@ -14,11 +14,9 @@
     int e = s.length() % 3;
     if(e == 0) e = 3;
     sb.append(s.substring(0, e));
-    s = s.substring(e);
     //残りを3文字ずつ処理
-    while(s.length() >= 1) {
+    for(int j=e; j<s.length(); j+=3) {
-      sb.append(',').append(s.substring(0, 3));
+      sb.append(',').append(s.substring(j, j+3));
-      s = s.substring(3);
     }
     return sb.toString();