Не останавливается корутина. Что нужно дописать чтобы ее остановить?

Question

[deeda]

Суть такая, хотел написать код для получения урона от ловушки, но без использования Time.deltaTime, ибо тот медленный, решил окунуться в мир корутин, так что открыл для себя его недавно, буквально вчера утром.

Все думал как написать код, но все без толку, либо персонаж получает урон один раз в триггере, а потом не получает вовсе потому что цикл я останавливал через "break;", либо персонаж получал урон даже выйдя из триггера, как будто при отравлении, пока хп не кончится, либо при входе в триггер Юнити эдитор зависал напрочь.

Отчаялся так, что попросил помощь в дискорде Russian Gamedev, но там не дали дельного совета, один ответчик посоветовал использовать Time.deltaTime, а второй вовсе не понимал почему мое хп уходит в минус, так что теперь прошу помощи здесь, пожалуйста, помогите с решением данной проблемы, заранее спасибо огромное за ваши ответы, я очень вам благодарен!

Вот данный код:

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

public class TakeDamage : MonoBehaviour
{


    public HealthSystem Hsys;
    public PlayerController PCont;


    private Collider2D collision;


    private bool Trigger;


    public int DamageFromTraps;



    private void OnTriggerEnter2D(Collider2D collision)
    {
        if (collision.gameObject.tag.Equals("Trap"))
        {
            Trigger = true;
            DamageCondition();
        }
    }

    private void OnTriggerExit2D(Collider2D collision)
    {
        if (collision.gameObject.tag.Equals("Trap"))
        {
            Trigger = false;
            DamageCondition();
        }
    }


    private void DamageCondition()
    {
        if (Trigger == true)
        {
            StartCoroutine(Spikes());
        }

        else if (Trigger == false)
        {
            StopCoroutine(Spikes());

            Debug.Log("Корутина остановлена");
        }


   
    }

   IEnumerator Spikes()
    {
        while (Hsys.health > 0)
        {
            Hsys.health -= DamageFromTraps;
            yield return new WaitForSeconds(2f);
        }
    }


}

Comments

[freeExec]

Time.deltaTime, ибо тот медленный

А поподробней?

По поводу остановки, посмотри код и подумай какую курутину ты остановишь.

var a = Spikes();
StartCoroutine(a);

var b = Spikes();
StartCoroutine(b);

var с = Spikes();
StopCoroutine(c);

[deeda]

freeExec

По поводу поподробнее:

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

А по поводу корутин и того, какую я остановлю:

Я же стартую корутину через "StartCoroutine(Spikes());", неужели обязательно объявлять переменную для остановки корутины? Ведь для ее включения объявлять переменную не пришлось.

[freeExec]

deeda, я так и не понял, что значит "слишком долго"?

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

[deeda]

freeExec, под "слишком долго" я имел ввиду тот момент, когда я использовал Time.deltaTime, это происходило медленнее, чем через корутину.

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

upd. Попробовал объявить переменную, ничего не поменялось, комментарий как появлялся, так и появляется, а корутина так и не останавливается, такие дела.

[freeExec]

deeda, потому что ты ничего не понял. Ты запустил первый спайк, а остановил второй. Это разные объекты.

[freeExec]

Условно на задание "остановить выполнение команд если поступила команда 'stop'" ты пишешь следующий код:

var commands = new string[] { "one", "two", "stop", "wtf"};
foreach (var command in commands)
{
    if (command == "stop")
    {
        foreach (var command1 in commands)
            break;
    }
    OnCommand(command);
}

И сидишь удивляешься, почему команда 'wtf' выполнилась.

[deeda]

freeExec, то есть должно выглядеть примерно так?

[deeda]

freeExec, я разобрался, спасибо большое за помощь, я понял что вы имели ввиду.

[freeExec]

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

[deeda]

freeExec, она неверная, да. Я сделал и все работает немного по другому.

Вот так:

Answer 1

[K0TlK]

Hsys

PCont

Много времени сохранил себе, сократив названия?

TakeDamage

Что это вообще такое и зачем оно нужно?

DamageCondition

Метод назван так, будто он должен что-то возвращать, но возвращает он ничего.

Почему вообще урон от пик персонаж наносит сам себе, а не пики ему?

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

Отдельно здоровье

namespace Coroutines
{
    public interface IHealth
    {
        bool IsOver { get; }
        void Lose(float amount);
        void Restore(float amount);
    }
}

using System;
using UnityEngine;

namespace Coroutines
{
    public class Health : IHealth
    {
        private readonly float _max;
        private readonly float _min;
        private float _current;

        public Health(float max, float min = 0)
        {
            _max = max;
            _min = min;
            _current = _max;
        }

        public bool IsOver => _current <= _min;
        
        public void Lose(float amount)
        {
            if (amount <= 0) throw new ArgumentException();
            
            SetCurrent(_current - amount);
        }

        public void Restore(float amount)
        {
            if (amount <= 0) throw new ArgumentException();

            SetCurrent(_current + amount);
        }

        private void SetCurrent(float amount)
        {
            _current = Mathf.Clamp(amount, _min, _max);
        }
    }
}

Отдельно персонаж

namespace Coroutines
{
    public interface IDamageable
    {
        bool IsDead { get; }
        void ApplyDamage(float amount);
    }
}

using UnityEngine;

namespace Coroutines
{
    public class Character : MonoBehaviour, IDamageable
    {
        [SerializeField] private float _maxHp = 100f;

        private IHealth _health;

        public bool IsDead => _health.IsOver;

        private void Awake()
        {
            _health = new Health(_maxHp);
        }

        public void ApplyDamage(float amount) => _health.Lose(amount);
    }
}

И отдельно ловушка

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

namespace Coroutines
{
    public class Trap : MonoBehaviour
    {
        [SerializeField] private float _damage = 10f;
        [SerializeField] private float _delay = 2f;
        
        private readonly Dictionary<IDamageable, Coroutine> _targets = new();

        private void OnTriggerEnter2D(Collider2D other)
        {
            if (other.TryGetComponent(out IDamageable damageable))
            {
                var coroutine = StartCoroutine(Damaging(damageable));
                _targets.Add(damageable, coroutine);
            }
        }

        private void OnTriggerExit2D(Collider2D other)
        {
            if (other.TryGetComponent(out IDamageable damageable))
            {
                var coroutine = _targets[damageable];
                StopCoroutine(coroutine);
                _targets.Remove(damageable);
            }
        }

        private IEnumerator Damaging(IDamageable target)
        {
            while (target.IsDead == false)
            {
                target.ApplyDamage(_damage);
                yield return new WaitForSeconds(_delay);
            }
        }

    }
}

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

Comments

[deeda]

Много времени сохранил себе, сократив названия?

Хорошо, тут согласен, самому вырвеглазно на это смотреть.

Что это вообще такое и зачем оно нужно?

Класс для получения дамага от ловушек, да я не переименовал название класса, потому что было не до этого.

Метод назван так, будто он должен что-то возвращать, но возвращает он ничего.

Если до этого в целом все понятно, и там я согласен, то тут я не понял, "DamageCondition", то есть условие получения дамага, не вижу тут ничего такого. Я понимаю что возможно для каждого определенного метода, класса и так далее, должны быть определенные правильные названия, но на данный момент мне это не нужно, я программирую один и делаю игру тоже один.

Почему вообще урон от пик персонаж наносит сам себе, а не пики ему?

Ваша правда, проще будет, если я не буду вешать все скрипты на одного персонажа.

...чтобы потом собрать другого персонажа, тебе придется искать всё, из чего он состоит.

В данном проекте мне не нужны другие управляемые персонажи, только один управляемый игроком герой, нпс и противники с боссами, не более.

Делай все через интерфейсы.

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

Как именно должен дамаг наноситься ты не сказал, поэтому у меня каждому персонажу, что зашел в триггер, урон наносится отдельно.

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

Чтобы завершить определенную корутину, ее нужно кэшировать куда-то.

Хороший совет, спасибо за него и в целом за весь ответ, а также за предложенные вами скрипты, я воспользуюсь вашей помощью.

[K0TlK]

я программирую один и делаю игру тоже один

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

на данный момент мне это не нужно

Если на данный момент это тебе что-то не нужно, это не значит, что оно не пригодится потом. Гораздо проще изначально писать код по стандартам, нежели после переучиваться со своего какого-то мордорского наречия. Дотнетовский кодстайл.

[deeda]

K0TlK, ладно, справедливо, спасибо.