配列に乱数4桁を格納して画面出力

Question

前提・実現したいこと

Javaの課題で乱数4桁を重複なしで画面出力したいのですが
上手くできません

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

エラーメッセージ

該当のソースコード

System.out.println("ゲームを始めます");
        int answer[] = new int[4];
        Boolean isDuplicate=false;
        do{
            answer[0]=(int)(Math.random()*9)+1;
            for(int i = 1;i < answer.length;i++){
                answer[1] = (int)(Math.random() * 10) ;
            }
            isDuplicate=false;
            for (int j=0;j < answer.length-1;j++){
               for (int k=j+1; k < answer.length;k++){
                  if (answer[j]==answer[k]){
                     isDuplicate=true;
                  }
               }      
            }
         } while (isDuplicate);
           System.out.println();

Comments

[nskydiving]

コードタグで囲ってもらっていますが、閉じの```が抜けてしまっているようです。
質問欄を「編集」ボタンから修正してください。

[BeatStar]

>> うまくいきません

ではなくて、「どのようにうまくいかないのか」を明記してください。
たまに「質問者さんが頭の中で計算ミスしている」場合もあります。
例えば 7 × 4 は 28 なのに、21 としてしまっていて…とかですね。
あるいはコード的には正しくても見当違いのことをしている( 例えば「ユーザ入力を受け取っている」つもりで「標準出力」しているとかです ) 場合もあります。

「本来はこうあるべきだが、こうなる」みたいなものでもいいので書いてください。

[yureighost]

要望が少し分かりづらいので確認ですが、
処理を見たところでは４桁を形成する数字に同じ数字があってはならない。
例(OK:1234、NG:1224)
ってことでいいですかね？

とりあえず処理的には3桁目(answer[2])と4桁目(answer[3])に値を設定しないまま、
処理を回しているため、初期値の0で一致して
isDuplicate=true;
の文に絶対入ってしまうため無限ループとなっています。

[oisii]

したい処理はそれです！無限ループを抜けるにはどのようにしたらいいのでしょうか？ここら辺の理解が乏しくすいません。

Answer 1

int配列で前後比較する発想は悪くないと思います。
ただ何度もループさせるより一桁ずつ確定させてしまった方が簡単ですね。

System.out.println("ゲームを始めます");
int answer[] = new int[4];
answer[0]=(int)(Math.random()*9)+1;
int i = 1;
// 1から3まで繰り返す(answer[1～3]まで処理を行う)
while (i < answer.length) {
    answer[i] = (int)(Math.random() * 10) ;
    int j = i - 1;
    while (j >= 0) {
        // 前の桁の数字と同じ場合
        if (answer[i] == answer[j]){
        	break;
        }
        j--;
    }
    if (j == -1) {
    	i++;
    }
}
// int配列を一文字ずつ出力
Arrays.stream(answer)
    .forEach(System.out::print);

Comments

[oisii]

出来ました！ありがとうございます。

[kazuma-s]

できるわけありません。
一つ前と異なるだけなので、7202 や 9098 などが生成されます。

[yureighost]

申し訳ない。
その通りなので修正入れてます。

[kazuma-s]

その修正も変です。
i が 2 のとき、answer[2] を answer[0] と answer[1] と比較して、どちらとも異なっていたら
i は 2増えて 4 になり、answer[3] に値が設定されません。

Answer 2

class Game {
	public static void main(String[] args) {
        int answer[] = new int[4];
		int num[] = { 0, 1, 2, 3, 4, 5, 6, 7, 8, 9 };
		int n = num.length;
		for (int i = 0; i < answer.length; i++) {
			int r = (int)(Math.random() * n);
			answer[i] = num[r];
			num[r] = num[--n];
		}
		for (int i = 0; i < answer.length; i++)
			 System.out.print(answer[i]);	
		System.out.println();
	}
}

先頭に 0 が来ないようにするにはどうしたらよいと思いますか？

追記
先頭に 0 が来ないようにすると、

class Game {
	public static void main(String[] args) {
		int answer[] = new int[4];
		int num[] = { 0, 1, 2, 3, 4, 5, 6, 7, 8, 9 };
		int n = num.length - 1;

		int r = (int)(Math.random() * n) + 1;
		answer[0] = num[r];
		num[r] = num[n];

		for (int i = 1; i < answer.length; i++) {
			r = (int)(Math.random() * n);
			answer[i] = num[r];
			num[r] = num[--n];
		}
		for (int i = 0; i < answer.length; i++)
			 System.out.print(answer[i]);	
		System.out.println();
	}
}