Как исправить ошибку NullReferenceException: Object reference not set to an instance of an object?

Question

[soks_of_fate]

на камере есть код:

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

public class user : MonoBehaviour
{
    public float Distens = 0f;
    void Update()
    {
        if (Input.GetKeyDown(KeyCode.E))
        {
            Ray ray=new Ray (transform.position, transform.forward);
            RaycastHit hit;
            if(Physics.Raycast(ray, out hit, Distens))
            {
                if (hit.collider.CompareTag("door"))
                {
                    hit.collider.transform.parent.GetComponent<DoorScript>().count_v();
                }
            }
        }
    }
}

и на предмете:

using System.Collections;
using System.Collections.Generic;
using Unity.VisualScripting;
using UnityEngine;

public class DoorScript : MonoBehaviour
{
    private int count = 3;

    public void count_v()
    {
        count--;
        print(count);
        count--;
    }
}

но при звзаимодейстии возникает ошибка
NullReferenceException: Object reference not set to an instance of an object
user.Update () (at Assets/my/script/user.cs:19)

Возможно я сильно туплю, но вроде все делал по ютубу

Answer 1

[GavriKos]

Надо делать не по ютубу, а по уму.
99% что не туда повесили скрипт DoorScript, его не находит в объекте столкновения и все.

Comments

[soks_of_fate]

неа, скрипт висит просто на кубике. Делал по ютубу, так как учусь с него

[GavriKos]

soks_of_fate, так спрашивайте на этом своем ютубе, капец.

hit.collider.transform.parent. - вот судя по этому коду должен висеть не на кубике.

Answer 2

[Nik Faraday]

Ошибка NullReferenceException возникает, когда у вас какой-то объект, к которому вы обращаетесь, равен null. Почему он null? Потому что вы где-то не вызвали конструктор (Ключевое слово new)

Comments

[GavriKos]

Если не знаете - зачем лезете?
Ну во первых null может быть не только из за невызванного конструктора, а банально из за не переданной зависимости.
Во вторых - в юнити НЕЛЬЗЯ вызывать конструктор для MonoBehaviour.
Вредные советы

[Nik Faraday]

GavriKos, серьёзно? А зависимость конструктор не вызывает? Там объект сам материализуется, или его туда арабы заносят?

Причём тут MonoBehaviour? Там же автор вопроса написал ниже об ошибке:

NullReferenceException: Object reference not set to an instance of an object
user.Update () (at Assets/my/script/user.cs:19)

Где тут вам указан MonoBehaviour?

Вы не знаете, то не лезьте, умники нашлись. Ошибка NullReferenceException возникает во время кривой зависимости? Вот ошибка DI: "Could not create an instance of type '...' "

Если не знаете - зачем лезете?

....

[GavriKos]

Nik Faraday, Нет, зависимость не всегда вызывает конструктор - зависимость можно просто передать.

И в 19 строке четко указан монобех - DoorScript. GetComponent вернул null потому что не нашел скрипт, а дальше к нему пошло обращение. И никаким new вы это не решите.

И да, в юнити объект "материализуется" - например через инспектор задается.

Answer 3

[Мимокрокодил]

Ошибка заключается в том, что в этой строке:

hit.collider.transform.parent.GetComponent<DoorScript>().count_v();

Вы пытаетесь вызвать родительский объект у объекта, который не имеет родительского объекта. В таком случае получаете null. Если вы хотите обратиться к корневому объекту, используйте свойство root. Типа:

hit.collider.transform.root.GetComponent<DoorScript>().count_v();

Что, к слову, можно сократить до

hit.transform.root.GetComponent<DoorScript>().count_v();

Если же у вас есть вероятность, что скрипта DoorScript вообще не будет на объекте с данным тегом, то от него проще отказаться:

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

public class user : MonoBehaviour {

	public float Distens = 0f;

	void Update() {
		if (Input.GetKeyDown(KeyCode.E)) {
			Ray ray = new Ray (transform.position, transform.forward);
			RaycastHit hit;
			if (Physics.Raycast(ray, out hit, Distens))
				if (hit.transform.root.TryGetComponent(out DoorScript door))
					door.count_v();
		}
	}
}