Как обработать нанесение урона объектам бомбой которые находятся в радиусе ее взрыва?

Question

[vova_20]

Добрый день! Уже второй день сижу и не могу понять почему не работает как надо мой код:

void Update()
    {
        StartCoroutine(DamageObjects());
    }
    
    IEnumerator DamageObjects()
    {
        Collider2D[] collider2D = Physics2D.OverlapCircleAll(transform.position, explosionRadius);

        foreach (Collider2D col in collider2D)
        {
            if (col.tag == enemyName)
            {
                yield return new WaitForSeconds(0.3f);
                DamageEnemy(col.transform);
                Destroy(gameObject);
            }
        }
    }
    
    void DamageEnemy(Transform enemy)
    {
        Enemy e = enemy.GetComponent<Enemy>();

        if (e != null)
        {
            e.TakeDamage(damage);
        }
    }

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

Comments

[freeExec]

Это точно тут и для того Destroy(gameObject);?

[ColdSpirit]

Проставьте табуляцию и добавьте подсветку тэгом "code" - ваш код нечитаемый

[vova_20]

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

[vova_20]

ColdSpirit, Исправил.

[freeExec]

vova_20, я вас не понимаю, то вы жалуетесь почему урон наносится только одному объекту, то сами же это считаете нормальным

если этого не сделать, то бомба уничтожает почти все объекты

.
Вы уничтожаете объект, поэтому код после yeldreturn далее не будет выполняться, ибо объекта уже нет.

[vova_20]

freeExec, допустим есть здоровье у юнита 100 у бомбы урон 10, то бомба наносит не 10 урона всем объектам которые попали у радиус взрыва а 100 если не удалить бомбу после нанесения урона.

[freeExec]

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

[vova_20]

freeExec, я это понял, но пока не знаю как это сделать.

Answer 1

[vova_20]

Спасибо всем за помощь!!! Решил проблему:) Если кому интересно и нужно, то смотрите:

void Update()
    {
        ...
        transform.position = nextPos;

        if (nextPos == endPoint)
        {
            DamageObjects();
        }   
    }

    void DamageObjects()
    {
        Collider2D[] collider2D = Physics2D.OverlapCircleAll(transform.position, explosionRadius);
        if (collider2D.Length > 0)
        {
            foreach (Collider2D col in collider2D)
            {
                if (col.tag == enemyName)
                {
                    DamageEnemy(col.transform);
                    Destroy(gameObject);
                }
            }
        }
        else
        {
            Destroy(gameObject);
        }
        
    }

    void DamageEnemy(Transform enemy)
    {
        Enemy e = enemy.GetComponent<Enemy>();

        if (e != null)
        {
            e.TakeDamage(damage);
        }
    }

Answer 2

[ColdSpirit]

Поставьте Destroy(gameObject); после цикла foreach:

// проходим по каждому задетому коллайдеру
foreach (Collider2D col in collider2D)
{
	// если это враг
	if (col.tag == enemyName)
	{
		// ждем 0,3с
		yield return new WaitForSeconds(0.3f);
		// наносим ему урон
		DamageEnemy(col.transform);
	}
}

// уничтожаем бомбу
Destroy(gameObject);

Comments

[vova_20]

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

[ColdSpirit]

Пропустил мимо глаз: Update на Start замените.

Если все еще будет уничтожаться - ищите проблему в другом месте.

[vova_20]

ColdSpirit, В начале вызывать эту функцию нету смысла, в коллайдеры нечего не попадет. Просо не пойму почему перестал работать этот код, раньше то работал.

[ColdSpirit]

vova_20, в Update тем более нету смысла - пока вы ждете эти 0,3с, бомба взрывается еще раз, и еще раз, до тех пор пока бомба не уничтожится.
Вам нужно выбрать момент взрыва, и убедиться, что функция StartCoroutine(DamageObjects()); вызывается исключительно единожды.

###
Либо уберите yield return new WaitForSeconds(0.3f);, замените IEnumerator на void, а StartCoroutine(DamageObjects()); на DamageObjects(); и выполняйте где угодно - бомба нанесет урон всем одновременно и удалится после.

Но всё это с моей правкой выше.

[ColdSpirit]

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

[vova_20]

ColdSpirit, Спасибо Вам! Я так и сделал.

[ColdSpirit]

vova_20, прошу пометить вопрос решенным если всё работает хорошо.