Как можно сократить код в OnTriggerEnter2D?

Question

[WasTabon]

private bool _isPlayerInside;

    private void Update() 
    {
        Press();
    }

    private void OnTriggerEnter2D(Collider2D coll) 
    {
        if (coll.gameObject.CompareTag("Player"))
        {
            _isPlayerInside= true;
        }
    }
    private void OnTriggerExit2D(Collider2D coll) 
    {
        if (coll.gameObject.CompareTag("Player"))
        {
            _isPlayerInside= false;
        }
    }

    private void Press()
    {
        if (_playerInput.Player.E.WasPressedThisFrame() && _isPlayerInside)
            {
                Debug.Log("Pressed");
            }
    }

Нужно чтобы если игрок вошел в триггер и нажал на кнопку, то метод сработал.
Если просто метод Press() закинуть в OnTriggerEnter2D то ничего не будет (что логично, потому что он срабатывает только при входе)
Пробовал ещё Stay, но он работает когда игрок внутри триггера передвигается

Answer 1

[Alex Maximovich]

_isPlayerInside = coll.gameObject.CompareTag("Player");

Вопрос только зачем? Какие задачи это решит?)

Comments

[K0TlK]

Если так сделать, у него все сломается, когда в триггер зайдет объект, у которого есть тэг, а следом за ним объект, у которого этого тэга нет. В его случае лучше вовсе ничего не менять.

[Alex Maximovich]

Про отсутсвие тэга не подумал)

[WasTabon]

Я имел ввиду костыли, типо что там булевую переменную меняю, там тоже меняю, потом в каком-то методе идет проверка на неё, а вызов метода вообще в Update

Answer 2

[K0TlK]

Убрать фигурные скобочки. А если использовать TryGetComponent, то еще больше сократишь.

private void OnTriggerEnter2D(Collider2D coll) 
    {
        if (coll.gameObject.CompareTag("Player"))
        {
            _isPlayerInside= true;
        }
    }

==>

private void OnTriggerEnter2D(Collider2D coll) 
    {
        if (coll.TryGetComponent(out Player player) _isPlayerInside = true;
    }

А можно пойти дальше и вообще всё в одну строку уместить.

private void OnTriggerEnter2D(Collider2D coll) { if (coll.TryGetComponent(out Player player)) _isPlayerInside = true; }

Ну как? Читабельно? Не нужно всё пытаться сократить. У тебя был читабельный метод из четырех строк, но я, путем сокращения, превратил его в нечитабельное месиво символов. Сокращать нужно, когда у тебя явно есть повтор кода или в одном методе столько строк, что, чтобы разобраться как он работает, тебе приходится делать комментарии в этом методе.

Comments

[WasTabon]

Я имел ввиду костыли, типо что там булевую переменную меняю, там тоже меняю, потом в каком-то методе идет проверка на неё, а вызов метода вообще в Update

[K0TlK]

WasTabon, Ты хочешь сделать какой-то интерактивный предмет? Чтобы, когда игрок находится рядом с ним, он мог взаимодействовать с ним? Тогда инпут тебе здесь вообще не нужен. Выделяешь интерфейс с методом Interact, реализуешь его в каком-нибудь абстрактном монобехе, впихиваешь свой код, наследники этого монобеха будут перезаписывать то, что вызывается при взаимодействии.

Что-то типа этого:

using UnityEngine;

namespace Interaction
{
    [RequireComponent(typeof(Collider2D))]
    public abstract class InteractiveItem<TComponent> : MonoBehaviour, IInteractive
    where TComponent : class
    {
        protected TComponent Component { get; private set; }
        
        private void Awake()
        {
            GetComponent<Collider2D>().isTrigger = true;
        }

        public void Interact()
        {
            if (Component != null) InteractInternal();
        }

        protected abstract void InteractInternal();

        private void OnTriggerEnter2D(Collider2D other)
        {
            if (other.TryGetComponent(out TComponent character))
            {
                Component = character;
            }
        }

        private void OnTriggerExit2D(Collider2D other)
        {
            if (other.TryGetComponent(out TComponent character))
            {
                Component = null;
            }
        }
    }
}

А инпут это вообще другое. Человек нажимает кнопку, скрипт, который обрабатывает инпут, говорит, что кнопка нажата, вызывается команда и эта команда уже говорит объекту персонажа попытаться поднять предмет. OnTriggerEnter здесь нужен только лишь, чтобы разрешить зависимость предмета от игрока. Ты либо делаешь это, либо делаешь так, чтобы игрок сам определял с чем он хочет взаимодействовать и раскидываешь все зависимости при инициализации игры.

Answer 3

[FireDrago19]

Не понял суть темы, но если вся суть в том что написано под скринами..

Нужно чтобы если игрок вошел в триггер и нажал на кнопку, то метод сработал.
Если просто метод Press() закинуть в OnTriggerEnter2D то ничего не будет (что логично, потому что он срабатывает только при входе)
Пробовал ещё Stay, но он работает когда игрок внутри триггера передвигается

Можно в Update проверять на _isPlayerInside, и в случае есть оно истинно, то вызывать Press()