Question

我制作了这个c ++代码：

std::string const &     Operand::toString() const
{
  std::ostringstream    convert;
  convert << this->value;
  return convert.str();
}

编译器告诉我：returning reference to temporary

我是否被迫将convert.str()放入Operand班级？

编辑：这是学校运动，我不能改变原型

Answer 1

convert.str();

这将返回一个std::string对象，该对象将在Operand::toString()返回后被销毁。因此，这是临时变量，其寿命仅限于此函数的范围。您应该只返回string本身，按值：

std::string Operand::toString() const
{
  std::ostringstream    convert;
  convert << this->value;
  return convert.str();
}

或：

const std::string Operand::toString() const
{
  std::ostringstream    convert;
  convert << this->value;
  return convert.str();
}

Answer 2

只需更改功能即可返回std::string而不是std::string const &。你想在这里按价值返回。

Answer 3

只需删除&和const

即可

即

std::string Operand::toString() const
{
  std::ostringstream    convert;
  convert << this->value;
  return convert.str();
}

Answer 4

如果您无法更改原型 - 它必须返回引用而不是字符串本身 - 您必须将字符串保留在某处并返回对它的引用。也许创建一个静态字符串，为其分配convert.str()并返回对此静态字符串的引用。

std::string const & Operand::toString() const
{
  static std::string s;
  std::ostringstream convert;
  convert << this->value;
  s = convert.str();
  return s;
}

David Schwartz在评论中建议使用另一种解决方案（在不同答案中），但这取决于您是否可以将该成员字符串添加到Operand。

c ++：返回对临时的引用

4 个答案: