WaitForSeconds Not Working

Question

I need to delay an Inventory's open time, but "yield return WaitforSeconds" is not working. Here is the coroutine's code:

 public IEnumerator CheckIfStillHover() {
         yield return new WaitForSecondsRealtime (1.5f); // change to options class variable once it is created
         print ("Fine. I admit it. I cut in line. Happy?");
     }

In case this has something to do with the way I called it, which I also had difficulty with (if anyone knows why I had to store it in a variable to call it, please tell me), here is the code from a separate script which I used to call the coroutine:

 using System.Collections;
 using System.Collections.Generic;
 using UnityEngine;
 
 [RequireComponent (typeof (InventoryRenderer))]
 public class ToggleInventoryHover : MonoBehaviour {
 
     //name the inventory to match this string (or vice-versa)
     public string inventoryName = "Inventory";
     GameObject inventory;
     IEnumerator coroutine;
 
     bool isHover = false;
 
     void Start() {
         inventory = GameObject.Find(inventoryName);
         coroutine = inventory.GetComponent<InventoryRenderer> ().CheckIfStillHover ();
     }
 
     void OnMouseOver() {
         isHover = true;
         inventory.GetComponent<InventoryRenderer> ().StartCoroutine(coroutine);
     }
 
     void OnMouseExit() {
         isHover = false;
 
     }
 }

Answer 1

Answer by nviverosb · Apr 23, 2018 at 11:25 PM

Here's a possible solution you're not keeping in mind:

WaitForSeconds DOES NOT WORK AS A DELAY OUTSIDE THE COROUTINE.

When you start a Coroutine, it executes itself, but the script inmediately executes the following instruction. For a WaitForSeconds to work, you need to put the code to open the inventory inside the coroutine method.

It would look something like this:

 public IEnumerator CheckIfStillHover() {
          yield return new WaitForSecondsRealtime (1.5f); // change to options class variable once it is created
         //Whatever you want to delay goes in here
  print ("Fine. I admit it. I cut in line. Happy?");
      }

Add comment · Share

Answer 2

Answer by Karsten · Apr 22, 2018 at 07:38 PM

i would use the Invoke mehod for this its much more straight forward and in the end is doing the same behind the scenes https://docs.unity3d.com/ScriptReference/MonoBehaviour.Invoke.html Also check if OnMouseOver is really called

Add comment · Show 4 · Share

kaplica · Apr 23, 2018 at 09:22 AM 0

Share

How is the invoke method doing a coroutine? Invoke, as the name implies, invokes a method, on the main thread of course, couroutin is being ran over multiple frames, which is different to a normal method invoke.

Karsten kaplica · Apr 23, 2018 at 07:40 PM 0

Share

he wants to wait 1,5 seconds, no need for a coroutine in this case

foureyes44 · Apr 23, 2018 at 11:14 PM 0

Share

Thanks, I will actually do that. (Forgot about Invoking methods) But, I still would like to find out why this wouldn't work.

Karsten foureyes44 · Apr 24, 2018 at 06:17 PM 0

Share

What happens when you change this line inventory.GetComponent ().StartCoroutine(coroutine); to StartCoroutine(coroutine); ? A future problem i see is that the code in OnMouseOver will call the routine probably unintended multible times.

I dont know what dev tools you use but you maybe should debug this proper by setting a breakpoint and step-in with Visual Studio for example, and the breakpoint set in the OnMouseOver() in line 2 would make sense

Answer 3

Answer by LilGames · Apr 22, 2018 at 08:44 PM

What do you mean when you say "not working" ? It would help if you identified what problem you're having.

Is there a reason you are using WaitForSecondsRealtime and not WaitForSeconds(5) ??

Add comment · Show 1 · Share

foureyes44 · Apr 22, 2018 at 08:49 PM 0

Share

My problem is simply that WaitForSeconds is not working. I tried both RealTime and the normal one, but it just doesn’t wait.

Answer 4

Answer by Instant_Dave · Apr 23, 2018 at 03:50 PM

I'm new but I'd like to help if I can.

public class Whatever : MonoBehaviour { public float delay = 1.5;

     void Start()
     {

OnMouseOver(); // I use Button also StartCoroutine (LoadStuffAfterDelay (delay)); } IEnumerator LoadStuffAfterDelay(float delay) { yield return new WaitForSeconds (delay); // //Do this next directions here
} }

Add comment · Show 2 · Share

Instant_Dave · Apr 23, 2018 at 03:52 PM 0

Share

Well that didn't post right lol.

     public float delay = 75;
     void Start()
     {
         StartCoroutine (LoadAfterDelay (delay));
     }
     IEnumerator LoadAfterDelay(float delay)
     {
         yield return new WaitForSeconds (delay);
         // Do next directions here
          }
 }

Instant_Dave · Apr 23, 2018 at 03:58 PM 0

Share

Add to On Mouse Over and modify scripts a little. 2nd script pointed to from first script after coroutine delay something like

  public Class CheckIfStillHover() {
          if still hover = true
          print ("Fine. I admit it. I cut in line. Happy?");
      }

WaitForSeconds Not Working

4 Replies

Your answer

Welcome to Unity Answers

Follow this Question

Related Questions