为何-Wparentheses对if中用作真值的无括号赋值告警？双层括号作用解析

先看触发警告的代码片段：

long n = -1;
bool asking = true;
do
{
  ask_into("Enter the order of this Bezier curve (# of points):", n);
  if (asking = n <= 0) // <-- WARNING: "suggest parentheses around assignment used as truth value"
  {
    std::cout << "Please enter a number > 0.\n";
  }
}
while (asking);

这种看似多余的括号是否有合理依据？ #

当然有。-Wparentheses警告的核心是消除代码语义的歧义：C++允许把赋值表达式直接放在条件判断里，但这种写法很容易和相等判断==混淆——不管是写代码时的笔误，还是读代码时的误解。

加双层括号((asking = n <=0))不是多余，而是显式告诉编译器和其他开发者：我就是故意把赋值的结果作为判断条件，不是写错了。它相当于给这个表达式加了一个“意图声明”，既让编译器停止警告，也让代码可读性更强，避免后续维护时被当成bug修改。

是否仅为防止误将==写为=？ #

不全是。防止笔误是这个警告最常见的应用场景，但它的本质是避免隐式的、容易被误解的表达式逻辑。哪怕你确实是故意这么写，不加括号的话，其他读代码的人第一反应可能是“这里是不是打错了？”，而加括号能直接打消这种疑惑。所以它不止是防笔误，更是提升代码的可维护性。

有没有无需拆分赋值与条件的更优写法满足该警告要求？ #

有几种可选的写法，其中最推荐的还是双层括号：

• 双层括号：if ((asking = n <=0))，这是最简洁、通用的写法，符合所有主流编译器的警告要求，语义也清晰。
• 逗号表达式：if (asking = n <=0, asking)，利用逗号表达式返回最后一个值的特性，但可读性不如双层括号。
• 显式bool转换：if (static_cast<bool>(asking = n <=0))，通过强制转换明确告诉编译器这是一个布尔判断，但写法偏啰嗦，没必要。

另外不推荐依赖编译器特定的语法（比如GCC的__builtin_expect），会降低代码的可移植性。

内容的提问来源于stack exchange，提问作者Lily-Heather Crawford