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] 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

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* (last edited 4 months ago)