ArgumentOutOfRangeExceptionというエラーの対処とListの使い方があっているかわかりません

Question

前提・実現したいこと

Unityでカードゲームを作ろうとしています。
現在、デッキエディット画面を作ろうとしています。
前回の質問で長文になっていた検索機能を簡略化できそうなのですが、試しで一部のカードだけ表示しようとしたら下記のエラーが出ました。
Listにエクセルのデータをfor文で入れていこうとしています。
どのように対処すれば改善されますか？

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

ArgumentOutOfRangeException: Index was out of range. Must be non-negative and less than the size of the collection.
Parameter name: index
System.ThrowHelper.ThrowArgumentOutOfRangeException (System.ExceptionArgument argument, System.ExceptionResource resource) (at <fb001e01371b4adca20013e0ac763896>:0)
System.ThrowHelper.ThrowArgumentOutOfRangeException () (at <fb001e01371b4adca20013e0ac763896>:0)
CardSearch.Start () (at Assets/DeckEdit/Script/CardSearch.cs:176)

176行目は下記のコードのStart()の閉じカッコ　｝に位置しています。
おそらくListの取り扱いが間違っているのだろうということしかわかりません。

該当のソースコード

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.UI;
using System;
using System.Linq;
public class CardModelU
{
    public string id;
    public string name1;
    public string name2;
    public int level;
    public string type;
    public int attack;
    public int defence;
    public string tag;
    public string effect1;
    public string effect2;
    public string effect3;
 //   public string effect4;
}
public class CardModelS
{
    public string id;
    public string name1;
    public string name2;
    public int level;
    public string type;
    public string kind;
    public string tag;
    public string effect1;
    public string effect2;
    public string effect3;
    public string effect4;
}
public class CardModelI
{
    public string id;
    public string name1;
    public string name2;
    public int level;
    public string type;
    public string kind;
    public string tag;
    public string effect1;
    public string effect2;
    public string effect3;
    public string effect4;
}
public class CardSearch : MonoBehaviour
{
    List<CardModelU> allUnitList = new List<CardModelU>();
    List<CardModelS> allSpelList = new List<CardModelS>();
    List<CardModelI> allItemList = new List<CardModelI>();
    public InputField Level;
    public InputField ATK;
    public InputField DEF;
    public InputField Text;
    public Dropdown kindDD1;
    public Dropdown kindDD2;
    public Dropdown Type;
    //int Lv, atk, def,i;
    public GameObject SearchArea;
    public GameObject CardData;
    public GameObject CardID_;
    GameObject Obj;
    [SerializeField] private unit unit;
    [SerializeField] private item item;
    [SerializeField] private spel spel;
    
    public void CardSEARCH()
    {
        
        foreach (Transform child in SearchArea.transform)
        {
            Destroy(child.gameObject);
        }
        IEnumerable<CardModelU> result = allUnitList;
        
        if (Level.text != "")
        {
            result = result.Where(s => s.level.ToString() == Level.text);
        }
        if (ATK.text != "")
        {
            result = result.Where(s => s.attack.ToString() == ATK.text);
        }
        if (DEF.text != "")
        {
            result = result.Where(s => s.defence.ToString() == DEF.text);
        }
        if (Text.text != "")
        {
            result = result.Where(s => s.name1.ToString().Contains(Text.text) ||
                                       s.name2.ToString().Contains(Text.text) ||
                                       s.effect1.ToString().Contains(Text.text) ||
                                       s.effect2.ToString().Contains(Text.text) ||
                                       s.effect3.ToString().Contains(Text.text));
        }
        foreach (CardModelU modelU in result) ShowU(modelU);
    }
    public void ShowU(CardModelU modelU)
    {
        CardData.GetComponent<CardD>().ShowU(modelU);
        Obj = Instantiate(CardData, this.transform.position, Quaternion.identity);
        Obj.GetComponent<CardID>().cardID = modelU.id;  //この辺のIDの取り回しは要確認
        Obj.transform.parent = SearchArea.transform;
    }
    void Start()
    {
        int UD = unit.Sheet1.Count;
        int SD = spel.Sheet1.Count;
        int ID = item.Sheet1.Count;
        for (int i = 0; i < UD; i++)
        {
            allUnitList[i].id = unit.Sheet1[i].id;
            allUnitList[i].name1 = unit.Sheet1[i].name1;
            allUnitList[i].name2 = unit.Sheet1[i].name2;
            allUnitList[i].level = unit.Sheet1[i].level;
            if (String.IsNullOrEmpty(unit.Sheet1[i].type) == false) 
            { 
                allUnitList[i].type = unit.Sheet1[i].type;
            }
            else
            {
                allUnitList[i].type = "なし";
            }
            allUnitList[i].attack = unit.Sheet1[i].attack;
            allUnitList[i].defence = unit.Sheet1[i].defence;
            allUnitList[i].tag = unit.Sheet1[i].tag;
            allUnitList[i].effect1 = unit.Sheet1[i].effect1;
            if (String.IsNullOrEmpty(unit.Sheet1[i].effect2)==false)
            {
                allUnitList[i].effect2 = unit.Sheet1[i].effect2;
            }
            else
            {
                allUnitList[i].effect2 = "A";
            }
            if (String.IsNullOrEmpty(unit.Sheet1[i].effect3) == false)
            {
                allUnitList[i].effect3 = unit.Sheet1[i].effect3;
            }
            else
            {
                allUnitList[i].effect3 = "B";
            }
            // allUnitList[i].effect4 = unit.Sheet1[i].effect4;
        }
       /* for (int i = 0; i < SD; i++)
        {
            allSpelList[i].id = spel.Sheet1[i].id;
            allSpelList[i].name1 = spel.Sheet1[i].name1;
            allSpelList[i].name2 = spel.Sheet1[i].name2;
            allSpelList[i].level = spel.Sheet1[i].level;
            allSpelList[i].type = spel.Sheet1[i].type;
            allSpelList[i].tag = spel.Sheet1[i].tag;
            allSpelList[i].effect1 = spel.Sheet1[i].effect1;
            allSpelList[i].effect2 = spel.Sheet1[i].effect2;
            allSpelList[i].effect3 = spel.Sheet1[i].effect3;
            allSpelList[i].effect4 = spel.Sheet1[i].effect4;
        }
        for (int i = 0; i < ID; i++)
        {
            allItemList[i].id = item.Sheet1[i].id;
            allItemList[i].name1 = item.Sheet1[i].name1;
            allItemList[i].name2 = item.Sheet1[i].name2;
            allItemList[i].level = item.Sheet1[i].level;
            allItemList[i].type = item.Sheet1[i].type;
            allItemList[i].tag = item.Sheet1[i].tag;
            allItemList[i].effect1 = item.Sheet1[i].effect1;
            allItemList[i].effect2 = item.Sheet1[i].effect2;
            allItemList[i].effect3 = item.Sheet1[i].effect3;
            allItemList[i].effect4 = item.Sheet1[i].effect4;
        }*/
    }
}

試したこと

エクセルの空白入力がだめなのかと思い、空白セルのときに代入処理を任意の文字を入れようとしてみましたがだめでした。

Comments

[sakura_hana]

質問の回答としてはtakapi_csさんので合っていますが気になった所を。
同じようなカードモデルクラスを3つ作る意味が分からないので１つに統一するか、せめて「クラスの継承」を調べて派生クラスにした方がいいと思います。（以前の質問の私の回答では統一させていた筈です）

[wata3]

エクセルのデータとの兼ね合いですね。
カード種ごとにエクセルのデータを作っており、IDはカード種ごとに通し番号になっています。
（現在Unitの場合CUU00001～CUU00182、Spellの場合CUS00001～CUS00166、Itemの場合CUI00001～CUI00066）
あとはUnitとSpellとItemの持つ情報が少し異なるためです。
（Unitは攻撃力等がありSpellとItemにはなく、Spellには通常、永続、付与、Itemには通常、永続、装備など）

[sakura_hana]

エクセルのデータが別々でも同じカードモデルクラスを用いることは可能です。どのタイプか＋通し番号でIDが割り出せますし。
種類毎に情報が異なる場合、気にせず変数を全部乗せる（SpellとItemは攻撃力0扱いとする等）か、前述のクラス継承をした方がいいです。
でなければ今後もコードを書く量が3倍になるだけです（例えば現状のコードで検索出来るのはUnitだけで、SpellとItemを探す為には同じ量のコードを書く必要があります）。それで構わないというならいいですが。

[wata3]

調べてみてできそうであれば挑戦してみます。ありがとうございます。

Answer 1

Listにエクセルのデータをfor文で入れていこうとしています。

リストに対してデータを新規追加しようとしているのですね。
現在の実装では、allUnitListはnewのみ実行されており、Start()実行時のデータ個数は0です。
その状態でインデックスを使用した場合、allUnitList[0]は存在しないのでエラーとなります。
リストにデータを新規追加したい場合は、1つ適当な変数を用意してその中にデータを入れていき、データが整ってからリストに追加すると良いでしょう。
下記に例を記載しています。

var cardModelU = new CardModelU()
{
    id = unit.Sheet1[i].id,
    name1 = unit.Sheet1[i].name1,
    ...
};
allUnitList.Add(cardModelU);

Comments

[wata3]

うまく行きそうです。ありがとうございました。