[Java初心者]1~6の数字を重複しないようにランダムに3つ表示させる方法

題名のとおり1~6の数字の中から重複しないように3つ数字を取り出すという問題です。
自分が考えた回答は以下の通りなのですが、回答とかなり違うためあっているのかこれでいいのかもあまり判断がつきません。ここはこうしたほうがよいというアドバイスご指摘をお願いします。

Java
1//答えを入れる配列
2 int[] answer = new int[3];
3        //1~6までの数字を格納
4        ArrayList<Integer> list = new ArrayList<Integer>();
5        for(int i = 1;i <= 6;i++){
6        	list.add(i);
7        }
8        //listをシャッフル
9        Collections.shuffle(list);
10
11        //３つ取り出しanswer配列に入れる
12        for(int i = 0;i < answer.length; i++){
13        	answer[i]=list.get(i);
14        	System.out.println(answer[i]);
15        }

ちなみに課題の回答は以下の通りです。

Java
1//ランダムな答えを作成。
2        //ただし、仕様通り、同じ数字がないようにする。
3        for (int i = 0; i < answer.length; i++) {
4            //自分より前の要素にかぶるやつがないか確かめる。
5            //あったらもう1回random
6            boolean flag = false;
7            answer[i] = (int) (Math.random() * 6 + 1);
8            do {
9                flag = false;
10                for (int j = i - 1; j >= 0; j--) {
11                    if (answer[i] == answer[j]) {
12                        flag = true;
13                        answer[i] = (int) (Math.random() * 6 + 1);
14                    }
15                }
16
17            } while (flag == true);
18        }

行動規範の内容に同意します

回答2件

ベストアンサー

この要件を満たすのであればあなたの作成したコードで十分だと思います．
むしろ後半の課題の回答ではある不備があります．

java
1            do {
2                flag = false;
3                for (int j = i - 1; j >= 0; j--) {
4                    if (answer[i] == answer[j]) {
5                        flag = true;
6                        answer[i] = (int) (Math.random() * 6 + 1);
7                    }
8                }
9
10            } while (flag == true);

answer[i]とanswer[j]が同じ数値だった場合answer[i]を振りなおしていますが，
これが違う数値になる保証がありません．
そしてそのまま次の数値との比較になってしまい，次に比較するのは一旦whileループを抜けてからになり，無駄が多い構成に見えます．
その点，はじめから6つの数字を入れて配列を作り，並べ替える方法は確実にかぶらない3つの数字を取り出せます．

投稿2015/07/31 14:01