Confused with something...

Unity Engine
1

After some experimenting I’ve stumbled on a minor bug that’s confusing me. I’ve no idea why it’s an issue, but basically, each level has a Screen Fader object in the Hierarchy. It’s essentially a canvas, which has my Screen Fader script on and has a Black Screen child object. When each scene loads, a coroutine is called and fades the screen to black for a second and then unfades. If I deactivate one of them in a level via the Inspector, pause the game, select Restart (restarts the entire game), the screen remains black and has an error in the console saying ‘object reference not set to an instance of an object’. Clicking on this highlights line 49 in my PauseMenu script, which is where it tries to call the Screen Fader’s coroutine. I don’t understand why this is a problem. As I say, each scene has its own Screen Fader object set up, the black screen is ‘on’ by default (apart from one scene), so surely once the level is destroyed, that coroutine will be called from the first level’s own Screen Fader…?

These are the two scripts:

PauseMenu:

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.SceneManagement;

public class PauseMenu : MonoBehaviour
{
    public static bool GamePaused;
    public GameObject pauseMenuUI;

    private ScreenFader theScreenFader;

    void Start()
    {
        theScreenFader = FindObjectOfType<ScreenFader>();
        ControllerDetection();
        GamePaused = false;
    }

    void Update()
    {
        if (Input.GetKeyDown(KeyCode.Escape) || Input.GetKeyDown("joystick button 7") || Input.GetKeyDown("joystick button 9"))
        {
            if (GamePaused)
            {
                Resume();
            }
            else
            {
                Pause();
            }
        }
    }

    public void Resume()
    {
        pauseMenuUI.SetActive(false);
        Time.timeScale = 1f;
        GamePaused = false;
        //theScreenFader.blackScreen.enabled = true;
        PlayerController.canMove = true;
    }

    public void Restart()
    {
        SceneManager.LoadScene("start_area");
        Debug.Log("level loaded");
        pauseMenuUI.SetActive(false);
        Debug.Log("pause menu disabled");
        ScreenFader.black = true;
        Debug.Log("screen is black");
        theScreenFader.StartCoroutine("ScreenFade");
        Time.timeScale = 1f;
    }

    public void Pause()
    {
        Cursor.visible = true;
        pauseMenuUI.SetActive(true);
        Time.timeScale = 0f;
        PlayerController.canMove = false;
        GamePaused = true;
        //theScreenFader.blackScreen.enabled = false;
    }

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

    public void ControllerDetection()
    {
        string[] names = Input.GetJoystickNames();
        for (int x = 0; x < names.Length; x++)
        {
            //print(names[x].Length);
            if (names[x].Length == 19)
            {
                //print("PS4 CONTROLLER IS CONNECTED");
                SceneManagement.ps4Controller = 1;
                SceneManagement.xbox360Controller = 0;
                if (SceneManagement.ps4Controller == 1)
                {
                    //Debug.Log("PS4 controller detected");
                }
            }
            else if (names[x].Length == 33)
            {
                //print("XBOX 360 CONTROLLER IS CONNECTED");
                SceneManagement.ps4Controller = 0;
                SceneManagement.xbox360Controller = 1;
                if (SceneManagement.xbox360Controller == 1)
                {
                    //Debug.Log("Xbox 360 controller detected");
                }
            }
            else
            {
                SceneManagement.ps4Controller = 0;
                SceneManagement.xbox360Controller = 0;
            }

            if (SceneManagement.xbox360Controller == 0 && SceneManagement.ps4Controller == 0)
            {
                //Debug.Log("No controllers detected");
            }
        }
    }
}

ScreenFader:

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.UI;
using UnityEngine.SceneManagement;

public class ScreenFader : MonoBehaviour
{
    public Image blackScreen;
    public float fadeSpeed;
    public float waitForFade;
    //public static ScreenFader instance;
    public static bool black;

    private bool isFadeToBlack;
    private bool isFadeFromBlack;
    //private bool isBlack;

    /*void Awake()
    {
        if (instance != null)
        {
            Destroy(gameObject);
        }
        else
        {
            instance = this;
            DontDestroyOnLoad(gameObject);
        }
    }*/

    void Start()
    {
        StartCoroutine("ScreenFade");
    }

    IEnumerator ScreenFade()
    {
        if (SceneManager.GetActiveScene() == SceneManager.GetSceneByName("level 1, room 1"))
        {
            isFadeToBlack = false;
        }
        else if (black)
        {
            Debug.Log("black");
            yield return new WaitForSeconds(fadeSpeed);
            isFadeToBlack = true;
            yield return new WaitForSeconds(waitForFade);
            isFadeFromBlack = true;
            yield return new WaitForSeconds(2f);
        }
        else
        {
            yield return new WaitForSeconds(fadeSpeed);
            isFadeToBlack = true;
            yield return new WaitForSeconds(waitForFade);
            isFadeFromBlack = true;
            yield return new WaitForSeconds(2f);
        }
    }

    void Update()
    {
        if (isFadeToBlack)
        {
            blackScreen.color = new Color(blackScreen.color.r, blackScreen.color.g, blackScreen.color.b, Mathf.MoveTowards(blackScreen.color.a, 1f, fadeSpeed * Time.deltaTime));
            if (blackScreen.color.a == 1f)
            {
                isFadeToBlack = false;
            }
        }
        if (isFadeFromBlack)
        {
            blackScreen.color = new Color(blackScreen.color.r, blackScreen.color.g, blackScreen.color.b, Mathf.MoveTowards(blackScreen.color.a, 0f, fadeSpeed * Time.deltaTime));
            if (blackScreen.color.a == 0f)
            {
                isFadeFromBlack = false;
            }
        }
        if (black)
        {
            blackScreen.color = new Color(blackScreen.color.r, blackScreen.color.g, blackScreen.color.b);
            if (blackScreen.color.a == 1f)
            {
                black = false;
            }
        }
    }
}
2

Deactivating the object as in toggling the GameObject off?

FindObjectOfType only checks components on active objects. The components you’re looking for can be enabled or disabled, which doesn’t matter for this particular type of query, but the GO must be active, activeInHierarchy to be more exact.

3

Yeah, the GameObject off. But again, surely if a scene has its own GameObject and it’s on, it would use/find that…?

4

Have you checked whether the object is active or inactive when you enter the scene again?

5

If I turn it off before hitting Play, it remains off. If I pause the game and select restart, that scene is destroyed, along with its Screen Fader, and the first level loads up. If I select the Screen Fader in the first level, it’s enabled and the screen is black. If I set it to clear, the scene is visible, but the game is paused, there’s no pause menu, and that error comes up.

6

So follow-up question: How often does the pause menu component exist? Is there only one instance of it and you carry it over to the other scenes by marking it to stay alive? Or does it exist in every scene?

If you carried it over, Start wouldn’t be called again. Then you’d keep the old reference, which is - in the context of the engine - no longer a valid object even though its managed part still exists.

But, unless something has changed recently, if you were to keep that destroyed reference, you’d get a different exception. That one used to say something along the meaning of you’re trying to access an object that no longer exists (sort of, not the exact message).

So actually, I believe you have multiple PauseMenu instances and then there should only be two cases in which you get ‘null’ and the corresponding exception + message:

  1. Start is not called, which means either the PauseMenu instance is disabled, or its GameObject is
  2. There is no such component that it attempts to find.

You said that there is the component, and that it is active, so I believe we’re getting closer if we take another look at the #1.

Potential bug:
Since the pause menu GO most-likely holds the object with the PauseMenu component, and the pause menu is likely disabled at the beginning, there’s probably no chance the attached components can run their Start method before the Restart method is called.
Then, you’d not have the reference for the screen fader right in time.

Try to replace the reference access with the FindObjectOfType method, or link the screen fader via the inspector. Check what happens.

7

I did have it carry over in every scene, but then decided to have it exist in every scene. I might have it carry over again, we’ll see…

Line 52 of the PauseMenu starts the Coroutine in the ScreenFader. Now, is that calling it independently of the ScreenFader’s Start function, which I’m guessing it does, or is that calling the line from the Start function? If the latter, then that makes sense if the Pause Menu objects are disabled once Restart is initialised.

I mean, this isn’t a major deal or anything, but it’s just something I’d like to try and understand why it’s happening. :slight_smile: