将多个类调用存储在一个变量中总是更快吗？

Question

如果你有这样的方法：

float method(myClass foo)
{
    return foo.PrivateVar() + foo.PrivateVar();
}

这样做总是更快/更好吗？：

float method(myClass foo)
{
    float temp = foo.PrivateVar();
    return temp + temp;
}

我知道您不应该将像 foo.PrivateVar() 这样的调用放在 for 循环中，因为当您实际上只需要使用一次值时（在某些情况下），它会多次评估它。

for (int i = 0; i < foo.PrivateInt(); i++)
{
    //iterate through stuff with 'i'
}

据此，我假设将第一个示例中的代码更改为第二个示例中的代码，但后来人们告诉我不要试图比编译器更聪明！并且它可以很好地内联调用。

我不想描述任何东西，我只是想要一些简单的规则，以便在这方面进行良好的实践。我正在为一个工作申请编写一个演示，我不希望任何人看到代码并看到一些菜鸟错误。

Answer 1

这完全取决于PrivateVar() 正在做什么以及它的定义位置等。如果编译器可以访问PrivateVar() 中的代码，并且可以通过调用该函数来保证没有副作用，它可以执行 CSE，即基本上是您在第二个代码示例中所做的。

对于您的 for 循环，情况完全相同。因此，如果您想确保它只被评估一次，因为它是一个非常昂贵的函数（这也意味着即使没有任何副作用，也很难保证没有副作用）明确地编写它。

如果 PrivateVar() 只是一个 getter，编写更清晰的代码 - 即使编译器可能不执行 CSE，性能差异在 99.9999% 的所有情况下都无关紧要。

编辑：CSE 代表 Common Subexpression eliminiation 并且完全符合它的含义；) wiki 页面显示了一个简单乘法的示例，但我们也可以对较大的代码结构执行此操作，例如函数调用.

在所有情况下，我们都必须保证只评估一次代码不会改变语义，即为此代码执行 CSE：

a = b++ * c + g;
d = b++ * c * d;

并将其更改为：

tmp = b++ * c;
a = tmp + g;
d = tmp * d;

显然是非法的（对于函数调用，这显然有点复杂，但原理相同）。