Problem with WaitForSeconds() inside a Coroutine(), it just doesn't wait

Hello,

I'm trying to figure out how WaitForSeconds works inside a coroutine. As I understand it just delays a fixed amount of seconds the return of a coroutine. So to test it, I wrote a very basic script:

Code (CSharp):

1. protected  void Start ()
2.     {  
3.  
4.         MoveItMoveIt();
5.     }
6.  
7. protected void MoveItMoveIt()
8.     {
9.         for(int i = 0; i < 5; i++)
10.         {
11.             Debug.Log ("Going to Coroutine", gameObject);
12.  
13.             StartCoroutine(ColdDown());
14.  
15.             Debug.Log ("Coming back from Coroutine", gameObject);
16.         }
17.     }
18.  
19. private IEnumerator ColdDown()
20.     {
21.         yield return new WaitForSeconds(2.0f);
22.     }

I was expecting to watch the console display the "Coming back from Courutine" message 2 seconds later than "Going to Coroutine", but instead the console display inmediately the 10 messages of the for loop.

Can anyone tell me what am I doing wrong? Thank you for your feedback.

Just add: I'm working with C# and the gameObject that contents this script is instantiated by a gameManager as soon as the "game" starts.

 

Put the loop and your debug messages inside the coroutine.

The way it's currently set up, you're telling this behaviour to start 5 coroutines all at once, that simple wait for 2 seconds, and then do nothing.

 

You're just starting the coroutine, not yielding on it. Note that there's no reason to even have a coroutine that only has WaitForSeconds, just replace the coroutine call with WaitForSeconds.

--Eric

 

Thanks for your reply. I understand what you mean.

So my goal was to call the coroutine 5 times, with a 2 second delay between each call. I though the coroutine was like a thread and the yield was some kind of sleep (I'm still very noob ). I will try once again using flags, and post the results.

 

Thanks for the feedback Eric, I tried what I think you said, but it is still not dealy between messages in the console. Just all of them appear together at the beggining.

Code (CSharp):

1. protected override void Start ()
2.     {
3.         MoveItMoveIt();
4.     }
5.      
6.  
7. protected void MoveItMoveIt()
8.     {
9.         for(int i = 0; i < 5; i++)
10.         {
11.             Debug.Log ("Going to Courutine", gameObject);
12.             new WaitForSeconds(3.5f);
13.             Debug.Log ("Comming back from Courutine", gameObject);
14.         }
15.     }

 

http://docs.unity3d.com/Manual/Coroutines.html

--Eric

 

Ok, I got what I wanted. Thank you very much for your comments guys.

My final goal was to make a gameObject move every X seconds, for example an enemy. Changing the delay of WaitForSeconds I make it move faster or slower depenging on the type of enemy. So here it is what I wrote:

Code (CSharp):

1. private void Start ()
2.      {
3.         StartCoroutine(ColdDown());
4.  
5.      }
6.  
7. private void MovingThings()
8.     {
9.         Debug.Log ("Moving Enemy", gameObject);
10.     }
11.  
12.     private IEnumerator ColdDown()
13.     {
14.         //Assuming the enemy is always moving
15.         for(;;)
16.         {
17.             //Moving every 1.5 seconds
18.             yield return new WaitForSeconds(1.5f);
19.             MovingThings();
20.         }
21.  
22.     }

 

Hope it can help you

Code (CSharp):

1. using System;
2.  
3. public void YourFunc()
4.     {
5.            StartCoroutine(DelayFunc(()=>
6.             {
7.                 for(int i = 0; i < 5; i++)
8.                 {
9.                    Debug.Log("HI");
10.                 }
11.             }, 1.0f));
12.     }
13.  
14.  
15.     IEnumerator DelayFunc(Action func, float delay)
16.     {
17.         yield return new WaitForSeconds(delay);
18.         func();
19.     }

 

That won't work at all; the OP already posted working code (that's much more straightforward).

--Eric

 

What do u mean of won't work at all?
because it works on me?
it will print hi * 5 after 1 seconsd

what is the problem?
ps: just want to learn more things


edit: i guess what you mean is it isn't print 5 times each with a delay 1 secs
my bad that i didn't read through his post, sry