如何删除已取消引用的对象？

Question

我有以下代码

MyObject * func1() {
    MyObject * obj = new MyObject();
    // lots of stuff here
    return obj;
}

MyObject func2() {
    MyObject * obj = func1();
    // even more stuff here
    return *obj;
}

void main() {
    MyObject obj = func2()
}

，正如我从 得到的这里这段代码正在泄漏。这会：

MyObject * func1() {
    MyObject * obj = new MyObject();
    // lots of stuff here
    return obj;
}

MyObject func2() {
    MyObject * obj = func1();
    // even more stuff here
    MyObject obj_r(*obj);
    delete obj;
    return obj_r;
}

void main() {
    MyObject obj = func2()
}

解决问题吗？或者还有其他一些不错的解决方案吗？

在 b4 中：不，我不能从一开始就引用它，因为 func1() 在某些情况下返回 NULL。

upd：添加了一些评论，这样人们就不会认为我非常愚蠢

Answer 1

一个更优雅的解决方案（并且更“正确”）是使用智能
指针：（

MyObject func2()
{
    return *std::auto_ptr<MyObject>(func1());
}

使用更现代的编译器，请使用 std::unique_ptr。或者如果您使用
Boost，你也可以使用 boost::scoped_ptr。）

我说得更“正确”，因为如果 MyObject 的复制构造函数
抛出异常，该解决方案仍然会删除该对象，其中
因为你的会泄漏。

Answer 2

是的，这将解决内存泄漏问题。

一般来说，这不是一个很好的模式。但我不确定你想在这里实现什么目标！

Answer 3

避免内存泄漏的最佳方法是使用智能指针：

#include <memory>

MyObject func2() {
    std::unique_ptr<MyObject> obj(func1());
    // stuff here
    return *obj;
}

int main() {
    MyObject obj = func2();
}

这与您的解决方案几乎相同，但修复了在复制对象或执行“操作”时引发异常时您的内存泄漏问题。如果您不使用 C++11，请使用 auto_ptr 而不是 unique_ptr。

Answer 4

只需将 func2() 更改为：

MyObject func2() { return MyObject(); }

动态分配在您的情况下完全没有用。

Answer 5

是的，您的第二个解决方案将解决该问题，但代价是制作更多副本。

我的问题是：为什么要在堆上分配一个对象（用 new ）只是为了将其复制到堆栈分配的对象，然后立即删除它。为什么不一开始就在堆栈上创建它呢？

你可以这样写：

void main() {
    MyObject obj;
}

最后你会得到完全相同的结果。而且会简单得多！

Answer 6

恕我直言，最好的方法是删除 func1 中的一个对象（如果它必须删除 NULL）。
在 func2 中，如果我们收到空指针，我们无法以任何方式取消引用它。

Answer 7

如果您不确定 func1 是否会返回有效对象，则 boost::Optional 可能更合适，而不是返回指针 - 这会让 func2 检查 func1 是否返回了有效的内容，而不需要动态分配任何内容。