Почему у персонажа в Unity при столкновении с врагом не отнимается последняя жизнь?

Question

[fleshik]

Здравствуйте такая проблема курс "Unity Базовый курс: 2D платформер с нуля" когда у персонажа остается 1 жизнь он становится бессмертным сколько бы не сталкивался с врагами эта жизнь не отнимается (первые две жизни отнимаются нормально) в чем может быть проблема? (В каком из скриптов можно поискать ошибку скрипты названы также как в курсе)

Answer 1

[freeExec]

Проблема очевидна в коде отнимания жизни или умирания. К 80 годам стану как Ванга, тогда отвечу конкретно в какой строке проблема.

Comments

[fleshik]

Ошибка где то в этом куске но что именно нужно исправить не знаю

bool canHit = true;
int curHp;
 int maxHp = 3;
bool isHit = false;
public void RecountHp(int deltaHp) 
    {
        if (deltaHp < 0 && canHit) {
            curHp = curHp + deltaHp;
            StopCoroutine(OnHit());
            canHit = false;
            isHit = true;
            StartCoroutine(OnHit());
        }

        else if (curHp > maxHp) { 
            curHp = curHp + deltaHp;
            curHp = maxHp;
        }
        print(curHp);
        if (curHp <= 0)
        {
          GetComponent<CapsuleCollider2D>().enabled = false;
          Invoke("Lose", 2.5f);
        }
  }

Answer 2

[Мимокрокодил]

На курсах обычно дают один из двух вариантов:

1) Из текущего здоровья отнимают дельту и затем проверяют, не упало ли здоровье в условие:

public int healthCurrent;

public void OnHealthChanged(int healthDelta)
	healthCurrent -= healthDelta;
	// update health bar
	if (healthCurrent <= 0) {
		// death
	}
}

В этом варианте у вас может быть ошибка в условии. Например в примере ниже реально можно попасть в ситуацию, когда персонаж стал бессмертным:

public int healthCurrent;

public void OnHealthChanged(int healthDelta)
	if (healthCurrent > 0) {
		healthCurrent -= healthDelta;
		// update health bar
		if (healthCurrent < 0) {
			// death
		}
	}
}

2) Через Mathf.Clamp:

public int healthCurrent;
public int healthMaximum;

public void OnHealthChanged(int healthDelta)
	healthCurrent = Mathf.Clamp(healthCurrent - healthDelta, 0, healthMaximum);
	// update health bar
	if (healthCurrent == 0) {
		// death
	}
}

В этом варианте помимо неправильного условия также можно случайно ошибиться в аргументах функции Clamp и вместо 0 указать 1, а проверять будет именно 0.

Comments

[fleshik]

Проблема вот с этим куском кода, но что именно здесь не так не пойму

bool canHit = true;
int curHp;
 int maxHp = 3;
bool isHit = false;
public void RecountHp(int deltaHp) 
    {
        if (deltaHp < 0 && canHit) {
            curHp = curHp + deltaHp;
            StopCoroutine(OnHit());
            canHit = false;
            isHit = true;
            StartCoroutine(OnHit());
        }

        else if (curHp > maxHp) { 
            curHp = curHp + deltaHp;
            curHp = maxHp;
        }
        print(curHp);
        if (curHp <= 0)
        {
          GetComponent<CapsuleCollider2D>().enabled = false;
          Invoke("Lose", 2.5f);
        }
  }

[Мимокрокодил]

fleshik, выложите, пожалуйста, весь скрипт.