配列へランダムに数字を入れたいです。

Question

###前提・実現したいこと
javaで156個の配列にランダムでそれぞれ1~13の数字を同じ数だけ入れたいです。
マークが関係なくジョーカーが入っていないトランプを3組混ぜるイメージです。
mainのクラスで実行してみたら、無限ループしてしまい、試行錯誤の末、原因は被らないように判定するfor文せいで14個目以降、無限ループしてしまうことに気付きました。
が、これをどう直したらいいのかわからず躓いてしまいました。
初心者ゆえ読みづらいソースコードになっているかもしれませんがアドバイスお願いいたします。

###発生している問題・エラーメッセージ
無限ループ

###該当のソースコード

class Gamesystem{
	Random r = new Random();
	private int[] card = new int[156];
	public void setcard(){
		int i = 0;
		int count = 13;
		for( ; count < card.length ; count += 13)
			for( ; i < count ; i++){
				boolean flag = false;
				card[i] = r.nextInt(13) + 1;
				do {
					flag = false;
					for (int j = i - 1; j >= 0; j--) {
						if (card[i] == card[j]) {
							flag = true;
							card[i] = (r.nextInt(13)+1);
						}
					}
				} while (flag == true);
			}
	}

Answer 1

通常このような「決まった内容のものをランダムに並べる」ものは、「内容を予め入れて置いてシャッフルする」ほうが簡単で確実です。

int[] cards = new int[156]
for (int i = 0; i < cards.length; i++) {
    // 配列に1~13の各数を12ずつ格納する
    cards[i] = i / 12 + 1;
}

// シャッフル(156枚のカードの中から無作為に選んで順番に並べていくイメージ)
for (int i = cards.length - 1; i > 0; i--) {
    int index = (int)(Math.random() * (i + 1));
    if (index == i) continue;
    int temp = cards[i];
    cards[i] = cards[index];
    cards[index] = temp;
}

Comments

[mckkon]

解決しました。そういうやり方があるのですね！回答ありがとうございました！

Answer 2

直接的な回答ではないのですが、
Collections.#shuffle()メソッドを使う方法があります。

        List<Integer> cards = new ArrayList<>();

        for (int i = 0; i < 156; i++) {
            cards.add(i % 13 + 1);
        }

        System.out.println("シャッフル前");
        System.out.println(cards);

        // シャッフル
        Collections.shuffle(cards);

        System.out.println("シャッフル後");
        System.out.println(cards);

Comments

[swordone]

ArrayListに対するtoStringも配列化してArrays.toStringも同じ形式の文字列になるので
System.out.println(cards);
でいいと思いますがね。

[java-beginner]

おっしゃる通りでした。ご指摘ありがとうございます。
コメントを修正しました。