Question

我正在尝试创建一个动态的二维数组（在这种情况下，我几乎成功完成）。我有一个类，其中包括一些构造函数，析构函数和其他处理矩阵算术运算的函数。所以这是整个代码：

  #include <iostream>
#include <stdint.h>
using namespace std; 
int n,m,input,i,j;

class Matrix{
    int **matrix;
    int i,j,codeError;
    public:
        Matrix();
        Matrix(int);
        Matrix(int, int);
        Matrix(const Matrix&, int, int);
        ~Matrix();
        void setIJ(int, int);
        int getIJ(int, int);
        void set(int,int);
        void position();
        void plus(const Matrix&,const Matrix&,int, int);
        void minus(const Matrix&,const Matrix&,int, int);
        void multiply(const Matrix&,const Matrix&,int, int);
        //void multiply(int,int,int);
        void print(int, int);
    };
Matrix a,b,c,d,e;
Matrix::Matrix(){

        matrix=NULL;
        i=0;
        j=0;
    }
Matrix::Matrix(int n){
    matrix=new int*[n];
    for(i=0;i<n;i++)
    matrix[i]=new int[n];
    for (int i=0;i<n;i++){
        for(int j=0;j<n;j++){
            if(i==j)
            matrix[i][j]=1;
            else matrix[i][j]=2;
            }
        }
    }

Matrix::Matrix(int m, int n){
    matrix=new int*[n];
    for(i=0;i<n;i++)
    matrix[i]=new int[m];
    for (int i=0;i<n;i++){
        for(int j=0;j<m;j++){
            if(i==j)
            matrix[i][j]=0;
            else matrix[i][j]=1;
            }
        }
}


Matrix::Matrix(const Matrix &obj,int n, int m)

    {
        matrix=new int*[n];
    for(i=0;i<n;++i)
    matrix[i]=new int[m];
        for (int i=0;i<n;i++){
            for (int  j=0; j<n; j++){
                    matrix[i][j]=obj.matrix[i][j];
                }
            }
        }

Matrix::~Matrix(){
    delete [] matrix;
}


void Matrix::set(int n, int m){
     matrix=new int*[n];
    for(i=0;i<n;++i)
    matrix[i]=new int[m];

    for ( i=0;i<n;i++){
        for( j=0;j<m;j++){
            cin>>matrix[i][j];
            }
        }
    }
void Matrix::setIJ(int i, int j){
        int num;
        cout<<"matrix[i][j]="<<endl;
        cin>>num;
        matrix[i][j]=num;
        }


int Matrix::getIJ(int i, int j){
    return matrix[i][j];
    }

void Matrix::plus(const Matrix &obj,const Matrix &tmp,int n,int m){
    matrix=new int*[n];
    for(i=0;i<n;i++)
    matrix[i]=new int [m];
    for(i=0;i<n;i++){
        for(j=0;j<m;j++){
            matrix[i][j]=tmp.matrix[i][j]+obj.matrix[i][j];
        }
    }
}

void Matrix::minus(const Matrix &obj,const Matrix &tmp,int n,int m){
    matrix=new int*[n];
    for(i=0;i<n;i++)
    matrix[i]=new int [m];
    for(i=0;i<n;i++){
        for(j=0;j<m;j++){
            matrix[i][j]=tmp.matrix[i][j]-obj.matrix[i][j];
        }
    }
}

void Matrix::multiply(const Matrix &obj,const Matrix &tmp,int n,int m){
    matrix=new int*[n];
    for(i=0;i<n;i++)
    matrix[i]=new int [m];
    int temp1=0,temp2=0;
    for(i=0;i<n;i++){
        for(j=0;j<m;j++){
            for(int k=0;k<n;k++){
            temp1=tmp.matrix[i][k];
            temp2=obj.matrix[k][j];
            matrix[i][j]+=temp1*temp2;
            }
        }
    }
}

/*void Matrix::multiply(int n,int m,int scalar){

    matrix=new int*[n];
    for(i=0;i<n;i++)
    matrix[i]=new int [m];
    for(i=0;i<n;i++){
        for(j=0;j<m;j++){
            matrix[i][j]*=scalar;
        }
    }
}*/

void Matrix::print(int n, int m){
    for(int i=0;i<n;i++){
        cout<<endl;
        for(int j=0;j<m;j++){
            cout<<" "<<matrix[i][j];
            }
        }
    }

int main(){
        menu:
    cout<<"menu"<<endl;
    cout<<"1.Constructor without parameters"<<endl;
    cout<<"2.Constructor 1 parameter"<<endl;
    cout<<"3.Constructor 2 parameters"<<endl;
    cout<<"4.copy constructor"<<endl;
    cout<<"5.change number on IJ position"<<endl;
    cout<<"6.what is the number on IJ position"<<endl;
    cout<<"7.adition"<<endl;
    cout<<"8.substraction"<<endl;
    cout<<"9.multiply"<<endl;
    cout<<"10.print"<<endl;

    cin>>input;
    switch(input){
        case 1:{
            Matrix a;
            cout<<"write n si m"<<endl;
            cin>>n>>m;
            a.set(n,m);
            goto menu;
            }
        case 2:{
            Matrix b(3);
            goto menu;
            }
        case 3:{
            Matrix c(3,4);
            goto menu;
            }
        case 4:{
            Matrix d(a);
            goto menu;
            }   
        case 5:{
            cout<<"write i,j"<<endl;
            cin>>i>>j;
            a.setIJ(i,j);
            goto menu;
            }
        case 6:{
            cout<<"itnrodu i,j"<<endl;
            cin>>i>>j;
            a.getIJ(i,j);
            goto menu;
            }
        case 7:{
            e.plus(a,d,n,m);
            goto menu;
            }
        case 8:{
            e.minus(a,d,n,m);
            goto menu;
            }
        /*case 9:{

            }*/
        case 10:{
            a.print(n,m);
            b.print(3,3);
            c.print(3,4);
            d.print(n,m);
            e.print(n,m);

            goto menu;
            }
        }
    }

所以在主函数中包含switch语句之前，一切都通过手动调用每个函数完全正常。然而，我猜，前4个开关语句正常工作，但是当我进入5-10个案例时，我得到了分段故障（核心转储139）。我试着调试代码，它告诉我，当我将矩阵[i] [j]分配给其他东西时问题总是存在，例如：

void Matrix::setIJ(int i, int j){
    int num;
    cout<<"matrix[i][j]="<<endl;
    cin>>num;
    **matrix[i][j]=num;**<-here
    }

或在这里：

void Matrix::plus(const Matrix &obj,const Matrix &tmp,int n,int m){
matrix=new int*[n];
for(i=0;i<n;i++)
matrix[i]=new int [m];
for(i=0;i<n;i++){
    for(j=0;j<m;j++){
        **matrix[i][j]=tmp.matrix[i][j]+obj.matrix[i][j];**<-here
    }
}

}

我知道有些东西需要指针，但我仍然是编程的新手请帮帮我

Answer 1

这是问题所在。您在a范围内创建了Matrix个对象。但正如Lightness Races in Orbit所述：

[...]具有自动存储持续时间的对象不会被泄露＆＃34;当你case超出范围时

和：

[n3290：6.6 / 2]：退出范围（无论多么已完成），具有自动存储持续时间（3.7.3）的对象在这个范围内建造的东西按其相反的顺序销毁施工。 [..]

因此，当goto完成并且您离开goto的范围后，在范围下创建的case个对象将被销毁。这就是为什么当您在之后使用它时，您获得了Matrix

分段故障（核心转储139）

1 个答案: