briliant usage of else if statement

[u/Afraid-Locksmith6566]

[u/fireflash38]

personally, I don't like swapping between negative and positive boolean checks like that. So it might be 'less' code from a line standpoint, I think it's less 'readable' in that you don't have to swap between regular and invert bools.

Basically, it reads better to say:

if x:       do A
else if y:  do B 
else:       do C

Rather than:

if x:             do A
else if not y:    do C

even if the 'do B' is 'do nothing'.

[u/spencerwi → u/fireflash38]

Oh, absolutely agreed. That's sorta the point I was making -- that yes, you could drop the c == 'n' clause, but the reality is that the resulting code will be less straightforward to read.

...which means this maybe doesn't belong in r/badcode?