javaでwhile文を使い６が出るまでサイコロを振り続けるプログラムを作成したい。

Question

前提・実現したいこと

javaでjavaでwhile文を使い６が出るまでサイコロを振り続けるプログラムを作成したい。

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

作成してみたはものの、6が出ても終わらない時がある。

該当のソースコード

class Ex1 {
public static void main (String[] args) {

	while( 1 + (int)(Math.random() * 6.0) < 6 ){
		int diceNumber = 1 + (int)(Math.random() * 6.0);
		System.out.println( diceNumber );
		
	}
	
	System.out.println( 6 );
	System.out.println( "「6が出たので終了します」" );
	
}

}

試したこと

while文の繰り返し条件と繰り返し処理にある1 + (int)(Math.random() * 6.0を同値とするために、while文外で変数に代入してみた。しかし今度は無限ループに入ってしまった。

補足情報（FW/ツールのバージョンなど）

saiなどまだ習っていないことは使わないでwhile文で作成したいです。
windows /java se8 221

Answer 1

while( 1 + (int)(Math.random() * 6.0) < 6 ){
    int diceNumber = 1 + (int)(Math.random() * 6.0);
    System.out.println( diceNumber );

}

表示する乱数と、判定する乱数が別です。
表示タイミングで6だったとしても、whileの判定時に再び乱数を発生させるので、そこで6にならない限りループを抜けません。

Comments

[nabeko]

ありがとうございました。解決しました。

Answer 2

Math.randomを呼ぶ度に乱数が生成されているからです。

int diceNumber = -1;  // とりあえず -1
while( diceNumber < 6 ){
    diceNumber = 1 + (int)(Math.random() * 6.0);
    System.out.println( diceNumber );
}

do-while文を使うのも良いアイデアです。

---

while文の繰り返し条件と繰り返し処理にある1 + (int)(Math.random() * 6.0を同値とするために、while文外で変数に代入してみた。

方針としては正しいです。
おそらく実装に問題があったのでしょう。ループ内で値を更新し忘れていたり。

コードの書き方について

teratailには、コードを見やすく表示する機能があります。
質問編集画面を開き、コードを選択した状態で<code>ボタンを押して下さい。

Comments

[nabeko]

ありがとうございました。int diceNumber = -1としてしまう発想はありませんでした。
コードの載せ方のアドバイスまでありがとうございます！