Unity: ошибка «NullReferenceException» при взаимодействии с объектом. В чём может быть причина?

Question

[Дмитрий Кулаков]

Задача: игра от 1-го лица. Управление персонажем по джойстикам, пишется под смартфон. Персонаж должен подойти к объекту, увидеть кнопки и брать / бросать объект по тапу на кнопки.

Алгоритм:
1) Игрок (Player) подходит к объекту, с которым нужно взаимодействовать - мяч (Ball)
2) Игрок вступает в коллайдер, дочерний мячу – активируются UI-кнопки подбора (grabButton) и броска (throwButton) мяча:

private void OnTriggerExit(Collider other)
    {
        if (other.gameObject.name == "Player")
        {
            grabButton.SetActive(false);
            throwButton.SetActive(false);
        }
    }

3) При нажатии кнопки «Взять» - активируется метод «Grab»:

public void Grab()
    {
        _rb.useGravity = false;
        _rb.isKinematic = true;
        _rb.MovePosition(_arm.position);
        _fpc.take = true;
    }

4) При нажатии кнопки «Бросить» - активируется метод «Throw»:

public void Throw()
    {
        _rb.AddForce(Camera.main.transform.forward * 800);
        _rb.useGravity = true;
        _rb.isKinematic = false;
        _fpc.take = false;
        //_throw.throwAbility = true;
    }

Вопрос: при нажатии на Throw – ошибка:
NullReferenceException: Object reference not set to an instance of an object
ColliderGrab.Throw () (at Assets/ColliderGrab.cs:48)

Пробовал и метод OnClick, и через if (Input.Getkey) – результат тот же. Пожалуйста, подскажите, в чём проблема. Весь скрипт висит на коллайдере-триггере, дочернем к мячу.

P.S.: я начинающий и это мой первый вопрос, сори за возможные недочёты. Заранее спасибо!

Код целиком:

using UnityEngine;

public class ColliderGrab : MonoBehaviour
{
    public GameObject grabButton;
    public GameObject throwButton;

    public Rigidbody _rb;
    private FpcParam _fpc;
    public Transform _arm;

    private void Start()
    {
        _rb = GameObject.Find("Ball").GetComponent<Rigidbody>();
        _fpc = GameObject.Find("Player").GetComponent<FpcParam>();
    }

    private void OnTriggerStay(Collider other)
    {
        if (other.gameObject.name == "Player")
        {
            grabButton.SetActive (true);
            throwButton.SetActive(true);
        }
    }

    private void OnTriggerExit(Collider other)
    {
        if (other.gameObject.name == "Player")
        {
            grabButton.SetActive(false);
            throwButton.SetActive(false);
        }
    }

    public void Grab()
    {
        _rb.useGravity = false;
        _rb.isKinematic = true;
        _rb.MovePosition(_arm.position);
        _fpc.take = true;
    }

    public void Throw()
    {
        _rb.AddForce(Camera.main.transform.forward * 800);
        _rb.useGravity = true;
        _rb.isKinematic = false;
        _fpc.take = false;
    }

    private void FixedUpdate()
    {
        if (_rb.isKinematic == true)
        {
            _rb.gameObject.transform.position = _arm.position;
        }
    }
}

Comments

[#]

как совет - спойлеры для кода не помешали бы.. читать и так уже не плохо.. а со спойлерами проще обдумывать ;)

Answer 1

[Nik Faraday]

public Rigidbody _rb;
    private FpcParam _fpc;
    public Transform _arm;

    private void Start()
    {
        _rb = GameObject.Find("Ball").GetComponent<Rigidbody>();
        _fpc = GameObject.Find("Player").GetComponent<FpcParam>();
    }

public void Throw()
    {
        _rb.AddForce(Camera.main.transform.forward * 800);
        _rb.useGravity = true;
        _rb.isKinematic = false;
        _fpc.take = false;
    }

Таки инициализация на 10 строке:
public Transform _arm;

Объявление переменной это когда вы в коде говорите, что есть такая переменная, что бы в будущем к ней обратиться и работать с ней.

Инициализация переменно это когда вы даёте переменное первоначальное значение. Для ссылочных типов (Классы и т.д.) стандартное значение это null. Когда вы вызываете конструктор к классу либо передаёте ему ссылку (Через конструктор) такого же типа, ваша переменная уже считается инициализиваронной

Ссылочные классы так называются, поскольку они не хранят в себе информацию о внутренним состоянии, а ссылаются на фрагмент памяти, где хранится эта информация. Стандартное значение null указывается, что ссылка существует, но она никуда не указывается (Аналогично, когда вы в поисковике переходите по ссылке на страницу, которой не существует).

Comments

[Дмитрий Кулаков]

Спасибо за ответ!

Поразмышлял над вашим ответом и понял, что стоит "уточнить" камеру, от которой идёт бросок:

spoiler

public Camera _cam;

    public void Throw()
    {
        _rb.useGravity = true;
        _rb.isKinematic = false;
        _fpc.take = false;
        _rb.AddForce(_cam.transform.forward * 1000);
    }

Всё заработало, ещё раз - спасибо!

[Nik Faraday]

Дмитрий Кулаков, пожалуйста) Но тут проблема больше не в самом Unity, а в вашем понимании языка C#, потому что в представленном коде я так и не увидел инициализацию переменной _arm, с которой у вас реально была проблема

Answer 2

[Сергей Соловьев]

Не вижу инициализации _arm

Comments

[Дмитрий Кулаков]

Сергей, спасибо за ответ!

Таки инициализация на 10 строке:
public Transform _arm;

[Сергей Соловьев]

Дмитрий Кулаков, это называется объявление. Инициализация - это когда значение ей присваивают

[Дмитрий Кулаков]

Сергей Соловьев, да, спасибо за подсказку!

Поступил с ней так:

private Transform _arm;

    private void Start()
    {
        _arm = GameObject.Find("Arm").GetComponent<Transform>();
    }

Ошибка осталась, на той же строке (

[Сергей Соловьев]

Дмитрий Кулаков, остальные инициализации оставил? или остальные удалил?

[Дмитрий Кулаков]

Сергей Соловьев, остальное не трогал.

[Сергей Соловьев]

Дмитрий Кулаков, ну тогда дебажить надо

[Дмитрий Кулаков]

Сергей Соловьев, да уже всё везде посмотрел, все мной обозримые варианты попробовал

[Сергей Соловьев]

Дмитрий Кулаков, а что null в итоге-то?

[Дмитрий Кулаков]

Сергей Соловьев, оказалось, что камера. Я её "пояснил" - и всё заработало!

public Camera _cam;

    public void Throw()
    {
        _rb.useGravity = true;
        _rb.isKinematic = false;
        _fpc.take = false;
        _rb.AddForce(_cam.transform.forward * 1000);
    }