ランダムで重複しない配列

Question

###前提・実現したいこと
Java初心者です。
javaでヒットアンドブローを作る過程で
ランダムで重複しない配列を作る必要があり、
以前こちらで見たコードを使おうと思ったのですが、
重複してしまうことがあり悩んでいます。

・Math.random()を使うこと
・乱数は一文字ずつ配列に格納すること
を条件として提示されています。

list等は使わずにコードを書き換えるとしたらどうしたら良いでしょうか？
どうか皆さまのお力添えをお願いします。

###該当のソースコード
int[] ans = new int[4];
for (int i = 0; i < ans.length; i++) {
while (true) {
ans[i] = (int) (Math.random() * 10);
for (int j = 0; j < i; j++) {
if (ans[i] == ans[j])
continue;
}
System.out.print(ans[i]);
break;
}
}

Comments

[koko_u]

なぜそんな縛りプレイをしているのですか？

[sava]

いまいちlistの使い方がわかっていないというのが理由です…。

Answer 1

何でもいいなら、Collections.shuffle() を使うのが簡単なのでは？

Comments

[swordone]

List使っちゃいけないならそれも使えない…

[koko_u]

使い方がわからないだけなら、覚えればよいと思ってな...

Answer 2

こんな感じでしょうか？

int[] ans = java.util.stream.IntStream
     .generate(() -> (int)(Math.random() * 10))
     .distinct().limit(4).toArray();

Answer 3

ヒント:continueの行き先は？

まずはコードをマークダウンしましょう。

int[] ans = new int[4];         
for (int i = 0; i < ans.length; i++) { 
    while (true) { 
        ans[i] = (int) (Math.random() * 10); 
        for (int j = 0; j < i; j++) { 
            if (ans[i] == ans[j]) 
                continue; // このcontinueが実行されたら次にどこに行く？
        } 
        System.out.print(ans[i]); 
        break; 
    } 
}

###答え
continueはその地点から見て**最も内側のループ文(for,while)**に戻り、次のループに入ります。
つまり、2行前のfor (int j = 0; j < i; j++)に戻ります。
そしてこれはifに入ろうが入るまいが戻るので、実質このif文とcontinue文は無くても同じです。

このコードでやりたいのは、数字がダブったら数字の設定をやり直す、つまりwhile(true)に戻りたいのですよね？
なので、
0. while(true)にラベルを付けて、ラベル付きcontinueにする(あまりお勧めされない)
0. 数字が重複したかどうかのフラグを用意し、whileの継続条件をそのフラグの状態で判定する

のいずれかを取る必要があります。

また、このような重複を避けて選びたい場合、選ぶ候補を配列に用意してシャッフルし、先頭からいくつか(この場合4つ)選択するのが早くて確実です。
配列のシャッフルは意外に簡単に実装できます。「シャッフル　アルゴリズム」で検索してみてください。

Comments

[sava]

breakせずに再度forを回す…ということだと思っていたのですがこの時点で間違っているのでしょうか…。
お手数でなければご説明頂けると幸いです。