3桁ごとにカンマをつけたい。

Question

以前にも質問させていただきましたが、アンドロイドスタジオで電卓を作っています。
画面に表示させている文字列を数値に変換した変数と、数字ボタンが押された際に表示される文字列を数値に変換した、変数とでTextView内で連結させています。
連結させた所、それまで3桁毎についていたカンマが、4桁毎につくようになってしまいました。
「12,345」と表示させたいが、「1,2345」となっている。

前提・実現したいこと

3桁毎にカンマが入るように修正をし、「12,345」と表示させたい。

発生している問題・エラーメッセージ

4桁毎にカンマが入っている。

該当のソースコード

	private String number = new String();
	private int hengeAtai = 0;
	private int num;
	
// 0~9のボタンを押した時
public void pushNumbers(View view) {
	switch (view.getId()) {
		case R.id.imageButton:
			if (number.length() == 9) {
				return;
			} else {
				number = "1";
			}
			break;
		case R.id.imageButton2:
			if (number.length() == 9) {
				return;
			} else {
				number = "2";
			}
			break;
　　　　case R.id.imageButton10:
	if (number.length() == 0) {
		return;
	} else if (number.length() == 9) {
		return;
	} else {
		// それ以外の数字キーが、押されたら0を入力
		number = "0";
	}
		break;
    }
chengeAtai = Integer.parseInt(number);
String valueTwo = text01.getText().toString().replace(",", "");
num = Integer.parseInt(valueTwo);
	text01.setText(String.format("%,d", num) + chengeAtai + "");
}

Java

試したこと

String.format内の変数や、()の位置を変更すれば対処出来るんじゃないかと思って試しました。
結果として、数字のボタンを続けて押しても桁が増えず、押されたボタンを基にどんどんと数値が足されていくだけでした。

Comments

[gentaro]

> 「12.345」と表示させたい
それカンマじゃないですやん
というツッコミだけ入れておきます。

[退会済みユーザー]

ロケール問題じゃねきっと
あとデシマルフォーマットとかナンバーフォーマットつかわんのはなぜだ?

[Orlofsky]

他の質問もだけど、コードは https://teratail.com/help#about-markdown の [コードを入力] に修正してください。

[退会済みユーザー]

ちなみに "1,000"+1 = "1,0001" は書いてある処理としてはまちがえてないからな

[Orlofsky]

markdownの使い方が間違っています。

[plight03]

markdownの使用方法が、間違っておりました。
ソースコードを、 [コードを入力] 欄に修正しました。

[plight03]

書かれている上記２つの内、どちらかを使わなかったのかについては、知らなかったためです。

Answer 1

String.format内の変数や、()の位置を変更すれば対処出来るんじゃないかと思って試しました。
結果として、数字のボタンを続けて押しても桁が増えず、押されたボタンを基にどんどんと数値が足されていくだけでした。

バグは通常, 勘で対処するものではありません.
そのような対応になってしまうのは, コードの意味を把握出来ていないためと思います.

String.format("%,d", num) + chengeAtai + ""

は, num = 1234, changeAtai = 5 の時,
・num を "%,d" でフォーマットした "1,234"
・changeAtai を文字列化した "5"
・""(空文字列)
を繋げた "1,2345" を表します.
つまり「3桁毎についていたカンマが、4桁毎につくように」なったのでは無く, 「3桁毎にカンマのついている文字列に, 1文字の文字列を繋げている」だけです.

"12,345" を得たいのでしたら, num の桁を上げ(10倍)て一の位に changeAtai を入れてからフォーマットする

String.format("%,d", num * 10 + changeAtai)

となるかと思います.

Comments

[plight03]

回答していただき、ありがとうございます。
回答内容を基に試してみた所、3桁毎にカンマが入りました。
指摘されている通り、把握できていない点はありました。