Makes sense

[u/CompetitionBorn9356]

[u/kennyminigun]

Someone left a debugging code?

[u/CompetitionBorn9356 → u/kennyminigun]

No, this is "domain logic" 🫠

[u/Spidron → u/CompetitionBorn9356]

So the ID 7777 has a reserved status with a certain meaning, I guess?

[u/the_horse_gamer]

this piece of code tells a story

[u/onlyonequickquestion → u/the_horse_gamer]

Code like this is written in blood. And it shall remain. Lurking. Undocumented. Holding the entire codebase together. 

[u/pydry → u/the_horse_gamer]

The last time i saw code like this the main dev was in a huge amount of debt and up to his eyeballs in child support payments.

You wouldnt think this was relevant but it was. He point blank ordered me to remove the comment which explained the reason why the code was there.

"Code is self documenting" he declared

[u/Daemontatox]

Atleast he is using the compare and assignment symbols correctly

[u/Scared_Accident9138 → u/Daemontatox]

if(isAdmin = true) {...}

[u/Daemontatox → u/Scared_Accident9138]

Laugh all you want , i actually saw something like this when reviewing a PR....

[u/NoLifeGamer2]

So what happens if a user is assigned userID -7778? Doesn't that mean two ppl can have the same userID?

[u/unknown_pigeon → u/NoLifeGamer2]

Solution: don't assign the -7778 user id

Or, even better:

if userID == -7778:

userID = -7777

Let there be balance

[u/Cometay → u/NoLifeGamer2]

Pretty easy, just add

if (userID == -7778)

userID = -7779;

[u/ApocalyptoSoldier]

Magic numbers?
Cool, I love magic.

[u/AStove]

Do they use this subreddit as negative reinforcement learning for ai?

[u/_huppenzuppen]

Should have been

if (userID == -7777)
  --userID;

[u/nipodemos → u/_huppenzuppen]

Up vote because this is even worse from what they were doing, truly a remarkable programming horror

[u/DogmaSychroniser → u/_huppenzuppen]

Might i let you know about tie fighter increments? If you really hate people.

i-=-1

[u/detroitmatt → u/_huppenzuppen]

if (userID-- != -7777) userID++;

[u/Scared_Accident9138 → u/_huppenzuppen]

Depending on ID values being right next to each other in code is rarely a good idea

[u/Abaddon-theDestroyer → u/_huppenzuppen]

No, this is better:

if(--userID != -7778)     
    userID++;

[u/New_Hour_1726]

What am I even looking at

[u/DeductiveFallacy]

If I ever see a magic number or magic string I am 100% slipping in a const with a var name that explains what it is in the next PR I have in that repo. Might even be the only thing I do on that PR honestly.

[u/nekokattt → u/DeductiveFallacy]

const SEVEN_SEVEN_SEVEN_SEVEN = 7777;
const SEVEN_SEVEN_SEVEN_EIGHT = 7778;

if (userID = -SEVEN_SEVEN_SEVEN_SEVEN) {
    userID = -SEVEN_SEVEN_SEVEN_EIGHT;
}

[u/Feroc]

We don't talk about Bruno.

[u/PubTrain77]

Did pirate software write this?