ファイルの処理について。

Question

###前提
書籍で勉強している学生です。
書籍の解答がないため問題のヒントや解説をしていただけると嬉しいです。

###問題
キーボードからファイル名を読み込んで、その名前のファイルの中身を消去するプログラムを作成せよ。
＊ファイルを書き込みモードでオープンするとよい（ファイルを書き込みモード”ｗ”でオープンすると、ファイルの中身がすべて消去される。）

###該当のソースコード

#include <stdio.h>

#define MAX	100	//ファイルの名前

int main (void){
	FILE *fp;
	char fname[MAX];

	printf("file name : ");
	scanf("%s", fname);

	fp = fopen(fname, "r");

	if(fp == NULL){
		printf("\afile %s is not exist.\n", fname);
	}else{
		printf("file %s is exist.\n", fname);
		fclose(fp);    /*---<A>---*/
		printf("erase data...\n");
		fopen(fname, "w");
		fclose(fp);
	}
	
	return 0;
}

###疑問点
<A>の部分のfclose()関数を使用してもしなくても、動作をしたのですが、
どちらが、より安全にファイルを処理することが出来るかが知りたいです。

一回クローズしてからの方が安全なような気がします...

###補足情報
書籍　：　新明解c言語　入門編
演習　１３－２

英語力不足のため、英文に誤りがあればそれもご指摘願います。

Answer 1

一回クローズしてからオープンするのは正しいですが、二回めのクローズがおかしいので見なおしてください。
二回めのcloseに渡しているファイルポインタfpですがfopen(fname, "w")で開いたものになってないですよね？

    if(fp == NULL){
        printf("\afile %s is not exist.\n", fname);
    }else{
        printf("file %s is exist.\n", fname);
        fclose(fp);    /*---<A>---*/
        printf("erase data...\n");
        fopen(fname, "w");
        fclose(fp);
    }

Comments

[kamecha]

本当ですね。
２回目のfpは、恐らく読み取りモードでオープンしたファイルに繋がっているのだと思います。

Answer 2

fopenでオープンしたら、必ずfcloseでクローズする必要があります。動作したように見えたのは、fcloseしなかったことによる不都合が表面化する前にプログラムが終了したからです。

fopenすると、管理用のメモリ領域が確保されます。たとえ同じファイルに対するオープンでも、別々の領域が割り当てられます。もし、fcloseをしないままでいると、その管理領域がゴミとしてメモリに残ったままになり、場合によっては悪さを引き起こします。ただ、そうなる前にプログラムが終了すれば、一応「動作」はします。

一回クローズしてからの方が安全なような気がします...

その通りですので、そうしてください。

Comments

[kamecha]

有り難うございます！