在c ++中重新分配指针的正确方法

Question

编辑:我知道在这种情况下,如果它是一个实际的类,我最好不要将字符串放在堆上.但是,这只是一个示例代码,以确保我理解该理论.实际代码将是一棵红黑树,所有节点都存储在堆上.

我想确保在继续之前我有这些基本想法是正确的(我来自Java/Python背景).我一直在网上搜索,但还没有找到这个问题的具体答案.

当您重新指定指向新对象的指针时,是否必须先在旧对象上调用delete以避免内存泄漏？我的直觉告诉我是的,但在继续之前我想要一个具体的答案.

例如,假设您有一个存储指向字符串的指针的类

class MyClass
{
private:
    std::string *str;

public:
MyClass (const std::string &_str)
{
    str=new std::string(_str);
}

void ChangeString(const std::string &_str)
{
    // I am wondering if this is correct?
    delete str;
    str = new std::string(_str)

    /*
     * or could you simply do it like:
     * str = _str;
     */ 
}
....

在ChangeString方法中,哪个是正确的？

我想如果你不使用new关键字第二种方式我会被挂起,它仍然会像你预期的那样编译和运行.这只会覆盖此指针指向的数据吗？或者它做了别的什么？

任何建议都会受到很大的批评:D

Answer 1

如果必须解除分配旧实例并创建另一个实例,则应首先确保创建新对象成功:

void reset(const std::string& str)
{
    std::string* tmp = new std::string(str);
    delete m_str;
    m_str = tmp;
}

如果先调用delete,然后创建一个新的抛出异常,那么类实例将留下一个悬空指针.例如,您的析构函数可能最终再次尝试删除指针(未定义的行为).

您也可以通过将指针设置为NULL来避免这种情况,但上述方法仍然更好:如果重置失败,对象将保持其原始值.

至于代码评论中的问题.

*str = _str;

这是正确的做法.这是正常的字符串赋值.

str = &_str;

这将指定指针并完全错误.你会泄漏之前指向的字符串实例str.更糟糕的是,传递给函数的字符串很可能不是首先分配的new(你不应该将指针混合到动态分配和自动对象).此外,您可能正在存储其生命周期以函数调用结束的字符串对象的地址(如果const引用绑定到临时).