JAVA ２次元配列 shuffle

Question

前提・実現したいこと

JAVAを用いて、２次元配列の要素をシャッフルしたいです。
以下のコードで行ってみた結果、
隣同士の要素がセットで動いてランダムにシャッフルできていません。
どのようにすればよいか教えていただけますでしょうか。

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

下が実行結果になります。
3 3 4 4 
5 5 6 6
A A 2 2
自分が考えているのは、
3 6 2 A
5 4 A 6
2 3 5 4
のようにしたいと考えています。

該当のソースコード

import java.util.*;


public class Main {
    public static void main(String[] args) {
        String[][] type = CardType();
    
        for(int i=0; i<3; i++){
            for(int j=0; j<4; j++){
                System.out.print(type[i][j]);
                System.out.print(" ");
            }
            System.out.println();
        }
    }

    static String[][] CardType(){
        String[][] type = {{"A","A","2","2"},{"3","3","4","4"},{"5","5","6","6"}};
        List<String[]> list = Arrays.asList(type);
        Collections.shuffle(list);
        return type;
    }
}

Comments

[dodox86]

提示のコードが違うのではないでしょうか。CardType()メソッドでは typeをreturn しているので、せっかくのshuffle実行した結果が反映されず、提示の実行結果にはなりません。

[seele]

解答ありがとうございます。
return listに変えてみたのですが、
エラー: 不適合な型: List<String[]>をString[][]に変換できません:
return list;
このようなエラーが出てしまいました。

[dodox86]

いえ、そういうことではなく、とりあえず今現在、提示の実行結果になるJavaのソースコードを正しく貼りましょう、ということです。

[seele]

いえ、このコードであっています。

[dodox86]

ん？　あれ、OKなのかな？

[ozwk]

OKですね

[dodox86]

そうですね。OKですね。これは私の見誤りです。大変失礼しました。

Answer 1

隣同士の要素がセットで動いてランダムにシャッフルできていません。

いいえ、列ごとに動いています。({"A","A","2","2"}を一つのまとまりとしてシャッフルをしている)

どのようにすればよいか

もととなる配列は1次元で生成してシャッフルし、先頭から4つずつとって2次元配列にすればよいです。

Comments

[seele]

すみません。列ごと動いているのは確認不足でした。
なるほどですね、１次元で作成してから２次元に格納すればいいんですね。

思いつきませんでした。解答いただきありがとうございます。
やってみます。

[dodox86]

> もととなる配列は1次元で生成してシャッフルし、

こちらの解法が良いですね。試しに（安易に）行ごとにシャッフルしても、ランダム性（？）に欠けました。

static String[][] CardType(){
String[][] type = {{"A","A","2","2"},{"3","3","4","4"},{"5","5","6","6"}};
List<String[]> list = Arrays.asList(type);
for (int i = 0; i < list.size(); i++) {
Collections.shuffle(Arrays.asList(list.get(i)));
}
Collections.shuffle(list);
return type;
}

で試した例、
6 5 5 6
2 A A 2
3 4 3 4
とか、

A 2 A 2
4 3 4 3
6 6 5 5

などで、とてもダメです。