我应该删除函数中的本地指针吗？ （C++）

Question

一个问题：

我应该删除在函数中获取的指针（不是创建，只是提取）？示例：

#include <SomeObject> 

#define SAFE_DELETE(p) { if (p) { delete (p); (p) = NULL; } } 

class DraftObject 
{ 
public: 

    DraftObject() : _x(0) {} 
    ~DraftObject(){} 

    int CalculateSomething() 
    { 
     AnotherObject* aObj = SomeObject::getInstance()->getAObjPointer(); 

     /* Do some calculations and etc... */ 
     _x += aObj->GetSomeIntValue(); 

     SAFE_DELETE(aObj) // <-- Would you recomend this here? 

     return _x; 
    } 

protected: 
    int _x; 
}; 

aObj在其他情况下也会在SomeObject实例中重用。我可以继续，并且总是拨打SomeObject::getInstance()->getAObjPointer()来满足我需要的所有内容，但我个人认为SomeObject::getInstance()->getAObjPointer()->GetSomeIntValue()的可读性不如aObj->GetSomeIntValue()。 我知道，如果我使用boost（shared_ptr，weak_ptr或auto_ptr），我不需要担心，但是我对它的工作方式更加好奇。不会删除指针会产生内存泄漏情况，或者删除指针会将其从内存中删除，以便在其他作用域（实例对象以及可能使用的任何其他位置）中消失？

有什么想法？

干杯。

Answer 1

这取决于。

如果SomeObject::getInstance()->getAObjPointer();每个调用返回一个不同的对象，可能是。否则，不。 这应该记录在。

而且，你的 “安全删除”：

#define SAFE_DELETE(p) { if (p) { delete (p); (p) = NULL; } } 

是毫无用处。而且丑陋。如果我在代码中看到这个，我会去嘲笑编写它的程序员。如果p是NULL，则删除是安全的。

Answer 2

不，请不要使用delete，因为这会为对象释放内存，并且以后将无法使用它。

Answer 3

它应该记录在API中。

一些库返回的指针不应该被用户删除，因为它们也保存在内部数据结构中。其他人正在创建应该由用户删除的指针。

假设你实际上已经为自己写了这个类和函数，如果你没有在其他任何地方（包括内部函数）使用这个实例，你可能想要删除函数末尾的指针。