C++宏以确保释放已用内存？

Question

所以我有点不懂行，正在阅读有关 new、delete 和指针的内容，而且我确信在我的一生中我会忘记太多的删除操作。所以我想知道像下面这样的宏是否会比它的价值更麻烦。

#define withObject(ptr, value, BODY)                       \
{                                                           \
     ptr = value;                                           \
     BODY                                                   \
     delete ptr;                                            \ 
     ptr=NULL                                               \
}

这个宏会导致一些问题或者以意想不到的方式运行吗？

编辑：哎呀，我在释放时留下了 d 。好的，谢谢大家的回答。

Answer 1

不，不要这样做。它不会防止内存泄漏或杂散指针。相反，请阅读智能指针。当前的标准提供了一种 auto_ptr 的形式，但还有更多（和更好的）可用，具体取决于您的 C++ 平台。

另外，我怀疑您可能过度使用 new 和 delete - 它们应该在您的代码中很少使用。您应该更喜欢使用值。因此，不要像这样：

string * s = new string ( "foobar" );
....
delete s;

您应该简单地编写：

string s( "foobar" );

并让编译器为您管理字符串的生命周期。

Answer 2

无需在这里重新发明轮子，您正在寻找智能指针。

非常有用的 Boost 库有 智能指针功能。

Answer 3

google RAII 习惯用法；它是一样的，但不那么老套；然后，您可以使用无处不在的智能指针（当它们超出使用/范围时自动释放内存）将其应用于动态分配。

Boost 包含最著名/最广泛的智能指针版本（几种风格）。

C++ 标准一直有一种有缺陷的智能指针，名为 auto_ptr。这是一个有警告的地方，但它本身很有用（RTFM！）。

C++0x 采用了几个 Boost TR1 类，包括最流行的智能指针（我希望我的措辞正确，因为标准语言通常在细节上非常具体）

HTH

Answer 4

您正在寻找智能指针模式。它是标准 C++，在此站点中有详细解释。

Answer 5

除了您确实应该使用智能指针（即使是低级的 auto_ptr 也能满足您的目的）这一事实之外，BODY 还可能存在问题。使用可能很大的代码段作为类似函数的宏的参数有其自身的陷阱。例如，如果 BODY 中存在未加括号的逗号，则 BODY 会变成两个参数。我不确定进一步的问题，因为我从未见过有人尝试过。

不要在 C++ 中使用类似函数的宏。这很少值得。

Answer 6

看起来您希望分配一个对象，对其进行一小部分工作，然后删除它。为此，我建议使用 std::auto_ptr。

Answer 7

执行此类操作的惯用 C++ 方法不是使用宏，而是使用使用 资源的“智能指针”类获取就是初始化。例如，Boost 的 scoped_ptr 在上下文中可以很好地工作你的宏在哪里有意义。

我将在这里放置一个智能指针类的草图 - 但我省略了很多细节。这只是为了让您了解它们是如何工作的。使用scoped_ptr代替，他们没有遗漏任何东西。

template <typename T>
class smartptr<T> {
public:
    smartptr() : ptr(new T) {}
    smartptr(T* p) : ptr(p) {}
    ~smartptr() { delete ptr }
    T& operator*() { return *ptr; }
    T* operator->() { return ptr; }
private:
    T* ptr;
}

Answer 8

这个宏会导致一些问题或者以意想不到的方式运行吗？

如果 BODY 抛出异常，那么删除将被跳过。

解决方案是使用“RAII”习惯用法。它是现代 C++ 编程中最重要的概念之一。

Answer 9

我认为更好的解决方案是创建一个分配和删除宏而不是一个包罗万象的宏。这是一件非常混乱、没有多大帮助的事情

当我在 CI 中编程时，通常会执行如下操作：

#define ALLOC_ARY(type, sz) (type*)calloc(sz, sizeof(type))
#define ALLOC_STR(sz) ALLOC_ARY(char, sz)

char *string = ALLOC_STR(128);
int *array = ALLOC_ARY(int, 20);

但记住在 vanilla C 中释放只是其中的一部分：

free(string); string = NULL;
free(array); array = NULL;

编辑：对于 C++，智能指针正如其他人所建议的那样，这是一个好方法。