c++ - 在 IF 语句中读写变量

Question

我希望在单个 IF 语句中执行以下步骤以节省代码编写:

如果ret 为TRUE，则将ret 设置为函数lookup() 的结果。如果 ret 现在是 FALSE，打印错误信息。

我为此编写的代码如下:

BOOLEAN ret = TRUE;

// ... functions assigning to `ret`

if ( ret && !(ret = lookup()) )
{
    fprintf(stderr, "Error in lookup()\n");
}

我觉得这并不像看起来那么简单。在 IF 语句中读取、分配和再次读取同一变量。据我所知，编译器总是会根据优先级将这样的语句拆分成它们的组成操作，并一次评估一个 union ，在将操作数评估为 false 时立即失败，而不是评估所有操作数。如果是这样，那么我希望代码遵循我上面写的步骤。

我在 IF 语句中使用过很多赋值，我知道它们有效，但不能与事先读取的另一个一起使用。

这不是好的代码有什么理由吗？就我个人而言，我认为它很容易阅读并且意思很清楚，我只是担心编译器可能出于某种原因不会产生等效的逻辑。也许编译器供应商的差异、优化或平台依赖性可能是一个问题，但我对此表示怀疑。

Answer 1

...为了节省代码编写 这几乎从来都不是一个有效的论点。不要这样做。特别是，不要将您的代码混淆成有错误、不可读的困惑以“节省输入”。这是非常糟糕的编程。

I've got a feeling that this isn't as simple as it looks. Reading from, assigning to and reading again from the same variable in an IF statement.

正确。不过，它与 if 语句本身关系不大，而与所涉及的运算符有关。

As far as I'm aware, the compiler will always split statements like this up into their constituent operations according to precedence and evaluates conjuncts one at a time

嗯，是的......但是有operator precedence and there is order of evaluation of subexpressions ，它们是不同的东西。让事情变得更复杂的是sequence points .

如果您不知道运算符优先级和求值顺序之间的区别，或者如果您不知道序列点是什么，则需要立即停止将尽可能多的运算符塞入一行，因为在在这种情况下，您将到处写出可怕的错误。

在您的特定情况下，您可以摆脱糟糕的编程，只是因为作为一种特殊情况， && 运算符的左右评估之间恰好有一个序列点。如果您使用不同的运算符编写了一些类似的困惑，例如 ret + !(ret = lookup()，您的代码将具有未定义的行为。需要数小时、数天或数周才能找到的错误。嗯，至少你节省了 10 秒的打字时间!

此外，在 C 和 C++ 中都使用标准的 bool 类型，而不是一些自制的版本。

您需要将您的代码更正为更具可读性和安全性的代码:

bool ret = true;

if(ret)
{
  ret = lookup();
}

if(!ret)
{
  fprintf(stderr, "Error in lookup()\n");
}