StartCoroutineで呼び出せない。

Question

UnityでC#を使って作っています。
コールチンを使ってsound Play();を３０秒ごとに実行させたいのですが、StartCoroutine(Sleep));からIEnumerator Sleep()に移行しません。
そのためにaddText textの表示が　１　、３０秒後に　２　を表示したいのですが、すぐに　２　を表示してしまいます。
また、このスクリプトで　１　、３０秒後に　２　を表示出来るのかも分からないので訂正箇所を教えてもらえますか。
下記にそのスクリプトを添えて置きます。
・・・・
void Update()
{
ST = GameObject.Find("Stage");
scripta1 = ST.GetComponent<Stage2>();
T = scripta1.total;
if (Input.anyKey)
{
if (T < 0)
StartCoroutine(Sleep());

        {
            addText.text = " " + 1.ToString();
            sound.PlayWide();
            Debug.Log("add");
        }
        if (T < 1)
            StartCoroutine(Sleep());

        {
            addText.text = " " + 2.ToString();
            sound.PlayWide();
            Debug.Log("add");
        }
    }
}
IEnumerator Sleep()
{
    yield return new WaitForSeconds(30.f);
}

}
＊T=5と考えてください。

Answer 1

StartCoroutine(実行したい内容)　は、実行したい内容を「やっておいてね！」と言って、すぐ次の行に進んでしまうので、質問のような挙動になります。

また、すぐに次の行に進んでしまうので、複数回StartCoroutine()で待つ処理を呼び出しても、何回か連続して待つ(30秒待ったあとに、さらに30秒待つ)という処理にはなりません。

質問にある仕様を正しく理解できているか分かりませんが、
0. 「1」を表示する
0. 30秒待つ
0. 「2」を表示する
0. 30秒待つ
・・・

というような挙動を期待している場合は、StartCoroutine()で呼び出すSleep()側で、WaitForSeconds()とaddText.text = " " + 1.ToString();・・・のループを処理する必要があるかと思います。
また、引数としてループ回数などを渡す必要もあるかと思います。

Unity スクリプトリファレンス MonoBehaviour.StartCoroutine
https://docs.unity3d.com/jp/540/ScriptReference/MonoBehaviour.StartCoroutine.html
にあるサンプルを参考にするとよいかと思います。

---

追記：
質問にあるコード(抜粋)を読みやすく(バグを仕込みにくく)書き直してみると、次のようになっているかと思います。
不等号の向きなどが期待している動作と異なっているような気がしますので、一旦見直してみては？

if (T < 0)
{
    StartCoroutine(Sleep()); 
}
addText.text = " " + 1.ToString();

if (T < 1)
{
    StartCoroutine(Sleep()); 
}
addText.text = " " + 2.ToString();

Comments

[YukioMaki]

toris-birdsさん回答ありがとうございます。
書かれていたサンプルを参考に下記のスクリプトに書き換えたところ、[ 'void' は反復子インターフェイス型ではないため、'Addsistem.Update()' の本体は反復子ブロックにできません。]エラーになりました。
訂正を教えていただきたいのですが、
・・・
void Start()
{
addText.text = " ";
sound = GameObject.Find("SoundManager").GetComponent<SoundManager>();

}

void Update()
{
ST = GameObject.Find("Stage");
scripta1 = ST.GetComponent<Stage2>();
T = scripta1.total;

if (Input.anyKey)
{
StartCoroutine("DoSomething", 2.0f);
yield return new WaitForSeconds(1);
StopCoroutine("DoSomething");

if (T < 0)
{
addText.text = " " + 1.ToString();
sound.PlayWide();
}
if(T<1)
StartCoroutine("DoSomething", 2.0f);
yield return new WaitForSeconds(1);
StopCoroutine("DoSomething");
{
addText.text = " " + 1.ToString();
sound.PlayWide();
}

}
}
IEnumerator DoSomething(float someParamerter)
{
while (true)
{
print("DoSomething Loop");
yield return null;
}
}
}
void Start()
{
addText.text = " ";
sound = GameObject.Find("SoundManager").GetComponent<SoundManager>();

}

void Update()
{
ST = GameObject.Find("Stage");
scripta1 = ST.GetComponent<Stage2>();
T = scripta1.total;

if (Input.anyKey)
{
StartCoroutine("DoSomething", 2.0f);
yield return new WaitForSeconds(1);
StopCoroutine("DoSomething");

if (T < 0)
{
addText.text = " " + 1.ToString();
sound.PlayWide();
}
if(T<1)
StartCoroutine("DoSomething", 2.0f);
yield return new WaitForSeconds(1);
StopCoroutine("DoSomething");
{
addText.text = " " + 1.ToString();
sound.PlayWide();
}

}
}
IEnumerator DoSomething(float someParamerter)
{
while (true)
{
print("DoSomething Loop");
yield return null;
}
}
}
void Start()
{
addText.text = " ";
sound = GameObject.Find("SoundManager").GetComponent<SoundManager>();

}

void Update()
{
ST = GameObject.Find("Stage");
scripta1 = ST.GetComponent<Stage2>();
T = scripta1.total;

if (Input.anyKey)
{
StartCoroutine("DoSomething", 2.0f);
yield return new WaitForSeconds(1);
StopCoroutine("DoSomething");

if (T < 0)
{
addText.text = " " + 1.ToString();
sound.PlayWide();
}
if(T<1)
StartCoroutine("DoSomething", 2.0f);
yield return new WaitForSeconds(1);
StopCoroutine("DoSomething");
{
addText.text = " " + 1.ToString();
sound.PlayWide();
}

}
}
IEnumerator DoSomething(float someParamerter)
{
while (true)
{
print("DoSomething Loop");
yield return null;
}
}
}

[toris-birds]

YukioMakiさん

まず、質問やコメントは、Markdownを使ってフォーマットしましょう。
インデントも抜けているので、非常に読みにくいです。
また、同一のコードが繰り返しペーストされているようです。
投稿、コメント後には、一度内容を見直しましょう。
・・・と思ったら、コメント欄は、Markdownがうまく動作しないみたいですね。

ざっくり打ち直しておきました。質問にあるコードと若干仕様が変わっているかもしれません。
(最初の「1」を一定時間待った後に出す仕様になっています。)
また、動作確認、コンパイル等通していないので、typo等あるかもしれませんが、ご容赦を。

----

```c#
void Start()
{
addText.text = " ";
sound = GameObject.Find("SoundManager").GetComponent<SoundManager>();
}

void Update()
{
ST = GameObject.Find("Stage");
scripta1 = ST.GetComponent<Stage2>();
T = scripta1.total;

if (Input.anyKey)
{
StartCoroutine(DoSomething(T));
}
}

IEnumerator DoSomething(int T)
{
for (int i = 0; i < T; i++)
{
yield return new WaitForSeconds(1); // 指定時間待つ
String text = String.format(" {0}", i); // " " + i.ToString(); でも作れると思いますが・・
print(text);
sound.PlayWide();

// (UIスレッドとは別のスレッドで動作するはずなので)ここで呼ぶと例外が発生するかと思います。
// 必要な場合は、別途。
// addText.text = text;
}
}
```

[YukioMaki]

toris-birdsさんコードを書き直し構想どうりに動くようになりました。
ありがとうございます。
また、ご指導よろしくお願いします。

[toris-birds]

@YukioMaki さん
動くようになったようで、よかったです。
今回の質問の範囲からは外れますが、Update()は、設定したフレームレート(例えば60FPS)などに応じて、1秒間に数10回呼び出されるかと思います。また、キーは押してすぐ手を離したとしても、0.1秒程度は押したままになっているかと思います。

なので、Update()の中でキー入力に応じて呼ばれる処理は、1回のキー入力でかなりの回数呼ばれることになるかと思います。(似たようなタイミングで同じ処理が走るので、表示上は問題なさそうに見えるかもしれません。)　実装に慣れて来たら、その当たりを考慮して設計を変えた方がよいかと思います。

参考になれば。

Answer 2

サウンドのPlayと30秒待つをコルーチン内で行えばできると思います。

IEnumerator SoundLoop()
{
    while(true)
    {
        // サウンド再生
        sound.PlayWide();

        // 30秒待つ
        yield return new WaitForSeconds(30.f);
    }
}

Comments

[toris-birds]

おそらく、質問者が想定している動作は、yield return new WaitForSeconds(30.f); sound.PlayWide(); の順かと思います。

[YukioMaki]

StudioAresさん回答ありがとうございます。