保留和删除向量？

Question

我在删除向量数据时遇到问题。

当我创建数据时，我首先在数组中保留空间，然后调整向量的大小并复制数组的地址：

//Create the vertices
verts.reserve(vn); verts.resize(vn);
TriVertex *vertsaux = new TriVertex[vn];

for(int i=0, c=0; i<vn; i++, c+=3)
{
     vertsaux[i].SetId(i);
     vertsaux[i].SetCoords0(Vector3(vs[c], vs[c+1], vs[c+2]));

     //Inicializate texture vertices
     vertsaux[i].SetTextureCoords(Vector2(0.0f, 0.0f));
}

for(int i=0; i<vn; i++)
{
     verts[i] = &vertsaux[i];
}

但是在我的类的析构函数中，当我这样做时，它会给我一个运行时错误：

for (i=0; i < this->verts.size(); i++) {
     delete this->verts[i];
}

任何人都知道为什么可以这正在发生吗？

顺便说一句，由于一些实现细节，我不能只在 for 中创建 new TriVertex ...

Answer 1

您分配了一个对象数组，

TriVertex *vertsaux = new TriVertex[vn];

要再次删除该对象，您需要执行以下操作：

delete[] vertsaux;

您不能单独删除元素。

Answer 2

您并没有真正向我们提供足够的信息。它是什么
顶点的类型？为什么要使用 versaux？从什么
我几乎看不到，最合乎逻辑的解决方案是
std::vectorverts; 作为类成员，并且
直接初始化，无需中间动态
分配。也许：

verts.reserve(vn);

for ( int i = 0; i < vn; ++ i ) {
    verts.push_back( TriVertex() );
    TriVertex& current = verts.back();
    current.SetId( i );
    current.SetCoords( Vector3( *vs, *(vs+1), *(vs+ 2) ) );

    current.SetTextureCoords( Vector2( 0.0, 0.0 ) );
    vs += 3;
}

我没有看到任何中间动态的原因
分配的数组。 （如果有的话，乔的回答是正确的。）

Answer 3

当您使用分配数组时，

TriVertex *vertsaux = new TriVertex[vn];

必须通过调用使用 delete[] 来取消分配

delete[] vertsaux;

Answer 4

由于您创建了一个由 vertsaux 指向的 TriVertex 数组，因此您应该使用以下方法删除该数组：

delete[] vertsaux;

您无法删除数组中的各个元素。

完成上述 delete[] 后，您应该像这样删除 verts 的所有元素：

verts.clear();

这是因为自从您删除了 vertsaux 后， verts 内的指针无效。

我假设 verts 类似于：std::vector

Answer 5

delete this->verts[i];

这会删除您创建的动态分配数组中的一项，实际上是 UB 以这种方式删除它。

像这样的数组应该使用 delete[] 语句删除，这会删除整个数组。

Answer 6

这里有两件事：

1. 正如许多人指出的那样，分配数组需要将其作为数组删除。然而，我认为它掩盖了你真正想要在这里做的事情。更改您的分配，为每个项目执行“new TriVertex”。

2. 假设您的向量正在存储 TriVertex* 指针，您确实必须删除析构函数中的每个项目（如您所写）。

因此，修复 TriVertex 对象的初始分配策略应该可以解决问题。

for(int i=0, c=0; i<vn; i++, c+=3)
{
    TriVertex *vertsaux = new TriVertex;

    vertsaux->SetId(i);
    vertsaux->SetCoords0(Vector3(vs[c], vs[c+1], vs[c+2]));

    //Initializing texture vertices
    vertsaux->SetTextureCoords(Vector2(0.0f, 0.0f));
    verts[i] = vertsaux;
}