分配二维数组的问题

Question

我还是 C、malloc 和所有爵士乐的新手，所以我决定写这篇文章来学习更多技能。这个想法是，我正在从文件中读取一堆整数并将它们放入矩阵（二维数组）中。文件的开头说明有多少行和列，因此它会读取这些数字并使用 malloc 设置二维数组。

int read_matrix(FILE *mat, int ***Z, int *x, int *y) 
{
    int i = 0;
    int x_temp = 0;
    int y_temp = 0;

    if (fscanf(mat, "%d %d", &(*x), &(*y)) == EOF){
        printf("File is not big enough to contain a matrix\n");
        return -1;
    }

    printf("About to malloc %d\n", *x);

    *Z = (int**) malloc(*x * sizeof(int*));
    while (i < *x) {
        printf("mallocing %d\n", i);
        *Z[i] = (int*) malloc(*y * sizeof(int));
        printf("malloced\n");
        ++i;
    }

    printf("Malloc complete\n");

    /*Other unimportant code*/
}

输出如下：

About to malloc 3 
mallocing 0 
malloced 
mallocing 1
Segmentation fault

所以它在 Z 中除了一个 int** 之外没有分配任何东西。我认为？

我对 C 很陌生，所以我不确定我是否犯了一些小错误，或者我是否真的把整个事情都搞错了。有什么想法吗？谢谢！

Answer 1

[] 运算符比一元 * 运算符绑定得更紧密。尝试将 *Z[i] 更改为 (*Z)[i] 并查看您的代码是否正常运行。

附带说明一下，在 C 中，将 (sizex*sizey) 大小的单个数组 malloc 用于矩阵然后将其索引为 arr[x*sizey + y] 或 arr[y*sizex + X]。这更接近于模仿语言对静态数组所做的事情（例如，如果您声明 int foo[10][10]，则所有 100 个 int 在内存中都是连续的，并且没有存储 10 个 int* 的列表。

Answer 2

我同意 Walter 和 AndreyT 的观点。这只是一些附加信息。

请注意，您可以只使用两个 malloc() 调用，而不是 *x + 1 - 一个大块用于 ints 本身，一个用于行索引。

*Z = malloc(*x * sizeof (*Z)[0]);
(*Z)[0] = malloc(*x * *y * sizeof (*Z)[0][0]);
for (i = 1; i < *x; i++) {
    (*Z)[i] = (*Z)[0] + i * *y;
}

Answer 3

正如沃尔特在他的回答中正确指出的那样，它应该是 (*Z)[i] = ...，而不是 *Z[i] = ...。

最重要的是，我建议摆脱源代码中存在的取消引用/类型转换地狱。不要转换malloc 的结果。不要在sizeof 下使用类型名称。表示如下

 *Z = malloc(*x * sizeof **Z);
 ...
 (*Z)[i] = malloc(*y * sizeof *(*Z)[i]);

将使您的代码与类型无关并且更具可读性。

另一个问题是到底是什么让你在fscanf 中使用&(*x)。这是某种奇怪的编码标准吗？