javaのランダム数字を集計してその数を表示したい

Question

前提・実現したいこと

ここに質問の内容を詳しく書いてください。
java でランダムに１～６の数字を２０回出現させ、その出現した数分を「+」で表現したい。
例）
1：+++
2：++
3:++++
4:+++
5:+++++
6:+++
■■な機能を実装中に以下のエラーメッセージが発生しました。

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

途中までですが、とりあえず１の数を「+」で表示したいのに
1：1：1
と、表示され「+」の表示できません。
初心者で申し訳ないのですがご教授頂ければ幸いです。
エラーメッセージ

### 該当のソースコード

java
ソースコード
for (int i=0; i<20; i++){
            int dice =rand.nextInt(6) + 1;
       int num1 =0;
        if(dice==1){
    num1++;
           System.out.print("1:");
            for(int g =1;g<num1;g++){
            System.out.print("+");
        	}
    	}
	}
}
}
### 試したこと

ここに問題に対して試したことを記載してください。

### 補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Answer 1

ランダムに１～６の数字を２０回出現させ、その出現した数分を「+」で表現したい。

それぞれの目の頻度を+で表示するんですよね？
1だけならこうですね。（書く場所を入れ替えただけ）

import java.util.Random;

class Main {
    public static void main(String[] args) {
        Random rand = new Random();
        int num1 = 0;
        for (int i = 0; i < 20; i++) {
            int dice = rand.nextInt(6) + 1;
            if (dice == 1) {
                num1++;
            }
        }

        System.out.print("1:");
        for (int g = 0; g < num1; g++) {
            System.out.print("+");
        }
        System.out.println();
    }
}

6個ならnum2、num3とやっていってもいいですが、配列を使うとスッキリ書けます。

Comments

[dodox86]

質問者さんのオリジナルのコードに寄り添ったご回答でしたので、高評価させていただきました。

Answer 2

追記で修正しました：
私の最初の回答は題意を間違えていましたね。20回サイコロを振って、1～6の目についてそれぞれ何回出たかを集計するのが目的の問題でした。修正したものを追記させていただきました。すみません、これは低評価も止む無し、です。

---

当初の回答（※誤ったもの）

途中までですが、とりあえず１の数を「+」で表示したいのに
1：1：1
と、表示され「+」の表示できません。

とりあえず書いても要望通りには動きません。このレベルでしたら初心者とかあまり関係ありません。落ち着いて考えてみましょう。

if (dice == 1) {...}などとしてしまったら、サイコロの出た目が1の場合しか出力しないと思いませんか。まず、Javaから離れて頭の中、必要なら手を使って紙に書いて考えてみましょう。サイコロの出た目、そのままの回数で+を出力すれば良いということが分かると思います。

import java.util.Random;

public class t7 {
    public static void main(String[] args) {
        Random rand = new Random();
        for (int i = 1; i <= 20; i++) {
            // サイコロを振る
            int dice = rand.nextInt(6) + 1;

            // 左端の"n:" を出力
            System.out.print(i + ":");

            // サイコロの出た目分、'+' をループで出力
            for (int j = 0; j < dice; j++) {
                System.out.print("+");
            }

            // 最後に改行
            System.out.println();
        }
    }
}

実行例です。

$ javac t7.java
$ java t7
1:+++++
2:+++
3:+
4:++++
5:+++
6:+++++
7:+++++
8:+
9:++++
10:++++
11:++++
12:++++++
13:+++++
14:+++
15:+
16:+++
17:++++
18:++++++
19:++++++
20:+++++
$

---

修正した回答（題意に沿ったもの）

import java.util.Random;

public class t7b {
    public static void main(String[] args) {
        Random rand = new Random();
        int[] freqs = new int[6 + 1];
        for (int i = 0; i < 20; i++) {
            // サイコロを振る
            int dice = rand.nextInt(6) + 1;
            // 集計
            freqs[dice]++;
        }

        for (int i = 1; i < freqs.length; i++) {

            // 左端の"n:" を出力
            System.out.print(i + ":");

            // 1～6の目のそれぞれの回数分、'+' をループで出力
            for (int j = 0; j < freqs[i]; j++) {
                System.out.print("+");
            }

            // 最後に改行
            System.out.println();
        }
    }
}

再度の実行例です。

$ java t7b
1:++
2:++++
3:++++
4:+++++++
5:++
6:+
$

Comments

[TN8001]

コメントするつもりだったのですが他にも回答がついてしまい、迷った末に回答にしてしまいました。
余計なお気遣いをさせてすいません。

私も全く同じ答えを用意していましたが、サイコロの目と合わすために配列の0番目（freqs[0]）を使わないのがポイントですね。

[dodox86]

＞TN8001さん
> 余計なお気遣いをさせてすいません。
とんでもございません。誤りに気付かせていただいて感謝いたします。どうもありがとうございます。最初に問題を読み、考えるときに少し違和感はあったのでしたが、明らかにする前に投稿してしまいした。配列インデックスについては0から使うか1から始めてシンプルにするか、この問題に限らずいつも少しだけ悩むところですね。

[dodox86]

あれ？ 当初は間違った回答であったののに、BAをいただいてしまって恐縮しています。。。※私的にはTN8001さんのnum1、num2...と配列について言及した回答がベストと思ったものでしたが。

Answer 3

for(int g =1;g<num1;g++){

for(int g =0;g<num1;g++){
ではどでしょ

元のコードでは、
forの動作的に、num1が１のときは、g<num1が成立しないので、ループのナカミが全く実行されません

Answer 4

import java.util.Random;

public class Main {
	public static void main(String[] args) {
		final int LOOP_COUNT = 5;
		final String BARS[] = { ":+", ":++", ":+++", ":++++", ":+++++", ":++++++" };
		Random r = new Random();
		for (int i = 0; i < LOOP_COUNT; i++) {
			int n = r.nextInt(6);
			System.out.print(n + 1);
			System.out.println(BARS[n]);
			/**
			n++;
			String line = "" + n + ":";
			for (int j = 0; j < n; j++) {
				line += "+";
			}
			System.out.println(line);
			**/
		}
	}
}

２つの方法を書いてみました。

• サイコロの目ごとに表示する＋の文字列を配列でつくっておいて、それを表示する。
• サイコロの目の数だけ + をつなげてるようにして、それを表示する。

実行例；

追記：
私も問題を取り違えてました。
頻度を表示するようにつくりなおしました。

import java.util.Random;

public class Main {
	public static void main(String[] args) {
		final int LOOP_COUNT = 100;
		String bars[] = { 
				"1:", "2:", "3:", "4:", "5:", "6:" };
		Random r = new Random();
		for (int i = 0; i < LOOP_COUNT; i++) {
			int n = r.nextInt(6) + 1;
			bars[n - 1] += "*";
		}
		System.out.println(String.join("\n", bars));
	}
}