Unity - クラスのインスタンスを呼び出すたび、異なる結果を返してほしい

Question

２つ目のスクリプトからWeaponクラスのインスタンスを複数回呼び出し、リストに格納します。
Weapnクラスの中のweaponPowerが毎回違う値で格納されることを期待していますが、毎回同じになってしまいます。

public class Weapon : MonoBehaviour
{
    public int weaponPower;

    private void Start()
    {
        weaponPower = Random.Range(1, 100 + 1);
    }
}

public class WeaponList : MonoBehaviour
{
    public List<Weapon> weaponDate = new List<Weapon>();

    public void WeaponCreate() //これを呼び出す
    {
        for (int i = 0; i < 5; i++)
        {
            //こう書くとエラーが出て、weaponPowerが0になる
            //weaponDate.Add(new Weapon());

            //objはWeaponスクリプトがついたオブジェクト
            weaponDate.Add(GameObject.Find("obj").GetComponent<Weapon>());
            Debug.Log(weaponDate[i].weaponPower);
        }
    }
}

このように動いてほしい

コンソール画面に56,83,12,98,44のようにランダムな数字が5つ並んでほしい

実際の動作

24,24,24,24,24のように同じ数字が5つ並んでしまう

追記

objは空のオブジェクトでWeaponスクリプトが１つついています。
Unityのバージョンは2020.3.8f1です。

その他

２つ目のスクリプトで
weaponData.Add(new Weapon());　とすると

You are trying to create a MonoBehaviour using the 'new' keyword.
This is not allowed. MonoBehaviours can only be added using AddComponent().
Alternatively, your script can inherit from ScriptableObject or no base class at all

MonoBehaviourを継承したクラスをnewでインスタンス化するなと警告が出ます。
なので、
weaponDate.Add(GameObject.Find("obj").GetComponent<Weapon>());　としました。
(WeaponスクリプトでTransformやGetComponentを使用するのでMonoBehaviourは継承したいです。)

Comments

[ku__ra__ge]

> ここで詰まってしまいました。
どのような動作を想定しており、実際にはどのような動作になってしまったのかを書いてください。

[退会済みユーザー]

修正依頼ありがとうございます。
質問内容をできるだけ簡潔にし、想定する動作と、実際の動作を載せました。

[bboydaisuke]

修正前の内容を知らないと、修正後の内容だけ見ても意味がわからないのでは？

[退会済みユーザー]

修正後の内容だけでも意味がわかるように全てを書き換えました。
新しく質問するべきだったかもしれません。すみません。

[BluOxy]

obj に Weapon オブジェクトはいくつアタッチされていますか

[BluOxy]

また、Unityのバージョンを記載してください

[退会済みユーザー]

objにアタッチされているWeaponオブジェクトは１つだけです。
Unityのバージョンは2020.3.8f1です。
本文にも追記しました。

Answer 1

GameObject.Findはシーン内でゲームオブジェクトの名前を検索し、一致する名前を持つ最初のゲームオブジェクト1つを取得します。
以下のコードは5回ほどWeapon.Start()を実行してweaponPowerが決定している、同じゲームオブジェクトを取得してweaponDateリストに追加してweaponPowerを表示します。
同じオブジェクトのweaponPowerを何度も表示しているため同じ数字が何度も出力されています。

for (int i = 0; i < 5; i++)
{
    weaponDate.Add(GameObject.Find("obj").GetComponent<Weapon>());
    Debug.Log(weaponDate[i].weaponPower);
}

コンソール画面に56,83,12,98,44のようにランダムな数字が5つ並んでほしい

シーン内にWeaponスクリプトをアタッチしたオブジェクトが複数あって、その複数のゲームオブジェクトからそれぞれweaponPowerを取得したい場合、GameObject.FindGameObjectsWithTagなどを利用する方法が考えられます。
GameObject-FindGameObjectsWithTag - Unity スクリプトリファレンス

Comments

[退会済みユーザー]

回答ありがとうございます！
お二方をベストアンサーにしたかったのですが、
僅かではありますが先に答えてくださった方を選ばせていただきました。
申し訳ありません。

[BluOxy]

> シーン内にWeaponスクリプトをアタッチしたオブジェクトが複数あって、その複数のゲームオブジェクトからそれぞれweaponPowerを取得したい場合

上記がもし unipon さんの求めていたものであればこちらの回答が正解ですから、時系列を理由に選ぶのではなく、解決に最も貢献した回答をベストアンサーを選んでください。
（スクリプトを複数貼り付ける方法がまさに求めていたもの とのことなので、今回は違ったようですが）

※ 要件を出来るだけ言語化して質問に記載しておくと、このように回答が何通りもぶれることはなくなりますから、難しいかもしれませんが、今後に活かして頂けると回答する側として嬉しいです

[退会済みユーザー]

アドバイスありがとうございます。
今後質問する際は、気をつけます。

Answer 2

objは空のオブジェクトでWeaponスクリプトが１つついています。

現在は obj に1つだけ Weapon インスタンスが存在する状態なので、そのインスタンスを参照しても同じ weaponPower しか取得できないでしょう。

5つ持たせたい場合は obj に対して Weapon オブジェクトを5つアタッチする必要があります。

そして、複数アタッチされた Weapon オブジェクトは Component.GetComponents で取得することができますから、下記のようなコードを記述して問題が解決するか試してみてください。

public class WeaponList : MonoBehaviour
{
    public List<Weapon> weaponData = new List<Weapon>();

    public void WeaponCreate()
    {
        var weapons = GameObject.Find("obj").GetComponents<Weapon>();
        foreach(var weapon in weapons)
        {
            weaponData.Add(weapon);
            Debug.Log(weapon.weaponPower);
        }
    }
}

仮に同一のオブジェクトを参照したことが原因で同じ乱数が出力されるのであれば、このメソッドを利用することで解決するでしょう。

---

そうではなく Weapon が持つ weaponPower を5回変更したい場合はメソッドを定義して利用します。
MonoBehavior.Start はスクリプトが有効になったときに Unity 側で呼び出される特別なメソッドなので、別途ユーザが利用するためのメソッドを用意する必要があります。

public class Weapon : MonoBehaviour
{
    public int weaponPower;

    private void Start()
    {
        UpdateWeaponPower();
    }

    public void UpdateWeaponPower()
    {
        weaponPower = Random.Range(1, 100 + 1);
    }
}

public class WeaponList : MonoBehaviour
{
    public List<Weapon> weaponDate = new List<Weapon>();

    public void WeaponCreate()
    {
        var weapon = GameObject.Find("obj").GetComponent<Weapon>();
        weaponDate.Add(weapon);
        // 5回処理をループしたいだけなので、i は使用しない
        for (int i = 0; i < 5; i++)
        {
            weapon.UpdateWeaponPower();
            Debug.Log(weapon.weaponPower);
        }
    }
}

Comments

[退会済みユーザー]

回答ありがとうございます！
スクリプトを複数貼り付ける方法がまさに求めていたものなので、それを使おうと思います。
とても助かりました。