オブジェクトに当たったら、当たらないようにして、一定時間たったら、当たるようにする

Question

こんにちは、詰まってしまったので質問させていただきます。

やりたい事は、題名の通りなのですが、ブロックに特定のオブジェクトが当たったら、見えなくして（当たらないようにする)一定時間たったら、また当たるようにするというものです。

やった事

実装をやってみたのですが、うまくいきません。
コードです

親についているコード

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

public class QuestionDaddy: MonoBehaviour
{
    /// <summary>
    /// なくなったら一定時間後生成
    /// </summary>
    /// <returns></returns>
    public IEnumerator QuestionGenerator(GameObject question)
    {
        question.SetActive(false);
        print("当たったP");
        var nowTime = 0f;
        var timeLimit = 4f;
        while (nowTime < timeLimit)  {
            nowTime += Time.deltaTime;

            yield return null;
        }
        print("終わった");
        question.SetActive(true);
    }
}

子についているコード

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

public class HatenaBox : MonoBehaviour
{
    [SerializeField] private QuestionDaddy daddy;
    private void OnTriggerEnter(Collider other)
    {
        if(other.tag == "Cart")  {
            other.GetComponent<ItemCatch>().GetItem();
            StartCoroutine(daddy.QuestionGenerator(this.gameObject));
        }
    }
}

この方法でやってみたのですが、コルーチンが一回しか呼ばれないため、カウントが溜まりません、
何かいい方法はないでしょうか？回答お願いします。

追記です。

階層構造がこれで

QuestionDaddy が親についており

HatenaBox が子についております。

Answer 1

・コルーチンの仕様として呼び出し元であるオブジェクト（StartCoroutineを行うオブジェクト）を非アクティブにすると止まります。

コルーチンは一見呼び出し先のクラスに管理を委譲しているように思えるのですが、
実はそうではないので呼び出し元を非表示にする場合は設計を見直す必要が出てきます。

例として下記の様にコルーチンを始める関数を親に追記してそれを呼び出せば止まりません。

親についているコード

using System.Collections;
using UnityEngine;

public class QuestionDaddy : MonoBehaviour
{
    /// <summary>
    /// なくなったら一定時間後生成
    /// </summary>
    /// <returns></returns>
    public IEnumerator QuestionGenerator( GameObject question )
    {
        question.SetActive( false );
        print( "当たったP" );

        var nowTime = 0f;
        var timeLimit = 4f;
        while( nowTime < timeLimit )
        {
            nowTime += Time.deltaTime;

            yield return null;
        }

        print( "終わった" );
        question.SetActive( true );
    }

    /// <summary>
    /// コルーチンの呼出し開始
    /// </summary>
    /// <returns></returns>
    public void QuestionGeneratorStart( GameObject question )
    {
        StartCoroutine( QuestionGenerator( question ) );
    }
}

子についているコード

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

public class HatenaBox : MonoBehaviour
{
    [SerializeField] private QuestionDaddy daddy;
    private void OnTriggerEnter( Collider other )
    {
        if( other.tag == "Cart" )
        {
            other.GetComponent<ItemCatch>().GetItem();
            daddy.QuestionGeneratorStart( this.gameObject );
        }
    }
}

若しくは以下の二つの設計に作り変えると良いです。

・表示・非表示切り替えないCartオブジェクトにクラスを移してStartCoroutineを呼び出す
・下記の参考サイトの様にコルーチン実行用オブジェクトを生成する

・参考サイト
https://qiita.com/naoK/items/55fb18bd348cfaa92708

Comments

[Um_kok]

ご回答ありがとうございます。
お返事遅れました。なるほどです。やはり1回しか呼ばれていなかったのですね。StartCroutineしてるオブジェクトが非アクティブだったら呼ばれないのは良いお勉強になりました。ありがとうございます。

Answer 2

親についているコードです↓

public IEnumerator QuestionGenerator(GameObject question)
    {
        question.SetActive(false);
        print("当たったP");

        yield return new WaitForSeconds(4);

        print("終わった");
        question.SetActive(true);
    }

これで４秒たった後に、元に戻るはずです。
意図に添えていなかったらごめんなさい。

Comments

[Um_kok]

やはり return してしまうと　一回しか呼ばれないので、
print("終わった");
question.SetActive(true); は通らなくなってしまいます。

[hehebeb]

yield return new WaitForSeconds(4)　以降の処理が呼ばれない理由がわかりません。
普通なら４秒後に通るはずです。

もしかして表示非表示させたいオブジェクトにスクリプトをあてはめて、questionを代入していませんか。

[Um_kok]

あっていると思うのですが、、ご確認お願いします。
質問に追記しました。