this post was submitted on 28 Mar 2024
616 points (98.1% liked)

Programmer Humor

32561 readers
488 users here now

Post funny things about programming here! (Or just rant about your favourite programming language.)

Rules:

founded 5 years ago
MODERATORS
 
you are viewing a single comment's thread
view the rest of the comments
[–] cookie_sabotage@sh.itjust.works 159 points 8 months ago (4 children)
public class GameManager : MonoBehaviour
{
    public bool EnableHighContrast;
    public bool PlayerWon;
    public float PlayerUnitsMoved;
    public int PlayerDeathCount;
    public float PlayerHealth;

    public void PlayerTakeDamage(float damage)
    {
        PlayerHealth -= damage;
        if (PlayerHealth < 0)
        {
            PlayerDieAndRespawn();
        }
    }

    public void PlayerDieAndRespawn()
    {
        return;
    }
}

I couldn't contain myself.

[–] wise@feddit.uk 57 points 8 months ago (5 children)

Should it be

PlayerHealth <= 0

?

Otherwise the player could have 0 health and not die? I’m sleep deprived so forgive me if I’m wrong

[–] bassomitron@lemmy.world 27 points 8 months ago (1 children)
[–] wise@feddit.uk 54 points 8 months ago (1 children)

Counting this meme as my first FOSS contribution

[–] SidewaysHighways@lemmy.world 12 points 8 months ago

Holy shit I was there with you sir! With the zeros and stuff

[–] mryessir@lemmy.sdf.org 26 points 8 months ago

Open up ticket first, please. Thanks Codemonkey.

[–] vithigar@lemmy.ca 12 points 8 months ago* (last edited 8 months ago)

You are correct about it allowing you to have zero health and not die, but whether or not that's the correct behavior will depend on the game. Off the top of my head I know that Street Fighter, some versions at least, let you cling to life at zero.

[–] joshfaulkner@lemmy.world 5 points 8 months ago* (last edited 8 months ago) (2 children)

I know this is /c/Progammerhumor, but I wanted to pull on this thread a little bit for my own edification. I'm a Python guy and have been a while, but I've dabbled in other languages. The screenshot says "MonoBehaviour" which makes me assume this is mono or a .Net-like language (you know what happens when you assume).

If your player health is a float, would mono or .Net have an issue comparing the float with integer zero "0"? I mean, it seems like floating point precision may make it impossible for it to ever "equal" integer zero, but it also seems like the code isn't accounting for that precision error.

Am I overthinking this?

[–] herrvogel@lemmy.world 7 points 8 months ago (1 children)

Floating point errors are a product of how floating points work as a mathematical concept. So they're independent of the programming language and can happen everywhere.

In this case though, I doubt it's a critical issue. So the player "died" when they actually had 0.000000000027 hp left or whatever. Who cares? Do you need to be that precise?

[–] TipRing@lemmy.world 14 points 8 months ago

Hanging on with 1.70E-31 health.

[–] Melobol@lemmy.ml 1 points 8 months ago

As a noob in unity and programming, my understanding is that MonoBehavior only means that this script has to be attached as a component to a game object to function. And has no other meaning - but correct me if I'm wrong please.

[–] Randomocity@sh.itjust.works -4 points 8 months ago

This won't work if you can ever take more than 1 damage. If you were at 1 and received 2 damage you would become invincible. You'd want to do less than or equal to.

[–] blarth@thelemmy.club 34 points 8 months ago (2 children)

Yay, escaped the fight with 0 health!

[–] TheOakTree@lemm.ee 15 points 8 months ago

Well if you have a "down but not dead" condition then yes, you could escape a fight with 0 health (assuming you have teammates/pawns that can save you).

[–] andrew@lemmy.stuart.fun 5 points 8 months ago

This is floating point. We also need to know what happens when you escape with -0.

[–] Sakychu@lemmy.world 10 points 8 months ago (1 children)

I called the takeDamage function and my player disappeared: send 'elp everything foobar

[–] cookie_sabotage@sh.itjust.works 15 points 8 months ago (1 children)

Don't worry! this issue will be fixed in the next patch. In the meantime just try not getting hit.

[–] mac@infosec.pub 6 points 8 months ago

The doctor prescribed "getting good"

[–] Ironfacebuster@lemmy.world 5 points 8 months ago

Too readable, please make each name a paragraph describing its function and how it relates to the other variables/functions around it