質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.13%

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

解決済

回答 9

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 2,627

score 12

        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桁毎にカンマを追加していく

        //文字列の長さの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()など、そのままズバリできてしまうメソッドの利用は禁止

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • m.ts10806

    2019/11/14 19:12

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

    キャンセル

  • ta.tatu

    2019/11/14 19:14

    頑張って見ます。

    キャンセル

  • swordone

    2019/11/14 21:11

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

    キャンセル

  • 退会済みユーザー

    2019/11/15 19:19

    複数のユーザーから「やってほしいことだけを記載した丸投げの質問」という意見がありました
    「質問を編集する」ボタンから編集を行い、調査したこと・試したことを記入していただくと、回答が得られやすくなります。

回答 9

+2

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/11/14 18:58

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

    キャンセル

  • 2019/11/14 19:06

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

    キャンセル

+2

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/11/14 19:08

    それ以外とかいってるw

    キャンセル

  • 2019/11/14 19:09

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

    キャンセル

  • 2019/11/14 19:11

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

    キャンセル

  • 2019/11/14 19:14

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

    キャンセル

checkベストアンサー

+1

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

    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;
    }

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

+1

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

数字の後ろから見るのが面倒だったので、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();
}

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/11/15 10:59

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

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

    キャンセル

  • 2019/11/15 12:07 編集

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

    キャンセル

+1

末尾から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;
    }

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/11/15 02:46

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

    キャンセル

  • 2019/11/15 02:49

    すみません、つい癖で…

    キャンセル

+1

方針だけ。

  • 数字を 10 進での文字列にするときの方法を書く。そこに 3 桁ごとに "," を追加する処理ち追加する。
    (数字の先頭には ","  を付け加えないとか、 - 符号のことも考慮する)

  • 数字を 1000 で % と / で処理しながら文字列をつくっていく。そこに "," を売れこむ処理を追加する。
    この場合も 数字の先頭の特別処理あ 符号のことも考慮する)

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

-1

まあ、適当に書いたもの

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

        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);

清書版(ステップ数削減w)

        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);

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

-1

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

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;
}

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/11/15 02:49

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

    キャンセル

  • 2019/11/15 07:16

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

    キャンセル

  • 2019/11/15 07:40

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

    キャンセル

-1

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();
  }

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/11/15 02:47

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.13%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る