スクリプトをアクティブにする

Question

unityでゲームを作ろうと思っています。
ボールを動かし、そのボールから最も近い距離にあるキューブの色を変更したいです。
色を変更するスクリプト(colorc)はキューブにアタッチしており、ボールを動かして最も距離が近くなった際にそのスクリプトをアクティブにしようと思っています。(シーン開始時そのスクリプトは非アクティブ)
以下を実行したのですが、Object reference not set to an instance of an objectとエラーが出てしまいます。
何が原因でしょうか？

using UnityEngine;
using System.Collections;


    public class Distance_detection : MonoBehaviour
    {
        private GameObject nearObj;
        private float searchTime = 0;

        // Use this for initialization
        void Start()
        {

            nearObj = serchTag(gameObject, "Player");
        }

        // Update is called once per frame
        void Update()
        {
            searchTime += Time.deltaTime;
            if (searchTime >= 1.0f)
            {
                nearObj = serchTag(gameObject, "Player");
                searchTime = 0;
            }
            //nearObj = GameObject.Find("Sphere").GetComponent<Renderer>();
            //nearObj.material.color = Color.yellow;
            //nearObj.transform.LookAt(nearObj.transform);
            //nearObj.transform.Translate(Vector3.forward * 0.01f);
             nearObj.GetComponent<colorc>().enabled = true;

        }

        GameObject serchTag(GameObject nowObj, string Player)
        {
            float tmpDis = 0;
            float nearDis = 0;
            GameObject targetObj = null;


            foreach (GameObject obs in GameObject.FindGameObjectsWithTag(Player))
            {
                tmpDis = Vector3.Distance(obs.transform.position, nowObj.transform.position);
                if (nearDis == 0 || nearDis > tmpDis)
                {
                    nearDis = tmpDis;
                    targetObj = obs;
                }

            }

            return targetObj;
        }
    }

Answer 1

Object reference not set to an instance of an objectとエラーが出てしまいます。何が原因でしょうか？

おそらくnullチェックしていないのが原因です。
GetComponent<colorc>()をしてもnearObjにnullが代入された場合、オブジェクトが存在していないのでエラーが出るということだと思います。

	void Update()
	{
		searchTime += Time.deltaTime;
		if (searchTime >= 1.0f)
		{
			nearObj = serchTag(gameObject, "Player");
			searchTime = 0;

			//nearObjが存在するなら
			if (nearObj != null)
			{
				//nearObj = GameObject.Find("Sphere").GetComponent<Renderer>();
				//nearObj.material.color = Color.yellow;
				//nearObj.transform.LookAt(nearObj.transform);
				//nearObj.transform.Translate(Vector3.forward * 0.01f);
				nearObj.GetComponent<colorc>().enabled = true;
			}
		}
	}

nullが存在する言語は常にnullを意識する必要があります。
nullはクラッシュの原因になるのでできる限り初期値を設定するのがおすすめです。設定できないところは必ずnullチェックをするといいと思います。

//数値は初期値が0なので初期化の必要はありません
int SumNum;
//stringは初期値がnullなので初期化
string SumTxt = string.Empty;
//配列もできれば初期化
string[] SumAry = new string[5];
List<string> SumAry2 = new List<string>();

###おまけ
Updateは毎フレーム実行されるので少々高コストです。OO秒毎の実行でしたらCoroutineを使うといいと思います。Coroutineの注意点はゲームオブジェクトが非アクティブの時に実行するとエラーが出るので注意が必要です。

//ループ間隔をfloatで指定
static readonly float LOOP_INTERVAL = 1f;
Coroutine m_distanceCorutine;

void Start ()
{
	//処理のスタート
	StartDistanceCorutine ();
}

void StartDistanceCorutine ()
{
        StopDistanceCorutine ();
	m_distanceCorutine = StartCoroutine ( DistanceCorutine() );
}

//ストップしたいときはこの関数を呼ぶ
void StopDistanceCorutine ()
{
	//nullの可能性があるのでnullチェック
	if (m_distanceCorutine != null) StopCoroutine ( m_distanceCorutine );
}

IEnumerator DistanceCorutine()
{
	var waitTime = new WaitForSeconds( LOOP_INTERVAL );
	while (true)
	{
		yield return waitTime;

		//LOOP_INTERVAL毎にここに記述した処理が実行されます

	}
}

###(追記) 最も近いオブジェクトとそれ以外を分ける

※コードは未検証です。挙動がおかしければご自分で修正してみてください。

//serchTagが返すオブジェクトを定義
struct serchCallBackObj
{
	public GameObject neartObj;
	public GameObject[] otherObj;
}

void Start ()
{
	serchCallBackObj objs = serchTag(gameObject, "Player");

	//objs.neartObjで近いオブジェクトを取得
	//objs.otherObjでそれ以外のオブジェクトが配列で格納
}

serchCallBackObj serchTag(GameObject nowObj, string Player)
{
	float tmpDis = 0;
	float nearDis = 0;

	serchCallBackObj callBackOnj = new serchCallBackObj();
	callBackOnj.otherObj = GameObject.FindGameObjectsWithTag(Player);
	
	if (callBackOnj.otherObj == null
	   || callBackOnj.otherObj != null && callBackOnj.otherObj.Length == 0) return callBackOnj;
	
	int neartObjIdx = 0;

	for (int i = 0; i < callBackOnj.otherObj.Length; i++)
	{
		tmpDis = Vector3.Distance(callBackOnj.otherObj[i].transform.position, nowObj.transform.position);
		if (nearDis == 0 || nearDis > tmpDis)
		{
			nearDis = tmpDis;
			callBackOnj.neartObj = callBackOnj.otherObj[i];
			neartObjIdx = i;
		}
	}

	//一番近い要素を削除
	System.Array.Clear(callBackOnj.otherObj, neartObjIdx, 1);

	return callBackOnj;
}

Comments

[tyint121]

ありがとうございます！
解決できました！
もし、最も近いオブジェクトとそれ以外の処理を分けるといった場合はどういった方法がありますでしょうか？教えていただけましたら幸いです。

[IShix]

追記したので見てみてください。あと、次からでいいので質問する際はご自分でやってみてできなかったソースを提示するようにしてください。それと、今回2重質問になっているので次からは質問が変わる場合は別の質問として投稿してください。

[tyint121]

ありがとうございます。
以後気をつけます。