使用流的字符串连接返回垃圾

Question

我可能应该开始声明我对C++很陌生，而且，由于我来自更高级别的语言，我可能会遗漏一些我无法识别的技术问题。

我有以下方法：

const char * Point::toString() const {
    std::ostringstream stream;
    stream << "[" << x << ", " << y << "]"; //[3, 5] for example
    return stream.str().c_str();
}

然后我这样称呼它：

Point p1 (3, 5);
std::cout << p1.toString() << std::endl;

但是，这会打印一些垃圾。

我做错了什么？另外，我的toString()方法真的有效吗？我是否通过分配新char *而从未释放它来泄漏内存？

Answer 1

string::c_str()返回的 char 指针仅在字符串的生存期内有效（并且仅在未修改时有效）。

但是stream.str()返回一个临时字符串对象，您需要将其存储在某个地方。否则，它将在您调用该函数的确切语句之后被销毁。

但即使你写

string result = stream.str();
return result.c_str();

然后销毁函数内部的字符串对象，使 c_str() 返回的 char 指针无效，并在使用（取消引用）后立即导致未定义的行为。

最后，最好的方法是只返回一个字符串：

return stream.str();

Answer 2

你可以只做 返回strdup (stream.str().c_str()) ，但你应该自己管理这个内存。