エラーメッセージの意味が分かりません

Question

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

実際のコード
using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class myscript : MonoBehaviour
{
GameObject[] cubes = new GameObject[4];
GameObject[] gos = new GameObject[4];

// Use this for initialization
void Start()
{
for(int i = 0;i < 4; i++)
    {
        cubes[i] = GameObject.Find("Cube" + i);
        gos[i] = GameObject.Find("GameObject" + i);
    }
}


// Update is called once per frame
void Update()
{
   Rigidbody rigidbody = GetComponent<Rigidbody>(); 
    foreach(GameObject obj in cubes){
        obj.transform.Rotate(new Vector3(1f, 1f, 1f));
    }

    Vector3 v = transform.position;
    v.y += 2;
    v.z -= 7;
    Camera.main.transform.position = v;　　　←この部分がダメ見たいです

   
    if (Input.GetKey(KeyCode.LeftArrow))
    {
        rigidbody.AddForce(new Vector3(-1f, 0f, 0f));
    }
    if (Input.GetKey(KeyCode.RightArrow))
    {
        rigidbody.AddForce(new Vector3(1f, 0f, 0f));
    }
    if (Input.GetKey(KeyCode.UpArrow))
    {
        rigidbody.AddForce(new Vector3(0f, 0f, 1f));
    }
    if (Input.GetKey(KeyCode.DownArrow))
    {
        rigidbody.AddForce(new Vector3(0f, 0, -1f));
    }
}

private void OnTriggerEnter(Collider collider)
{
    if (collider.gameObject.name.StartsWith("Cube"))
    {
      for(int i = 0; i < 4; i++)
        {
            if(cubes[i] == collider.gameObject)
            {
                ParticleSystem ps = gos[i].GetComponent<ParticleSystem>();
                ps.Play();
                cubes[i].SetActive(false);

            }
        }
    }
}

}

Comments

[Orlofsky]

コードは https://teratail.com/help#about-markdown の[コードを入力]を使ってください。

Answer 1

####エラーメッセージの意味について

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

myscript.Update ()メソッドの33行目でNullReferenceExceptionという例外が発生しているということです。

この回答で分からない単語があれば、1個1個以下のワードから検索してみてください。

• メソッド
• NullReferenceException
• Null
• 例外(Exception)
• 参照(Reference)

####実際の問題点

Camera.main.transform.position = v;　　　←この部分がダメ見たいです

ここがmyscript.csの33行目を指しているからだと推測します。

この中で、CameraかmainかtransformかpositionかvがNullだということかもしれません。

ただ、
vはVector3 v = transform.position;で通っており、
Camera.main.transform.positionのpositionはまだmyscript.cs内のどこにも使われていないため、Nullでも問題ないと思います。

なので、残った
CameraかmainかtransformのどれかがNullであることが原因です。
確認してみてください。

また、何かわからないことがあれば質問してください。

Comments

[Bongo]

BluOxyさんのアドバイスに加えまして、怪しい点が絞り込めましたら、やはり本家リファレンスも参照してみるのがいいかと思います。

Camera.main - Unity スクリプトリファレンス（https://docs.unity3d.com/jp/current/ScriptReference/Camera-main.html）によれば...

Camera.main

public static Camera main ;

説明
"MainCamera" にタグ付けされている最初の有効なカメラ(読み取り専用)。

The primary Camera in the scene. Returns null if there is no such camera in the scene. This property uses FindGameObjectsWithTag internally and doesn't cache the result. It is advised to cache the return value of Camera.main if it is used multiple times per frame.

...との記述があります。タグを「MainCamera」に設定されたカメラがシーン内に存在しているかどうかはご確認済みでしょうか?

Answer 2

「Cameraって何だよ！」って言っていますね。
CameraというGameObjectを探しているのならば、こうすることで大丈夫だと思います。

//省略
GameObject camera;
void Start()
{
for(int i = 0;i < 4; i++)
    {
        cubes[i] = GameObject.Find("Cube" + i);
        gos[i] = GameObject.Find("GameObject" + i);
    }
camera = GameObject.Find("Camera");
}
//v.z…まで省略
Camera.transform.position = v;

省略しすぎて、すみません。

Answer 3

結論として、以下のコードで実行可能です。

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class myscript : MonoBehaviour
{
	GameObject[] cubes = new GameObject[4];
	GameObject[] gos = new GameObject[4];

	Camera _camera;//カメラ
	Rigidbody _rigidbody;//Rigidbody

	// Use this for initialization
	void Start()
	{
		_camera = Camera.main; //キャッシュすれば書き込み可能

		_rigidbody = GetComponent<Rigidbody>();// キャッシュする
		
		for(int i = 0;i < 4; i++)
		{
			cubes[i] = GameObject.Find("Cube" + i);
			gos[i] = GameObject.Find("GameObject" + i);
		}	
	}

	// Update is called once per frame
	void Update()
	{
		//Rigidbody rigidbody = GetComponent<Rigidbody>();  これは負荷が高くなるのでStart()内でキャッシュする。
		foreach(GameObject obj in cubes){
			obj.transform.Rotate(new Vector3(1f, 1f, 1f));
		}

		Vector3 v = transform.position;
		v.y += 2;
		v.z -= 7;
		// Camera.main.transform.position = v; こうではなく...

                _camera.transform.position = v;//こうすればカメラが移動する。

		//以下、割愛
	}

}

公式リファレンスには、

"MainCamera" にタグ付けされている最初の有効なカメラ(読み取り専用)。

と書いてあります。
NullReferenceExceptionが起きたのは、Camera.mainが読み取り専用だったためです。
そのためCamera.main.transform.positionを変更しようとするとこのエラーが出ます。
読み取り専用の変数は、一度別の変数に代入することで操作できるようになるので、覚えておくと良いですよ。

また、ついでですが、GetComponentをUpdate内でいちいちしてると重くなるので、クラス変数を用意して、Start内でGetComponentすると良いですよ。

Answer 4

よくわからないエラーが出たときは、
エラーメッセージをコピーしてgoogle検索。
「NullReferenceException: Object reference not set to an instance of an object」
このとき、そのソース特有の文言は省いた方がいいです。（myscript.Update ()とか）
それでたいてい解決します。

Answer 5

nullに対して何かをやろうとした時に出る例外です。

Answer 6

Null Reference Exception
エラーメッセージの意味を知りたいという質問ですね？
ここを見てください。