Draw player health to screen with fonts/numbers

[BeasTie]

You're right Bloodbane the anim trick is almost perfect duration, but pain and get animations are very fast/short, I just want them to display just a tiny bit longer without messing with their durations.  I could make them a bit longer but that would mean slower reaction times when grabbing a weapon during a battle or being attacked.

Mirror the sprite already there? Clever thinking! it still needs the normal facing forward face between each face looking left and right thou.

This is kinda what the face does in original Doom when idle. (was studying it a bit, he may also do it as enemies approach)

When he's shot he faces the direction of his attacker

This is the faces I'm using for pain at the moment, but these are also for when he is attacking with chainsaw or in close quarters fight. I already added them to run during ATTACK anims.


EDIT: Thanks for the script Bloodbane, just tried it and it's basically working already just adding your code.  Still needs tweaking but another step closer,

[MatMan]

what you want is a method that checks if the enemy is to your right or to your left....


void self = getlocalvar("self");
void target = getentityproperty(self, "opponent");

void selfxposition = getentityproperty(self, "x") ;
void targetxposition = getentityproperty(target, "x");

if ( targetxposition >= selfxposition )
{
  //show right icon
} else {
  //show left icon
}

[BeasTie]

Cheers, I'll try that for the left and right pain faces.

[BeasTie]

Something I'm still messing up is just the logic for when it displays the face, it works but you can tell some are still being drawn underneath the next sprite.

Code: [Select]

if(p1disphealth >= 100) drawsprite(getglobalvar("faceb"), 456, 2, 2000, 0);
else if(p1disphealth >= 80) drawsprite(getglobalvar("faceb"), 456, 2, 2001, 0);
else if(p1disphealth <= 80) drawsprite(getglobalvar("facef"), 456, 2, 2001, 0);
if(p1disphealth <= 55) drawsprite(getglobalvar("facei"), 456, 2, 2002, 0);
if(p1disphealth <= 35) drawsprite(getglobalvar("facel"), 456, 2, 2003, 0);
if(p1disphealth <= 15) drawsprite(getglobalvar("faceo"), 456, 2, 2004, 0);
if(p1disphealth == 1) drawsprite(getglobalvar("faceo"), 456, 2, 2005, 0);

I'm kinda been stumped for ages how to do it better.  it's after the <= 80 I think it's flawed, I think it still runs the 80% part when it runs the others. (or something)

This is a sprite showing what it does.  Like I said mine works but it's not perfect.


Can someone show me how to fix the <=

(edit - typo)

[MatMan]

If you have a IF block of code that is considered to be a group(like the face) then always use the else statement..

Code: [Select]

if(p1disphealth >= 100) { drawsprite(getglobalvar("faceb"), 456, 2, 2000, 0); } else
        if(p1disphealth >= 80) { drawsprite(getglobalvar("faceb"), 456, 2, 2001, 0); } else
if(p1disphealth <= 80) { drawsprite(getglobalvar("facef"), 456, 2, 2001, 0); } else
if(p1disphealth <= 55) { drawsprite(getglobalvar("facei"), 456, 2, 2002, 0); } else
if(p1disphealth <= 35) { drawsprite(getglobalvar("facel"), 456, 2, 2003, 0); } else
if(p1disphealth <= 15) { drawsprite(getglobalvar("faceo"), 456, 2, 2004, 0); } else
if(p1disphealth == 1) { drawsprite(getglobalvar("faceo"), 456, 2, 2005, 0); }
This also prevents problems that you just suggested and also prevents on cpu wastage.

Also try to use the curly brackets aswell.

[BeasTie]

Cheers dude, I was about to try some brackets and change to else if, but I thought my logic was still flawed anyway.
Oh yeah I forgot the updated screenshot.

Decided why have a revamped STBAR when you can just have the real thing  

For when 2-4p are present then I'll make it use smaller versions.

EDIT: - I tried what you had but now it doesn't show the other faces, just the 100% and 80% ones, might need to tidy up the rest of my code thou, will try that first.

[Bloodbane]

 I found the flaw in your code Beastie:

if(p1disphealth >= 100) { drawsprite(getglobalvar("faceb"), 456, 2, 2000, 0); } else
        if(p1disphealth >= 80) { drawsprite(getglobalvar("faceb"), 456, 2, 2001, 0); } else
   if(p1disphealth <= 80) { drawsprite(getglobalvar("facef"), 456, 2, 2001, 0); } else
   if(p1disphealth <= 55) { drawsprite(getglobalvar("facei"), 456, 2, 2002, 0); } else
   if(p1disphealth <= 35) { drawsprite(getglobalvar("facel"), 456, 2, 2003, 0); } else
   if(p1disphealth <= 15) { drawsprite(getglobalvar("faceo"), 456, 2, 2004, 0); } else
   if(p1disphealth == 1) { drawsprite(getglobalvar("faceo"), 456, 2, 2005, 0); }

 With else, when one of the IFs is run, the others will be neglected.
 That's what happened here. When health is less than 80, the 3rd IF is run. It doesn't matter if health is less than 15, 35,  55 or 1, it fits 3rd IF. That explains the problem you have here.

 To solve this, you can either define the group strictly like this:

if(p1disphealth >= 80 && p1disphealth < 100)

 or reorder them like this:

if(p1disphealth <= 1) { drawsprite(getglobalvar("faceo"), 456, 2, 2005, 0);
 } else if(p1disphealth <= 15) { drawsprite(getglobalvar("faceo"), 456, 2, 2004, 0);
 } else if(p1disphealth <= 35) { drawsprite(getglobalvar("facel"), 456, 2, 2003, 0);
 } else if(p1disphealth <= 55) { drawsprite(getglobalvar("facei"), 456, 2, 2002, 0);
} else if(p1disphealth <= 80) { drawsprite(getglobalvar("facef"), 456, 2, 2001, 0);
 } else if(p1disphealth <= 100) { drawsprite(getglobalvar("faceb"), 456, 2, 2001, 0);
 } else if(p1disphealth > 100) { drawsprite(getglobalvar("faceb"), 456, 2, 2000, 0);
 }

 The former defines clearly health area so there won't be 'overlapping case' like this.
 The latter is advanced reorder trick which works by checking the less possible event to the most possible one. Getting health 1 or below (negative values included) is less possible that's why it's define earliest.

 HTH

[BeasTie]

Ahh that makes a sense and explains a few things that were going on when I tried to fix it.

Worked great thanks heaps again.

[BeasTie]

I finally sorted the IDLE face, I had an idea how to do it before, but I'm getting a little better with script now and managed to work it out this time.

Code checks if player is idle and also what frame number, the script displays looking left on frame 1 and looking right on frame 3.
Player has extra frames in his idle anim to control the timing of the eyes. (Currently about 4-5 frames with 80 delay.) 



Here a line of code to show how it runs, crude no doubt, but it works

Code: [Select]

if(anim == openborconstant("ANI_IDLE") && (aniF == 1) && (p1disphealth <= 1)) drawsprite(getglobalvar("faceu"), 137, 4, 3008, 0);

Next step is to expand on it so it shows the varying speeds of eye movements like in the original game. 

[Bloodbane]

 Great!

 I think you should try playing with time so face displaying code is controlled by time instead of player's IDLE animation.

[BeasTie]

Agreed, but I don't how to do that yet thou.   
I tried to use openborvariant("elapsed_time") at first but I couldn't work it out.

MatMan's showed me some stuff from his scripts so I can do the whole GUI a lot better, I've got some of the text objects running from another script now, I just can't work out how to properly display sprites without update script, I'm just using drawsprite command.  But without update script they only display for a moment.

Will try to move it all to this other script eventually,
sticking with this one for now thou just so I can see it working while I'm modding

Messing with scripts and stuff in another version of  the mod so I can keep one version kinda presentable.  Making some advances thou,  more than I could pull off a few months ago.