13

我有一个返回这样一个数组的函数:

vector<string> GetString()
{
   vector<string> s;
   s.push_back("one");
   s.push_back("two");
   s.push_back("three");
   return s;
}

我这样称呼它:

   vector<string> mystrings=GetStrings();

我也可以按如下方式实现:

void GetString(vector<string> & s)
{
   s.push_back("one");
   s.push_back("two");
   s.push_back("three");
}

并以这种方式调用它:

 vector<string> mystrings;
 GetStrings(mystrings);

哪一个更好?

版本一是否将向量复制到另一个?如果是,那么如果向量很大,它就会很慢。

4

2 回答 2

17

哪一个更好?

他们做不同的事情。如果您想要一个仅包含这些字符串的向量,请使用第一个;如果您希望能够将这些字符串附加到现有向量,请使用第二个。

如果您想要第一个版本的语义,那么从更容易正确使用和更难错误使用的意义上说,这是“更好”的。

版本一是否将向量复制到另一个?

在现代 C++ 中,绝对不是:返回一个局部变量和从一个临时变量初始化都是通过移动而不是复制来完成的。即使您坚持使用 C++11 之前的编译器,也应该忽略这两个副本。如果你的编译器不支持移动语义或复制省略,那么你真的应该把它扔掉。

于 2013-10-22T13:43:24.020 回答
2

这主要是个人偏好问题,以及您可能需要使用的任何编码约定。

在 C++11 之前,有时会认为第二种方法更可取。性能保证良好,因为没有不必要的复制。另一方面,第一种方法有可能(理论上)调用一个相当昂贵的复制构造函数。

但在实践中,称为复制省略的优化通常会避免复制构造函数,这意味着无论哪种方式性能都一样好。但不能保证,因为它可能取决于编译器设置/功能等因素。

不过,C++11 引入了移动语义。它提供了一种消除复制的替代方法,并且内置于语言规范中(而不是可选的编译器优化)。从可读性的角度来看,这可以使第一个选项更可取,因为您的代码在做什么可能更明显。

于 2013-10-22T13:57:31.000 回答