我正在像这样在堆上创建一个整数向量:
std::vector<int> vec = *new std::vector<int>();
然后我到了我的程序的结尾,我需要释放内存,但使用vec.clear()
不会释放内存。
我该如何正确地做到这一点?
谢谢,一切顺利-米切尔
我该如何正确地做到这一点?
替换这个:
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;
值得庆幸的是,动态分配不是必需的,因为它在其他语言中通常是必需的。声明一个对象会创建它,当它超出范围时它会被销毁。因此,我向您展示的简单代码行就足够了,不需要删除语句。
附带说明一下,如果您出于某种原因确定必须进行动态分配。使用智能指针(谷歌)。
您的程序正在泄漏内存。
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;