１〜１３の数字をシャッフルさせるプログラム。ただし、重複させない。

Question

実現したいこと

１〜１３の数字をシャッフルさせ、重複せずに実行させたい。

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

同じ数字が出てしまいます。

$ Javac Shuffle.java
$ java Shuffle
13 6 5 6 7 13 12 8 6 4 3 3 

該当のソースコード

public class Shuffle {
  public static void main(String[] args) {
    int[] card = {1,2,3,4,5,6,7,8,9,10,11,12,13};

    for (int i=1; i<card.length; i=i+1) {
      int x = (int)(Math.random() *13) + 1;
      System.out.print(x+ " ");
    }
    System.out.println();
  }
}

試したこと

ループ回数の配列と、乱数で発生した数字の配列を入れ替えてみたり、二重for文にしてみたりした。
書き方がおかしかったのか、同じ数字が出てしまった。

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

PC：Mac
エディタ：Atom

Comments

[amiya]

配列を入れ換えてやった場合のコードを書くと問題点がわかるかもしれません。

[momon-ga]

java list シャッフルで、検索するといろいろ出てきますが、配列を使うことが条件でしょうか？

[hona_tan]

きちんと確認したらできました！
ご指導ありがとうございました！

Answer 1

フィッシャー・イェーツのシャッフルの改良版を利用。
出力のコードは入れていません。

public class Shuffle {
  public static void main(String[] args) {
    int[] card = {1,2,3,4,5,6,7,8,9,10,11,12,13};

    for (int i = card.length; i > 1; i--) {
      int x = (int)(Math.random() * i);
      int t = card[x];
      card[x] = card[i - 1];
      card[i - 1] = t;
    }
  }
}

Answer 2

シャッフルするには、たとえば「用意済みの2つの値を交換する」処理の繰り返しを書けばいいので、

public class Shuffle {
    public static void main(String[] args) {
        int[] card = { 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13 };

        for (int i = card.length - 1; i >= 1; i--) {
            int j = (int) (Math.random() * i);
            int tmp = card[j];
            card[j] = card[i];
            card[i] = tmp;
        }

        for (int i = 0; i < card.length; i++) {
            System.out.print(card[i] + " ");
        }
        System.out.println();
    }
}

C>java Shuffle
2 11 10 5 7 13 6 4 12 9 3 8 1

C>java Shuffle
11 6 1 9 8 3 5 12 7 13 10 2 4

Answer 3

for ( int i=1; i<card.length; i=i+1) {
      card[i] = i;
}

↑この部分を追加したらできました！

Comments

[momon-ga]

上記の回答は不完全な連番を設定するだけで、シャッフルが実現できると思えないのですが・・・