|
|
|
|
|
|
by RHSeeger
978 days ago
|
|
|
I would add to this a bit in that 1. Given that "select " is considered something to avoid except when necessary in edge cases 2. And "select 1" will accomplish the same goal Anyone reading the "select " version of the code will have to stop and consider whether it is using "select " for a reason, because "select 1" would be the normal choice. Using "select " is assumed to be conveying some intent (that isn't there) _because_ it's not the expected way to do it. I kind of see it like if (thisField == thatField) ...
vs if ( (( true || false )) && ( 11 == 11 ) && thisField == thatField ) ...
Sure, they do the same thing... but you have to stop and look at the second one to make sure you're understanding what it does and if there's some reason its weird. |
|
|