Am I doing something wrong?

Unity Courses Ask
, ,
#1

Rick’s method to add the delay for loading the scene works without any issues, but I was wondering why my method isn’t working

My SceneLoader:

public IEnumerator LoadEndMenu()
    {
        Debug.Log("A");

        yield return new WaitForSeconds(delayToEndMenu);

        Debug.Log("B");

        SceneManager.LoadScene(2);
    }

my Player:

private void Die()
    {
        StartCoroutine(sceneLoader.LoadEndMenu());

        Destroy(gameObject);

        AudioSource.PlayClipAtPoint(playerDeathSFX, Camera.main.transform.position, playerDeathSFXVolume);
    }

everything that’s before: yield return new WaitForSeconds(delayToEndMenu);
works, but the things after it don’t, what am I doing wrong?

#2

Hi Haisp,

What does “my method isn’t working” mean? What did you expect to happen, what happened instead?

See also:

#3

There should be a delay before loading the new scene after the player dies, what ended up happening is A gets written in the console when the player dies and everything after yield doesn’t work, so there’s some sort of issue that’s doing something in the corotuine

#4

Hard to say with the amount of code shown, what is the value of delayToEndMenu.

#5

It’s set to 2f, this is the entire code:

public class SceneLoader : MonoBehaviour
{
    [SerializeField] float delayToEndMenu = 2f;

    public void LoadStartMenu()
    {
        SceneManager.LoadScene(0);
    }

    public void LoadGame()
    {
        SceneManager.LoadScene(1);
    }

public IEnumerator LoadEndMenu()
    {
        Debug.Log("A");

        yield return new WaitForSeconds(delayToEndMenu);

        Debug.Log("B");

        SceneManager.LoadScene(2);
    }

    public void QuitGame()
    {
        Application.Quit();
    }
}
#6

Your delayToEndMenu variable is exposed in your Inspector. Check its value there as the Inspector overrides the value in your code. Increase it for testing purposes, maybe to 5 or 10 in the Inspector.

#7

may also be that the game object the script is attached to is being destroyed befor the script finishes running.

#8

It was set to 2, I changed it to 5 and nothing happened still

#9

The object that it’s attached to isn’t getting destroyed.

#10

Can try doing something more simple.

Example:

float delay = 5f;
bool playerDead = false;

// Update is called once per frame 
void Update()
{ 
    if (playerDead == true)
    {
        delay -= Time.deltaTime;
 
        if (delay <= 0) 
        {
            // Load level code here.
        } 
    }
}

Sorry if its messy, on my phone.

#15

How are you getting on with this, @Haisp?

#16

I ended up following Rick’s method, but I still don’t know why my method isn’t working.

#17

Your SceneLoader looks fine. Maybe the problem is/was that LoadEndMenu was not called via StartCoroutine but directly. Or something else called one of your other public methods.

#21

Change the “StartCoroutine(sceneLoader.LoadEndMenu());” to

image
image1092×80 7.24 KB

#23

This topic was automatically closed after 14 days. New replies are no longer allowed.

Privacy & Terms