C++関数から動的に割り当てられた2D配列を返す方法

Question

これを実行するたびにメモリエラーが発生します。私も閉じていますか？

int **createField(int N, int ** ary) { 
    ary = new int*[N]; 
    for(int i = 0; i < N; ++i) 
     ary[i] = new int[N]; 

    for(int i = 0; i < N; ++i) 
     for(int j = 0; j < N+1; ++j) 
      ary[i][j] = 0; 

    return ary; 
} 

int playGame (int N, int ** ary) { 
    cout<<"Enter the coordinates of the " << N << " shots:"<<endl; 
    for(int i = 0; i < N; i++) 
    { 
     for(int j = 0; j < N; j++) 
     { 
      cout<<"Enter element ["<<i<<"]["<<j<<"]: "; 
      cin>>ary[i][j]; 
     } 
    } 
    return 0; 
} 

int main() { 
    int N; 
    cout << "Enter the number of Redshirts: "; 
    cin >> N; 
    int** ary; 
    createField(N, ary); 
    playGame(N, ary); 
} 

Answer 1

次はary

int** ary; 
createField(N, ary); 

を初期化していない、それは

int** ary; 
ary = createField(N, ary); 

する必要がありますが、実際にcreateFieldの第2引数は不要ですので、それは

int **createField(int N) { 
    int** ary = new int*[N]; 
    for(int i = 0; i < N; ++i) 
     ary[i] = new int[N]; 

    for(int i = 0; i < N; ++i) 
     for(int j = 0; j < N+1; ++j) 
      ary[i][j] = 0; 

    return ary; 
} 

だろう

その後、

int** ary = createField(N); 

しかし、より良いは、手動でメモリを扱えないためにstd::vectorを使用することです。