在这种情况下返回参考是一种好的风格吗？

Question

我刚刚偶然发现了 Evan Teran 在 Split a string in C++? 上的回答，他在其中通过引用获取了一个向量，对其进行了修改并返回了它的引用。

std::vector<std::string> &split(const std::string &s, char delim, std::vector<std::string> &elems) {
  std::stringstream ss(s);
  std::string item;
  while (std::getline(ss, item, delim)) {
    elems.push_back(item);
  }
  return elems;
}

这种风格好吗？我看到好处是直接使用结果，就像在

std::vector<std::string> vec;
unsigned int uiSize = split("ssad asda", ' ', vec).size();

而不是

std::vector<std::string> vec;
split("ssad asda", ' ', vec);
unsigned int uiSize = .size()

有什么缺点吗？据我所知，反对返回引用的主要原因是人们可能会返回一个在返回后被破坏的局部变量的引用。这在这里不适用，因为对象不是本地的，而是作为参数提供给方法的。

Answer 1

代码

std::vector<std::string> &split(const std::string &s, char delim, std::vector<std::string> &elems) {
  std::stringstream ss(s);
  std::string item;
  while (std::getline(ss, item, delim)) {
    elems.push_back(item);
  }
  return elems;
}

被误导了：它更像是一个append_lines 而不是split 函数。

但鉴于此，返回对向量的引用是有意义的，是的。按值返回通常不能从移动语义中受益，因此会产生一些不必要的开销。而void 结果可能会使调用代码变得非常笨拙和尴尬。