IEnumerableのforeach文でなぜNullReferenceExceptionエラーが表示されるのか

Question

プログラミング初心者です。unityとC#を使ってブラックジャックゲームを作るサイトを参考に開発しているのですが、サイトと全く同じコードでもなぜかDeck.CSのforeach文でNull？エラーが出てしまいます。

 foreach (int i in cards)//cardsの中の要素それぞれについて
        {
            yield return i;
        //要素を戻り値に返します
        }

unityのバージョンは「2020.3.23f1」です。必要な情報が載せられているかわかりませんが、どなたかなぜエラーがでているのか教えてくださるとありがたいです。

NullReferenceException: Object reference not set to an instance of an object
Deck+<GetCards>d__1.MoveNext () (at Assets/Scripts/Deck.cs:14)
DeckView.ShowCards () (at Assets/Scripts/DeckView.cs:24)
DeckView.Start () (at Assets/Scripts/DeckView.cs:17)

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

public class Deck : MonoBehaviour
{

    List<int> cards; 



    public IEnumerable<int> GetCards() //戻り値に列挙可能なリストを持つメソッド
    {
        foreach (int i in cards)//cardsの中の要素それぞれについて
        {
            yield return i;
        //要素を戻り値に返します
        }
    }

    

    public void Shuffle() // デッキをシャッフルする
    {

        if (cards == null)
        {
            cards = new List<int>();
        }
        else
        {
            cards.Clear();
        }

        for(int i = 0; i < 75; i++)
        {
            cards.Add(i);
        }
        

        
        // 整数 n の初期値はデッキの枚数
        int n = cards.Count;
        
        // nが1より小さくなるまで繰り返す
        while (n > 1)
        {
            n--;
        
            // kは 0 ～ n+1 の間のランダムな値
            int k = UnityEngine.Random.Range(0, n + 1);
        
            // k番目のカードをtempに代入
            int temp = cards[k];
            cards[k] = cards[n];
            cards[n] = temp;
        }

    }

    // Start is called before the first frame update
    void Start()
    {
        Shuffle();
    }

}

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

[RequireComponent(typeof(Deck))]
public class DeckView : MonoBehaviour
{
    Deck deck;

    public Vector3 start;
    public float cardOffset;
    public GameObject cardPrefab;

    void Start()
    {
        deck = GetComponent<Deck>();//Deck.csの取得
        ShowCards();//下記メソッドの実行
    }

    void ShowCards()//メソッド本体
    {
        int cardCount = 0;//内部で使う値cardCountの宣言

        foreach (int i in deck.GetCards())
        {
            float co = cardOffset * cardCount; //オフセット幅の計算

            GameObject cardCopy = (GameObject)Instantiate(cardPrefab);
           //カードプレファブのコピー
            Vector3 temp = start + new Vector3(co, 0f);
           //tempというオフセットした位置の計算
            cardCopy.transform.position = temp;
            //現在の位置にtempを代入
            
            CardModel cardModel = cardCopy.GetComponent<CardModel>();
            // //コピーしたカードプレファブのCardModelクラスを取得
            cardModel.cardIndex = i;
            // //インデックスにiを代入
            cardModel.ToggleFace(true);
            // //表面をレンダー（カードゲームを作りたい第2回目で作成したスクリプトを使用）
            cardCount++;//cardCountをインクリメント
        }
    }

}

補足：Cardsリストのシャッフル処理はunity上のdebug表示でインスペクターをみたところうまくいっており、問題はforeachの時点でcardsの中身がNullになっていることだとわかりました。しかし、なぜforeachの時点でNullなのか、調べても自分では理解できませんでした。

Comments

[退会済みユーザー]

> Deck.CSの14行目

そういう言い方は NG です。行番号が振ってあるわけではないので。

[dyer]

確かに仰る通りです。
今後該当箇所かわかるように質問します。
質問の仕方の訂正、ありがとうございます。

[退会済みユーザー]

質問文は編集できますので今やりましょう。Teratail は「質問と回答を通してお互いに知識や情報を交換・共有する場所」ということですので、後から検索などでここにたどり着いた人などにも分かりやすく書くようお願いします。

Answer 1

Shuffle してはじめて cards が設定されるのに、その前に GetCards メソッドを呼び出しているからでは？

Comments

[dyer]

ご教授ありがとうございます。
その可能性が高そうですが、このコードだとShuffle()はGetCards()より前にDeck.cs上で行われることになると思ったのですが、この認識が間違っているのでしょうか？
void Start()
{
deck = GetComponent<Deck>();//Deck.csの取得
ShowCards();//下記メソッドの実行
}

[退会済みユーザー]

デバッガを使ってどういう順で実行されているか調べてはいかが？

[dyer]

デバッガなるものを調べて使ってみます。
ありがとうございました

[退会済みユーザー]

Deck クラスの Start ⇒ Shuffle ⇒ GetCards メソッドの順で実行されるはずだから GetCards が実行される時点では cards は null にはならないはずと思っているということですか？　そうであれば、思っている通りになっているかは各メソッドの中の最初の一行目にブレークポイントを設定して Visual Studio からデバッグ実行すれば分かります。まずそれをやりましょう。

[dyer]

visual studioをインストールしてデバッグのやり方検索通りにしてみると正しくできたのかわかりませんが、Deckview.csのStart処理から始まり「ShowCard()→ DeckView.csのforeach文→Deck.csのforeach文」とforeachの順番が自分の想像と真逆になっていました。

[dyer]

最終的にスクリプトのIDを変更してDeck.csがDeckView.csより先に動くように設定したらうまく行きました！
デバッグという手法を教えていただきありがとうございました！

Answer 2

修正前: List<int> cards;
修正後: List<int> cards = new List<int>();

としたらいいんじゃないでしょうか。

Comments

[dyer]

ご教授ありがとうございます。
確かにエラーはうまく消えてくれましたが、Debug.Logをforeachの手前に入れて確認したところ、GetCards()がスルーされており、意図した挙動にはなりませんでした。
cardsに値が設定されていなかったのがエラーの原因だとわかりましたが、なぜShuffle()内で設定されているのにエラーになるのでしょうか。
if (cards == null)
{
cards = new List<int>();
}
else
{
cards.Clear();
}

for(int i = 0; i < 75; i++)
{
cards.Add(i);
}

[退会済みユーザー]

デバッガを使ってどういう順で実行されているか調べてはいかが？　←すみません、書く場所を間違えました。でも、デバッガを使って調べようという話は間違ってないので消さずに置いときます。