Как передать ссылку на двумерный массив в функцию?

Question

[BiProg]

Пытаюсь передать ссылку в функцию, заполнить там, потом другой функцией вывести, но заполняет только одну строку, и при переходе на следующую крашит с ошибкой segmentation fault.

#include <stdio.h>
#include <stdlib.h>
#include <malloc.h>
#define XMAX 10
#define YMAX 5

int mat(int ***matrix, int m, int n) {
  for (int i = 0; i < n; i++)
    for (int j = 0; j < m; j++) {
      printf("x: %d / y: %d /  ", i, j);
      *matrix[i][j] = 1;
    }

  return 0;
}

int mato(int ***matrix, int m, int n) {
  for (int i = 0; i < n; i++)
    for (int j = 0; j < m; j++)
      if (j == m - 1) {
          printf("%d\n", *matrix[i][j]);
      } else {
          printf("%d ", *matrix[i][j]);
      }
  return 0;  
}

int main() {
    int **matrix = malloc(XMAX * YMAX * sizeof(int) + XMAX * sizeof(int*));
    int *ptr = (int*) (matrix + XMAX);
    for (int i = 0; i < XMAX; i++) {
        matrix[i] = ptr + YMAX * i;
    }
    
    if (mat(&matrix[0][0], XMAX, YMAX) != 0) {
      free(matrix);
      return 1;
    }
    
    mato(&matrix, XMAX, YMAX);
    
    free(matrix);
    return 0;
}

Answer 1

[jcmvbkbc]

int mat(int ***matrix, int m, int n) {

Зачем три звёздочки? Для двумерного массива достаточно было бы двух:

int mat(int **matrix, int m, int n) {
  for (int i = 0; i < n; i++)
    for (int j = 0; j < m; j++) {
      printf("x: %d / y: %d /  ", i, j);
      matrix[i][j] = 1;
    }

  return 0;
}

int mato(int **matrix, int m, int n) {
  for (int i = 0; i < n; i++)
    for (int j = 0; j < m; j++)
      if (j == m - 1) {
          printf("%d\n", matrix[i][j]);
      } else {
          printf("%d ", matrix[i][j]);
      }
  return 0;  
}

int main() {
    int **matrix = malloc(XMAX * YMAX * sizeof(int) + XMAX * sizeof(int*));
    ...
    if (mat(matrix, XMAX, YMAX) != 0) {
      free(matrix);
      return 1;
    }
    
    mato(matrix, XMAX, YMAX);
    ...
}

В коде с тремя звёздочками конкретно здесь:
mat(&matrix[0][0], XMAX, YMAX)
передаётся неправильный указатель, потому что matrix --- это не настоящий двумерный массив, и matrix и &matrix[0][0] указывают в разные места.

Ну и порекламирую свой давнишний обзорный ответ на подобный вопрос.

Comments

[BiProg]

Вот как раз бы понять что на что указывает, при разном указании, меньше было бы проблем :)
И первоначально там было просто "&matrix", без [0][0], уже экспериментировал

[jcmvbkbc]

Вот как раз бы понять что на что указывает

BiProg, для этого надо внимательно прочитать и понять вот этот фрагмент кода:

int *ptr = (int*) (matrix + XMAX);
    for (int i = 0; i < XMAX; i++) {
        matrix[i] = ptr + YMAX * i;
    }

Первые XMAX элементов массива matrix -- это указатели на строки массива, это пространство зарезервировано при вычислении указателя ptr, который указывает на данные первой строки.

[jcmvbkbc]

И первоначально там было просто "&matrix", без [0][0], уже экспериментировал

BiProg, это было правильно и работало бы с тремя звёздочками, если бы в функциях обращение к элементам массива было написано с учётом приоритетов операций * и [], т.е. не *matrix[i][j] а (*matrix)[i][j].