C++의 클래스 함수에서 2 차원 배열 참조

Question

printMatrix 클래스 함수의 생성자에서 만든 2D 배열에 액세스 할 수 없습니다. 내가 간단한 cout으로 main에서 함수를 호출 할 때 < < "test"; 그것을 인쇄합니다. matrixArray [] []의 값을 인쇄하려고하면 아무 것도 인쇄되지 않고 프로그램이 종료됩니다. 2d 배열을 올바르게 참조하지 않습니까?

class matrix 
{ 
    int **matrixArray; 

public: 
    int matrixSize = 0; 
    matrix(int matrixSize); 
    void printMatrix(); 
    void makeMagicSquare(); 
}; 
matrix::matrix(int const matrixSize) 
{ 
    this->matrixSize = matrixSize ; 


    int** matrixArray = new int*[matrixSize]; 
     for(int i = 0; i<matrixSize; i++){ 
      matrixArray[i] = new int[matrixSize]; 
     } 


    for(int row = 0; row < matrixSize ;row++) 
      { 
       for(int col = 0; col < matrixSize; col++) 
       { 
        matrixArray[row][col] =0; 
        cout << matrixArray[row][col] << " "; 
       }//End for Col 
       cout << endl; 
      }//End for Row 

} 
//printMatrix Function 
void matrix::printMatrix(){ 

for(int row = 0; row < matrixSize;row++) 
     { 
      for(int col = 0; col < matrixSize; col++) 
      { 
       cout << "test" << " "; 
       //Not able to print from print function 
       cout << matrixArray[row][col] << endl; 
      }// end col 
      cout << endl; 
     }//end row 

} 

Answer 1

어떻게 할 수 있습니까?

int** matrixArray = new int*[matrixSize]; 

따라서 matrixArray 생성자 내부에서만 볼 생성자 내부 로컬 변수이다.

대신 데이터 멤버를 사용하면 잘 될 것입니다, 당신은 이미 정의 된 이후 회원, 당신은이 작업을 수행해야 :

matrixArray = new int*[matrixSize]; 

는 소멸자에서 메모리를 삭제하는 것을 잊지 마세요! ;)

Answer 2

매우 일반적인 실수를 범했습니다. 로컬 범위 변수/매개 변수와 멤버 변수를 구별하지 못했습니다.

int** matrixArray = new int*[matrixSize]; 

이 표현은 형태로 시작하기 때문에

, 그것은 반원 (당신이 그것을 액세스 할 수 this->matrixArray를 사용 할 것 "숨어"인 새로운, 지역 변수의 선언이며, 그것은 것 초기화되지 않음). 귀하의 컴파일러 당신에게 "그림자 변수"경고를 제공해야합니다.

일반적인 방법은 멤버 변수를 구별 자의 제공하는 것입니다 : 어떤 사람들은 "M_"로 접두사, 일부는, 그래서 당신의 코드가 될 것 뒤에 "_"를 넣어 :

class matrix 
{ 
    int **matrixArray_ = nullptr; //1 
    int matrixSize_ = 0; //2 

public: 
    matrix(int matrixSize); 
    ~matrix(); //7 
    void printMatrix() const; 
    void makeMagicSquare(); 
    int size() const { return matrixSize_; } //3 
}; 

matrix::matrix(int const matrixSize) 
    : matrixSize_(matrixSize) //4 
{ 
    matrixArray_ = new int*[matrixSize_]; //5 
    for(int i = 0; i<matrixSize_; i++) { 
     matrixArray_[i] = new int[matrixSize_](); //6 
    } 
} 

matrix::~matrix() //7 
{ 
    if (!_matrixArray) 
     return; 
    for (int i = 0; i < matrixSize_; ++i) { 
     delete [] matrixArray_[i]; 
    } 
    delete [] matrixArray_; 
} 

void matrix::printMatrix() const 
{ 
    for(int row = 0; row < matrixSize_; row++) { 
     for(int col = 0; col < matrixSize_; col++) { 
      cout << "test" << " "; 
      cout << matrixArray_[row][col] << endl; 
     }// end col 
     cout << endl; 
    }//end row 
} 

1 : 추가 된 '_ '접미사뿐만 아니라

2, 우리가 실수로 여기에 쓰레기하지 않도록 할 수 있습니다 추가'가 외부에서 불통되지 않도록 _ '접미사 내가 비공개,

3 : 추가 된 'size()'접근 자 멤버 함수,

4 : 이니셜 라이저 목록 "matrixSize"매개 변수를 복사에 "matrixSize_"회원

5 : 제거 '내부 **'우리는 'matrixArray_'사용하고주의 - 회원

(6) : new int[matrixSize_] 뒤에있는 ()은 컴파일러에게 기본으로 을 지정합니다 (0으로 설정). 현대 C++에서는이 코드가 {}이되지만, 최신 C++ 컴파일러를 사용하고 있는지 잘 모르겠습니다. 작업.

7 : 비슷한 이유로

를 사용하는 메모리를 해제하기위한 소멸자를 추가, 당신의 유형 이름 구별하게 고려하는 것이 좋습니다 것입니다 : class Matrix 또는 class matrix_t을.