Listに、生成されたPrefabの初期座標を代入する方法。

Question

前提・実現したいこと

UnityC#についての相談です。
家と学校に見立てた四角形（順にHome,School）を、人に見立てたボール(Human)が行き来するというスクリプトを目指しています。
具体的には、以下のようなものを目指しています。

・HumanとHomeのプレハブを1から始まる連番で、どちらも同じ座標に生成。
・Vector3型のListであるhuman、homeを宣言し、生成されたHumanとHomeの初期座標をそれぞれ代入（human[0]にはHuman1の座標のように）
・Humanは、左クリックの後、Humanと同じ連番を振ったHome（Human1ならHome1というように）からSchoolの座標に向かう。
・その後、右クリックで、今度は同じ連番を振ったHomeの座標に戻ってくる。

しかし、色々試してみたものの、どのようにして初期座標を代入するか、悩んでいます。
どうか知恵をお貸しいただけませんでしょうか。
よろしくお願いいたします。
今までのスクリプトは以下の通りです。

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

NullReferenceException: Object reference not set to an instance of an object
HumanMover.Update () (at Assets/HumanMover.cs:36)

該当のソースコード

＜ObjectMaker.cs＞＝＝＝＝＝＝＝＝＝＝
using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class ObjectMaker : MonoBehaviour {

    public GameObject Home;
    public GameObject Human;

    public GameObject HomeClone;
    public GameObject HumanClone;

    public int homenumber = 1;
    public int humannumber = 1;
    
	void Update () {

        if (Input.GetMouseButtonDown(0))
        {
            GameObject HomeClone = Instantiate(Home, new Vector3(Random.Range(40.0f, 45.0f), 0.5f, Random.Range(40.0f, 45.0f)), Quaternion.identity);
            GameObject HumanClone = Instantiate(Human, HomeClone.transform.position, Quaternion.identity);

            HomeClone.name = "Home" + homenumber;
            HumanClone.name = "Human" + humannumber;

            homenumber += 1;
            humannumber += 1;
        }
	}
}
＜HumanMover.cs＞＝＝＝＝＝＝＝＝＝＝
using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class HumanMover : MonoBehaviour
{

    public GameObject ObjectMaker;

    GameObject Human;

    GameObject Home;

    GameObject School;

    public List<Vector3> human = new List<Vector3>();
    public List<Vector3> home = new List<Vector3>();

    int arraynumber = 0;

    void Update()
    {
        if (Input.GetMouseButtonDown(0))
        {
            GameObject ObjectMaker = GameObject.Find("ObjectMaker");

            int humannumber = ObjectMaker.GetComponent<ObjectMaker>().humannumber;
            int homenumber = ObjectMaker.GetComponent<ObjectMaker>().homenumber;

            human[arraynumber] = GameObject.Find("Human" + humannumber).transform.position;
            home[arraynumber] = GameObject.Find("Home" + homenumber).transform.position;

            Vector3 school = GameObject.Find("School").transform.position;

            transform.localPosition = Vector3.MoveTowards(human[arraynumber], school, Time.deltaTime * 3.0f);

        }

        if (Input.GetMouseButtonDown(1))
        {
            transform.localPosition = Vector3.MoveTowards(human[arraynumber], home[arraynumber], Time.deltaTime * 3.0f);
        }

        arraynumber += 1;
    }
}
＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝

試したこと

human、homeをListとして宣言して、その配列にGameObject.Find関数で探したHuman、Homeの初期座標を代入しようとしました。

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

Visual Studio 2017
Unity 2018.1.0b1

Comments

[退会済みユーザー]

コードは ``` と ``` で囲ってください。インデントされて見やすくなるので。

インデントされてないコードは質問者さんも読む気がしないのでは？　回答者・閲覧者はなおさらです。読んでもらえないと話が始まらないのですから、読んでもらえる努力をしましょう。

[Kamine]

大変申し訳ございません。確認がおろそかになっておりました。先ほど修正いたしました。

[simapiko]

いまいち何が問題なのか読み取れていないのですが、エラー文を見るとHumanMover.csの36行目で、取得出来ていないObjectの値を使おうとしています。該当行で指定しているオブジェクトが存在しているかどうか、スペルミスも含め今一度確認してみるといいかと思います。

[Kamine]

大変申し訳ございません。こちらのコメントに気付かずベストアンサーを選んでしまいました。確認してみます。ありがとうございました。

[simapiko]

内容が被ってしまった上に、時間もこちらの方が後でしたのでむしろ、こちらこそ申し訳ございませんでした。解決法含めsakura_hanaさんの回答が間違いなくBAなのでお気になさらず。

[Kamine]

かしこまりました。

Answer 1

まずはこちらをご覧ください。
Null Reference Exception - Unity マニュアル

HumanMoverの36行目にある変数のどれかがnullなので、
Debug.Log(human); Debug.Log(arraynumber); Debug.Log(human[arraynumber]); Debug.Log(school);
などとして、何がnullなのか確認後、対策を行ってください。

---

ただし今回のコード、問題の部分も含めて、そもそもHumanMoverの大半の処理が不要だと思います。
（その上、今のままだとクリック時にObjectMakerの処理より前にHumanMoverの処理が実行される可能性がある）

説明が面倒臭いのでコードで書きますが、例えばこんな感じでいけるかと思います。
（コンパイル通してないので細かい誤字脱字あったら適宜修正してください）

＜ObjectMaker.cs＞＝＝＝＝＝＝＝＝＝＝
using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class ObjectMaker : MonoBehaviour {
    // 必要そうな型で宣言する（GameObject型でなくてもInstantiate時はGameObjectが生成される）
    // インスペクタからプレハブをドラッグ＆ドロップでセットは同様
    public Transform Home;
    public HumanMover Human;

    // もしリストが必要ならここで持つべき（作り手＝管理者 という考え方をする）
    // 数が欲しかったら humanList.Count で取れる
    public List<Transform> homeList = new List<Transform>();
    public List<HumanMover> humanList = new List<HumanMover>();

    // 学校オブジェクトもこっちで持っておく
    //（とりあえずインスペクタからセットする形にしたが、StartでFindしてももちろんOK）
    public Transform School;

    void Update () {

        if (Input.GetMouseButtonDown(0))
        {
            Transform HomeClone = Instantiate(Home, new Vector3(Random.Range(40.0f, 45.0f), 0.5f, Random.Range(40.0f, 45.0f)), Quaternion.identity);
            HumanMover HumanClone = Instantiate(Human, HomeClone.transform.position, Quaternion.identity);

            // リストに入れて数取って命名する
            homeList.Add(HomeClone);
            humanList.Add(HumanClone);
            HomeClone.name = "Home" + homeList.Count;
            HumanClone.name = "Human" + humanList.Count;

            // Humanに自分のHomeとSchoolを渡してやる（この為HumanCloneはHumanMover型でないといけない）
            HumanClone.Init (HomeClone, School);

            //（ちなみにHomeCloneがTransform型なのは、必要なのはpositionの為。HomeClone.transform.positionよりHomeClone.positionの方が文字数が短い、というだけなので、GameObjectである必要性があるならGameObjectでももちろんOK）
        }
    }
}

＜HumanMover.cs＞＝＝＝＝＝＝＝＝＝＝
using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class HumanMover : MonoBehaviour
{
    // 目的地だけあればいい
    Transform Home;
    Transform School;

    // 学校へ向かっているかフラグ
    bool isGotoSchool;

    // 生成時に呼ぶ初期設定メソッド
    public void Init (Transform _home, Transform _school) {
        Home = _home;
        School = _school;
        isGotoSchool = true;
    }

    void Update()
    {
        // 目的地が未セットなら何もしない（Initより早くUpdateが実行されるとNullReferenceExceptionが出る為）
        if (Home == null || School == null) return;

        // if(Input.~) の中に transform.localPosition = ~ を入れてしまうと、クリックした時だけしか移動しない。
        // 多分そういう想定ではないと思うので、移動のコードはif文の外に出し、if文では目的地がどっちかだけ決める。
        if (Input.GetMouseButtonDown(0))
        {
            isGotoSchool = true;
        }
        if (Input.GetMouseButtonDown(1))
        {
            isGotoSchool = false;
        }

        // localPositionでいいのかは要確認（自分とHomeとSchoolの親が違うと多分ずれる）。
        transform.localPosition = Vector3.MoveTowards(transform.localPosition, (isGotoSchool) ? School.position : Home.position, Time.deltaTime * 3.0f);

        // ちなみに (isGotoSchool) ? School.position : Home.position ←こいつは
        //「isGotoSchoolがtrueならSchool.position、falseならHome.position」となる。
        //「三項演算子」というので詳しくはググってください。
    }
}
＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝

Comments

[Kamine]

ご丁寧なコードまで書いて下さり、誠にありがとうございます。精進いたします。