(Unity)シャッフルしたカードを並べたいのですがNullReferenceExceptionがでてしまいます。

Question

前提・実現したいこと

プログラミング初心者です。
作り方を載せてくださっているサイトのやり方やスクリプトを丸々真似して、Unityで試験的にカードゲームを作っています。52枚のシャッフルしたカードを1枚目から順に横に並べるための操作をしている途中です。スクリプトや手順を全部真似しているつもりなのですが、実行するとNullReferenceExceptionが出てしまいます。

ネットでエラーメッセージについて調べてみたのですが、この場合は何がNullになってしまっているのか、初心者の自分の力だけでは分かりませんでした。

どこに原因があるのか、何がNullになってしまっているのかを教えていただきたいです。

ご回答宜しくお願い致します。

【参考にさせていただいたサイトのURL】https://tomoarch.com/2019/04/10/unityでできるみんなが知りたいカードゲーム作りの/
https://tomoarch.com/2018/09/12/カードゲームをunityとcで作りたい%EF%BC%9Aカードシャッフ/

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

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

該当のソースコード

Deck.cs
【参考にさせて頂いたサイトのページ】https://tomoarch.com/2018/09/12/カードゲームをunityとcで作りたい%EF%BC%9Aカードシャッフ/

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)  //10行目
        {
            yield return i;
        }
    }

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

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

        int n = cards.Count;
        while (n > 1)
        {
            n--;
            int k = Random.Range(0, n + 1);
            int temp = cards[k];
            cards[k] = cards[n];
            cards[n] = temp;
        }
    }
    void Start()
    {
        Shuffle();
    }

}

DeckView.cs
【参考にさせて頂いたサイトのページ】https://tomoarch.com/2018/09/13/カードゲームをunityとcで作りたい%EF%BC%9Aカードシャッフ-2/

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>();
      ShowCards();  //19行目
   }

   void ShowCards()
   {
      int cardCount = 0;

      foreach(int i in deck.GetCards())  //26行目
      {
         float co = cardOffset * cardCount;

         GameObject cardCopy = (GameObject)Instantiate(cardPrefab);
         Vector3 temp = start + new Vector3(co, 0f);
         cardCopy.transform.position = temp;

         CardModel cardModel = cardCopy.GetComponent<CardModel>();
         cardModel.cardIndex = i;
         cardModel.ToggleFace(true);

         cardCount++;

      }
   }

}

試したこと

スクリプトやUnity上の手順を何回も確認してみたのですが、サイトと同じことをやっているつもりなのにうまくいきませんでした。

エラーコードについてネットで調べてみて、ある程度理解できたのですが、自分だけでは今回の場合に適用できませんでした。

最初は2019.3.12f1のバージョンでやっていたので、もしかしたらバージョンの違いかと思い2018.3.7f1でやってみても結果は変わりませんでした。

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

（Windows　10）
・Unity　2019.3.12f1
・Unity 2018.3.7f1
・Visual Studio Code

Answer 1

Start関数の呼び出し順はランダム（と言われてる）なので、
DeckViewのStart関数が先に呼ばれた場合、Shuffle関数が呼ばれる前に、GetCards関数が呼ばれ、
結果的にcards配列が初期化される前に呼ばれているのが原因だと思われます。

Comments

[Naoto96]

迅速なご回答ありがとうございます！
呼ばれる順番に着目していませんでした。今一度そこに気を付けて確認してみます。

[Naoto96]

Deck.csのStartをAwakeに変更したら解決しました。ご回答してくださったyuuyu様ありがとうございました。
"Awake"を用いて一番最初に"Shuffle"をして、"cards"配列を初期化した状態で、"Start"関数を呼び、次に"GetCards"を呼ぶということなんですね。良く理解することができました。次からは呼ばれる順番に注意してプログラミングを進めていきたいと思います。

迅速なご対応ありがとうございました。

Answer 2

多分DeckViewのdeckがnullです。
deck = GetComponent<Deck>();に失敗している気がするので、この次の行に
Debug.Log(deck);と入れて取得出来ているか確認してみてください。
（同じオブジェクトにDeckコンポーネントを付け忘れているとかだと思います）

---

↑かと思いましたがエラー発生箇所的にyuuyuさんの回答の方が正しそうです。
DeckのStart()をAwake()にしてみてください。

Comments

[Naoto96]

迅速なご回答ありがとうございます！
早速確認してみたのですが、
UnityEngine.Debug:Log(Object)DeckView(Deck)
UnityEngine.Debug:Log(Object)
と出ました。

DeckViewというObjectにDeck(Script)とDeckView(Script)を付けている状態です。
この状態で間違いはないでしょうか？
ご回答宜しくお願いします。

[Naoto96]

Deck.csのStartをAwakeに変更したら解決しました。ご回答してくださったsakura_hana様ありがとうございました。
"Awake"を用いて一番最初に"Shuffle"をして、"cards"配列を初期化した状態で、"Start"関数を呼び、次に"GetCards"を呼ぶということなんですね。良く理解することができました。次からは呼ばれる順番に注意してプログラミングを進めていきたいと思います。

迅速なご対応ありがとうございました。

[sakura_hana]

解決して何よりです。
Debug.Logについて、状態はそれでOKです。
もしnullの場合は「Null」と出るので、これを使うとどこがnullなのか自分で調べられます。
変数の中身を確認する他、本当にその部分に到達しているのかのチェックが出来るので次回以降はまずこれを使ってデバッグしてみてください。