Как сделать чтобы Player наносил урон Enemy?

Question

[LeBrone]

Я новичок. Написал скрипт чтобы Player атаковал Enemy, но урон не проходит по Enemy. Помогите пожалуйста.
Скрипт атаки:

void Attack()
    {
        anim.SetTrigger("Attack");

        Collider2D[] hitEnemies = Physics2D.OverlapCircleAll(attackPoint.position, attackRange, enemyLayers);
         
        if (collision.tag == "Enemy")
        {
            collision.GetComponent<Health>().TakeDamage(damage);
        }
    }

Скрипт Damage:

public void TakeDamage(int damage)
    {
        currentHealth -= damage;

        if (currentHealth <= 0)
        {
            Die();
        }
    }

Comments

[freeExec]

У тебя ничего не вызывается.

[LeBrone]

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

[freeExec]

LeBrone, приведённый код

[Dmitry_Sel]

LeBrone, Добавь Debug.Log чтоб понять работает ли это вообще

[LeBrone]

Dmitry_Sel, проверял не работает(

Answer 1

[Ente]

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

У этих объектов будет скрипт, назовем его Health.cs

[SerializedField] private int hitpoints = 100;
public Action<int, int> OnChange;

public void Change(int amount)
{
           hitpoints += amount;
           OnChange?.Invoke(hitpoints, amount);
}

В этом скрипте можно будет менять здоровье через функцию Change, причем в любую сторону, может проходить и урон, так и лечение, все зависит от знака в переданной переменной amount. Также скрипт бросает событие, что здоровье изменилось, чтобы другие скрипты могли подписаться и отреагировать. Например, скрипт, который рисует полосу здоровья или скрипт, который отвечает за смерть персонажа.

Второй скрипт - Damage.cs. Он будет работать на триггерах в Unity, и когда объекты пересекаются между собой, то происходит вызов функции урона, например, если персонаж встанет в огонь, его затронет пуля или меч. Само собой нужно проставить им триггеры нужного размера, как объекту с уроном, так и тому, кому наносится урон + поставить правильно матрицу коллизий, чтобы OnTriggerEnter2D не затрагивал другие объекты, которые не являются его целью.

[SerializedField] private int amount = -10;
public Action<int> OnDamage;

private void OnTriggerEnter2D(Collider2D collider)
{
      collider.gameObject.GetComponent<Health>().Change(amount);
      OnDamage?.Invoke(amount);
}

Comments

[LeBrone]

Ente, ого, спасибо тебе большое ;)

[LeBrone]

Ente, не мог ли бы ты еще подсказать по поводу твоего скрипта. У меня Unity почему-то жалуется на скрипт
NullReferenceException: Object reference not set to an instance of an object
PlayerAttack.OnTriggerEnter2D (UnityEngine.Collider2D collision)

[Ente]

Есть ли на объекте с коллайдером скрипт Health?

[Ente]

Скрипт Damage должен висеть на объекте, который наносит урон.

[LeBrone]

[Ente]

Нет, Attack это другое, в системе участвуют два скрипта, Health и Damage

[Ente]

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

[LeBrone]

Ente, ааа понял)

[Ente]

PlayerHealth также не совсем верно, так как Health это скрипт для абсолютно всех объектов, которые хоть как то лечатся или калечатся. Если тебе нужно как то уникально обработать Health для игрока, ты либо наследуешься от базового Health, либо создаешь еще один скрипт и подписываешься на OnChange в Health

[LeBrone]

[LeBrone]

Ente, вот, но выдает ту же самую ошибку(

[LeBrone]

Ente, ок сейчас переделаю

[LeBrone]

Ente, создал новый проект что бы не было каши в голове. Я создал два объекта одному дал скрипт Health, а другому Damage (в скрипт добавил Debug). При столкновении объекта об объект ничего не происходит. Может я что-то не так сделал вы меня простите)

[Ente]

Проверь, что у коллайдера стоит IsTrigger включено галочкой

[Ente]

и что за поле коллайдера у Attack, его там не должно быть

[LeBrone]

Ente, Заработала)

[LeBrone]

Ente, а что бы персонаж атаковал. Это надо ему дать скрипт Damage и все будет работать? или для него надо писать отдельно

[Ente]

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

[Ente]

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

[LeBrone]

Ente, хорошо сейчас попробую

[LeBrone]

Ente, у меня все заработало, но есть над чем еще поработать) спасибо вам что помогли