IEnumerator issues

Question

I'm having issues with using IEnumerators, more specifically WaitForSeconds. So I have t$$anonymous$$s script that's supposed to play a bgm into, wait until it's done, then play the music loop:

 public IEnumerator playMusic()
 {
     audio.Play ();
     yield return new WaitForSeconds(audio.clip.length);
     audio.clip = musicLoop;
     audio.loop = true;
     audio.Play ();
 }

But what's going on instead is it doesn't play the loop until after twice the length of the intro.

I've had t$$anonymous$$s problem before with stopping the music, playing a jingle, and resuming the music.

Here is the full script if it'll be any help:

 using UnityEngine;
 using System.Collections;
 [RequireComponent(typeof(AudioSource))]
 public class AudioDirector : MonoBehaviour {
     public AudioClip musicIntro;
     public AudioClip musicLoop;
     GameObject musicSource;
     public AudioClip[] Jingles;
     public int JingleType;
 
     // Use t$$anonymous$$s for initialization
     void Start () {
         musicSource = gameObject;
         audio.clip = musicIntro;
         playMusic ();
     
     }
     
     // Update is called once per frame
     void Update () {
         DontDestroyOnLoad (musicSource);
     }
     public IEnumerator playMusic()
     {
         audio.Play ();
         yield return new WaitForSeconds(audio.clip.length);
         audio.clip = musicLoop;
         audio.loop = true;
         audio.Play ();
     }
     public IEnumerator PlayJingle(int jingletype)
     {
         JingleType = jingletype;
         audio.Stop ();
         audio.loop = false;
         audio.clip = Jingles [JingleType];
         audio.Play ();
         yield return new WaitForSeconds(audio.clip.length);
         audio.clip = musicLoop;
         audio.loop = true;
         audio.Play ();
     }
 }

Answer 1

Best Answer

Answer by liortal · Jul 11, 2014 at 09:07 PM

You should be starting a coroutine, passing the IEnumerator into that method.

So instead of:

 playMusic();

you should be calling:

 StartCoroutine(playMusic());

An IEnumerator is not$$anonymous$$ng magical, and as such, must be "registered" wit$$anonymous$$n Unity (by passing it into StartCoroutine), t$$anonymous$$s will make sure Unity will periodically run it, and also let you yield from it and return after parts of it are complete (e.g: wait for some time, etc).

Add comment · Show 1 · Share

Bunny83 · Jul 11, 2014 at 09:32 PM 0

Share

Answer 2

Answer by Kiwasi · Jul 11, 2014 at 09:06 PM

I'm surprised it waited at all.

In C# you need to use StartCouroutine

Add comment · Share

Answer 3

Well, what you first should check is:

What's the actual length of your intro outside of Unity?
What does "musicIntro.length" report?
Try starting the playMusic coroutine manually for testing and measure the actual time it takes.
Also it makes no sense to call DontDestroyOnLoad every frame. If you call it once it will never be destroyed automatically. The only way is using Destroy(). Keep in mind that DontDestroyOnLoad only works on root objects w$$anonymous$$ch don't have a parent. If a c$$anonymous$$ld is marked with DontDestroyOnLoad but the parent is not, it's useless.

You can do somet$$anonymous$$ng like that:

 void Start ()
 {
     musicSource = gameObject;
     DontDestroyOnLoad (musicSource); 
 }
  
 public IEnumerator playMusic()
 {
     Debug.Log("Intro length: " + musicIntro.length);
     var startTime = Time.realtimeSinceStartup;
 
     audio.clip = musicIntro;
     audio.Play ();
 
     Debug.Log("audio.clip.length : " + audio.clip.length);
 
     yield return new WaitForSeconds(audio.clip.length);
     
     var timeTaken = Time.realtimeSinceStartup - startTime;
     Debug.Log("time taken: " + timeTaken);
     
     audio.clip = musicLoop;
     audio.loop = true;
     audio.Play ();
 }
 
 
 void OnGUI()
 {
    if (GUI.Button(new Rect(10,10,100,40), "Start BGM"))
    {
        StartCoroutine(playMusic());
    }
 }

IEnumerator issues

3 Replies

Your answer

Welcome to Unity Answers

Follow this Question

Related Questions