0

我有两个简单的课程。一切都很好，当我在 A 类中有一个空的析构函数（空 ~A(){};）。当我在那里添加行以释放数组时，我的程序给了我分段错误。

#include <iostream>
#include <cstdlib>
#include <ctime>
using namespace std;

class A
{
    private :

        int n;
        int m;
        int **array;

    public :

        A();
        A(int nn, int mm);
        A(const A& a);
        A& operator=(A a);
        int getAt(int i, int j);
        int getN(){return n;}
        int getM(){return m;}
        ~A();
};

A::A()
{
    n = 0;
    m = 0;
    array = NULL;
}

A::A(int nn, int mm)
{
    n = nn;
    m = mm;

    array = new int*[n];
    for(int i = 0; i <n; i++)
        array[i] = new int[m];

    for(int i=0; i<n; i++)
        for(int j=0; j<m; j++)
            array[i][j] = rand()%2;
}

A::A(const A& a)
{
    for(int i=0; i <n; i++)
        delete[] array[i];
    delete [] array;
    array = NULL;

    n = a.n;
    m = a.m;

    array = new int*[n];
    for(int i = 0; i <n; i++)
        array[i] = new int[m];

    for(int i=0; i<n; i++)
        for(int j=0; j<m; j++)
            array[i][j] = a.array[i][j];
}

A& A::operator=(A a)
{
    for(int i=0; i <n; i++)
        delete[] array[i];
    delete [] array;
    array = NULL;

    n = a.n;
    m = a.m;

    array = new int*[n];
    for(int i = 0; i <n; i++)
        array[i] = new int[m];

    for(int i=0; i<n; i++)
        for(int j=0; j<m; j++)
            array[i][j] = a.array[i][j];

    return *this;
}

int A::getAt(int i, int j)
{
    return array[i][j];
}

A::~A()
{
    for(int i=0; i <n; i++)
        delete[] array[i];
    delete [] array;
    array = NULL;
}

class B
{
    private :

            A a;

    public :

            B(){}
            B(A aa);
            void setA(A aa);
            void showA();
};

B::B(A aa)
{
    a = aa;
}

void B::setA(A aa)
{
    a = aa;
}

void B::showA()
{
    int n = a.getN();
    int m = a.getM();

    for(int i=0; i<n; i++){
        for(int j=0; j<m; j++)
            cout << a.getAt(i,j) << " ";
        cout << "\n";
    }
    cout << "\n";
}

int main()
{
    A a(3, 4);
    B b(a);

    b.showA();

    return 0;
}

如何解决这个问题？为什么会发生？

解决了：

#include <iostream>
#include <cstdlib>
#include <ctime>
using namespace std;

class A
{
    private :

        int n;
        int m;
        int **array;

    public :

        A();
        A(int nn, int mm);
        A(const A& a);
        A& operator=(const A& a);
        int getAt(int i, int j);
        int getN(){return n;}
        int getM(){return m;}
        ~A();
};

A::A()
{
    n = 0;
    m = 0;
    array = NULL;
}

A::A(int nn, int mm)
{
    n = nn;
    m = mm;

    array = new int*[n];
    for(int i = 0; i <n; i++)
        array[i] = new int[m];

    for(int i=0; i<n; i++)
        for(int j=0; j<m; j++)
            array[i][j] = rand()%2;
}

A::A(const A& a)
{
    n = a.n;
    m = a.m;

    array = new int*[n];
    for(int i = 0; i <n; i++)
        array[i] = new int[m];

    for(int i=0; i<n; i++)
        for(int j=0; j<m; j++)
            array[i][j] = a.array[i][j];
}

A& A::operator=(const A& a)
{
    for(int i=0; i <n; i++)
        delete[] array[i];
    delete [] array;
    array = NULL;

    n = a.n;
    m = a.m;

    array = new int*[n];
    for(int i = 0; i <n; i++)
        array[i] = new int[m];

    for(int i=0; i<n; i++)
        for(int j=0; j<m; j++)
            array[i][j] = a.array[i][j];

    return *this;
}

int A::getAt(int i, int j)
{
    return array[i][j];
}

A::~A()
{
    for(int i=0; i <n; i++)
        delete[] array[i];
    delete [] array;
    array = NULL;
}

class B
{
    private :

            A a;

    public :

            B(){}
            B(A aa);
            void setA(A aa);
            void showA();
};

B::B(A aa)
{
    a = aa;
}

void B::setA(A aa)
{
    a = aa;
}

void B::showA()
{
    int n = a.getN();
    int m = a.getM();

    for(int i=0; i<n; i++){
        for(int j=0; j<m; j++)
            cout << a.getAt(i,j) << " ";
        cout << "\n";
    }
    cout << "\n";
}

int main()
{
    A a(3, 4);
    B b(a);

    b.showA();

    return 0;
}
4

2 回答 2

2

n您的复制构造函数首先用 new覆盖旧的n，然后尝试删除n旧数组的元素。但是如果数组的大小不同怎么办？循环n 后覆盖。

或者，更好的是，摆脱这个可怕的指针怪物并std::vector改用它。

于 2013-05-10T09:34:35.317 回答
1

首先，您违反了“三原则”（在任何分配内存的类中，您需要有一个复制构造函数和赋值运算符以适当的方式“处理”分配的内存）。

在“数组”为 NULL 的情况下，您的析构函数也会被破坏 - 您需要在运行循环以删除内部部分之前检查数组是否为 NULL。

于 2013-05-10T09:29:10.843 回答