我的问题很简单:如果我有一些类Man,并且我想定义返回man名字的成员函数,我应该选择以下两个变体中的哪一个?
首先:
string name();
第二:
void name(/* OUT */ string &name);
第一个变体效率不高,因为它复制了不必要的副本(赋值左侧的局部变量、返回值、->变量)。
第二个变体看起来相当有效,但它让我忍不住想写
string name;
john.name(name);
而不是简单
string name(john.name());
那么,我应该选择哪种变体,在效率和便利性/可读性之间的适当权衡是什么?
提前谢谢。
发布于 2012-05-11 22:16:30
这是一个很好的问题,你问它的事实表明你正在关注你的代码。然而,好消息是,在这种特殊情况下,有一种简单的解决方法。
首先,干净的方法是正确的方法。The compiler will eliminate unnecessary copies,在大多数情况下(通常是在有意义的地方)。
编辑(2016-06-25)
不幸的是,David Abaraham的网站似乎已经离线几年了,这篇文章已经丢失给了ethers (没有可用的archive.org副本)。我已经采取了自由上传我的本地副本作为存档的目的,它can be found here。
发布于 2012-05-11 22:17:03
使用第一个变体:
string name();
编译器很可能会优化掉任何不必要的副本。参见return value optimization。
在C++11中,移动语义意味着即使编译器不执行RVO,也不会执行完整复制。参见move semantics。
还要记住,如果另一种选择是
void name(std::string& s);
然后,您必须非常清楚地指定s
会发生什么情况,以及它在传递给函数时可以具有什么值,并且可能要么进行大量的有效性检查,要么简单地完全覆盖输入。
发布于 2012-05-11 22:23:32
既然您想要为您的类的一个字段创建一个getter,也许您应该这样做:inline const std::string& name() const { return this->name; }
由于名称是作为常量引用返回的,因此不会在类外部进行修改,也不会通过返回名称来创建副本。
在那之后,如果你想篡改这个名字,你就必须复制一份。
https://stackoverflow.com/questions/10553091
复制相似问题