一つのスクリプトに複数のオブジェクトの制御をさせたい

前提・実現したいこと

まだUnity初心者で、今軽いアクション系のFPSを作っています。
シューティング要素を入れておりRayで操作しています。敵として、ロボットを作成し、ヘッドショット、ボディショットとして分けています。ヘッドとボディでColliderを分け、それぞれにスクリプトを入れて、ヘッドボディを分割させています。

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

一つのロボットが倒れると他のも同じプログラムで動かしているので同時に倒れてしまいます。

該当のソースコード

ロボットのダメージ管理のスクリプト

C#
1public class RobotDamage : MonoBehaviour
2{
3    public static float currentHealth = 3f;
4    public GameObject thisBot;
5    private void Update()
6    {
7        if (currentHealth <= 0)
8        {
9            thisBot.SetActive(false);
10        }
11    }
12}

ロボットヘッドコライダのスクリプト

C#
1public class RobotHead : MonoBehaviour
2{
3    public void Damage(float damageAmount)
4    {
5        RobotDamage.currentHealth -= damageAmount;
6
7    }
8}

ロボットボディコライダのスクリプト

public class RobotBody : MonoBehaviour
{
    public void Damage(float damageAmount)
    {
        RobotDamage.currentHealth -= damageAmount;
    }
}

Rayのスクリプト

       if(Physics.Raycast(rayOrigin,fpsCam.transform.forward,out hit, weaponRange))
       {
           laserLine.SetPosition(1, hit.point);

           RobotHead head = hit.collider.GetComponent<RobotHead>();

           RobotBody body = hit.collider.GetComponent<RobotBody>();
           if(head != null)
           {
               head.Damage (headDamage);
           }
           if(head != null && hit.rigidbody != null)
           {
               hit.rigidbody.AddForce(-hit.normal * hitForce);
           }
           if (body != null)
           {
               body.Damage(bodyDamage);
           }
           if (body != null && hit.rigidbody != null)
           {
               hit.rigidbody.AddForce(-hit.normal * hitForce);
           }
       }

試したこと

一つ一つのロボットにつきスクリプトを与えましたが、数が多くなってきて、これだと重くなってしまうと思って質問しました。

行動規範の内容に同意します

回答1件

ベストアンサー

こんにちは。
コードについてですが、staticが悪さしてるのだと思います。
これをつけた変数はかんたんに言うならすべてのオブジェクトで共有されてしまい、一体が攻撃されると全員のHPが減ります。

なのでstaticを外したあとRobotDamageクラスをそれぞれのオブジェクトのDamageクラス他スクリプトたちにGetComponentでインスタンスを作らせて、そこから被弾ごとに引いていくようにすればいいと思います。

下のコードみたいな感じで…

C#
1public class RobotBody : MonoBehaviour
2{
3　　　　　RobotDamage rd;
4　　　　　//インスタンスを入れる箱の変数
5void Start()
6     {
7        rd = GetComponent<RobotDamage>();
8        //RobotDamageのインスタンス作成
9     }
10    public void Damage(float damageAmount)
11    {
12        rb.currentHealth -= damageAmount;
13        //インスタンスから情報取得して処理
14　　　　//rb.currentHealthはRobotDamageクラスのrbに含まれるcurrentHealthの意味
15    }
16}

しかしこれはおそらく最善ではありません。
もう少し学習が進んだらScriptableObjectというものを調べてみてください。
敵パラメータはこちらで管理するのがいいです。
しかし少し込み入ったモノなのでもう少し学習が進んでからでいいでしょう。

これで解決すればよいのですが……。
なんにせよ応援しております！

追記：質問文にある一つ一つのスクリプトを与えるという場合が今自分が書いたようなコードだとお思いならすみません。
しかしこの程度の数ならおそらく大丈夫です。
ですがやはり気になるということでしたら、そのような懸念を払拭するためのものがScriptableObjectです。
データの量をかなり節約しつつ個別に管理できます。
こちらの記事など参考になりますゆえ調べてみてください。
意図を取り違えていたらすみませんでした。
エクスポラボ様の　【Unity】ScriptableObjectってなんなん? って時に読む記事【解説】

投稿2020/09/08 14:38

編集2020/09/08 15:04