Как сделать проверку нахождения объекта в триггере?

Question

[a]

День добрый! Суть вопроса: У меня есть кнопка, и если войти в её триггер она активируется(OnButtonStay()), а если выйти, собственно, деактивируется(OnButtonExit()), но если объект войдёт в её триггер и затем исчезнет(SetActive(false)), то кнопка останется активной(OnButtonStay()). Так вот, как сделать так, чтобы скрипт проверял нахождение объекта в триггере, и в случае его исчезновения или пропажи, просто деактивировал кнопку(OnButtonExit()).
Буду очень благодарен за помощь!

using UnityEngine;

public class ButtonTrigger : MonoBehaviour
{

    [SerializeField] private Animator _anim;
    [SerializeField] private MeshRenderer _buttonbase;
    [SerializeField] private Texture _materialOff;
    [SerializeField] private Texture _materialOn;

    [SerializeField] private AudioSource _onSound;
    [SerializeField] private AudioSource _offSound;

    private bool _isDown;

    private void Start()
    {
        _anim.SetBool("isDown", !_isDown);
    }

    private void OnTriggerStay(Collider other)
    {
        if (other)
        {
            OnButtonStay();
        }
    }

    private void OnTriggerExit(Collider other)
    {
        if (other)
        {
            OnButtonExit();
        }
    }

    private void OnButtonStay()
    {
        _anim.SetBool("isDown", _isDown);

        _buttonbase.material.mainTexture = _materialOn;
    }

    private void OnButtonExit()
    {
        _anim.SetBool("isDown", !_isDown);
        _buttonbase.material.mainTexture = _materialOff;
    }

Comments

[K0TlK]

Насколько мне известно, если объект отключить во время нахождения в триггере, то вызывается OnTriggerExit, соответственно ваш код должен работать. Зачем проверки if(other)? Если OnTrigger... вызывается, то он уже подразумевает под собой то, что в other что-то есть, поэтому null оно не будет равно. Зачем вам поле _isDown? Оно используется для хранения false? Не вижу, чтобы вы его где-то в коде изменяли или использовали как-то, оно и не нужно вам, вместо _isDown и !_isDown используйте просто true false. Используйте Debug.Log или дебаггер, чтобы определить, вызывается ли метод OnButtonExit корректно.

[a]

K0TlK, Я бы не просил помощи если при отключении объекта он бы вызывал метод OnTriggerExit. Debug.Log использовал, и как следовало ожидать - он пишет в консоль только после того как объект выйдет из триггера, а не после отключение этого же самого объекта. Но всё равно, Спасибо за подсказку с if(other) и true false вместо _isDown.

[K0TlK]

a, Странно, в 2д все работает, в 3д нет. Нагуглил пару похожих вопросов:
https://stackoverflow.com/questions/41716663/detec...,
https://forum.unity.com/threads/fix-ontriggerexit-...

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

[a]

K0TlK, Спасибо за ответ! Я сделал по другому, так сказать костыль: создал скрипт, который повесил на куб (ниже будет пояснение, если, конечно, хотите посмотреть на самый костыльный костыль из всех которые существуют в интернете)
Ещё раз, спасибо за вашу помощь и за то, что подметили у меня некоторые ошибки в скрипте. Всего хорошего!

using UnityEngine;

public class KostylScript : MonoBehaviour
{
    public ButtonTrigger _button;
    public bool _isPressed;

    public void ButtonOff()
    {
        if (_isPressed)
        {
            _button.OnButtonExit();
            Debug.Log("Jest");
        }
    }
}

на него ссылается скрипт кнопки:

using UnityEngine;
using UnityEngine.Events;

public class ButtonTrigger : MonoBehaviour
{

    [SerializeField] private Animator _anim;
    [SerializeField] private MeshRenderer _buttonbase;
    [SerializeField] private Texture _materialOff;
    [SerializeField] private Texture _materialOn;

    [SerializeField] private ButtonTrigger _base;

    [SerializeField] private AudioSource _onSound;
    [SerializeField] private AudioSource _offSound;

    public UnityEvent _onPressed;
    public UnityEvent _onReleased;

    private void OnTriggerStay(Collider other)
    {
        OnButtonStay();

        if(other.tag == "Cube")
        {
            other.GetComponent<KostylScript>()._isPressed = true;
            other.GetComponent<KostylScript>()._button = _base;
            OnButtonStay();
        }
    }

    private void OnTriggerExit(Collider other)
    {
        OnButtonExit();
    }

    private void OnButtonStay()
    {
        _anim.SetBool("isDown", false);
        _onPressed.Invoke();
        _buttonbase.material.mainTexture = _materialOn;
    }

    public void OnButtonExit()
    {
        _anim.SetBool("isDown", true);
        _buttonbase.material.mainTexture = _materialOff;
        _onReleased.Invoke();
        Debug.Log("Exit");
    }

И уже в скрипте который выключает куб написал:

hit.collider.transform.GetComponent<KostylScript>().OnButtonExit();
hit.collider.transform.GetComponent<KostylScript>()._isPressed = false;

[K0TlK]

a, Что-то вы перекостылили. Гораздо проще выделить интерфейс, который будут реализовывать все отключаемые объекты и работать через него посредством событий. Что-то типа этого:

using System;
using UnityEngine;

namespace Assets.Scripts
{
    public class ObjectDisabler : MonoBehaviour, IDeactivatable
    {
        public event Action Deactivated;
        public event Action Activated;


        public void Activate()
        {
            gameObject.SetActive(true);
            Activated?.Invoke();
        }

        public void Deactivate()
        {
            gameObject.SetActive(false);
            Deactivated?.Invoke();
        }
    }

    public interface IDeactivatable
    {
        event Action Deactivated;
        event Action Activated;
        void Activate();
        void Deactivate();
    }
}

ObjectDisabler - это объект, который вы выключаете, а не какой-то отдельный объект, названия нормальные у меня нет желания придумывать.
Таким образом, нам не нужно получать целый объект, чтобы вызвать у него метод SetActive, тем самым нарушив инкапсуляцию. Мы просто получаем интерфейс и через него работаем:

using UnityEngine;

namespace Assets.Scripts
{
    public class TriggerTest : MonoBehaviour
    {
        private IDeactivatable _deactivatable;
        private void OnTriggerEnter(Collider other)
        {
            if (other.TryGetComponent(out IDeactivatable deactivatable))
            {
                _deactivatable = deactivatable;
                _deactivatable.Deactivated += ExitTrigger;
            }
        }

        private void OnTriggerStay(Collider other)
        {
            Debug.Log("TriggerStay");
        }

        private void OnTriggerExit(Collider other)
        {
            ExitTrigger();
        }

        private void ExitTrigger()
        {
            Debug.Log("TriggerExit");
            _deactivatable.Deactivated -= ExitTrigger;
        }
    }
}

Интерфейс IDeactivatable можно (и нужно) разделить на 2 разных, чтобы были отдельно ивенты и отдельно методы. Теперь все объекты, которые нужно отключать извне, должны реализовывать этот интерфейс и делать в них то, что нужно, чтобы отключить объект. Таким образом, не будет каких-то неожиданных отключений через SetActive и состояние объекта не будет ломаться. Также, я вижу, что вы делаете проверки по тэгу, этого делать не следует, т.к. это проверки по строковому литералу и они ненадежны (можно ошибиться и написать вместо Cube - cube и т.д и потом сидеть искать где же ошибка), гораздо надежнее проверять по типу через GetComponent. Также вы в OnTriggerStay делаете GetComponent вместо того, чтобы просто получить компонент единожды в OnTriggerEnter и закешировать его, это может вызвать проблемы с производительностью, если объектов будет очень много на сцене. Также вам, по всей видимости, абсолютно не нужен метод OnTriggerStay, т.к. все, что там делается не нужно делать пока объект находится в триггере, замените его на OnTriggerEnter. (OnTriggerEnter вызывается единожды, при входе в триггер, OnTriggerStay много раз пока объект находится в триггере, OnTriggerExit единожды при выходе из триггера)

[a]

K0TlK, хоть я и продолжу использовать свой костыль (мои нынешние навыки программирования способствуют понимать и работать только с кодом, который мною написанный), но я всё же ценю ваш труд и время, которые вы вложили в решение этого вопроса в качестве рекомендаций, поправок ошибок, написания кода и небольшого мануала к нему. Опять же, спасибо вам огромное за всё, что вы сделали в этой ветке! Всего доброго!!!