Как сделать уничтожение пули при столкновении в Unity?

Question

[NickNovicov]

Добрый вечер! Скажите, где в коде пули:

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


public class Bullet : MonoBehaviour
{

    public float speed = 100f;
    public Rigidbody2D rb;
    public GameObject bullet;
    // Start is called before the first frame update
    void Start()
    {     
        rb.velocity = transform.right*speed;
    }
    void OnTriggerEnter2D(Collider2D collider)
    {
        if(collider.gameObject.name != "Player")
        {
            Destroy(gameObject);
        }
    }

Что-то сделано неправильно? При запуске игры выдаёт ошибку:
The object of type 'GameObject' has been destroyed but you are still trying to access it.
Your script should either check if it is null or you should not destroy the object.
В уроках в интернете такой-же код.

Comments

[50Dreams]

NickNovicov, как уже отметил Homo Ludens, Destroy и Instantiate ресурсоёмкий процесс, да и тем более создавать и прикреплять класс к патронам в сцене является плохой затеей. Стоит только создать миниган с темпом стрельбы ≈66 патронов в секунду и вы убьете всю производительность игры. Поэтому лучше создать Object Pool.
(О том как создать простой Object Pool было сказано -> https://habr.com/ru/post/275091/).

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

void OnTriggerEnter(Collider col) {
	if (col.gameObject.layer == "_BULLETS") {
		/* Ваш код по возвращению игрового объекта в пул */
	}
}

[NickNovicov]

Добрый день ещё раз. Спасибо всем за ответы. Я всё исправил, моей проблемой было то, что я не добавил пулю в префабы

Answer 1

[Homo Ludens]

Что бы не было подобных глюков нужно не уничтожать объект, а помечать как уничтоженный.
Ну а потом, уже после отработки всех столкновений, или уничтожать его или откладывать в пул для переиспользования .

Answer 2

[xmoonlight]

1. Попробуйте проверить наличие объекта до обращения к его свойству:

if(gameObject && collider.gameObject.name != "Player")
        {
            Destroy(gameObject);
        }

2. Если в момент столкновения пули идёт выстрел, то объект пули не уничтожается, а помещается перед стволом - меняются только координаты.
Как бы, образуется цепной конвейер, что позволяет сильно снизить нагрузку клиента при быстром создании и удалении объектов.

Comments

[Homo Ludens]

проверять наличие это конечно хорошо... но может получится некрасиво.
допустим у нас встретились юнит с последним хитом и пуля.
проверяем юнита- он потерял последний хит и исчез
проверяем пулю- а она ни с чем не столкнулась поскольку юнита больше нет.
т.е. пуля убила и летит дальше.
При обратном порядке проверки получим бессмертных юнитов, поскольку пуля будет исчезать до того как они вычтут хиты.

[xmoonlight]

Homo Ludens, с таким подходом, как в вашем примере, лучше вообще к программированию не допускать.