[C言語]ファイルの入出力[main関数の外]

Question

前提・実現したいこと

ここに質問の内容を詳しく書いてください。
ファイルの入出力をmain関数の外で行いたいため，そのテストプログラムを作成しました．

発生している問題・エラーメッセージ

](384d4f32d820bd43a00ce08cac901e9f.png)

エラーメッセージ

該当のソースコード

34行目は空白です．

C言語
Microsoft Visual Studio Community 2019
Version 16.4.3

#include <stdio.h>
#include <math.h>

int n = 1;

int fileopen(FILE* fp) {
	char filename[100];
	errno_t error = 0;
	while (error == NULL) {
		sprintf_s(filename, 100, "test%d.txt", n);
		if (error != fopen_s(&fp, filename, "r") == NULL) {
			n++;
		}
		else {
			error = fopen_s(&fp, filename, "w");
			fprintf(fp, "%d\n", n);
		}
	}
	return 0;
}

int main() {
	FILE* fp1;
	FILE* fp2;

	fileopen(&fp1);
	fileopen(&fp2);

	fclose(fp1);
	fclose(fp2);

	return 0;
}

試したこと

インターネットで調べたが，わからなかった．
fprint_sなど情報が複雑でわかりにくかったため．

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Comments

[y_waiwai]

このままではコードが読みづらいので、質問を編集し、<code>ボタンを押し、出てくる’’’の枠の中にコードを貼り付けてください

[kotobuki]

変更しました．

Answer 1

戻り値でファイルポインタを返していいのであればFileopen()に引数として渡す必要ないので
こんな感じになると思います。

#include <stdio.h>

FILE* Fileopen(char namehead[100]) {
    FILE *fp = NULL;

    char filename[100];
    int fnumber = 0;
    errno_t err = 0;

    while (err == 0) {
        fnumber++;
        sprintf_s(filename, sizeof(filename), "%s(%d).txt", namehead, fnumber);

    	err = fopen_s(&fp, filename, "r");
        if (err == 0) {
            fclose(fp);
        }
    }

    err = fopen_s(&fp, filename, "w");
    if (err == 0) {
        fprintf(fp, "%d\n", fnumber);
    } else {
        fp = NULL;
    }

    return fp;
}

int main() {
    FILE *fp1, *fp2;

    fp1 = Fileopen("f1");
    fp2 = Fileopen("f2");

    if (fp1 != NULL) { 
        fprintf(fp1, "1行追加\n"); 
        fclose(fp1);
    }

    if (fp2 != NULL) { 
        fprintf(fp2, "1行追加\n"); 
        fclose(fp2);
    }

    return 0;
}

---

fileopen()の引数はFILE**で受けてfopen_s()の第1引数には&fpではなくfpをそのまま渡します。
※質問記載のコードをコンパイルするとmain()でfileopen()の呼び出し時に型が違う旨warning出てると思うんですが。。。

int fileopen(FILE** fp) {
//略
  while (error == 0) { // ← これもNULLじゃなくて0
//略
    if (error != fopen_s(fp, filename, "r") /* これ何？→ == NULL */) {
//略
    error = fopen_s(fp, filename, "w");
    if (error == 0) {  // エラーじゃないときだけ出力しましょう。
        fprintf(*fp, "%d\n", n);
    }
//略
}

Comments

[kotobuki]

回答ありがとうございます．修正したところ，新しくエラーが発生してしまいました．

>#include <stdio.h>
>#include <math.h>
>
>int n = 1;
>
>int fileopen(FILE** fp) { //変更行
> char filename[100];
> errno_t error = 0;
> while (error == NULL) {
> sprintf_s(filename, 100, "test%d.txt", n);
> if (error != fopen_s(fp, filename, "r") == NULL) { //変更行
> n++;
> }
> else {
> error = fopen_s(fp, filename, "w"); //変更行
> fprintf(fp, "%d\n", n);
> }
> }
> return 0;
>}
>
>int main() {
> FILE* fp1;
> FILE* fp2;
>
> fileopen(&fp1);
> fileopen(&fp2);
>
> fclose(fp1);
> fclose(fp2);
>
> return 0;
>}

>0x77223CB3(ntdll.dll)で例外がスローされました．(Testapp.exe内)
>0cC0000005:場所0x00000015への書き込み中にアクセス違反が発生しました

[Y.H.]

あぁ見逃してました。
fprintf()は第1引数はFILE*型なので*fpを渡しましょう。
これもコンパイル時にwarning出てると思うんで直ぐに気づけると思うんですが・・・

[kotobuki]

16行目
fprintf(*fp, "%d\n", n);
修正しました．
返信時のエラーがなくなり，質問時のエラーが再発いたしました．

[kotobuki]

重大度レベル コード 説明 プロジェクト ファイル 行 抑制状態
警告 C4047 '==': 間接参照のレベルが 'int' と 'void *' で異なっています。 Testapp D:..\Testapp.c 11
警告 C6387 '*fp' は '0' である可能性があります: この動作は、関数 'fprintf' の指定に従っていません。 Testapp D:..\TESTAPP.C 16
警告 C4047 '==': 間接参照のレベルが 'errno_t' と 'void *' で異なっています。 Testapp D:..\Testapp.c 9

現在発生している警告メッセージです．

[Y.H.]

> main関数の外で行いたいため，そのテストプログラムを作成しました
main関数の中では動いているんでしょうか？

> 現在発生している警告メッセージです
で？メッセージを見てkotobukiさんは何もしないのでしょうか？

とりあえずコードは修正しておきましたがここまでにします。

[kotobuki]

修正ありがとうございます．
回答者に丸投げする質問をしてしまい，申し訳ありませんでした．

Answer 2

Fileopen関数の型をFIlE型にすることで解決しました．
回答ありがとうございました．

#include <stdio.h>

FILE* Fileopen(FILE** fp ,char namehead[100]) {
    char filename[100];
    int flag = 0, fnumber = 1;
    sprintf_s(filename, 100, "%s(%d).txt", namehead ,fnumber);
    while (flag == 0) {
        errno_t error = fopen_s(&fp, filename, "r");
        if (error == 0) {
            fclose(fp);
            fnumber++;
            sprintf_s(filename, 100, "%s(%d).txt", namehead, fnumber);
        }
        if (error != 0) {
            errno_t error = fopen_s(&fp, filename, "w");
            flag = 1;
            if (error == 0) {
                fprintf(fp, "%d\n", fnumber);
        }
    }
    return fp;
}

int main() {
    FILE *fp1, *fp2;
    
    fp1=Fileopen(&fp1,"f1");
    fp2=Fileopen(&fp2,"f2");

    fclose(fp1);
    fclose(fp2);

    return 0;
}

Comments

[Y.H.]

fopen_s()の第1引数はFILE**型であってFILE***型ではない。
Fileopen()の戻り値はFILE*型で定義しているのにFILE**型の値を返している。
fprintf(fp, "%d\n", n);は不要になったの？

[kotobuki]

fprintfの項を追記しました．申し訳ありません．

>fopen_s()の第1引数はFILE**型であってFILE***型ではない。
>Fileopen()の戻り値はFILE*型で定義しているのにFILE**型の値を返している。
すいません．勉強不足のため，警告は表示されていますがエラーはなくなり意図していた動作が確認できたため，解決済みとしてしまいました．

[kotobuki]

このサイトの仕組みを理解していなくて申し訳ありません．BAを変更しました．

[Y.H.]

あぁすみません。
BA催促するつもりはありませんでした。