是否有基于参数设置变量的约定？答案

【问题标题】：Is there a convention for setting variables based on arguments?是否有基于参数设置变量的约定？
【发布时间】：2012-04-19 20:46:03
【问题描述】：

假设我有一个带有标志或其他东西的函数：

void foo (Param p1, Param p2, bool setVariable)
{
    //if setVariable is true, set some bool var to true, else false
}

是否有强烈偏好以下哪一项？

if (setVariable)
    _someClassVariable = true;
else
    _someClassVariable = false;

或

_someClassVariable = setVariable;

显然，第二个打字更少，但第一个让我觉得更具可读性。哪一个会更受欢迎？

【问题讨论】：

【解决方案1】：

我通常更喜欢第二种。对我来说，第一个是强烈警告，写代码的人充其量只是勉强胜任。

也就是说，我也倾向于建议不要将布尔值作为参数传递。 foo(true); 与 foo(false); 的真正含义很少立即显而易见。通常最好使用枚举，这样您就可以得到foo(do_this); 与foo(do_that);

【讨论】：

【解决方案2】：

我会选择第二个，它同样具有可读性。可读性更多的是变量的名称，而不是两个选项的选择。如果变量和参数有好的名字，那么赋值就会很自然。这类似于从您不会执行的函数返回布尔值：

bool conditionHolds() {
    if (condition)
       return true;
    else
       return false;
}

（如果您正在考虑这样做，请重新考虑）

【讨论】：

【解决方案3】：

我更喜欢第二个。如果减少打字对你来说还不够，还要考虑你同事的意见。如果我看到类似的东西，我个人会笑

if (condition)
   return true;
else
   return false;

在生产代码中。（假设变量是 bool，并且您没有使用它来实现某些转换机制）。

【讨论】：

【解决方案4】：

后者要好得多，if else 看起来毫无用处，并在代码和编译器生成的代码生成中引入了额外的复杂性（尽管可能已优化掉）。

我也会避免使用前导下划线符号，其中一些名称是为标准库和编译器保留的。

【讨论】：