Skip to content
This repository has been archived by the owner on Jul 17, 2018. It is now read-only.

Change healing logic #179

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Change healing logic #179

wants to merge 3 commits into from

Conversation

MatthiasMoore
Copy link
Contributor

The player will be healed when they are not full hp during boss battles.

Resolves #177

The player will be healed when they are not full hp during boss battles.
@meepen
Copy link
Owner

meepen commented Jul 4, 2018

Correct me if im wrong - but the healing attack heals everyone, not just you, correct?

@MatthiasMoore
Copy link
Contributor Author

Yea, the healing skill does heal others, and it seems the boss will randomly damage one of the ten players with its laser attack. It would be more efficient to check all players health values, and then apply it in an optimal way, but I was not thinking about that at the time.

If more than half of the allies are not full hp, use the heal skill.
Reference proper variable within game state.
Return false in the event that m_bBossLevel is false
Change check to be greater than or equal to 5 allies not at full health.
@MatthiasMoore
Copy link
Contributor Author

The new commits should now prefer healing when half the allies are not full hp. I will not have time to test it for a few hours, but looking over the logic a few times, it should function properly unless I am overlooking something simple.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants