回答編集履歴

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

2019/11/15 03:04

投稿

スコア12648

test CHANGED Viewed

@@ -12,17 +12,19 @@
     //負数は正数化
+    long l = i;
-    if(i < 0) {
+    if(l < 0) {
       sb.append('-');
-      i *= -1;
+      l *= -1;
     }
     //文字列化
-    String s = ""+i;
+    String s = ""+l;
     //最初の1～SEPARATE_WIDTH文字を処理

"3" を定数化

2019/11/15 03:04

投稿

スコア12648

test CHANGED Viewed

@@ -3,6 +3,8 @@
 ```java
+  static final int SEPARATE_WIDTH = 3;
   private static String commaSeparatedString(int i) {
@@ -22,19 +24,19 @@
     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));
     }

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

2019/11/14 16:43

投稿

スコア12648

test CHANGED Viewed

@@ -30,15 +30,11 @@
     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);
     }