ファイル入出力とプロフ

Question

#define _CRT_SECURE_NO_WARNINGS 1

#include <stdio.h>
#include <string.h>
#include <stdlib.h>


typedef struct profile{
	char name[20];
	char age[10];
	char sex[10];
}Pro;

int main(){
	
	int i;
	char filename[256];
	FILE *fp;


	puts("プロフィールを設定します。\n"
		 "以下の項目を入力してください。");
	printf("何人分作りますか？>>>");
	scanf("%d", &i);

	for (int h = 0; h != i ; h++){
		printf("%d人目のプロフィールを作ります。"
			"以下の項目を入力してください。\n", h + 1 );

		Pro hito;
		getchar();
		puts("ファイル名を入力してください。");
		fgets(filename, sizeof(filename), stdin);
		filename[strlen(filename) - 1] = '\0';
		strcat(filename, ".txt");
		fp = fopen(filename, "w");

		puts("名前を入力してください。");
		fgets(hito.name, sizeof(hito.name), stdin);
                hito.name[strlen(hito.name) - 1] = '\0';
		fprintf(fp,"名前は%s\n", hito.name);

		puts("年齢を入力してください。");
		fgets(hito.age, sizeof(hito.age), stdin);
                hito.age[strlen(hito.age) - 1] = '\0';
		fprintf(fp,"年齢は%s\n", hito.age);

		puts("性別を入力してください。");
		fgets(hito.sex, sizeof(hito.sex), stdin);
                hito.sex[strlen(hito.sex) - 1] = '\0';
		fprintf(fp,"性別は%s\n", hito.sex);

		fclose(fp);
	}
	return EXIT_SUCCESS;
}

このプログラムを走らせると、人数を1人に設定すると、.txtが作られ何もできずに終わり、2～にすると入力した人数-1人分のファイルを生成しようとし、名前が入力されたものだけ作られます。
また、ファイルは作られますが、ファイルの中に書き込む所はできません。puts→fscanfの処理が飛ばされます。

そして初心者なので一部、入門書に書かれてるファイルを生成するコードをコピーしています。
ファイル名を入力してください→fgets、strcatなどの処理のところです。

このコードをどうすれば人数分のファイル生成ができ、中身を書き込むことができますか？

説明が下手なのでもしかしたら意図が通じてない所もあるかもしれません。
それと、もともとコード自体を誰かに見せるつもりはなかったので、変数の名前はあまり気にしないでほしいです。すみません。

追記
コードを編集しました。
ここが間違ってる、ここはこうすると良いなどがあれば教えてください。
実行してみた感じでは思った通りに動いてくれました

Answer 1

こんにちは。

fscanf(fp, ...)はfpが示すファイルからデータを読み込みます。書き出しではないです。
fgets()で標準入力から文字列を読み込み、fpus()でfpへ出力すれば良いと思います。

また、fp=fopen()に成功したらfclose(fp);する必要が有ります。（開いたら閉じる。鉄則です。）

Comments

[nisuko]

fscanfではファイルに書き込めないのですね、、わかりました。fgetsなどを一度確認してきます。
fcloseは最後にしてたので大丈夫だと思いましたが、他の方と貴方の回答を見て納得しました。ありがとうございます。

追記
コード編集しました。確認をお願いします。

[Chironian]

> コード編集しました。確認をお願いします。

基本的には良さそうです。（バッファオーバーフロー等の細かい問題はありますが、それは追々学習を進めて下さい。）
ただ、名前等の後１行余分に改行されると思います。fgets()は改行まで入力されるからです。下記はその改行を0で上書きして削除する処理です。文字列の最後に改行('\n')か入るのです。
> filename[strlen(filename) - 1] = '\0';

[nisuko]

確認ありがとうございます。
１行空くのは確かに気になっていたので、処理を追加しました。
こうでいいのですよね？動作は一応考え通りに動いてくれていますし、、。

バッファオーバーフローって取ってあった分のメモリからあふれるってやつですよね？
改善するにはunsigned int だったり配列でとる分を増やすだけでよいのでしょうか？
効果的な処理手法があるのでしょうか？

[Chironian]

それで良いと思います。

> バッファオーバーフローって取ってあった分のメモリからあふれるってやつですよね？

その通りです。

> 改善するにはunsigned int だったり配列でとる分を増やすだけでよいのでしょうか？

増やすだけですと、それを超えて入力されるとやはりバッファオーバーフローしてしまいます。

> 効果的な処理手法があるのでしょうか？

方法は色々あります。今回のケースの場合は、メモリ以上の長さの入力があったら残りを'\n'まで読み捨てると良いと思います。

あ、nisukoさんのプログラムはバッファオーバーフローは起きないですね。
長過ぎる名前等を入力すると、動作が可笑しくなるだけです。（どうなるかは実際にやってみるのが良いです。19文字の名前や20文字以上の名前など。）
「長過ぎる入力＋配慮不足≒バッファオーバーフロー」なので短絡的に書いてしまいました。ごめんなさい。

[nisuko]

ありがとうございます。
やってみると、名前で溢れた分は年齢へ、それも溢れると性別へ、という動作しました。
どのように対処するかまで教えていただきありがとうございます。

Answer 2

for 内の fp = fopen(filename, "w"); で出力ファイルを人数分オープンしていますが、このfpのクローズは for の外(48行目)でしていますよね。
クローズもfor内で毎回、人数分する必要があります。

また、"名前を入力してください" の箇所も fscanf で fp を指定しているのも変と思います。

Comments

[nisuko]

fcloseは毎回必要なのですね、確かに言われてみて、よく考えてみると毎回閉じなきゃおかしいですね。ありがとうございます。

fscanfを、独学なのですが、本で習った時には
fscanf(fp,"%s",name)
と書かれていたので必要だと思ったのですが、、
一度リファレンスを確認してみます、確認する癖をつけるようにします。

追記
コード編集しました。確認をお願いします。

Answer 3

fscanf(fp, "%s", hito.namae);

は、fpでオープンしたファイルから読み込むということですが、やりたいことと違いますよね？
また、どこでもファイルに書いている箇所がありません。どこで書いているつもりですか？
プログラムを音読してみてはどうでしょうか。

Comments

[nisuko]

説明を見る限りやりたいことと違います。
音読しました。わからないので３回やりましたがわかりません。
大体３５か３６行目辺りでfopenでファイルを生成（"w"で、なければ生成する、と聞きました。）しており、そのしたのfscanfで、そのファイルに情報を書いてるつもりでした。
どうすればよいでしょうか？fscanfは入力関数じゃないのですか？

追記
コード編集しました。確認をお願いします。