this post was submitted on 10 Jul 2024
645 points (94.0% liked)

Programmer Humor

19623 readers
507 users here now

Welcome to Programmer Humor!

This is a place where you can post jokes, memes, humor, etc. related to programming!

For sharing awful code theres also Programming Horror.

Rules

founded 1 year ago
MODERATORS
 
you are viewing a single comment's thread
view the rest of the comments
[–] [email protected] 82 points 4 months ago* (last edited 4 months ago) (10 children)

WHY IS THE HEALTH INPUT PARAMETER A GODDAMN STRING?????????????

Why are you passing ‘%’ inside said goddamn string?!?! Not to mention the static reference instead of the actual instance.

Shame on you

[–] [email protected] 19 points 4 months ago

Also putting sudo in front of what looks like Java code not shell.

[–] [email protected] 14 points 4 months ago

Because the meme wasn't made by someone with a doctorate in CS or even a bachelor's.

[–] [email protected] 13 points 4 months ago (1 children)

OldMan.setHealth("Robert'); DROP TABLE Students;--")

[–] [email protected] 3 points 4 months ago

Finally someone with some wisdom

[–] [email protected] 11 points 4 months ago (1 children)

It's not his fault the world is made this way.

He just has to follow it or else that man dies.

[–] [email protected] 5 points 4 months ago

Found the legacy support person

[–] [email protected] 7 points 4 months ago* (last edited 4 months ago)

Smells like JavaScript.

[–] [email protected] 7 points 4 months ago (2 children)

They use a look-up table with every value from 0% to 100%

[–] [email protected] 1 points 4 months ago

Floating points included for thoroughness!

[–] [email protected] 6 points 4 months ago* (last edited 4 months ago)

I guess its just a reminder that getting a PhD is often more about dedication than it is about practical knowledge.

[–] [email protected] 5 points 4 months ago* (last edited 4 months ago)

Honestly, if someone were to try to safe my life. And I find out he uses a string as a parameter to do so. Just let me die right there.

[–] [email protected] 2 points 4 months ago (4 children)

The high level setter function should be made to handle both string and numeric values.

If it contains "%" it's a percentage value.

If it's a string without a "%" it's an absolute value and needs to be normalized.

If it's a numeric value, it's an absolute value.

If it's a numeric 100, it's 100%.

If it's a subunitary numeric value, it's a percentage.

[–] [email protected] 6 points 4 months ago (1 children)

yeah I’m gonna go ahead and reject your PR, please change this function to accept a decimal value between 0 and 1

[–] [email protected] 5 points 4 months ago (2 children)

Ironically, the worst thing I ever saw a coworker do was to change a function that accepted an Integer value between 0 and 32767 to one that accepted a Float between 0.0 and 1.0. Perfectly sensible change except that it resulted in a 120 mph knuckleball fired a foot above a 10 year old kid's head, followed by a fist fight between the client and my boss.

[–] [email protected] 4 points 4 months ago (1 children)

That sounds like something that should have been caught by QA, integration tests or unit tests long before it was launching balls at ten year olds.

[–] [email protected] 3 points 4 months ago (1 children)

Yes, testing the new Little League control module on a field full of Little Leaguers was not the best plan.

[–] [email protected] 0 points 4 months ago (1 children)

What is a little league control module?

[–] [email protected] 3 points 4 months ago

You push a button and it makes Little Leaguers do whatever you tell them to do. Very potent, should never be misused.

[–] [email protected] 1 points 4 months ago

yeah every engineer knows you gotta set KidHeadKnuckleballClearance waaay higher than that, it's compsci 101

[–] [email protected] 2 points 4 months ago (1 children)

If it’s a numeric 100, it’s 100%.

absolute lunacy

[–] [email protected] 1 points 4 months ago* (last edited 4 months ago)

Absolute (cm)

adding one 0:

100%, automatically changes unit to %

(Word table properties)

[–] [email protected] 1 points 4 months ago* (last edited 4 months ago)
[–] [email protected] 1 points 4 months ago

Oldman.setHealth("dicktits"); //normalize pls

Oldman.setHealth("-100±1%"); //make percentage pls

Oldman.setHealth(0.0); //it is subunitary, but undefined behavior - will it access the 'numeric value' overload, or the 'subunitary numeric value' overload?

Don't write your own code just yet.

[–] [email protected] 1 points 4 months ago (1 children)

Yeah, in Ren'py games usually it's character_health=100 or something.

[–] [email protected] 3 points 4 months ago

Yes absolutely, the parameter even if not in a strongly typed language should be a specific number and the unit should be implied. Overload the method to support different units if necessary or provide a unit as an additional parameter instead of forcing the method to parse the string for any unit type hints that may or may not be there