[Unity]変数が異なる同じ処理を省略する方法

Question

実現したいこと

Unityにて2Dゲームを作っています。
変数が異なるが同じ処理をするコードが多数あるので、何らかの方法を使って最適化したいと考えています。

void OnTriggerEnter2D(Collider2D collision)
{
    if(collision.CompareTag("target"))
    {
         isContact = true;
    }
}

void OnTriggerExit2D(Collider2D collision)
{
    if(collision.CompareTag("target"))
    {
         isContact = false;
    }
}

このコードと同じ処理で変数（このコードでのtarget(string)とisContact(bool)）だけが異なるコードがいくつもあります。該当するスクリプトでは変数のみを指定して、メソッドのコードは別のスクリプトに書いて省略したいと考えています。

試したこと

そこで継承を使った方法を思いつきました。

↓継承元

public class CollisionDitection : MonoBehaviour
{
    protected string targetTag;
    protected bool targetBool;

    void OnTriggerEnter2D(Collider2D collision)
    {
        if (collision.CompareTag(targetTag))
        {
            targetBool = true;
        }
    }

    void OnTriggerExit2D(Collider2D collision)
    {
        if (collision.CompareTag(targetTag))
        {
            targetBool = false;
        }
    }
}

↓継承先

public class Example : CollisionDitection
{
   void Start()
    {
        targetTag = "examTarget";
        targetBool = examBool;
    }
}

これだとエラーは発生はしませんが、継承元のメソッドは呼び出されませんでした。
また、そもそも継承をこういった形で使用するのは適切でないように感じています。

答えていただきたいこと

わがままですが、こういった状況での適切な最適化方法を教えていただきたいです。
よろしくお願いいたします。

Answer 1

上手く伝えられずすみません。
OnTriggerEnter2DとOnTrggerExit2Dの２つの処理がセットでいくつも存在するので省略できないかと考えていました。
自己解決として以下のコードを載せておきます。一つのスクリプトを多用する形を取っています。そもそもcol.Tagはレイヤーを設定すれば必要ないのですが、

public class CollisionDitection : MonoBehaviour
    {
        [SerializeField] string targetTag;
        public bool isOn;

        void OnTriggerEnter2D(Collider2D collision)
        {
            if (collision.CompareTag(targetTag))
            {
                isOn = true;
            }
        }

        void OnTriggerExit2D(Collider2D collision)
        {
            if (collision.CompareTag(targetTag))
            {
                isOn = false;
            }
        }
    }

コードの削減に囚われてはいけませんね、

Answer 2

void OnTriggerEnter2D(Collider2D collision,string target,ref bool result)

ってことでいいのでは

Comments

[fiveHundred]

これは「接触したときに呼び出される」というUnityの関数です。
https://docs.unity3d.com/ja/current/ScriptReference/MonoBehaviour.OnTriggerEnter2D.html

この引数を変更すると、ただの普通の関数になり動作しなくなります。

Answer 3

やりたい事がようわからんけど

isContact = collision.CompareTag("target");

みたいにワンライナーにすりゃいいんじゃないか、とかそういう話？

Comments

[Zin.Creat]

ごめんなさい。私のニュアンスが曖昧でした。
参考になる方法ですが、Tagが異なるときにisContactがfalseになってしまうので、それでは解決できませんでした。
複数の同じコードを削減する方法を知りたいです。

[gentaro]

イマイチよくわからんなぁ…。
trueになる条件とfalseになる条件は別物なんだからコードが別で当たり前じゃないのかと思うんだけども。
falseになるべき場所では
isContact = !collision.CompareTag("target");
こうすりゃいいだけで。