HashSetと乱数を生成し数を表示させたいです。

Question

以下、内容です
1から10までの乱数を10回発生させ、その結果を表示し、そのあとに、そこで出力された文字の種類を表示するプログラムを作りたいです。数値はHashSetに登録し、１つの数が複数出現しても、一つとしてカウントするものとします。

以下が私が書いたコードです。

public static void main(String[] args) {
		HashSet<String> hs = new HashSet<String>();
		int a = (int) (Math.random() * 10) + 1;
		// ハッシュセットにデータを追加
		hs.add("1");
		hs.add("2");
		hs.add("3");
		hs.add("4");
		hs.add("5");
		hs.add("6");
		hs.add("7");
		hs.add("8");
		hs.add("9");
		hs.add("10");
		// 追加した成分をすべて表示
		System.out.println("乱数:");
		System.out.println("出現した数:");
		for (String s : hs) {
			System.out.print(a);
			System.out.print(s);
		}

	}

}

乱数: 1 2 1 10 8 7 5 4 2 9
出現した数: 1 2 4 5 8 7 9 10
上記のようになるのですがなりません。
修正箇所をご指摘ご鞭撻賜りますよう、よろしくお願いします。

Comments

[m.ts10806]

現在のコードではそのような出方にならないので調整された方が良いかと思いますし、そのような数字の感じにはならないと思いますが・・

[m.ts10806]

また、「出力された文字の種類」とは何でしょうか？

[ssssa]

わかりました。

Answer 1

HashSetは重複する値が入らないので、

Set<Integer> result = IntStream.range(0, count).mapToObj(i -> (int)(Math.random() *10) +1).collect(Collectors.toSet());

System.out.println(result);

だけで事足りるでしょう。

Comments

[swordone]

みんなrangeからmapToObjでやるんだけど、なんでgenerateからlimitを誰も出さないのだろう

あと、「乱数」に相当する出力ができない

Answer 2

import java.util.ArrayList;
import java.util.HashSet;

public class memo_qa149091 {

    public static void main(String[] args) {
        HashSet<Integer> rndSet = new HashSet<Integer>();
         ArrayList<Integer> rndList = new ArrayList<Integer>();
        int maxCount = 10;//回数
        for(int i=0;i<maxCount;i++) {
            int rndNum = (int) (Math.random() * 10) + 1;
            // データを追加
            rndList.add(rndNum);//履歴
            rndSet.add(rndNum);//種類
        }
        // すべて表示
        System.out.print("乱数:");
        for (Integer i : rndList) {
            System.out.printf("%s ", i);
        }
        System.out.println();

        System.out.print("種類:");
        for (Integer i : rndSet) {
            System.out.printf("%s ", i);
        }
        System.out.println();
    }
}

//乱数:9 3 7 7 6 3 10 3 6 1 
//種類:1 3 6 7 9 10 

Comments

[ssssa]

丁寧でわかりやすいコードありがとうございます。
理解が深まりました。

[ssssa]

ありがとうございます。理解が深まりました。

Answer 3

後半を以下のようにするとわかりやすくなります。

        String ra = "";
        String rs = "";
        for (String s : hs) {
            ra +=a+"";
            rs +=s+"";
        }
        // 追加した成分をすべて表示
        System.out.println("乱数:"+ra);
        System.out.println("出現した数:"+rs);

いずれにしても提示のような形にはなりませんよ。

結果：

乱数:7777777777
出現した数:12345678910

「乱数」に表示されている数字がランダム関数で取得された数字になるのみです。

Comments

[m.ts10806]

あ、sはstringだから+""はなくてもよかったですね。ちょっとした癖です。

Answer 4

1. 出した乱数を記録するものがない、というか乱数をどこでも記録していない
2. 肝心のHashSetに1から10まで最初からすべて保存しているせいで、

何回やっても出現した数として1から10まですべて出現してしまう
0. 「乱数：」の後で数字を出力していない