java　解釈

javaでコードを見てもどうしてこんなコードになっているかわからないものがあるので解釈を教えて下さい。

int[] answer = new int[3];//答えが入る
//ランダムな答えを作成
//ただし同じ数字が無いようにする
for (int i = 0; i < answer.length; i++) {
//自分より前の要素にかぶる奴がないか確かめる
//あったらもう1回random
boolean flag = false;
answer[i] = (int) (Math.random() * 6 + 1);
do {
flag = false;
for (int j = i - 1; j >= 0; j--) {
if (answer[i] == answer[j]) {
flag = true;
answer[i] = (int) (Math.random() * 6 + 1);
}
}

        } while (flag == true);

これでどのようにしてかぶっている要素を見つけられるのでしょうか？？

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

回答2件

do文内の以下のif文で、かぶっているかを判定しているようです。

if (answer[i] == answer[j]) {

このときflagにtrueを代入し、再度乱数を取得しています。
do文はflagが'false'のときに抜けるようになっているので、doを抜けると要素はかぶっていない状態です。

ご期待の回答でしょうか。

Setを使って、要素数が3になるまで、乱数をセットする方法はいかがでしょうか。

lang
1Set<Integer> answerSet = new HashSet<Integer>();
2do {
3    answerSet.put((int) (Math.random() * 6 + 1));
4} while (answerSet.size() < 3);
5int[] answer = ArrayUtils.toPrimitive(answerSet.toArray(new Integer[0]));

すみません、この方法ですと、Integerのハッシュなので、取得した配列は値の昇順になってしまいます。
HashSetではなく、LinkedHashSetを使うことで解決すると思います。

投稿2015/01/10 00:41

jcs502ulf

総合スコア307

code_red_sky928

2015/01/10 03:10

迅速な回答有難うございました！！理解出来ました

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

ベストアンサー

まず、answerの初期状態は、[0, 0, 0]です。
ランダム値はサイコロと同じ1～6のどれかになります。

外側のforループは、for (int i = 0; i < answer.length; i++)
なので、最初はanswer[0]にランダム値を入れます。
5が入ったとすると、answerは[5, 0, 0]になります。
内側のループはfor (int j = i - 1; j >= 0; j--)ですが、
i = 0の時は最初の時点でj >= 0を満たさないので、
forループに入らずに進み、flag=falseのままなのでdo whileも抜けて
次に進みます。

次は、i = 1になるのでanswer[1]にランダム値を入れます。
4が入ったとすると、answerは[5, 4, 0]になります。
内側のforループは、i = 1の時はfor (int j = 1 - 1; j >= 0; j--)
になりますので、1回だけ、つまり
自分より前の要素answer[0]とだけ同じかどうか確かめます。
この場合は5と4ですから、同じでないので、flag=falseのままになり、
do whileを抜けて次に進みます。

★目印

次は、i = 2になるのでanswer[2]にランダム値を入れます。
5が入ったとすると、answerは[5, 4, 5]になります。
内側のforループは、i = 2の時はfor (int j = 2 - 1; j >= 0; j--)
になりますので、自分と自分より前の要素answer[1]とanswer[0]と順に確かめます。
この場合は、answer[2] == answer[1]は等しくありませんが、
answer[2] == answer[0]は等しいので、if文の中に入ります。
if文の中では、flag=trueにして、answer[2]にランダム値を入れます。
2が入ったとすると、answerは[5, 4, 2]になります。
flag=trueになったので、doの次（「★目印」）に戻ってからもう一度同じことをします。

i = 3になると、外側のfor文の条件を満たさなくなりますので、ループが終了し
すべての計算が終了します。

長くなりましたが、このようにして、値がかぶらないようにしています。

ただ、正直あまり良いコードでは無いですね。
randomするところが2箇所にあったりループが複雑すぎる気がします。
例：

lang
1loop1:
2for (int i = 0; i < answer.length;) {
3    answer[i] = (int)(Math.random() * 6 + 1);
4    for (int j = i - 1; j >= 0; j--) {
5        if (answer[i] == answer[j]) {
6            continue loop1; // かぶっていたら外側のループの最初に戻る
7        }
8    }
9    i++; // かぶっていないときだけ次に進む
10}