Unity: NullReferenceException: Object reference not set to an instance of an object в чём ошибка?

Question

[WalloutDay]

Выбивает ошибку. Уже даже и не знаю что делать. Скрипты если что висят на разных объектах. Первый на зомби, второй на игроке. Надо запустить с скрипта зомби функцию TakeDamage в скрипте игрока. Вот где ошибка.

А вот и два скрипта.
1-Скрипт зомби:

using UnityEngine;
using System.Collections;

public class ZombieAttack : MonoBehaviour
{

    public int zombieCol = 0;
    float timer;
    public float timeBetweenAttacks = 0.5f;
    public int attackDamage;
    HealthScript enemyHealth;
    HealthCharacter playerHealth;

    void Awake()
    {

        playerHealth = GetComponent<HealthCharacter>();
        enemyHealth = GetComponent<HealthScript>();

    }
    
    void OnTriggerEnter2D(Collider2D col)
    {

        if (col.gameObject.name == "Character")
        {
            zombieCol = 1;
        }

    }

    void OnTriggerExit2D(Collider2D col)
    {

        if (col.gameObject.name == "Character")
        {
            zombieCol = 0;
        }

    }

    void Update()
    {

        if (zombieCol == 1)
        {

            ZombieController move = GetComponent<ZombieController>();
            move.playerSpeedForOtherScript = 0;

            timer += Time.deltaTime;

            if (timer > timeBetweenAttacks && enemyHealth.HP > 0)
            {

                Attack();

            }
        }
    }

    void Attack()
    {

        timer = 0f;
        
        if (playerHealth.HP > 0)
        {

            playerHealth.TakeDamage(attackDamage);

        }

    }

}

2-Скрипт персонажа:

using UnityEngine;
using System.Collections;

public class HealthCharacter : MonoBehaviour
{

    public float HP;
    public bool isEnemy = true;
    public UISprite healthSprite;
    public float attackZombieRadi;
    public float timeBetweenAttack;

    float timer;
    public bool zombieAttack;

    void Awake()
    {

       healthSprite.fillAmount = 1;

    }

    void Update()
    {

        timer += Time.deltaTime;

        healthSprite.fillAmount = HP / 100;
        
        if (HP <= 0)
        {

            healthSprite.fillAmount = 0f;
            Destroy(gameObject);

        }
        
    }

    public void TakeDamage (int amount)
    {

        HP -= amount;

    }
}

Answer 1

[Алексей Павлов]

Я думаю, проблема как раз в том, что у вас разные компоненты у разных объектов - у зомби пытаетесь получить доступ к объекту скрипта жизней, но этот скрипт у персонажа, а не у зомби, поэтому playerHealth = GetComponent(); даст null.

Вы неправильно пытаетесь связываться с разными объектами. В одном объекте не нужно иметь ссылку на объект другого персонажа. А как же тогда уменьшить жизни? Через событие - у вас есть OnTriggerEnter2D - вот в нём есть ссылка на тот объект, с которым только что столкнулся, вот в нём и нужно выбивать жизни:

void OnTriggerEnter2D(Collider2D col)
{
    if (col.gameObject.name == "Character")
    {
        var pers = col.gameObject.GetComponent<HealthCharacter>();
        if (pers != null)
        {
            pers.TakeDamage(attackDamage);
        }
    }
}

И не надо проверять объект по его имени (я имею в виду код if (col.gameObject.name == "Character") ), в Юнити есть прекрасная штука - теги. Создайте тег Enemy и тег Hero (или как у вас там персонаж называется), и вот по ним проверяйте - с ним столкнулся, или нет.

Comments

[Алексей Павлов]

а, и ещё - вы пишете код в методе void Awake(). В нём ещё нет других объектов и других компонентов этого же объекта. В Awake можно писать код для этого объекта (или к статическим методам других объектов). А если вы хотите получить к другому объекту - то делайте код в Start (если вообще это нужно).

[WalloutDay]

Алексей Павлов: Я вас понял, но проблема в том что тогда зомби "укусит" человека только один раз, а мне надо чтобы он постепенно, понемногу его кусал. При этом я не могу писать такой вариант как у вас т.к. он вызывается только при входе триггера в триггер. Тоесть можно считать что это стоило написать в апдэйте, но там col уже не работает.

[Алексей Павлов]

WalloutDay: Возьмите OnTriggerStay2D вместо OnTriggerEnter2D, он будет вызываться каждый кадр. Поставьте только, пора ли кусать (сделать простейший таймер, уже прошло время кусания, или нет).

[WalloutDay]

Алексей Павлов: Поставил. Но он вызывается только первую секунду. Потом просто стоит. Т.к. поставил счётчик и число росло только при первой секунде и при выходе.
using UnityEngine;
using System.Collections;

public class ZombieAttack : MonoBehaviour
{

public int zombieCol = 0;
float timer;
public float timeBetweenAttacks = 0.5f;
public int attackDamage;
HealthScript enemyHealth;
HealthCharacter playerHealth;
public int i = 0;

void Awake()
{

playerHealth = GetComponent();
enemyHealth = GetComponent();

}

void OnTriggerStay2D(Collider2D col)
{

if (col.gameObject.name == "Character")
{
zombieCol = 1;
i++;
var pers = col.gameObject.GetComponent();

if (pers != null && zombieCol == 1 && timer > timeBetweenAttacks && enemyHealth.HP > 0)
{

pers.TakeDamage(attackDamage);
timer = 0f;

}
}

}

void OnTriggerExit2D(Collider2D col)
{

if (col.gameObject.name == "Character")
{
zombieCol = 0;
}

}

void Update()
{

if (zombieCol == 1)
{

ZombieController move = GetComponent();
move.playerSpeedForOtherScript = 0;

timer += Time.deltaTime;

}
}
}

[iundersun]

ставьте флаг zombieCol = 1 в методе:
void OnTriggerEnter2D(Collider2D collider){}
обнуляте zombieCol в методе:
void OnTriggerExit2D(Collider2D collider){}
обработку(удар по времени) делайте в методе void Update() если пришло время и zombieCol все еще = 1

[iundersun]

что-то вроде этого: ideone.com/1NWaRu

[Pparma]

WalloutDay:
Если вы вызываете GetComponent без указания GameObject'а, подразумевается что вы вызываете его у текущего GameObject.
Если вы хотите получить компонент другого GameObject, то, соответственно надо делать так:
someGameObject.GetComponent