yield WaitForSeconds Not Working?

Unity Engine
1

Hi, sorry again for my noobish question but im not sure why it is not working. Basically im on walkerboystudio lab 1:

http://www.walkerboystudio.com/html/unity_course_lab_1.html

i’ve finished the video but im confused what to do to finish the lab…

First of all, this is the original code for scriptEnemy.js:

var numberOfClicks : int = 2;
var respawnWaitTime : float = 2.0;
var shapeColor : Color[];
var explosion : Transform;
var enemyPoint : int = 1;

private var storeClicks : int = 0;

function Start ()
{
storeClicks = numberOfClicks;
var startPosition = Vector3 (Random.Range(-6,6), Random.Range(-4,4), 0);
transform.position = startPosition;
RandomColor ();	
}
function Update () {
	if (numberOfClicks <= 0)
	{
		if (explosion)
		{
			Instantiate (explosion, transform.position, transform.rotation);
		}
		if (audio.clip)
		{
			audio.Play();
		}
			var position = Vector3 (Random.Range(-3,3),Random.Range(-3,3), 0);
		WaitTime();
		transform.position = position;
		numberOfClicks = storeClicks;	
	}
}

	function WaitTime ()
	{
		renderer.enabled = false;
		RandomColor();
		yield WaitForSeconds (respawnWaitTime);
		renderer.enabled = true;
	}
	
	
	function RandomColor ()
	{
		var newColor = Random.Range(0, shapeColor.length);
		renderer.material.color = shapeColor[newColor];	
	}

this script aboveis applied to the “sphere”…there’s also one object called playerObject which applied with scriptPlayer.js, and here is what’s inside scriptPlayer:

var tagName : String;
var rayDistance : float = 0;
var score : int = 0;
var gameTime : float = 5.0;
var loadWaitTime : float = 3.0;
var waitbefore : float = 4.0;
var numberOfPointsToWin : int = 5;

function Start ()
{
	InvokeRepeating ("CountDown", 1.0, 1.0); // start the timer countdown 
}
 
function Update () 
{
if (Input.GetMouseButtonDown(0))
{
	anjing2();
print ("It works!");
var hit : RaycastHit;
var ray : Ray = Camera.main.ScreenPointToRay(Input.mousePosition);
if (Physics.Raycast (ray, hit, rayDistance))
{
	if (hit.transform.tag == tagName)
	{
var enemyScript = hit.transform.GetComponent(scriptEnemy);
enemyScript.numberOfClicks -= 1;
if (enemyScript.numberOfClicks == 0)
{
score += enemyScript.enemyPoint;
}
	}
	else
{
print ("This is not an enemy!");
}
}
}
}

function CountDown ()
{
	if (--gameTime == 0) // gametimer will keep reducing until 0
	{
	CancelInvoke("CountDown");	// to cancel the invoke when it reaches 0
	if ( score >= numberOfPointsToWin )
	{
	Application.LoadLevel ("sceneScreenWin");
	}
	else
	{
		Application.LoadLevel ("sceeneScreenLose");
		}
	}
}

function OnGUI ()
{
GUI.Label (Rect(10,10,100,20), "Score: " + score);	
GUI.Label (Rect(10,20,100,20), "Time: " + gameTime);	
}

	function anjing2 ()
	{
		yield WaitForSeconds (waitbefore);
	}

Now they told me to do this by myself:

How can i do that? i have tried to add another function, similar to “WaitTime ()”, i called it “WaitTime2 ()”, and use differet variable name for it. I added WaitTime2(); above if explosion() in scriptEnemy.js, but it doesn’t work?

2

The top one, you are not properly calling the coroutine, you cannot just call it like a normal routine, you MUST create a seperate process for it. To do that, you use StartCoroutine…

function Update () {
	if (numberOfClicks <= 0)
	{
		if (explosion)
		{
			Instantiate (explosion, transform.position, transform.rotation);
		}
		if (audio.clip)
		{
			audio.Play();
		}
			var position = Vector3 (Random.Range(-3,3),Random.Range(-3,3), 0);
		[COLOR="red"]StartCoroutine(WaitTime());[/COLOR]
		transform.position = position;
		numberOfClicks = storeClicks;	
	}
}

The bottom part… This:

InvokeRepeating (“CountDown”, 1.0, 1.0);

is not the way to do a countdown timer.

You would start a coroutine, display 3, yield for one second, then 2, then yield for one second, then 1, then yield for one second, the let the coroutine end. At the start, you simply set Time.timeScale to zero, at the end make it 1.

3

Hi,

I think setting the TimeScale to zero won t let the coroutine yield for one second (or three seconds) right?

sidev

4

It does, because I built a countdown timer in a game that did just that.

5

ok but this wasn’t my question. Even if i replace StartCoroutine(WaitTime()); with simply WaitTime();

it won’t make any difference. Atm, with this script:

if i click the object → it disappears → WaitTime → it reappears

What i want is not like that but like this…

if i click the object → it waits for x seconds → it disappear → WaitTime → it reappears

now if i want this “it waits for x seconds” to be done, what to add and how?

6

You could move the code into the Update to a CoUpdate. That is instead of void Update(){} you would put a IEnumerator CoUpdate(){}, with a while true loop inside. Then, your could do a yield return StartCoroutine instead of just a StartCoroutine. If you put yield return new WaitForSeconds(3.0) into this coroutine, then it will wait for 3 seconds before letting the code continue.
Does that make sense ?

Sidev

See “Coroutine Manager” on the Asset Store for easy and powerful coroutines !

7

OK how to do that? You mean i put:

IEnumerator CoUpdate()
{
StartCoroutine(TheWaitForXSecondsFunction());
}

before the if (explosion)? or how? Because it doesn’t work…

*PS: TheWaitForXSecondsFunction is the function for “it waits for x seconds”, check my previous post…

8

Hi pulento,

I took the liberty to write you the code skeleton you should ALWAYS use for your projects.
This should help you in solving your problems. Please let me know if something is still confusing.

using UnityEngine;
using System.Collections;

public class Punleto : MonoBehaviour {

	// Use this for initialization
	void Start () {
		StartCoroutine(CoUpdate()); //This is REALLY IMPORTANT
	}
	
	// Update is called once per frame
	void Update () {
	
	}
	
	//This is the best way to put non performance sensitive code. ALWAYS create a CoUpdate() !!! believe me.
	//Note: Coroutine Manager will help you stop, pause, resume or even reset this coroutine at will! Go for it.
	IEnumerator CoUpdate(){ //ALWAYS have a CoUpdate, you ll thank me later...
		while (true){ //Not necessary if you are using the "loop" mode in Coroutine Manager
			
			/* do some of your code there */
			//NOTE: this code will loop forever until you stop it based on a given condition.
			print("doing some stuff...");
			yield return StartCoroutine(WaitForXSeconds(3.0f));
			print("now we are done waiting. Let s move on");
			
			
			yield return 0; //Do not erase this without caution!
		}
	}
	
	IEnumerator WaitForXSeconds(float seconds){
		print("starting");
		yield return new WaitForSeconds(seconds);
		print("done!");
	}
}
9

Pulento? Are you confortable with C# instead of Javascript? There are similar in Unity but if you want to get into serious code I suggest you spend some time using it. Really.

10

hi sidev, thanks so much for helping. I actually dont have any experience with C#. I started learning javascript since about 1 month ago, so basically im a total noob. Thanks once again

11

Punleto,

No problem, enjoy Coroutines :wink:

See “Coroutine Manager” on the Asset Store for easy and powerful coroutines !

12

Wha… so what you want to do is add the wait for x seconds into the wait time… and I thought it was complicated…

Same script as before, just change your WaitTime coroutine to this:

	function WaitTime ()
	{
		yield WaitForSeconds (respawnWaitTime);
		renderer.enabled = false;
		RandomColor();
		yield WaitForSeconds (respawnWaitTime);
		renderer.enabled = true;
	}

If you want to specify the Wait time per click… then you would need this:

	function WaitTime (X : float)
	{
		yield WaitForSeconds (X);
		renderer.enabled = false;
		RandomColor();
		yield WaitForSeconds (respawnWaitTime);
		renderer.enabled = true;
	}