Скопируйте массив 2d int в двойной указатель в C

Question

Итак, у меня есть следующая функция. Функция принимает целевой int ** двойной указатель и копирует исходный массив.

void copyMatrix(int ** target, int m, int n, int source[m][n]) 
{ 
    int i,j; 

    target = (int **) malloc(m * sizeof(int *)); 
    for (i = 0; i < m; i++) 
    target[i] = (int *)malloc(n * sizeof(int)); 

    for(i=0; i < m; i++){ 
    for(j=0; j < n; j++){ 
    target[i][j] = source[i][j]; 
    } 
    } 
    printf("%d ", target[i][j]); 
} 

Когда я называю Printf после того, как для петель, я получаю ошибку сегментации, но если я называю Printf внутри для петель, он печатает мишень [я] [J] правильно. Почему это? Я разорву свои волосы за это ...

Answer 1

После петель, i == m и j == n. Они оба указывают 1 пункт за макс. Массивы в C индексируются с нулевой отметкой, поэтому доступ к target[n][m] массива с размером [n][m] даст доступ за пределы границ.

Answer 2

Непонятно, почему вы хотите сначала создать массив указателей. При таком подходе у вас возникает проблема изменения самого двойного указателя, который не будет отображаться в вызывающем. Вы можете просто выделить массив непосредственно для хранения входного 2D-массива.

Ваш код:

void copyMatrix(int ** target, int m, int n, int source[m][n]) 
{ 
    int i,j; 

    target = (int **) malloc(m * sizeof(int *)); 
    for (i = 0; i < m; i++) 
    target[i] = (int *)malloc(n * sizeof(int)); 

    for(i=0; i < m; i++){ 
    for(j=0; j < n; j++){ 
    target[i][j] = source[i][j]; 
    } 
    } 
    printf("%d ", target[i][j]); 
} 

Модифицированный код:

void copyMatrix(int ** target, int m, int n, int source[m][n]) 
{ 
    int i,j; 

    *target = (int *) malloc(sizeof(int) * (m * n)); 


    for(i=0; i < m; i++){ 
    for(j=0; j < n; j++){ 
    (*target)[i][j] = source[i][j]; 
    } 
    } 
    printf("%d ", (*target)[i][j]); 
}