ループ2回目以降が上手くいきません。ファイルの入出力 strtok_s関数

Question

アドレス帳のような機能を作っています。

不具合が起きているのは、profile_edit関数「mail.txtのデータ(元のデータ)をmail2.txtに書き写しながら、変更したい部分に新しい文字列を挿入する」のところです。
今の状態だと「元の文字列が消えずに残っているが表示されない」ので1回目のループはうまくいきます。
2回目以降だと、消えずに残っている文字列が邪魔をしてうまくいきません。1回目のループで、元の文字列を上書きもしくは削除したいです。

使う関数自体変えればいいのでしょうが、知識が浅くどの関数をどう使えばいいのか分からない状態です。（strtok_s関数とCSVファイルを使うとうまくいくのかなと考え程度です）

専門用語等分からないことが多いです。

よろしくお願い致します。

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

int profile_edit ( int , char [ 256 ] );

struct profile {			/* プロフィール構造体 */
	char name [ 256 ];		/* 氏名 */
	char code [ 256 ];		/* 郵便番号 */
	char add [ 256 ];		/* 住所 */
	char tel [ 256 ];		/* 電話番号 */
	char mail [ 256 ];		/* メールアドレス */
};
FILE* fp;
FILE* fp2;

int main ( )
{
	struct profile prof;
	char file_in [ 256 ];
	int num = 0;
	int end = 0;

	printf ( "【アドレス帳】\n" );
	do {
		printf ( "\nアドレスの\n\t[登録]：1\t[編集]：2\n\t[削除]：3\t[参照]：4\t数字を入力：" );
		scanf_s ( "%d" , &num );
		printf ( "\n" );
		if ( num == 1 ) {									/* 1の場合 */
			fopen_s ( &fp , "mail.txt" , "w" );
			if ( fp == NULL ) {
				return 0;
			}
			printf ( "氏名　　　　　：" );
			scanf_s ( "%s" , prof.name , 255 );
			printf ( "郵便番号　　　：" );
			scanf_s ( "%s" , prof.code , 255 );
			printf ( "住所　　　　　：" );
			scanf_s ( "%s" , prof.add , 255 );
			printf ( "電話番号　　　：" );
			scanf_s ( "%s" , prof.tel , 255 );
			printf ( "E-mailアドレス：" );
			scanf_s ( "%s" , prof.mail , 255 );
			fputs ( "氏名　　　　　：" , fp );
			fputs ( prof.name , fp );
			fputs ( "\n郵便番号　　　：" , fp );
			fputs ( prof.code , fp );
			fputs ( "\n住所　　　　　：" , fp );
			fputs ( prof.add , fp );
			fputs ( "\n電話番号　　　：" , fp );
			fputs ( prof.tel , fp );
			fputs ( "\nE-mailアドレス：" , fp );
			fputs ( prof.mail , fp );
			fputs ( "\n" , fp );
			fclose ( fp );
		}
		else if ( num == 2 ) {								/* 2の場合 */
			fopen_s ( &fp2 , "mail2.txt" , "w" );
			if ( fp2 == NULL ) {
				printf ( "編集ができません。\n" );
				break;
			}
			fopen_s ( &fp , "mail.txt" , "r" );
			if ( fp == NULL ) {
				printf ( "連絡先が見つかりません。\n" );
				break;
			}
			int edit = 0;	/* 編集番号を受け取る */
			int a = 0;		/* profile_edit関数の引数を受け取る */
			char newdata [ 256 ];
			printf ( "どの項目を編集しますか？\n\t[氏名]：1\t[郵便番号]：2\n\t[住所]：3\t[電話番号]：4\n\t[E-mailアドレス]：5\t\t数字を入力：" );
			scanf_s ( "%d" , &edit );
			printf ( "\n新しい情報を入力してください。\n\t：" );
			scanf_s ( "%s" , newdata , 255 );

			profile_edit ( edit , newdata );
			if ( a = profile_edit ( edit , newdata ) == 0 ) {
				printf ( "新しい情報に書き換えられました。\n" );
			}
			else {
				printf ( "編集に失敗しました。\n" );
				break;
			}
		}

		else if ( num == 3 ) {								/* 3の場合 */

			if ( remove ( "mail.txt" ) == 0 ) {
				printf ( "連絡先を削除しました。\n" );
			}
			else {
				printf ( "連絡先の削除に失敗しました。\n" );
			}
		}
		else if ( num == 4 ) {								/* 4の場合 */
			fopen_s ( &fp , "mail.txt" , "r" );
			if ( fp == NULL ) {
				printf ( "連絡先が見つかりません。\n" );
				do {
					printf ( "\n続けますか？[はい]：1　[いいえ]：2\t数字を入力：" );
					scanf_s ( "%d" , &end );
				} while ( ( end != 1 ) && ( end != 2 ) );
				if ( end == 2 ) {
					return 0;
				}
				else {
					continue;
				}
			}
			while ( 1 ) {
				fgets ( file_in , 255 , fp );
				if ( feof ( fp ) ) {
					break;
				}
				printf ( "%s" , file_in );
			}
			fclose ( fp );
		}
		else {
			printf ( "１～４の数字を入力し直してください。\n" );
			continue;
		}

		do {
			printf ( "\n続けますか？[はい]：1　[いいえ]：2\t数字を入力：" );
			scanf_s ( "%d" , &end );
		} while ( ( end != 1 ) && ( end != 2 ) );
		if ( end == 2 ) {
			break;
		}
	} while ( end == 1 );

	return 0;
}

// 関数名 : profile_edit
//テキストファイルの1部を書き換える関数
int profile_edit ( int row , char str [ 256 ] )		
{
	int ch = 0;
	int nr = 0;
	int nc = 0;

	while ( ( ch = fgetc ( fp ) ) != EOF ) {
		fputc ( ch , fp2 );
		if ( ch == '\n' ) {
			nr++;
		}
		if ( nr == row - 1 ) {
			nc++;
		}
		if ( nc == 17 ) {
			fputs ( str , fp2 );
			fputc ( '\n' , fp2 );
			fputc ( '\0' , fp2 );

			nr++;
			nc++;
		}
	}
	fclose ( fp );
	fclose ( fp2 );

	fopen_s ( &fp2 , "mail2.txt" , "r" );
	if ( fp2 == NULL ) {
		return -1;
	}
	fopen_s ( &fp , "mail.txt" , "w" );
	if ( fp == NULL ) {
		return -1;
	}
	while ( ( ch = fgetc ( fp2 ) ) != EOF ) {
		fputc ( ch , fp );
	}
	fclose ( fp );
	fclose ( fp2 );

	return 0;
}

Visuai studio

Comments

[y_waiwai]

提示のコードにはどういう不具合があって、それをどうしたいんでしょうか。
そこらへん詳しく説明しましょう。

いきなりコード出されても読む気にもなりません

[madanai]

失礼しました。
不具合が起きているのは、profile_edit関数「mail.txtのデータ(元のデータ)をmail2.txtに書き写しながら、変更したい部分に新しい文字列を挿入する」のところです。
今の状態だと「元の文字列が消えずに残っているが表示されない」ので1回目のループはうまくいきます。
2回目以降だと、消えずに残っている文字列が邪魔をしてうまくいきません。1回目のループで、元の文字列を上書きもしくは削除したいのです。

使う関数自体変えればいいのでしょうが、知識が浅くどの関数をどう使えばいいのか分からない状態です。（strtok_s関数とscvファイルを使うとうまくいくのかなと考え程度です）

[y_waiwai]

それを質問に追記しましょう。
修正依頼に書いてても読まれません

Answer 1

fopen_s ( &fp2 , "mail2.txt" , "r" );
	if ( fp2 == NULL ) {
		return -1;
	}
	fopen_s ( &fp , "mail.txt" , "w" );
	if ( fp == NULL ) {
		return -1;
	}
	while ( 1 ) {
		fgets ( file_in , 255 , fp2 );
		if ( feof ( fp2 ) ) {
			break;
		}
		fputs ( file_in , fp );	
}
fclose ( fp );
fclose ( fp2 );

Answer 2

１回目の編集で、mail.txtから目的の項目を変更してmail2.txtを作ってそのまま（作りっぱなし）ですよね？
これでは２回目の編集は何も変わっていないmail.txtから読み上げてmail2.txtを作り直すので１回目の編集内容が元に戻ってしまいます。
編集が終わって、ファイルをクローズしたら、
0. mail.txtを削除する。
0. mail2.txtをmail.txtにリネームする。
が必要です。

strtok_sとCSVを使用したい理由は、想像ですがアドレス帳(mail.txt)の各項目をカンマ区切りで格納して、編集の際カンマで区切ったトークンとして項目を扱いたいのかな？

Comments

[madanai]

mail2.txtはデータを一時保管しているだけで、最後にmail.txtにすべて上書きしています。

不具合が起きているのは、profile_edit関数「mail.txtのデータ(元のデータ)をmail2.txtに書き写しながら、変更したい部分に新しい文字列を挿入する」のところです。
新しい文字列を元の文字列に上書き、もしくは新しい文字列を挿入した後に元の文字列を削除したいです。
そのためには、データ（文字列）ごとに配列に格納するのがいいと思い、strtok_sで区切りCSVファイルを作成するのが手っ取り早いかな、と思ったのです。知識はありません。。