それまで入力した整数を逆から表示するプログラムを作りたい。

Question

public class Probex {

	/**
	 * @param args
	 */
	public static void main(String[] args) {
		List<Integer> list = new ArrayList<Integer>();
		while (true) {
			int a = (int) (Math.random() * 10) + 1;
			System.out.println("0から10の値を出力:" + a);
			if (a == 0) {
				break;
			}
			list.add(a);
		}
		for (int i = list.size() - 1; i >= 0; i--) {
			System.out.print(" ");
			System.out.print(list.get(i));
		}
	}
}

0～10の値を出力:2
0～10の値を出力:9
0～10の値を出力:1
0～10の値を出力:7
0～10の値を出力:4
0～10の値を出力:3
0～10の値を出力:1
0～10の値を出力:0 ← 0が出たら、出力を終える

1 3 4 7 1 9 2としたいのですが無限ループになってしまいわかりません。
ご教示いただきたいです。

Answer 1

int a = (int) (Math.random() * 10) + 1;

これは1から10の整数値となり、決して0になりません。
したがって

            if (a == 0) {
                break;
            }

この条件は永遠に満たされることなく、無限ループとなります。

Comments

[ssssa]

ご説明ありがとうございます。

Answer 2

int a = (int) (Math.random() * 10) + 1;
System.out.println("0から10の値を出力:" + a);

0から10の乱数を得たいなら、1を加えてはいけませんよね。

int a = (int) (Math.random() * 11);
System.out.println("0から10の値を出力:" + a);

また、目的を考えると、 Random#nextInt(int) の方が扱いやすいと思います。