Immortalised this actual legacy code for our Senior Devs leaving present (we all inherited this) with his last task bein

Ranter

Pepsi1x1

234

Comments

11

delegate212

1094

6y

"may he never have to work with code like this again" that was my line you arse was it not enough to steal my internet points :/
9

Codex404

16186

6y

If you are gonna write bad code at least make it work. Its only 8 possibilities to check for.
14

kescherRant

23842

6y

*ahem*

public string YesConverter(string yes)
{
if ("yes" == yes.ToLowercase())
return "yes";
return "no";
}
6

delegate212

1094

6y

@filthyranter

God I hope you haven't used the same approach that our legacy dev had done in any of your projects.

bool yes;

Solved.
10

kescherRant

23842

6y

@delegate212 This isn't even my final form:

public string YesConverter(string yes)
{
yes = yes.Trim().ToLowerCase();
if (yes.Contains("yes") return "yes";
return "no";
}

Huehuehuehue
3

kescherRant

23842

6y

Seriously tho, before replacing it by boolean logic when the surrounding system isn't fixed yet, I'd first change it to my first comment.
3

delegate212

1094

6y

@filthyranter It's long gone.
2

shaji

1246

6y

Ah, the good old yes converter!
7

d4ng3r0u5

4592

6y

this.YesConvert("Oui");
this.YesConvert("Ja");
this.YesConvert("Si");
6

aneesh

144

6y

Am I a noob or Did no one else notice that the function declaration name and invocation names do not match.
1

Codex404

16186

6y

@aneesh I indeed didn't notice that
2

sunfishcc

5205

6y

@filthyranter how about straight send it to server, let sql ignores the case 😼
5

kescherRant

23842

6y

@sunfishcc yes; DROP TABLE users;
1

electrineer

29809

3y

@scor I'm just mentioning you so you can read about the YesConverter once the mentions start flowing again

Add Comment

Immortalised this actual legacy code for our Senior Devs leaving present (we all inherited this) with his last task being to refactor it, may he never have to work with code like this again.

rant