0

我正在像这样在堆上创建一个整数向量:

std::vector<int> vec = *new std::vector<int>();

然后我到了我的程序的结尾,我需要释放内存,但使用vec.clear()不会释放内存。

我该如何正确地做到这一点?

谢谢,一切顺利-米切尔

4

2 回答 2

4

我该如何正确地做到这一点?

替换这个:

std::vector<int> vec = *new std::vector<int>();

有了这个:

std::vector<int> vec;

问题解决了。

与您可能遇到的其他语言不同,new在大多数情况下最好避免使用。它动态分配对象,就像在其他语言中一样。但与其他语言不同,C++ 没有垃圾收集器,因此您需要手动销毁动态分配的对象。但是,您编写代码的方式使这成为不可能。

您正在使用 new 动态分配一个对象,它返回一个指向该对象的指针。然后,您将取消引用该指针(通过 *),并将对象复制到vec. vec被正确销毁,但从中复制它的动态分配的对象没有。而且由于您没有存储该指针,因此您无法访问该对象,也无法处置它。为了销毁该对象,您必须捕获指针,如下所示:

std::vector<int>* vec_pointer = new std::vector<int>();

然后,您可以在指针上调用 delete,这会破坏对象并释放内存:

delete vec_pointer;

值得庆幸的是,动态分配不是必需的,因为它在其他语言中通常是必需的。声明一个对象会创建它,当它超出范围时它会被销毁。因此,我向您展示的简单代码行就足够了,不需要删除语句。

附带说明一下,如果您出于某种原因确定必须进行动态分配。使用智能指针(谷歌)。

于 2013-01-17T04:01:06.013 回答
2

您的程序正在泄漏内存。

std::vector<int> vec在堆栈上声明一个向量。您在堆上创建第二个(空)向量,并使用它来复制构造堆栈上的那个。由于它是空的,这实际上什么都不做。

但是,您丢失了指向在堆上创建的向量的指针(因为您从未存储它)。所以你不能删除它,那个内存也不能回收。然而,堆栈上的向量会自行清理干净。

你可能想要的只是:

std::vector<int> vec;    // Vector on stack, no manual memory management required

如果你出于某种原因真的想使用堆(堆栈更快,并且vector对象本身是一个小的固定大小,无论你放入多少元素,所以你不必担心堆栈溢出) , 你可以做:

// Declare pointer to vector, and initialize it with a new vector on the heap
std::vector<int>* vec = new std::vector<int>();

甚至(在 C++11 中):

auto vec = new std::vector<int>();

然后,当你完成它时:

delete vec;
于 2013-01-17T04:03:28.420 回答