使用指针向量矩阵对对象调用析构函数时程序崩溃

Question

我有一个class，它基本上是指向另一个对象的指针2D matrix 的包装，并且矩阵由vectors 组成。

由于某种原因，每当我的class 的析构函数被调用时，程序就会崩溃，而且它似乎试图delete 指针，即使它们是nullptr，这会导致崩溃。

这是我的.h 和.cpp 文件：

cpp 文件：

RotationSolution.h:

#ifndef PUZZLESOLVER_ROTATIONSOLUTION_H
#define PUZZLESOLVER_ROTATIONSOLUTION_H

#include <vector>
#include <fstream>
#include "PuzzlePiece.h"

using namespace std;

class RotationSolution {
private:
    int _height, _width;
    vector<vector<PuzzlePiece*>> _matrix;

public:
    RotationSolution();

    RotationSolution(int height, int width);

    vector<PuzzlePiece*>& operator[](int row);

    int get_height() const;

    int get_width() const;
};


#endif //PUZZLESOLVER_ROTATIONSOLUTION_H

RotationSolution.cpp：

#include "RotationSolution.h"

vector<PuzzlePiece*>& RotationSolution::operator[](int row) {
    return _matrix[row];
}

RotationSolution::RotationSolution() : RotationSolution(0, 0) {}

RotationSolution::RotationSolution(int height, int width) :
    _height(height), _width(width), _matrix(vector<vector<PuzzlePiece*>>(_height, vector<PuzzlePiece*>(_width, nullptr)))
{}

int RotationSolution::get_height() const {
    return _height;
}

int RotationSolution::get_width() const {
    return _width;
}

代码实际上在如下所示的部分崩溃：

    for (auto length: _rowLengths) {
        auto height = size / length;
        _sol = RotationSolution(height, length);

        ...
    }

在_sol = RotationSolution(height, length); 行的第二次迭代中。

调试时，发送崩溃信号的代码来自new_allocator.h（我很确定这是一个lib文件）：

  // __p is not permitted to be a null pointer.
  void
  deallocate(pointer __p, size_type)
  { ::operator delete(__p); }

我还是 c++ 的新手，所以请原谅任何新手错误和不良做法:)

Answer 1

RotationSolution 类中存在设计缺陷 - 它包含存储原始指针的成员变量 _matrix，并且它错过了正确定义的赋值运算符，该运算符将克隆存储在 _matrix 中的对象。 Default generated copy assigment operator 只会复制指针，这可能会导致双重释放内存和崩溃（here 是对正在发生的事情和原因的一些解释）。尝试使用“vector> > _matrix”（也可以使用#include “memory”）它应该可以解决大多数内存管理错误的问题。这里是tutorial 了解智能指针。