GameOverシーンへの移行

Question

前提・実現したいこと

PlayerのHP(fillAmount)が０になった瞬間にPlayerのgameObjectを破壊し、GameOverシーンに移行させたい。

発生している問題・エラーメッセージ

Playerの体力(fillAmount)が０になってもGameOverシーンに移行せず、体力０のままゲームが続行された。
エラーメッセージ
表示されない

該当のソースコード

ソースコード
using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.UI;
using UnityEngine.SceneManagement;

public class GameDirector : MonoBehaviour
{ 
    public GameObject hpGauge;
    float fillAmount;
    void Start()
    {
     this.hpGauge = GameObject.Find("hpGauge");
     fillAmount = 1.0f;
    }

    public void DecreaseHp()
    {
        this.hpGauge.GetComponent<Image>().fillAmount -= 0.1f;
        if(fillAmount <= 0)
        {
            Destroy(gameObject);
            Invoke("GoToGameOver", 1.0f);
        }
    }
     void GoToGameOver()
     {
      SceneManager.LoadScene("GameOver");
     }
}

試したこと

GameOverシーンを作成して、ネットの指示通りにBuildsettingsにて２つのシーンを設定した。しかしGameOverシーンに移行することはなかった。

補足情報（FW/ツールのバージョンなど）

Unity 2019.2.14f1 Personal

Comments

[yutampp]

アメリカでは、あなたがネットに指示を出す。
ソビエトロシアでは、ネットがあなたを指示する！
言いたかっただけです。

this.hpGauge.GetComponent<Image>().fillAmount -= 0.1f;
if(fillAmount <= 0)

この２行が気になりました。
すぐ上の行ではthis.hpGauge.GetComponent<Image>().fillAmountとしていて、
その一行下で、fillAmountとしている理由はなに？

[Neet-Saikou]

すいません、初心者なもので自分でも曖昧なんです
自分の解釈としましてはこのhpGaugeのfillAmountをGetComponentにて数値化させた、と考えているのですが…
もし間違っていましたら、ご教授していただけると幸いです。

[simapiko]

横から失礼します。あくまでGetComponentで参照しているのは、"ImageコンポーネントのfillAmount"です。ifの中で比較しているfillAmountは"GameDirectorクラスで宣言したfillAmount"なので、名前は同じでもその２つは別の物です。

「this.hpGauge.GetComponent<Image>().fillAmount -= 0.1f;」で行っているのは前者のImageコンポーネントのfillAmountに対してなので、後者であるif内の変数fillAmountの値は変化することは無く、同時にif条件も満たすことは無いという感じでしょうか。

参考にImageコンポーネントのリファレンスを下記に貼っておきます。Imageコンポーネントが持っている変数もそこで確認出来るかと思います。

https://docs.unity3d.com/ja/2018.4/ScriptReference/UI.Image.html

[Neet-Saikou]

ありがとうございます！

Answer 1

GameDirectorのfillAmount変数が変更されていないことが原因ではないでしょうか。

fillAmountはあくまで画像表示の埋め度合いを示す単語なので、HPを管理しているなら変数名もHPとしたほうが管理しやすいかもしれません。

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

public class GameDirector : MonoBehaviour
{
    public GameObject hpGauge;

    public int MaxHP = 10;
    int HP;

    void Start()
    {
        this.hpGauge = GameObject.Find("hpGauge");

        HP = MaxHP;
    }

    public void DecreaseHp()
    {
        HP -= 1;
        var currentHPPercentage = Mathf.Clamp01(HP / (float)MaxHP);
        this.hpGauge.GetComponent<Image>().fillAmount = currentHPPercentage;
        if (HP <= 0)
        {
            Destroy(gameObject);
            Invoke("GoToGameOver", 1.0f);
        }
    }
    
    void GoToGameOver()
    {
        SceneManager.LoadScene("GameOver");
    }
}

HPというデータ概念が「主」であって、ゲージ画像をどう表示するのかは「従」と考えてコードを整理すると読みやすくなると思います。

Comments

[Neet-Saikou]

ありがとうございます。
しかしGameOverシーンには何故か移行しませんでした…
また色々試してみます。

[tor4kichi]

・GameOverシーンがビルドに含まれてるかチェックしてみてください
https://www.sawalemontea.com/entry/2017/08/14/Unity-Hajimete-Script-5-Scene-Jump

・「Destroy(gameObject);」が呼ばれると以降の処理が実行されない？ ようです。
https://teratail.com/questions/93115

GoToGameOverメソッドのLoadSceneの後に「Destroy(gameObject);」を配置することも試してみてはどうでしょうか。

[Neet-Saikou]

Destroyのコードを消せば移行することに成功しました！
ありがとうございました！