2次元配列をmallocingするときの問題

Question

私はまだC、malloc、そしてそのすべてのジャズが新しいので、これを書いていくつかのスキルを学ぶことにしました。アイデアは、私はファイルから整数の束を読み込み、それらを行列（2次元配列）に入れています。ファイルの先頭には行数と列数があるので、それらの数値を読み込み、mallocを使って2次元配列を設定します。

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内の1つのint型**が、何もmallocingないのですか？

私は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が彼の答えで正しく述べたように、それは私があなたのソースコードでそのデリファレンス/型キャスト地獄の存在を取り除くことをお勧めしたい。mallocの結果をキャストしないでくださいその上で(*Z)[i] = ...、ない*Z[i] = ...。

でなければなりません。下型の名前を使用しないでくださいsizeof。以下のように表現する。

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

wilはコードを型に依存せず、はるかに読みやすくします。

fscanfには、&(*x)を使用した理由は別です。これは何らかの奇妙なコーディング基準ですか？

Answer 3

私はWalterとAndreyTの両方に同意します。これは単なる追加情報です。

​​（int自体の1つの大きなブロックと行インデックスの1つのブロック）ではなく、2つのmalloc()コールだけで離れていくことができます。

*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; 
}