エラー：IndexOutOfRangeException: Index was outside the bounds of the array.

Question

###前提・実現したいこと
Unityでデュエルマスターズプレイスを模倣したようなゲームを作ろうとしています。
IndexOutOfRangeException: Index was outside the bounds of the array.
上記のエラー発生の原因がわからず困っています。クラスのインスタンス宣言方法を

Player player1 = new Player;

から

GameObject PlayerObj1 = GameObject.Find("Player1")
Player player1 = PlayerObj1.GetComponent<Player>();

に変更したら急にエラーが出ました。

インスタンス宣言をnewを使ったものに変更すればエラーはなくなるのですが、そうすると今度はコルーチンが使えなくなってしまいます。エラーの原因、解決策を教えていただけると幸いです。

IndexOutOfRangeException: Index was outside the bounds of the array.
(wrapper stelemref) System.Object.virt_stelemref_class_small_idepth(intptr,object)
Player.SetDeck () (at Assets/Player.cs:131)
Main.Start () (at Assets/Main.cs:22)

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.UI;
using UnityEngine.EventSystems;
using System.Linq;
using System.Security.Cryptography;
/*メモ*/
/*コンストラクタを消したらインスペクタに変数表示されないバグが治った*/



public class Player : MonoBehaviour
{
   [SerializeField] DeckData pladeck;
   public Card[] hand = new Card[40]; //?????????????10?
   GameObject[] handobj = new GameObject[40];
   public Card[] deck = new Card[40]; //
   GameObject[] deckobj = new GameObject[40];
   public Card[] mana = new Card[40]; //?????
   GameObject[] manaobj = new GameObject[40];
   public Card[] shield = new Card[40]; //???????????????????????????
   GameObject[] shieldobj = new GameObject[40];
   public Card[] battlezone = new Card[10]; //?????????????10?
   GameObject[] battlezoneobj = new GameObject[40];
   public Card[] grave = new Card[40]; //??
   GameObject[] graveobj = new GameObject[40];

   [SerializeField] private GameObject HandZone; //?????
   [SerializeField] private GameObject DeckZone;
   [SerializeField] private GameObject ManaZone;
   [SerializeField] private GameObject ShieldZone;
   [SerializeField] private GameObject BattleZone;
   [SerializeField] private GameObject GraveZone;

   [SerializeField] Text InstructText;
   [SerializeField] Canvas mycanvas;
   [SerializeField] YesNoDialog dialog;


   /*  public Player(DeckData dek, GameObject h, GameObject d, GameObject m, GameObject s, GameObject b, GameObject g,
         Text text, Canvas can, YesNoDialog dia)
     {
         pladeck = dek;
         HandZone = h;
         DeckZone = d;
         ManaZone = m;
         ShieldZone = s;
         BattleZone = b;
         GraveZone = g;
         InstructText = text;
         mycanvas = can;
         dialog = dia;
     }*/

   public void Constrct(DeckData deck, GameObject h, GameObject d, GameObject m, GameObject s, GameObject b, GameObject g,
   Text text, Canvas can, YesNoDialog dia)
   {
       this.pladeck = deck;
       this.HandZone = h;
       this.DeckZone = d;
       this.ManaZone = m;
       this.ShieldZone = s;
       this.BattleZone = b;
       this.GraveZone = g;
       this.InstructText = text;
       this.mycanvas = can;
       this.dialog = dia;
       Debug.Log("Constructed!");
   }


   //YesNo??????????bool????????
   private bool dialog_clicked = false;
   public bool Ask_YesOrNo(string t)
   {
       bool ans = false;
       var _dialog = Instantiate(dialog, mycanvas.transform); //???????dialog???
       _dialog.FixDialog = result => Debug.Log(result);
       GameObject childYes = _dialog.transform.GetChild(1).gameObject.GetComponent<Transform>().transform.GetChild(1).gameObject;
       GameObject childNo = _dialog.transform.GetChild(1).gameObject.GetComponent<Transform>().transform.GetChild(2).gameObject;
       GameObject dialogtext = _dialog.transform.GetChild(1).gameObject.GetComponent<Transform>().transform.GetChild(0).gameObject;

       dialogtext.GetComponent<Text>().text = t; //?????????
       childYes.GetComponent<Button>().onClick.AddListener(() => { ans = true; dialog_clicked = true; Debug.Log("addlisnerYes"); });
       childYes.GetComponent<Button>().onClick.AddListener(() => { ans = false; dialog_clicked = true; });
       StartCoroutine(nonme());

       Debug.Log("Clicked");
       return ans;
   }

   IEnumerator nonme()
   {
       Debug.Log("StartCorutine");
       yield return new WaitWhile(() => !dialog_clicked);
       dialog_clicked = false;
   }





   //???????????
   public void DeckShuffle()
   {
       int nstart = 40;
       for (int i = 0; i < 40; i++)
       {
           if (deck[i] == null)
           {
               nstart = i;
               break;
           }
       }

       for (int i = 0; i < nstart; i++)
       {
           Card tmp = deck[i];
           int randomIndex = Random.Range(i, nstart);
           deck[i] = deck[randomIndex];
           deck[randomIndex] = tmp;
       }
   }

   //登録デッキリストから読み取り、デッキゾーンにカードをセットする
   public void SetDeck()
   {
       for (int i = 0; i < 40; i++)
       {
           deck[i] = pladeck.deckcard[i];
       }
       DeckShuffle();

       for (int i = 0; i < 40; i++)
       {
           deckobj[i] = Instantiate(deck[i].obj, DeckZone.transform);
       }

   }

   //DeckZone??????????????HandZone????
   public void Draw()
   {
       Card tmp = deck[0];
       GameObject tmpobj = deckobj[0];
       Array_add(ref hand, ref handobj, tmp, tmpobj, HandZone.transform);
       Remove(ref deck, ref deckobj, 0);
       if (deck[0] == null) Lose(); //??????????????
   }

   //DeckZone????????????????
   public void DeckToShield()
   {
       Card tmp = deck[0];
       GameObject tmpobj = deckobj[0];
       Array_add(ref shield, ref shieldobj, tmp, tmpobj, ShieldZone.transform);
       Remove(ref deck, ref deckobj, 0);
       if (deck[0] == null) Lose(); //??????????????
   }

   public void DeckToMana()
   {
       Card tmp;
       GameObject tmpobj;
       tmp = deck[0];
       tmpobj = deckobj[0];
       DestroyImmediate(deck[0].obj, true);
       Remove(ref deck, ref deckobj, 0);
       if (deck[0] == null) Lose(); //デッキが切れたら敗北
       Array_add(ref hand, ref handobj, tmp, tmpobj, HandZone.transform);
       Instantiate(tmp.obj, HandZone.transform);
       deck[0].obj.transform.SetParent(HandZone.transform);
   }


   public void ManaCharge()
   {
       InstructText.text = "????????????";
   }

   //マナチャージする
   public void FillinMana()
   {
       InstructText.text = "マ";
       EventTrigger[] triger = new EventTrigger[40];
       EventTrigger.Entry[] entryarry = new EventTrigger.Entry[40];

       //????????????EventTriger??????
       for (int i = 0; i < 40; i++)
       {
           if (hand[i] == null) break;
           handobj[i].AddComponent<EventTrigger>(); //??????????????
           triger[i] = handobj[i].GetComponent<EventTrigger>();
           EventTrigger.Entry entry = new EventTrigger.Entry();
           entry.eventID = EventTriggerType.PointerDown;
           // ???{}??????????????
           int n = i;
           entry.callback.AddListener((eventDate) => { HandToMana(n); });

           triger[i].triggers.Add(entry);
       }

   }

   //n???????????????????????EventTriger??????????????
   public void HandToMana(int n)
   {
       Debug.Log("Invoked HandToMana()" + n);
       Card tmp = hand[n];
       GameObject tmpobj = handobj[n];
       Array_add(ref mana, ref manaobj, tmp, tmpobj, ManaZone.transform);
       Remove(ref hand, ref handobj, n);

       for (int i = 0; i < 40; i++)
       {
           if (hand[i] == null) break;

           EventTrigger trg = handobj[i].GetComponent<EventTrigger>();
           trg.triggers.RemoveRange(0, trg.triggers.Count);
       }
       InstructText.text = " ";
   }


   public void Win()
   {

   }

   public void Lose()
   {

   }

   //Card????????????????(???????????????????????????)
   private void Array_add(ref Card[] array1, ref GameObject[] array2, Card cd, GameObject obj, Transform zone)
   {
       for (int i = 0; i < 40; i++)
       {
           if (array1[i] == null)
           {
               array1[i] = cd;
               array2[i] = Instantiate(obj, zone);
               break;
           }
       }
   }


   private void Remove(ref Card[] array1, ref GameObject[] array2, int x)
   {
       Destroy(array2[x]);
       for (int i = x; i < 39; i++)
       {
           array1[i] = array1[i + 1];
           array2[i] = array2[i + 1];
       }
       array1[39] = null;
       Destroy(array2[39]);
       array2[39] = null;
   }


}

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.UI;
using System;
using System.Linq;
using System.Security.Cryptography;


public class Main : MonoBehaviour
{
    void Start()
    {
        GameObject PlayerObj1 = GameObject.Find("Player1");
        Player player1 = PlayerObj1.GetComponent<Player>();

        GameObject PlayerObj2 = GameObject.Find("Player2");
        Player player2 = PlayerObj1.GetComponent<Player>();



        player1.SetDeck(); //デッキを設置
        player2.SetDeck();


        for (int i = 0; i < 5; i++) player1.DeckToShield();
        for (int i = 0; i < 5; i++) player2.DeckToShield();

        for (int i = 0; i < 5; i++) player1.Draw();
        for (int i = 0; i < 5; i++) player2.Draw();

        if (player1.Ask_YesOrNo("マナチャージしますか？"))
        {
            Debug.Log("Yesが選ばれた");
            player1.FillinMana();
        }
        else Debug.Log("Noが選ばれた");
    }


    void Update()
    {
        
    }


}

Comments

[y_waiwai]

エラーが出たなら、エラーメッセージを提示しましょう
エラーメッセージは、よけいな省略翻訳しないで出たそのママをコピペで提示してください

どこの行でそのエラーが出たのかぐらい書きましょうよ

[bboydaisuke]

エラーメッセージによると、エラーは Player.cs の 60 行目、Start() メソッドの中で起きていますが、貼ってあるコードには Start() メソッドがありません。コードを変えて違う状況になってから起きたエラーを教えてもらっても意味がないので、エラーを示す時はその時のコードを一行・一文字も変えずに示す必要があります。

あと、IndexOutOfRangeException の意味は、配列に指定した添え字が範囲を超えている（負の値を指定したり、存在しない要素を指定したりしている）という意味なので、それを元に調べれば多分解決できるでしょう。

Answer 1

エラーになっているとされる行で pladeck.deckcard[i]（i は 0~39）を読み込もうとしていますが、pladeck.deckcard の要素が 40 個ないのでしょう。

DeckData が何者で、deckcard プロパティにどのようなデータが入っているのか、それをどこからどのように設定しているのかはわかりません。

また、Unity で自分で書いたクラス（の内、GameObject に追加して使うもの、つまりコンポーネント）は new することは禁止されています。従ってそのようなクラスにはコンストラクタは書きません。コンポーネントを new すると実行時にその操作が許可されていない旨のエラーが出力されるはずです。

Answer 2

deck[]の要素数をインスペクター上から操作したら解決しました。助言をくださったみなさん、ありがとうございました。