c++ - 性能与可读性 : Local copies in functions

Question

考虑以下代码:

Vector2f Box::getCenter() const
{
    const float x = width / 2;
    const float y = height / 2;

    return Vector2f(x, y);
}

这样写会不会提高性能:

Vector2f Box::getCenter() const
{
    return Vector2f(width / 2, height / 2);
}

我更喜欢第一个，因为它漂亮且可读，但我开始怀疑如果我这样做太多，我是否会失去一些性能，因为它会创建一个额外的不必要的拷贝。

我知道你们中的一些人认为第二个函数同样可读，但这只是一个例子，我想问的更笼统，在这种情况下什么是好的编码实践。

Answer 1

根据经验:编写可读的代码。

作为一种语言，C++ 的设计以效率为主要目标之一。在理想世界中，可读代码是高效代码，而 C++ 非常接近于此。您很少会为了性能而被迫编写不可读的代码。即便如此，您也不应该从一开始就这样做。始终首先编写可读性代码。只有当你测量时，你才能知道什么是更多的，什么是更少的表现。

这是检查编译器输出的好工具:https://godbolt.org/当您打开优化时，我希望任何体面的编译器都能发出非常相似的程序集。

这是另一个可以让您轻松对代码进行基准测试的工具:http://quick-bench.com/ .在知道它会产生影响之前不要开始优化。

编译器在优化事物方面相当强大，而且大多数时候他们比你更了解。当然你不应该做“愚蠢”的事情，例如显然你不会 return Vector2f(width * sin(pi/2) * exp( ln( 0.5)) , height/2); 即使 super 聪明的编译器可以意识到它是相同的作为您的原创。

最后但同样重要的是，不要忘记您编写的代码主要是供人们阅读和理解的。

PS:可读性非常主观。有人可能会争辩说第二个更具可读性，我会更进一步写

Vector2f Box::getCenter() const
{
    return {width * 0.5 , height * 0.5};
}

这样做的额外好处是，如果您决定更改返回类型(或者如果您将值类型更改为 int)，则不一定需要更改主体。