同オブジェクトの他スクリプト内の変数をうまく取得できているかわからない。

Question

実現したいこと

同じオブジェクトの他スクリプトの変数を取得したい。

前提

C#、Unity歴２ヶ月未満。初心者。
Unityを使い、C#を使用しています。同じオブジェクトの他スクリプトの変数を取得したいのですがうまくいきません。

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

”Beam"というタグが衝突するごとにHpが－１されていくメソッドを使用し、Hpを更新しています。他スクリプトからHp変数を取得しているのですがOncollisionEnter()メソッド内で更新している値を返さないと意味がないということでしょうか？自分は変数の初期値を取得しているということなのでしょうか？

エラーメッセージは特にないです。

該当のソースコード

コード
using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.UI;

public class PlayerHitpoint : MonoBehaviour
{

    int maxHp = 10;
    public int Hp;

    public Slider slider;
    // Start is called before the first frame update
    void Start()
    {
        slider.value = 1;
        Hp = maxHp;
    }

    // Update is called once per frame
    void Update()
    {
        
    }

    void OnCollisionEnter(Collision collision)
    {
        if(collision.gameObject.tag == "Beam")
        {
            Hp = Hp - 1;
            slider.value = (float)Hp / (float)maxHp;
            
        }
    }
}


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

public class GameOver : MonoBehaviour
{
    public Text gameOverMessage;
    PlayerHitpoint HitPoint;
    float PlayerHp;


    // Start is called before the first frame update
    void Start()
    {
        PlayerHitpoint HitPoint = GetComponent<PlayerHitpoint>();
        float PlayerHp = HitPoint.Hp;
    }

    // Update is called once per frame
    void Update()
    {
        if(PlayerHp < -0.1)
        {
            gameOverMessage.text = "Game Over";
            Time.timeScale = 0f;
        }
    }

    
}


### 試したこと
PlayerHitpointクラスのOncollisionEnter()メソッド内でreturn HpとしましたがVoidで宣言しているためできずです。更新している値をreturnで返すという考え方は正解なのでしょうか。

Answer 1

やりたいこととしては、PlayerのHPが0になったら自動的にゲームオーバーになるようにしたいという感じでしょうかね？

自分は変数の初期値を取得しているということなのでしょうか？

現在の書き方だとそうですね。
float PlayerHpはStartの中で呼ばれて以降、一度も更新されません。
Updateの中でプレイヤーのリアルタイムのHPを見たければ、HitPoint.Hpを見る必要があります。

余談1

PlayerHpがオブジェクトをチェックしてダメージを計算する仕組みはおすすめしません。

    void OnCollisionEnter(Collision collision)
    {
        if(collision.gameObject.tag == "Beam")
        {

なぜなら障害物やオブジェクトを増やすたびにプレイヤーのif文を増やして処理を書かなかればいけないからです。
PlayerHpにvoid Damge(int damageAmount)といった、基本的なHP増減メソッドを定義し、ダメージを与えるオブジェクト側（今回だとBeam側）がPlayerHpを呼び出すといいです。
つまりこうです。

Beam.cs

    private void OnTriggerEnter(Collider other)
    {
        PlayerHp playerHp = other.GetComponent<PlayerHp>();
        if (playerHp)
        {
            playerHp.Damage(1);
        }
    }

最初の書き方では、PlayerHpはダメージを与えるオブジェクトを全て知らなければならないですが、
こちらの書き方は ダメージを与えるオブジェクトがPlayerHpだけを知っていればいい ので、オブジェクトが増えたときにコードがすっきり分かりやすくなります。

余談2

特定の値になったときに処理を呼び出すようにしたいのであれば、デリゲートを使うといい感じです。
UnityにはUnityEventという仕組みがあるので、今回のケースだと以下のように書けます。
（コードをアタッチしたあとに、インスペクターでonDeathにGameOver.OnGameOverを登録する必要があります）

PlayerHp.cs

using UnityEngine;
using UnityEngine.Events;

public class PlayerHp : MonoBehaviour
{
    private int maxHp = 10;
    public int Hp;
    public UnityEvent onDeath;

    void Start()
    {
        Hp = maxHp;
    }

    public void Damage(int damageAmount)
    {
        Hp -= damageAmount;

        if (Hp <= 0) // 死んだときに
        {
            Hp = 0;
            onDeath.Invoke(); // インスペクターで登録した他クラスの死亡時に呼び出したいメソッドを呼ぶ
        }
    }
}

GameOver.cs

using UnityEngine;

public class GameOver : MonoBehaviour
{
    public void OnGameOvver()
    {
        Debug.Log("Game Over!");
    }
}

余談1、2のコードを合わせると、

1. OnTriggerEnterでBeamがPlayerHpにダメージを与える
2. PlayerHpがHpを計算する
3. 計算した結果、Hp =< 0だったらGameOverを呼び出す

という処理の流れになり、Updateで常時監視する必要がなくなり、Hp =< 0の瞬間だけゲームオーバーが呼び出されるようになります。

Answer 2

    void Start()
    {
        PlayerHitpoint HitPoint = GetComponent<PlayerHitpoint>();
        float PlayerHp = HitPoint.Hp;
    }

このfloat PlayerHp = HitPoint.Hp;は変数を宣言して代入しています。
変数の宣言や代入に関しての文法、変数のスコープについて学習されることをお勧めします。

Comments

[wagato]

すみません、申し訳ないのですがその文だけですと具体的な解決方法がわかりません。

[YAmaGNZ]

まず
float PlayerHp = HitPoint.Hp;
これが何をしているのか理解されていますか？

[wagato]

PlayerHitpointクラスのHp変数をGameOverクラスのPlayerHp変数に代入しています！！

[YAmaGNZ]

違います。
float型のPlayerHpという名前の変数をStartメソッド内で定義し、その定義した変数にHitPoint.Hpを代入しています。
新しく宣言しているのでGameOverクラスのローカルで宣言しているfloat PlayerHp;とは別のものとなります。

こういった変数の宣言や代入、変数のスコープに関してはプログラムを作成するうえで基本的なこととなりますので「変数の宣言や代入に関しての文法、変数のスコープについて学習されることをお勧めします」と回答しました。

[wagato]

なるほど！！ありがとうございます！！