Динамические массивы внутри класса

Question

я делаю следующее упражнение на данный момент:

Общий класс Matrix (15 пт)

a) Create a class called Matrix, it should contain storage for M*N numbers of type double. Just like earlier, when choosing how to store your data it is often useful to know what we are going to use the data for later. In matrix operations we are going to access the different elements of the matrix based on their column and/or row, therefore it is useful to order the members of Matrix as an array. Also, we are going to need to change the size of the data stored in the matrix, therefore it should be dynamically allocated.

b) Create constructors for the matrix.

Create the following three constructors: Matrix() • Default constructor, should initialize the matrix into the invalid state.

explicit Matrix(unsigned int N) • Should construct a valid NxN Matrix, initialized as an identity matrix. (The explicit keyword is not in the syllabus, but it should be used here.)

Matrix(unsigned int M, unsigned int N) • Should construct a valid MxN Matrix, initialized as a Zero matrix. (All elements are zero.)

~Matrix() • The destructor of Matrix, should delete any dynamically allocated memory.

Мой класс далеко выглядит следующим образом:

class Matrix{ 
    private: 
     int rows; 
     int columns; 
     double* matrix; 
    public: 
     Matrix(); 
     explicit Matrix(int N); 
     Matrix(int M, int N); 
     ~Matrix(); 
}; 

А остальное мой код:

Matrix::Matrix(){ 
    double * matrix = NULL; 
} 

Matrix::Matrix(int N){ 
    double * matrix = new double[N * N]; 
    this->rows = N; 
    this->columns = N; 

    for(int i = 0; i < N; i++){ 
     for(int j = 0; j < N; j++){ 
      if(i==j) 
       matrix[i * N + j] = 1; 
      else 
       matrix[i * N + j] = 0; 
     } 
    } 
} 

Matrix::Matrix(int M, int N){ 
    double * matrix = new double[M * N]; 
    this->rows = M; 
    this->columns = N; 

    for(int i = 0; i < M; i++){ 
     for(int j = 0; j < N; j++) 
      matrix[i * N + j] = 0; 
    } 
} 

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

Я создал динамический массив и конструкторы правильно? Я позже в упражнении создаю три разных массива, используя три разных конструктора. Как это сделать? Если я пытаюсь что-то вроде этого

Matrix::Matrix(); 
Matrix::Matrix(3); 

или

Matrix::Matrix(3,4) 

я получаю следующее сообщение об ошибке:

Unhandeled exception at 0x773c15de in Øving_6.exe: 0xC0000005: Access violation reading location 0xccccccc0.

Что я делаю не так?

Answer 1

В ваших конструкторах, вы определяете локальную переменную

double * matrix = new double[N * N]; 

который Shadows переменную-член того же имени, так что член никогда не отформатирована.

Все вы должны это изменить его

matrix = new double[N * N]; 

И это очень не-C++ использовать this-> для доступа к члену, если это не абсолютно необходимо для устранения неоднозначности (который почти никогда)

Answer 2

В ваших трех конструкторах вы маскируете переменную-матрицу экземпляра локальной. Попробуйте это:

Matrix::Matrix(){ 
this->matrix = NULL; 
} 

Matrix::Matrix(int N){ 
this->matrix = new double[N * N]; 
this->rows = N; 
this->columns = N; 

for(int i = 0; i < N; i++){ 
    for(int j = 0; j < N; j++){ 
     if(i==j) 
      matrix[i * N + j] = 1; 
     else 
      matrix[i * N + j] = 0; 
    } 
} 
} 

Matrix::Matrix(int M, int N){ 
this->matrix = new double[M * N]; 
this->rows = M; 
this->columns = N; 

for(int i = 0; i < M; i++){ 
    for(int j = 0; j < N; j++) 
     matrix[i * N + j] = 0; 
} 

}

Answer 3

Вы будете найти более «C++» (а иногда единственный способ для инициализации членов):

Matrix::Matrix(int M, int N): rows (M), 
           columns (N), 
           matrix (new double[M * N]) 
{ 
    for(int i = 0; i < M; i++) 
     for(int j = 0; j < N; j++) 
      matrix[i * N + j] = 0; 

} 

Теперь попытайтесь понять это:

Matrix::Matrix(  int N): rows (N), 
           columns (N), 
           matrix (new double[N * N]) 
{ 
    for(int i = 0; i < N; i++) 
     for(int j = 0; j < N; j++) 
      matrix[i * N + j] = (i==j); 

} 

Если вы используете:

class Matrix{ 
private: 
    int rows; 
    int columns; 
    std::unique_ptr<double[]> matrix; 

вы обнаружите, что вам не нужно деструктор, и некоторые другие inerest вещь. Также прочитайте мой другой ответ.