Question

我的大学有一些项目，我需要将一些数据从文件转换成矩阵表示。

主要问题是在返回表单“returnNeighbours（int node）”之后，在邻居对象上调用析构函数（正如我从运行gdb得出的结论）。

我知道当函数中的局部变量被初始化时，总是会调用析构函数，但是neihbours是一个类成员。我不会发布所有内容，因为我认为没有必要。我有一些列出的结构。

representation.cpp

NodeContainer::NodeContainer(){ size = 0; array = nullptr; }
NodeContainer::~NodeContainer(){ size = 0; delete[] array; }
void NodeContainer::allocateMemoryAndSetSize(int n){ size = n; array = new int[size]; }

void MatrixRep::convertDataToMatrixRep(int** array)
{
  for(int i = 0 ; i != size; i++)
    for(int j = 0; j != size; j++)
        matrix[i][j] = array[i][j];
}

NodeContainer MatrixRep::returnNeighbours(int node)
{
    deleteNeighboursIfAny();
    if(!checkIfNotBeyondMatrix(node))
        return neighbours;

    neighbours.allocateMemoryAndSetSize(countNeighbours(node));

    for(int i = 0, j = 0; i < size; i++)
        if(matrix[node-1][i] != 0)
        {
            neighbours.array[j] = matrix[node-1][i];
            j++;
        }
    return neighbours;
}

void MatrixRep::deleteNeighboursIfAny(){ if(neighbours.array) neighbours.~NodeContainer(); }

bool MatrixRep::checkIfNotBeyondMatrix(int node)
{
    if(node == 0 || node > size)
    {
        std::cerr<<"There is no such a node!\n";
        return false;
    }
    else
        return true;
}

int MatrixRep::countNeighbours(int node)
{
    int count_non_zero = 0;

    for(int i = 0; i != size; i++)
        if(matrix[node-1][i] != 0)
            count_non_zero++;
    return count_non_zero;
}

representation.h

struct NodeContainer
{
    int size;
    int* array;

    NodeContainer();
    ~NodeContainer();
    void allocateMemoryAndSetSize(int);
};  

class MatrixRep
{
    int size;
    NodeContainer neighbours;
    int** matrix;

    public:

    MatrixRep(int);
    ~MatrixRep();
    void convertDataToMatrixRep(int**);
    NodeContainer returnNeighbours(int);
    void deleteNeighboursIfAny();
    bool checkIfNotBeyondMatrix(int);
    int countNeighbours(int);
    void setupMatrix();
    void deleteMatrix();
};

Answer 1

如果要返回NodeContainer的副本，必须为其实现复制构造函数和赋值运算符。如果您正在使用符合C ++ 11的编译器，那么也可以实现移动构造函数和移动赋值运算符。

另一方面，如果您不想创建副本，必须返回指向该成员的指针或引用。您也可以将该成员设为std :: shared_ptr，在这种情况下您可以返回。

但是，在当前的实现中，您实际上正在返回NodeContainer的浅副本。一旦你的副本超出范围，它的析构函数就被调用，它会释放它的内存，在这种情况下是你的成员的原始内存，有效地使你的成员无效。实施并不好。因此，根据您的目标，要么实施第一个建议的解决方案，要么实施第二个。

从函数返回后调用析构函数

1 个答案: