const 变量的条件初始化

Question

以下基本代码是一个相当大的过程的一部分：

int x = foo();
if (x == 0) x = bar();

x 没有在其他任何地方修改，所以我可以这样做：

const int x = foo() == 0 ? bar() : foo();

但是foo() 是一个非常昂贵且复杂的函数，因此我不能调用它两次，因为它可能会产生竞争条件并因此获得不同的值（它可能涉及读取外部资源）。

我想让代码尽可能可读，并且尽可能简短。一种选择是：

const int foo_ = foo(), x = foo_ == 0 ? bar() : foo_;

另一方面，我想避免这种时间变量，主要是因为foo() 可能依赖于外部资源，所以在其余代码中使用foo_ 作为缓存值是无效的。

我正在发布我现在正在使用的解决方案，但我想知道是否有更好的选择（没有或很少有代码混乱，同一范围内没有时间变量，可读性......）。提前致谢！

PS：它必须至少遵循 C++11 标准，因为它属于跨平台项目。

我知道这可能是基于意见的，但鉴于前面关于简单性（不是混乱代码）和避免时间变量（不是为了可读性而是为了代码安全）的陈述，我想知道解决这个问题的选项。

Answer 1

一个简单的呢：

const int temp_foo = foo();
const int x = (temp_foo == 0) ? bar() : temp_foo;

Answer 2

基本上，您想要Elvis operator，但 C++ 没有。这是您可能想要第二次使用的那种东西。因此，我选择了通用解决方案，而不是您提出的 lambda 解决方案。

例如

#include <functional>

int func_elvis (std::function<int ()> func1, std::function<int ()> func2) {
  int tmp = func1();
  return tmp ? tmp : func2();
}

然后像这样使用

const int x = func_elvis(foo, bar);

如果你喜欢自己调用函数，你可以这样做

#define ELVIS(A, B) func_elvis([](){ return A; }, [](){ return B; })

并像使用它

const int x = ELVIS(foo(), bar());

Answer 3

到目前为止我发现的解决方案是使用 lambda 函数，例如：

const int x = [](int n) { return n == 0 ? bar() : n; }(foo());

Answer 4

由于这是基于意见的问题，我会选择：

auto get_the_correct_x=[](){
    const auto temp=foo();
    return temp==0?bar():temp;
}
const auto x=get_the_correct_x();

一个好的名字而不是get_the_correct_x 会比仅仅尝试在没有命名的情况下使用 lambdas 更清晰。

Answer 5

除了 lambda，如果这种情况在您的程序中不会多次出现，您可以将函数 foo() 包装成一个函数，比如 conditionalFoo()：

int conditionalFoo() {
  int result = foo();
  if (result==0)
    result = bar();
  return result;
}
...
const int x = conditionalFoo();

Answer 6

如果你乐于使用gcc extensions，那么你可以写：

const int x = foo() ?: bar();