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.

Great. Now that my code is self-documenting it is somehow also not legible?

Make up your damn minds, peeps!

gestures at butterfly this code

Is this self-documenting code?

I would take this over int a; anyday

This is something that can easily get refactored, because the purpose of alia the variables is right there in the name. This is way better that spending three days to try to figure out what the purpose of var1 is.

I mean, this is overdoing it a bit and the "thisVarMakesItSoThat" part is redundant, but other than that those are very descriptive property- and method names, which is not a bad thing.

Noone yet?

https://xkcd.com/1960/

Deobfuscated code

yeah, PascalCase is the worst

I'll take this over the more "classic" styles, when people seed to believe they were paying the compiler by the character.

Self documenting!

Starting my day off with this absolutely cursed image, thank you OP.

Looks ugly until you need to implement something and realize you've been blessed with a description of business logic.

The real naming fail is calling the class "GameManager", still my number one pet peeve. With a class name as vague as that you would have to add tons of information into the variable name. (Also the class name begs for unorganized code. I mean name one function or variable that you could not justify putting into the "GameManager" class. After all if it's managing the game it could justifiably perform any process in the game and access any state in it.)

Once you put the first bool into a class with a name like AccessibilitySettings, calling it something like HighContrast is completely sufficient.

Reminds me of this https://lemmy.world/post/12487512

"Commenting is for n00bs"

You forgot to declare custom primitive types. You cannot create a bool you gotta declare a DoubleYouDoubleYouDoubleYouDotLemmyGradDotML_Bool

I see. So that’s why people put comments before variable declarations.

Self-explanatory code doesn't need comments!

Like... there's such a thing as comments my dude.

String if_you_must_please_at_least_separate_the_words