Указатель на указатель динамического массива в C++

Question

Мне не повезло с динамическими указателями, когда я хочу его закрыть. почему приложение записано в память после завершения кучи буфера? как я могу закрыть свой массив?

int main() 
{ 
    . 
    . 
    int **W; 
    W = new int* [n]; 
    for (int i=1; i <= n; i++) 
     W[i] = new int[n]; 
    . 
    . 
    . 
    ast(n,W); 

    for(int i = 1; i <=n ; i++) 
    { 
     delete W[i]; 
    } 
    delete W; 
    getch(); 
} 
void ast (int n,int **W) 
{ 
    int **D; 
    D = new int* [n]; 
    for (int i=0; i < n; i++) 
     D[i] = new int[n]; 

    D=W; 
    for (int k=1;k<=n;k++) 
     for (int i=1;i<=n;i++) 
      for (int j=1;j<=n;j++) 
       D[i][j]=min(D[i][j],D[i][k]+D[k][j]); 
    . 
    . 
    for(int i = 1; i <=n ; i++) 
    { 
     delete D[i]; 
    } 
    delete D; 
} 

Answer 1

Допустимый диапазон индексов массива с N элементов [0, N-1]. Таким образом, вместо того, чтобы, например, эта петля

for (int i=1; i <= n; i++) 
     ^^^^ ^^^^^^ 

вы должны написать

for (int i = 0; i < n; i++) 

Как вы использовали оператор new [] вы должны использовать оператор delete [] Так вместо

for(int i = 1; i <=n ; i++) 
{ 
    delete W[i]; 
} 

и

delete W; 

вы должны написать

for (int i = 0; i < n; i++) 
{ 
    delete [] W[i]; 
} 

и

delete []W; 

Функция ast не имеет смысла, потому что кроме других ошибок имеет место утечка памяти. Сначала вы выделить память и присвоить его адрес указателя D, а затем вы перезаписать это значение указателя индексов

void ast (int n,int **W) 
{ 
    int **D; 
    D = new int* [n]; 
    for (int i=0; i < n; i++) 
     D[i] = new int[n]; 

    D=W; // <== ???