从向量中删除取消引用的元素

Question

我使用 new 创建了对象，但在将它们添加到向量之前取消了它们的引用。尽管在互联网上搜寻，我仍无法弄清楚如何对这些项目调用删除。我想只使用标准 C++ 和 STL 来做到这一点，我不想（例如）使用 Boost 库。

正如你所看到的，a、b 和 c 失去了作用域，我留下了我认为是向量中的副本的内容。我该如何删除这些。我不想在数组中存储指针，因为我需要向 API 函数传递一个双精度数组。

请问有人 - 我如何删除这些对象？

#include <iostream>
#include <vector>

using namespace std;

vector<double> vectorDouble;
void createObjects();

void createObjects() {
    double* a=new double(13);
    double* b=new double(14);
    double* c=new double(15);
    //a,b and c are not contiguous memory blocks
    cout << "memory location of a: " << a << endl;
    cout << "memory location of b: " << b << endl;
    cout << "memory location of c: " << c << endl;

    vectorDouble.push_back(*a);
    vectorDouble.push_back(*b);
    vectorDouble.push_back(*c);
}

int main() {
    createObjects();
    //the memory addresses are contiguous 8 byte chunks
    cout << "vector memory at 0: " << &(vectorDouble[0]) << endl;
    cout << "vector memory at 1: " << &(vectorDouble[1]) << endl;
    cout << "vector memory at 2: " << &(vectorDouble[2]) << endl;

    //get pointer to the 2nd element
    double *P=&(vectorDouble[1]); 

    //dereference and look inside - two memory locations both contain the value 14
    cout << "vector Pointer P ["<< P <<"] contains " << *P <<endl;

    //Which should I call delete on? I have lost reference to the original pointers.
    //How should I call delete on the vector?

    cout << "deleting pointer that references 2nd vector element" << endl;
    delete P; //********* CRASH **********
    cout << "Done deleting" << endl;
}

Answer 1

你需要删除的对象不在向量中，因为你的向量元素不是用new创建的。当您在 push_back 中使用取消引用运算符时，您只是将 malloc 的双精度值的副本传递给向量，而不是将实际分配的双精度值传递给向量。

您实际上只是泄漏了它们 - 您的代码无需分配双精度数即可正常运行：

void createObjects() {
    vectorDouble.push_back(13);
    vectorDouble.push_back(14);
    vectorDouble.push_back(15);
}

Answer 2

正如每个人所指出的，没有任何理由在程序中调用new：

void createObjects() {
    vectorDouble.push_back(13);
    vectorDouble.push_back(14);
    vectorDouble.push_back(15);
}

但是，假设您确实有某种理由调用new >。 （我无法想象它可能是什么，但我们假设你是一个天才）。这样做的方法如下：

void createObjects() {
    double* a=new double(13);
    double* b=new double(14);
    double* c=new double(15);
    //a,b and c are not contiguous memory blocks
    cout << "memory location of a: " << a << endl;
    cout << "memory location of b: " << b << endl;
    cout << "memory location of c: " << c << endl;

    vectorDouble.push_back(*a);
    vectorDouble.push_back(*b);
    vectorDouble.push_back(*c);

    delete a;
    delete b;
    delete c;
}

您看，push_back 不会将指针的副本放入向量中，而是将对象的副本放入向量中。一旦你复制了你的对象，那么你的对象的内存就不再有任何持续的用途，并且可以被销毁。

Answer 3

函数 createObjects 并未将分配的值放入向量中；它会放入值，然后泄漏a、b 和c 指向的内存。调用：

vectorDouble.push_back(*a);

将 a 指向的值存储在向量中（*a 取消引用指针，您可能已经知道了）。一旦该函数返回，指针就会丢失。您无法从向量中检索它们。您要么需要创建一个指向双精度的指针向量，要么（更有可能）甚至不分配值；只需存储双打即可。

Answer 4

我认为我遇到了问题：

vectorDouble.push_back(*a);
vectorDouble.push_back(*b);
vectorDouble.push_back(*c);

您将 a、b 和 c 作为值传递，因此数组并不真正包含您创建的变量（这就是为什么它们现在具有不同的内存地址，它们是变量的内容！）。然后你可以删除方法内的变量，不要在方法内使用指针或使用 double* 向量。

Answer 5

您的泄漏发生在 createObjects() 中，因为 std::vector<>::push_back() 复制了其参数。您应该在 createObjects() 作用域结束之前删除指针。

也就是说，我不明白为什么你一开始就使用动态分配。如果您可以避免这种情况，请这样做（您可以使用像 std::unique_ptr<> 这样的智能指针，或者更好的是使用普通的旧 double ）。