変数が動的配列（C++）で使用されているときに変数が参照解除されているようです

Question

ここでも私たちは再びインターネットの良い人です。

//This is what is in the header file 
int *myArr[]; // A two-dimensional array representing holding the matrix data 

//This is what is in the definition file 
Matrix::Matrix(int n, int m) 
{ 
    myRows = n; 
    myColumns = m; 
    initialize(); 
} 

void Matrix::initialize() 
{ 
    *myArr = new int[myRows]; 

    for (int i=0; i < 3; i++)//Only set to 3 since myRows is acting crazy 
    { 
     myArr[i] = new int[myColumns]; 
    } 
} 

を何らかの理由で、私はちょうどそれが前を向いていた値を参照して停止するようだmyArr配列を作成するmyRows変数を使用する場合：

これは私が使用しているコードです。

たとえば、値3を与え、* myArr = new int [myRows]が実行された後、myRowsの値が9834496に変更されますが、これはわかりません。

「新しい」変数の参照を解除しますか？ または何か間違っていますか？

ああ、これは学校の練習プロジェクトです（回答しないとあなたのせいにしません）。私は将来のプロジェクトで間違っていたことを知るために、作業コードの回答が好きです。

Answer 1

int *myArr[]; 

これは間違っています。ポインタの配列のサイズもコンパイラに伝えなければなりません。もしあなたがint a[]と宣言すればどうですか？コンパイラに知られていないサイズのintの配列を作成するように指示していますが、これはC++では許可されていません。だからあなたはそれをすることができません。

私はこれを行うには、あなたをお勧めします：

int **myArr; 

void Matrix::initialize() 
{ 
    myArr = new int*[myRows]; //note int* here! 

    for (int i=0; i < myRows; i++) 
    { 
     myArr[i] = new int[myColumns]; 
    } 
} 

これが機能するようになりました。

Answer 2

std :: vector <>を使用する必要があります。メモリの割り当てと解放のすべての問題を扱います。 それはどんなバグもありません。

そして、アルゴリズムの本当の目標に集中してください。メモリ管理上

typedef std::vector<int> Ints; 
typedef std::vector<Ints> Matrix; 
Matrix myArray; 

Answer 3

:-)して交換してみてくださいません：

*myArr = new int[myRows]; 

myArr = new int*[myRows]; 

Answer 4

で、私はあなたがしている場合、プロジェクトがマルチレベルのポインタを使用する必要がわからないんだけど、もしあなたがこの問題に近づける別の方法でなければ、多次元配列を1つの大きなフラット配列として扱うだけです。

つまり、行の最後に到達すると、それ以降のインデックスは次の行の最初の要素になります。コードの外観は次のとおりです。

// In this approach the double pointer int** 
// is replaced with just a simple int* 
int *myArr; 

// Here's your Matrix ctor. Note the use of the initializer list 
Matrix::Matrix(int n, int m) : myRows(n), myColumns(m) 
{ 
    initialize(); 
} 

void Matrix::initialize() 
{ 
    myArr = new int[myRows * myColumns]; 

    /* This loop is no longer needed since we're allocating 
     one big chunk at once. 
    for (int i=0; i < 3; i++)//Only set to 3 since myRows is acting crazy 
    { 
     myArr[i] = new int[myColumns]; 
    } 
    */ 
} 

// To retrieve stuff from your array 
// you would do something like this: 
int Matrix::operator() (const int x, const int y) 
{ 
    return myArr[x * myRows + y]; 
}