编辑：我知道在这种情况下,如果它是一个实际的类,我最好不要将字符串放在堆上.但是,这只是一个示例代码,以确保我理解该理论.实际代码将是一棵红黑树,所有节点都存储在堆上.
我想确保在继续之前我有这些基本想法是正确的(我来自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
解决方法
如果必须解除分配旧实例并创建另一个实例,则应首先确保创建新对象成功：
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引用绑定到临时).
