删除队列对象数组

Question

我正在处理一个包含queues 数组的对象，该数组的长度在调用构造函数之前不会确定。基本上它看起来像下面这样

#include <queue>

class myClass{
public:
  //public functions
private:
  //private functions and variables

  queue<int>* myQueue;
};

它是这样初始化的：

myClass::myClass(int numOfQueues){
  myQueue = new queue<int>[numOfQueues];
}

看起来，这一切都很好。它的功能与我希望的完全一样，但是现在每次退出程序时都会出现分段错误。该类中还有一些其他数组，它们以相同的方式初始化，但它们的类型是bool 和int，而不是队列。我的析构函数看起来像：

myClass::~myClass(){
  delete boolArray;
  delete intArray;
  delete myQueue;
}

现在我假设这个析构函数适用于boolArray 和intArray 指针，因为直到我添加myQueue 才开始出现段错误。有谁知道编写析构函数的正确方法是什么？是否有可能这就是我所要做的，而析构函数没有在适当的时间被调用？

Answer 1

Naveen 已经解决了这个问题。我想添加一个良好的编程习惯。

下面的用例也会产生删除问题。

void foo()
{
    myClass a;
    myClass b(a);
}

当我们声明 a 时，将创建一个新的 myQueue 实例。然而，当声明 b 时，将调用复制构造函数而不是 myClass::myClass(int numQueues) 构造函数。因此 a.myQueue == b.myQueue。

退出函数foo时，a的析构函数会删除myQueue，然后b的析构函数会尝试删除未引用的指针，这将导致致命错误。

一个好的编程习惯是相应地实现 copy constructor 和 = operator 或者声明 copy constructor 和 = operator private 以避免此类问题。

private:
    myClass(const myClass&);
    const myClass& operator=(const myClass&);

另请参阅boost::NonCopyable

Answer 2

将delete 与new[] 一起使用不仅会导致内存泄漏，还会调用未定义的行为。

与new[] 一起使用的delete 的正确形式是delete[]。

但是，在惯用的 C++ 中，始终建议使用 std::vector 而不是使用 C 样式的数组。使用 STL 容器时，您无需自己显式管理内存。

Answer 3

因为您使用new[] 分配，所以您应该在析构函数中使用delete[] myQueue;。否则它将调用未定义的行为。顺便说一句，如果您不想遇到此类内存管理问题，可以使用std::vector<std::queue<int> >。

Answer 4

你为什么不使用 std::vector 而不是数组？

你需要delete[] 数组，而不是delete - 你分配了new[]