C言語 入力値の文字列を 関数に渡すと警告 うまくいかない

Question

int main(void) {

	char map[WORLD_H][WORLD_W];

	system("cls");

	COORD coord;
	HANDLE hStdout;
	hStdout = GetStdHandle(STD_OUTPUT_HANDLE);




	int checkno;
	char inva[50];   //入力された値
	int len;      //入力された文字数
	int hikaku;  // 入力値の後ろ4文字が .txt であるかどうか
	char* ap = inva;


	printf("数字を入力してください(乱数)  文字または文字列を入力してください(ファイル読み込み) 21行以上エラー");
	scanf("%s", &inva);

	checkno = check_number(inva);     //数値か文字、文字列かのチェック。　　　　

	len = strlen(inva);   //入力された文字数　取得

	



	if (checkno == -1) { //入力された値が文字か文字列だった場合

		hikaku = strcmp(inva + strlen(inva) - 4, ".txt");    //入力値　後ろ　４文字が .txt だったら　テキストファイルを読み込み
		if (hikaku == 0) {
			
			fileop(map,ap);
			exit(10);
		}
		else {                                                                 // 入力値　後ろ　４文字が .txt　ではなかったら 終了
			printf("テキストファイルではありません");
			exit(10);
		}


//ファイル読み込む
void fileop(char temp[WORLD_H][WORLD_W], char txtname[]) {

	FILE* fp;
	char txt;
	
	


	fp = fopen(txtname, "r");
	if (fp == NULL) {
		printf("%s  file not open!\n", txtname);

		return;
	}
fclose(fp);
}

scanf 入力された文字列を　　fileop 関数に　受け渡して　そのテキストファイルを開きたいのですが

受け渡しが上手くいきません。

警告が２つでます。
１．'fileop': 実引数が多すぎま　　　　　　　　この部分→ fileop(map,ap);

２．宣言されたパラメーターリストが定義と一致しません。　この部分→void fileop(char temp[WORLD_H]　[WORLD_W], char txtname[]) {

この二つの警告が出ます。
警告で言われていることは何となく分かるのですが、なかなか上手くいかないのでアドバイス頂けると助かります。

----------追記---------
警告　消えてました。すいません。

こうした方が良いなどコードにアドバイス頂けたら幸いです。

Comments

[mather]

> 警告　消えてました。すいません。
聞きたいことが終わっているなら自己解決にしましょう。
なんにせよ、提示されているコードがぐちゃぐちゃで何をしたいのかわかりません。

[kokok]

すいません。
今後はもっと整理して質問するようにします。

Answer 1

関数fileopのプロトタイプ宣言が，提示コードの末尾付近に書かれている実装と異なっているとか？

Comments

[kokok]

プロトタイプ宣言に間違いがありました。
ありがとうございます。

[fana]

できれば，低評価の理由をご教示願いたく．
（「？」で終わってる文章はとても回答とは呼べない，とか？）

[kokok]

自分は押していないので他の方が間違えて押されたのかと。
高評価押しておきます。

[fana]

低評価を押した方に向けて問うていますので，
あなた自身による評価に関しては，それとは無関係に行っていただきたく．

Answer 2

C言語のコンパイラというのは、ソースの最初から順番に解釈していきます

        fileop(map,ap);

の行の解釈のとき、どういう動作をするのかといえば、
・いままでfileop 関数の定義には出会わなかった
・となると、この関数の定義を、int fileop(char**,char*) として解釈を続けていく
という動作で、コードの解釈を続けていきます。

そして、

void fileop(char temp[WORLD_H][WORLD_W], char txtname[]) {

の関数宣言の行に来ると、あらかじめ読み込んでいた関数定義と型が違う、ということでワーニングが出ます

このワーニングを消すには、あらかじめ、コードの最初の方で関数定義をしておけばいいということになります

ということで、
void fileop(char temp[WORLD_H][WORLD_W], char txtname[]);
という定義行をコードの最初の方においてやってみてください

Comments

[kokok]

解決いたしました。
ありがとうございます。