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

16193

6y

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

kescherRant

23851

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

23851

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

23851

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

1248

6y

Ah, the good old yes converter!
7

d4ng3r0u5

4593

6y

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

aneesh

145

6y

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

Codex404

16193

6y

@aneesh I indeed didn't notice that
2

sunfishcc

5208

6y

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

kescherRant

23851

6y

@sunfishcc yes; DROP TABLE users;
1

electrineer

29784

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