Unity（C#）AddComponentを使わずに、クラスを生成しコルーチンを使用する方法

Question

前提・実現したいこと

Unity（C#）で、カードゲームを作っています。
カードの能力（カードを引くなど）をStrategyパターンで実装しました。

AddComponentでカード能力のクラスをアタッチし、コルーチンを動かすような作りになっています。

しかし、１個や２個の能力でしたら問題ないのですが将来的にカードの能力は100を超える可能性もあり、
100個クラスをAddComponentでアタッチするのはおかしな実装なのではないかと思っております。
（おかしくなければ解決です）

そこで、下記のコードでAddComponentが並ばないような実装方法などありますでしょうか？
よろしくお願いします。

該当のソースコード

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

public class AbilityManager : MonoBehaviour
{
    Dictionary<string,AbilityStrategy> strategyDic = new Dictionary<string,AbilityStrategy>();

    void Start()
    {
        // 【該当箇所】能力が１つ増えるごとにここのAddComponentが増えていきます……
        AbilityDraw ad = gameObject.AddComponent<AbilityDraw>();
        strategyDic.Add("draw", ad);
    }

    /// <summary>
    /// アビリティを実行する
    /// <summary>
    /// <param name="ability">アビリティが書かれた文字列</param>
    /// <param name="param">アビリティのパラメータ</param>
    /// <param name="card">使用カード</param>
    public IEnumerator RunAbility(string[] ability, string[] param, CardController card){
        bool isPlayer = GameManager.instance.IsPlayerTurn;
        // 省略

        // アビリティを実行する
        int i = 0;
        foreach(string abl in ability){
            if(abl != null && abl != ""){
                yield return StartCoroutine(strategyDic[abl].AbilityInterface(param[i],isPlayer));
            }
            i++;
        }

        // 省略
    }
}

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

abstract public class AbilityStrategy : MonoBehaviour
{
    public abstract IEnumerator AbilityInterface(string param,bool isPlayer);
}

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

public class AbilityDraw : AbilityStrategy
{
    public AbilityDraw()
    {
    }

    public override IEnumerator AbilityInterface(string param,bool isPlayer)
    {
        yield return StartCoroutine(DrawHand(int.Parse(param),isPlayer));
    }
    // DrawHandの中身は省略
}

試したこと

AbilityManagerのStart()で、
strategyDic.Add("draw", new AbilityDraw());
と書いたところ、MonoBehaviourを継承したクラスをnewしてはいけません、
と警告された上にnullが返ってきてしまうためコルーチンが実行できませんでした。

AddComponentを使いなさいとも警告に出ていたので、ほかに方法はないのかもしれませんが……

Answer 1

AbilityStrategyクラスがMonoBehaviourクラスを派生しなければいいのでは？

StartCoroutineはMonoBehaviourのメソッドなのでMonoBehaviourを継承したクラスに対して呼び出す必要がありますが、コルーチン自体は別にどのクラスだろうが関係ありません。

Comments

[moon_peta]

ありがとうございます！

AbilityStrategyクラスがMonoBehaviourクラスを派生しないようにしたところ、
AbilityDrawのAbilityInterfaceでStartCoroutineを使用していたため、エラーは出たのですが、
そもそもなぜ、AbilityInterfaceをStartCoroutineで呼び出したあとに、さらに
AbilityDrawのAbilityInterfaceでStartCoroutineを呼び出しているのだろう？
という自分の実装の変な箇所に気づき、AbilityInterfaceの中にあったStartCoroutineを外したところ
無事、動作することができました。

この方法ですとコンポーネントを追加していくことがなくなるため、
ベストアンサーとさせていただきます。

また、私のコードの中に不必要なStartCoroutine（AbilityInterfaceの中にあったStartCoroutine）も入っていて、
回答者の方にはご迷惑をおかけしました。
どうもありがとうございました！

Answer 2

1つのGameObjectに大量にAddComponentするのではなく、以下のようにPrefab化するのはどうでしょうか？

1. AbilityDrawをアタッチしたGameObjectをPrefabにする
2. AbilityManagerにフィールドを追加してInspectorからPrefabを入れる
3. RunAbility内のループでInstantiate、GetComponent<AbilityDraw>、StartCoroutineする

Comments

[moon_peta]

ありがとうございます！

もう一つ回答いただいた、MonoBehaviourを継承しない、という方法が
うまくいきそうなため、MonoBehaviourを継承しない方法で進めていこうと思いますが、
Prefab化という発想がなかったため、とてもためになりました。

[shiena]

補足するとカードゲームという事で、カードの画像とAbilityDrawがアタッチされたPrefabをイメージしたものでカード削除する時も Destroy(gameObject); で済むので楽そうという考えでした。ぜひやりやすい方法で進めてください。