動的な配列を作る関数

Question

目的
3次元配列を動的に作る関数を作成しようとしています。
mallocじゃなくてcallocを使うことで初期化を自動的にしてくれる事は理解できました。
しかし、「関数 ‘calloc’ へ渡す引数が少なすぎます」、「戻りでポインタからキャスト無しに整数を作成しています」、「代入で整数からキャスト無しにポインタを作成しています」が出て来て治すことができません。
この場合何がいけないのでしょうか。
※「関数 ‘calloc’ へ渡す引数が少なすぎます」は解決しました。ありがとうございます。

#include<stdio.h>
#include<stdlib.h>
#include<math.h>
#define pixel 128

/*動的な3次元配列を作る関数*/
char memory3(int nx,int ny,int nz){
	int j,k;
	char ***image;
	//image[nz][ny][nx]のうち'nz'を確保する
	image=(char ***)calloc(nz,sizeof(char **)*nz);
	if(image==NULL){
		printf("memory allocation error\n");
		exit(1);
	}
	for(k=0;k<nz;k++){
	//image[nz][ny][nx]のうち'ny'を確保する
	image[k]=(char **)calloc(ny,sizeof(char *)*ny);
		if(image[k]==NULL){
			printf("memory allocation error\n");
			exit(1);
		}
		for(j=0;j<ny;j++){
		//image[nz][ny][nx]のうち'nx'を確保する
		image[k][j]=(char *)calloc(nx,sizeof(char)*nx);
			if(image[k][j]==NULL){
				printf("memory allocation error\n");
				exit(1);
			}
		}
	}
	return image;
}

int main(void){
	FILE *fp1,*fp2;
	int nz=pixel,ny=pixel,nx=pixel,i,j,k;
	char ***image;
	image=memory3(nx,ny,nz);
	char ***nimage;
	nimage=memory3(nx,ny,nz);
	char proj[nx][ny];
}

Comments

[ozwk]

もとの問題を消してしまったら、修正前の回答が意味がわからなくなってしまいます。(なりました。)
もともとの問題を残した上で新たに発生した問題を追記する形にしてください。

[HYYH]

ありがとうございます。再度訂正してみました。

Answer 1

この場合何がいけないのでしょうか。

エラーメッセージのとおりです。callocはcalloc(size_t number, size_t size)というように、2つの引数を取ります。

本来はサイズと個数を指定するものですが、分けて指定せずに片方を1に固定するような使い方でも動くといえば動きます。

Comments

[HYYH]

ありがとうございます。上記のように訂正してみました！

Answer 2

calloc関数の仕様を調べよう
使い方が間違ってますよ