Integrate blood texture when the enemy dies. - Githubissues

NestorTejero / ES2016A

VideoJoc Curs Enginyeria Software 2016 - grup A de pràctiques (dijous)

https://nestortejero.github.io/ES2016A/

GNU General Public License v3.0

2 stars 0 forks source link

Integrate blood texture when the enemy dies. #257

Closed kazusaki1 closed 7 years ago

kazusaki1 commented 7 years ago

La rama será compartida con el Team A que realiza la Issue #233: Crear y mejorar texturas de sangre.

Definition of Done:

Blood prefab defined as a plane with a blood texture.
After being hit, enemies spawn at least one blood prefab.
Object spawned on the ground, near the X-Z position where the hit occurred.
Blood dissapears after N seconds. N assignable through Inspector. Default value: N=2 seconds.
Blood stain size proportional to damage suffered.

DEPRECATED: Blood stain spawned at death last more than those spawned when being hit.

Estimated time: 3 hours

Branch: dev_Issue233

Pull request: #264

sdiaz7 commented 7 years ago

I've uploaded the blood prefab. Since I do not dispose of the blood texture yet, I am using ashes texture, that being a distinguishable one.

sdiaz7 commented 7 years ago

I've made some adjustements to the definition of done. When hit, the stain will appear near the impact position. The scatter is generated randomly.

The changes have increased the number of hours needed for the issue to be completed.

ezendirak commented 7 years ago

Blood texture updated. #233

sdiaz7 commented 7 years ago

Last commit updates EnemyBehavior so that it performs blood spawning as described at DoD. Blood prevalence is not yet accessible through the Inspector. I'm considering moving some of the code to a new script BloodBehavior.

sdiaz7 commented 7 years ago

Small stains caused by low-damaging bullet projectiles:

Blood stain caused by heavy-hitting missile projectile:

sdiaz7 commented 7 years ago

Added BloodBehavior, which is responsible for depleting blood object. It also stores some relevant information: blood prevalence, blood maximum and minimum size.

sdiaz7 commented 7 years ago

For now, I've abandoned the idea of death stain due to some difficulties with the code. If I wanted to implement such feature, I would have to choose one of the following options, none of them appealing:

Bloating EnemyBehavior with new public variables for stain prevalence
Storing such variables at BloodBehavior, having to access to them from EnemyBehavior
Create 2 separate prefabs, one for death stain and another for hit stain

Since a hit must occur for an enemy to die, and that hit will already spawn a stain, I consider that the gains for implementation do not compensate for the effort.

sdiaz7 commented 7 years ago

Branch approved and integrated. Issue closed and moved to DONE.