hairinmybellybutt@lemmy.world to Programmer Humor@lemmy.mlEnglish · 1 year agoadvancedlemmy.worldimagemessage-square32fedilinkarrow-up1429
arrow-up1429imageadvancedlemmy.worldhairinmybellybutt@lemmy.world to Programmer Humor@lemmy.mlEnglish · 1 year agomessage-square32fedilink
minus-squarexmunk@sh.itjust.workslinkfedilinkarrow-up11·1 year agoIf you’re in a language that supports it, please don’t use if (false) use if ($disallowAllUsers = false && $whateverTheRealConditionIs)
minus-squarePlexSheep@feddit.delinkfedilinkarrow-up6·1 year agoNever seen this, what language or buildsystem is this?
minus-squarexmunk@sh.itjust.workslinkfedilinkarrow-up3·edit-21 year agoThat specific language is PHP, but the tip is applicable in any language that supports inline assignment.
minus-squareThe Octonaut@mander.xyzlinkfedilinkarrow-up6·1 year agoif (true === $wantToCauseErrorsForFun) { badOldFunction(); }
minus-squareSpaceNoodle@lemmy.worldlinkfedilinkarrow-up1·1 year agoIt seems much worse to use a setter in an if statement.
minus-squarexmunk@sh.itjust.workslinkfedilinkarrow-up1·1 year agoThink of it as inline attribution/documentation.
minus-squareKogasa@programming.devlinkfedilinkarrow-up1·1 year agoThe assignment syntax is too close to comparison, which is what is more typical in that position. I would recommend const bool _isFeatureEnabled = false; if (_isFeatureEnabled && ...) if not a proper feature flag (or just remove the code).
If you’re in a language that supports it, please don’t use
if (false)useif ($disallowAllUsers = false && $whateverTheRealConditionIs)Never seen this, what language or buildsystem is this?
That specific language is PHP, but the tip is applicable in any language that supports inline assignment.
It seems much worse to use a setter in an if statement.
Think of it as inline attribution/documentation.
The assignment syntax is too close to comparison, which is what is more typical in that position. I would recommend
const bool _isFeatureEnabled = false; if (_isFeatureEnabled && ...)if not a proper feature flag (or just remove the code).