数値に3桁ごとにカンマを付けるにはどうすれば良いでしょうか？

Question

		System.out.println(commaSeparatedString(123));
		System.out.println(commaSeparatedString(12345));
		System.out.println(commaSeparatedString(1234567));
		System.out.println(commaSeparatedString(-1234567));
		System.out.println(commaSeparatedString(123456789));
		System.out.println(commaSeparatedString(-123456789));

	}

	
	private static char[] commaSeparatedString(int i) {

		//文字列操作を行うためにStringBuilderを用意
		StringBuilder sb = new StringBuilder();

	    // numberを文字列にして追加
		sb.add(char.class, toString(number));
	//	文字列の後ろから３桁毎にカンマを追加していく

	    //文字列の長さの3文字前に初期値を設定

		int offset = sb.length() - 3;

	    //文字列の先頭まで繰り返す
		while(offset < 0) {
			// -123456等の場合に、 "-、123,456"となってしまわないように
						// offsetが1でnumberがマイナスの場合は、カンマを追加しないように対処する
			if(offset != 1 || number > 0) {

				sb.insert(offset,",");
			}

				//次のカンマの位置は3文字前
				offset -= 3;





		
			}
		}
		
	}

	
コード

ここまでしか出来ないし間違いが有ると思います。
教えてください。
※String.format()など、そのままズバリできてしまうメソッドの利用は禁止

Comments

[m.ts10806]

課題なら自力でやりましょう。

[ta.tatu]

頑張って見ます。

[swordone]

返り値がchar[]なのはなぜですか？メソッド名がcommaSeparated"String"であることと食い違います。

Answer 1

想像するものとは違うかもしれませんが、これだけでできます。

private static char[] commaSeparatedString(int i) {
    return String.format("%,d", i).toCharArray();
}

Comments

[退会済みユーザー]

それ以外とかいってるｗ

[LouiS0616]

後出しされてもどうしようも無いのですけどね。

[ta.tatu]

すいません。
※String.format()など、そのままズバリできてしまうメソッドの利用は禁止
追記しました。

[LouiS0616]

学習目的と言うことでしょうか。
しかしご提示のコードは唐突にnumberが表れていたり、ちょっと情報が足りないのでは。

Answer 2

NumberFormatを使えば解決しませんか。

https://www.javalife.jp/2018/01/20/post-243/

Comments

[mutsuki22]

あ！
もしかして自力のコードで実行したいってことですかね。
それだったらこの回答は駄目になりますね。

[ta.tatu]

すいません。
追記しました。

Answer 3

方針だけ。

• 数字を 10 進での文字列にするときの方法を書く。そこに 3 桁ごとに "," を追加する処理ち追加する。

(数字の先頭には "," を付け加えないとか、 - 符号のことも考慮する)

• 数字を 1000 で % と / で処理しながら文字列をつくっていく。そこに "," を売れこむ処理を追加する。

この場合も 数字の先頭の特別処理あ 符号のことも考慮する)

Answer 4

再帰呼出しを使ってみました。

	private static void comma(StringBuilder sb, long i) {
		if (i < 1000) sb.append(i);
		else {
			comma(sb, i / 1000);
			sb.append(',').append(i/100%10).append(i/10%10).append(i%10);
		}
	}
    private static String commaSeparatedString(int number) {
		StringBuilder sb = new StringBuilder();
		if (number >= 0) comma(sb, number);
		else comma(sb.append('-'), -(long)number);
		return sb.toString();
    }

愚直なやり方は、

    private static String commaSeparatedString(int number) {
		String s = Integer.toString(number);
		int n = s.length(), i = n % 3;
		if (i == 0) i = 3;
		else if (i == 1 && number < 0) i = 4;
		String t = s.substring(0, i);
		while (i < n) t += "," + s.substring(i, i += 3);
		return t;
    }

Answer 5

末尾から3桁ずつ戻していって、正の数(符号なし)なら終点を0に、負の数(先頭に符号)なら終点を1にすればいい。

    private static char[] commaSeparatedString(int i) {

        //文字列操作を行うためにStringBuilderを用意
        StringBuilder sb = new StringBuilder();
        sb.append(i);
        int end = sb.charAt(0) == '-' ? 1 : 0;
        for (int x = sb.length() - 3; x > end; x -= 3) {
            sb.insert(x, ',');
        }
        char[] arr = new char[sb.length()];
        sb.getChars(0, sb.length(), arr, 0);
        return arr;
    }

Comments

[kazuma-s]

i -= 3 は x -= 3 の間違いではありませんか？

[swordone]

すみません、つい癖で…

Answer 6

そのまま、普段あたまで考えていることを愚直に書けばええのでは？

数字の後ろから見るのが面倒だったので、StringBuilderのreverse使ってるけど
別にlengthとindex（offset）の計算でもいいと思います。

private static char[] commaSeparatedString(int i) {
	StringBuilder ret = new StringBuilder();
	StringBuilder num = new StringBuilder("" + i);
	num.reverse();

	for(int j = 1; j <= num.length(); j++) { // 仮引数は本当はiでなくnumberでは？
		ret.append(num.charAt(j - 1));
		// if(3桁毎に次の文字が存在してかつ、ハイフンでないなら) {
		if(j % 3 == 0 && j < num.length() && num.charAt(j) != '-')
			ret.append(',');
		}
	}
	return ret.reverse().toString().toCharArray();
}

Comments

[momon-ga]

ベストアンサー決まったので、if文の中身を記載

if(j % 3 == 0 && j < num.length() && num.charAt(j) != '-')

[swordone]

コメントしたあとで的外れだったことに気づき、削除

Answer 7

String を返す形にしています.

  static final int SEPARATE_WIDTH = 3;
  private static String commaSeparatedString(int i) {
    StringBuilder sb = new StringBuilder();
    //負数は正数化
    long l = i;
    if(l < 0) {
      sb.append('-');
      l *= -1;
    }
    //文字列化
    String s = ""+l;
    //最初の1～SEPARATE_WIDTH文字を処理
    int e = s.length() % SEPARATE_WIDTH;
    if(e == 0) e = SEPARATE_WIDTH;
    sb.append(s.substring(0, e));
    //残りをSEPARATE_WIDTH文字ずつ処理
    for(int j=e; j<s.length(); j+=SEPARATE_WIDTH) {
      sb.append(',').append(s.substring(j, j+SEPARATE_WIDTH));
    }

    return sb.toString();
  }

Comments

[kazuma-s]

commaSeparatedString(-2147483648) が正しく処理されません。

Answer 8

確認してないけど、こんなの保守しろと言われたらイヤな奴

private static char[] commaSeparatedString(int val)
{
	char[] chars = new char[14];
	int idx = 13;
	boolean minus = false;
	if (val < 0) {
		minus = true;
		val = -val;
	}
	do {
		if (idx % 4 == 2) chars[idx--] = ',';
		chars[idx--] = (char)('0' + (val % 10));
		val /= 10;
	} while(val != 0);
	if (minus) chars[idx--] = '-';
	if (idx == -1) return chars;
	char[] result = new char[13 - idx];
	for (int i = 0; i < result.length; ++i) result[i] = chars[++idx];
	return result;
}

Comments

[kazuma-s]

commaSeparatedString(-2147483648) が正しく処理されません。

[退会済みユーザー]

それは符号反転したときにあぶれちゃいますねｗ
他が動いてることに感動しました
ネタなのでバグらせときます

[退会済みユーザー]

回答多いので、特徴というか他との違いだけ書いときますね
・できあいのクラスを一切使っていない、完全独自処理
・速度特化

Answer 9

まあ、適当に書いたもの

末尾、もしくは, の前に数値が４つならんでいたら１文字目と２文字目の間に, を追加する

		long a = -304564L;
		String b = Long.toString(a);
		Pattern p1 = Pattern.compile("([0-9])([0-9]{3})$");
		Pattern p2 = Pattern.compile("([0-9])([0-9]{3},.*)$");

		b = p1.matcher(b).replaceFirst("$1,$2");
		while (p2.matcher(b).find()) {
			b = p2.matcher(b).replaceFirst("$1,$2");
		}

		System.out.println(b);

清書版（ステップ数削減ｗ）

        long a = -30456423424563463L;
        String b = Long.toString(a);
        Pattern p = Pattern.compile("([0-9])([0-9]{3}(,.*)*)$");

        while (p.matcher(b).find()) 
            b = p.matcher(b).replaceFirst("$1,$2");

        System.out.println(b);