• xmunk
    link
    fedilink
    arrow-up
    11
    arrow-down
    2
    ·
    1 year ago

    If you’re in a language that supports it, please don’t use if (false) use if ($disallowAllUsers = false && $whateverTheRealConditionIs)

      • xmunk
        link
        fedilink
        arrow-up
        3
        ·
        edit-2
        1 year ago

        That specific language is PHP, but the tip is applicable in any language that supports inline assignment.

    • Kogasa@programming.dev
      link
      fedilink
      arrow-up
      1
      ·
      1 year ago

      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).

      • xmunk
        link
        fedilink
        arrow-up
        1
        arrow-down
        1
        ·
        1 year ago

        Think of it as inline attribution/documentation.