c# リストの配列が分からぬ

Question

下のような感じでリスト作ってるんですがググってもうまい具合にシャッフルの機能を実装できず困っています。
助言お願いします。

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class cardGenerator : MonoBehaviour
{
public GameObject cardPrefab;//インスタンス化のための
public GameObject hand;//親にする要素指定

List<cardData> cardDataList = new List<cardData>() {

    new cardData(10,"ヤドラン","水鉄砲"),
    new cardData(1,"か","日"),
    new cardData(1,"1","1"),
    new cardData(1,"2","3"),




};
void Start()
{
    {
        for (int i = 0; i < cardDataList.Count; i++)
        {
            GameObject cardObj = Instantiate(cardPrefab);//生産
            cardObj.transform.SetParent(hand.transform);//親指定
            Card card = cardObj.GetComponent<Card>();//カードクラスのカード変数にスクリプト代入
            Debug.Log(card == null);
            card.Load(cardDataList[i]);
        }
    }
}
private void Update()
{
    if (Input.GetKeyDown(KeyCode.LeftShift))
    {
        GameObject cardObj = Instantiate(cardPrefab);//生産
        cardObj.transform.SetParent(hand.transform);//親指定
        Card card = cardObj.GetComponent<Card>();//カードクラスのカード変数にスクリプト代入
        Debug.Log(card == null);
        card.Load(cardDataList[0]);
        cardDataList.RemoveRange( 0, 1);

        if (Input.GetKeyDown(KeyCode.Space))
        {
            Shuffle(cardDataList);
        }
    }
}


//リストの要素をシャッフルする (Fisher-Yates shuffle)
public static void Shuffle<T>(List<T> list)
{
    for (int i = list.Count - 1; i > 0; i--)
    {
        int j = Random.Range(0, i + 1);  //[0]～[i]
        Swap(list, i, j);
    }
}

public static void Swap<T>(List<T> list, int i, int j)
{
    T tmp = list[i];
    list[i] = list[j];
    list[j] = tmp;
}

}

Comments

[papinianus]

Shuffle(とSwap)だけ確認しましたが、シャッフルのところはそれなりにできているように思います。具体的にどういう結果を求めていて、現状どうなるかを書いていただけませんでしょうか？

[ryo76588]

シャッフルとスワップのメソッド適当にコピペして持ってきただけで使い方が分からず適当にリストの引数を渡してみてみてもうまく動作することができず、というかまずリストをどのようにメソッドの引数として渡せばいいのかわからず困っておりこのプログラミングではどのように書けば動くようになるのか教えてほしいといったところです。

Answer 1

papinianusさん、pepperleafさんがチェックしていただいた通り、シャッフル部分はちゃんと機能しているようでした。あとは、シャッフル機能が正しく実行されているかご確認いただく必要がありそうですね。
下記のようにしてみると、コンソールに意図通りのタイミングで「Shuffle」と表示されるでしょうか?

    //リストの要素をシャッフルする (Fisher-Yates shuffle)
    public static void Shuffle<T>(List<T> list)
    {
        Debug.Log("Shuffle");
        for (int i = list.Count - 1; i > 0; i--)
        {
            int j = Random.Range(0, i + 1);  //[0]～[i]
            Swap(list, i, j);
        }
    }

私の試したところですと、シフトキーとスペースバーを同じタイミングで同時押ししないとシャッフルされませんでした。GetKeyDownはキーを離した状態から押し下げた状態に変わった最初の1フレームしかtrueになりませんので、このような挙動になったものと思われます。
たとえば下記のようにスペースバーの方をGetKeyに変えれば...

    private void Update()
    {
        if (Input.GetKeyDown(KeyCode.LeftShift))
        {
            GameObject cardObj = Instantiate(cardPrefab);//生産
            cardObj.transform.SetParent(hand.transform);//親指定
            Card card = cardObj.GetComponent<Card>();//カードクラスのカード変数にスクリプト代入
            Debug.Log(card == null);
            card.Load(cardDataList[0]);
            cardDataList.RemoveRange(0, 1);

            // GetKeyDownをGetKeyに変更
            if (Input.GetKey(KeyCode.Space))
            {
                Shuffle(cardDataList);
            }
        }
    }

まずスペースバーを押し下げておいて、その状態でシフトキーを押すとシャッフルが起こるようになりました。

---

※前回のご質問に引き続き「コードをタグでくくって...」とご指摘がありました。コードを貼り付けた部分の上下の行にバッククォート3つを入れてコード部分がそれらに挟まれるようにしてやると、投稿後の表示が読みやすくなってありがたいです（これがないと行頭の空白が削除されてしまい、コードの字下げがなくなって読みづらいのです...）。

Answer 2

Unity 無いので、一部、入れ替えて、シャッフルの部分のみテストしましたが、それらしく動いているようですが、どこが問題なのでしょうか?

public static void Shuffle<T>(List<T> list)
{
	var rnd = new Random();	// System.Random
	for (int i = list.Count - 1; i > 0; i--) {
		//int j = Random.Range(0, i + 1);  //[0]～[i]
		int j = rnd.Next(i + 1);	//[0]～[i]
		Swap(list, i, j);
	}
}

---

コードは、読みずらいので、コードタグで括ってください。