c++ - 在成员变量中存储 std::to_string(x).c_str() 会产生垃圾

Question

我有这个非常简单的 C++ 程序:

using namespace std;

class TheClass
{
private:
    const char *_numberString;

public:
    TheClass(int number)
    {
        _numberString = to_string(number).c_str();
    }

    operator const char *()
    {
        return _numberString;
    }
};

int main(int argc, const char * argv[])
{
   TheClass instance = 123;
   cout << (const char *)instance << endl;

   return 0;
}

当我在 Xcode 中运行它时，它会记录 \367\277_\377。但是，如果我将其更改为:

using namespace std;

class TheClass
{
public:   // Change 1/2
    const char *_numberString;

public:
    TheClass(int number)
    {
        _numberString = to_string(number).c_str();
    }

    operator const char *()
    {
        return _numberString;
    }
};

int main(int argc, const char * argv[])
{
    TheClass instance = 123;
    instance._numberString = to_string(123).c_str();   // Change 2/2
    cout << (const char *)instance << endl;

    return 0;
}

它会按应有的方式记录 123。我看不出我做错了什么。即使我将 123 更改为另一个数字，也会记录完全相同的内容。

Answer 1

此时

 _numberString = to_string(number).c_str();

您正在存储指向临时 std::string 值的驻留数据的指针，该值在该行代码之后无效。

访问 _numberString 有效地调用了未定义的行为。

---

如评论中所述，没有必要将 _numberString¹ 成员保留为 const char*。使用 std::string 成员代替:

class TheClass {
private:
     std::string  numberString_;

public:
    TheClass(int number) : numberString_(to_string(number)) {
    }

    operator const std::string& () {
         return numberString_;
    }
};

---

_{1) 您不应该为类成员名称使用前缀 _，这是为编译器和标准实现内在函数保留的。如果您不喜欢 m_ 等模式或其他前缀约定(例如我)，只需使用后缀 _ 即可，如我的示例所示。}