You're viewing a single thread.
32
comments
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.
if (true === $wantToCauseErrorsForFun) { badOldFunction(); }
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).
It seems much worse to use a setter in an if statement.
Think of it as inline attribution/documentation.
You've viewed 32 comments.