C++：如何安全地释放堆分配的向量数组？

Posted 2023-02-21

技术标签:

【中文标题】C++：如何安全地释放堆分配的向量数组？【英文标题】：C++ : How to safely deallocate a heap-allocated array of vectors? 【发布时间】：2020-12-13 17:14:20 【问题描述】：

我目前正在处理需要我制作向量数组的代码（我是 C++ 新手 - 如果这是一个绝对糟糕的想法，我将非常感谢反馈）。

假设我在堆上为我的向量分配内存，如下所示：

#include <iostream>
#include <vector>
#include <random>

int main() 
    typedef std::vector<double> doubleVec;
    long N = 1000;
    long M = 1000;

    doubleVec *array = new doubleVec[N];

    for (long i = 0; i < N; i++) 
        doubleVec currentVec = array[i];
        currentVec.resize(M);
        for (long j = 0; j < M; j++)
            currentVec[j] = std::rand();
    
    // ... do something with the data structure

   delete [] array;

当我完成了我需要对数据做的所有事情后，我应该如何安全地释放这个数据结构？

注意：我在最初的帖子中做错了一些我不打算成为讨论焦点的事情（未初始化的变量，没有调整矢量大小等）。我现在修好了。谢谢大家指出这些。

【问题讨论】：

使用智能指针，例如：std::unique_ptr 而不是原始指针，但“要点”是“为什么需要一堆指针”？你不使用std::vector<std::vector<double>>代替array的原因是什么？你用new[]创建的；你用delete[] 销毁它。不要想太多。 【参考方案1】：

问题不在于deallocating，而在于每个向量分配。您在代码中的哪个位置使用 M 值（访问元素时除外）？您的代码中还有其他问题，因此快速解决方法是：

    for (long i; i < N; i++) 
        doubleVec &currentVec = array[i];
        currentVec.resize(M);
        for (long j; j < M; j++)
            currentVec[j] = std::rand();

请特别注意currentVec 是一个引用：否则不会在array 中存储任何更改。

无论如何，每个人都会遇到的主要问题是：为什么需要一个向量数组？.. 向量向量是一个更优雅的解决方案。

更新：我错过了您忘记初始化i 和j 的事实。除了初始化它们的建议之外，我还建议使用 auto 关键字，这将使变量无法保持未初始化：

    for (auto i=0UL; i < N; i++) 
        doubleVec &currentVec = array[i];
        currentVec.resize(M);
        for (auto j=0UL; j < M; j++)
            currentVec[j] = std::rand();

0UL 表示 unsigned long 类型的零。

【讨论】：

因为我的判断一直严重失误。【参考方案2】：