ファイル中の選択１データの削除処理

[taz]

こんにちは。

ファイル中の選択１データを削除したいと思っております。

①キーボードから、削除したいNoを入力する。
②ファイルからＮｏ，名前、住所、電話番号、メールアドレス等のデータを読み込み、構造体に代入する。
③削除したいＮｏの構造体データを削除する。
④ファイルに構造体データを書き込む。

コード:

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

FILE *fp;
char *fname = "comma.csv";

struct member {
	int no;
	char name[20];
	char adress[20];
	char tel[20];
	char mail[20];
};

main()
{
	int no;
	int ret;
	printf("---会員管理システムメニュー---\n\n");
	printf("1.新規登録\n");
	printf("2.変更\n");
	printf("3.削除\n");
	printf("4.一覧表示\n");
	printf("No? ");
	ret=scanf("%d",&no);
	if(no == 1)
	{
		ret = add_record();
	}
	else if(no == 2)
	{
		ret = update_record();
	}
	else if(no == 3)
	{
		ret = del_record();
	}
	//else if(no == 4)
	//{
	//	ret = disp_record();
	//
	//}
}


int del_record()
{
	struct member member[20]={0};
	int no;
	int choice=0;
	int i=0;
	printf("\n会員番号（0-10数値）？ ");
	scanf("%d",&no);
	fp = fopen(fname,"r");
	if(fp == NULL)
	{
		printf("%sファイルが開けません\n",fname);
		return -1;
	}
	
	while(feof(fp)==0)
	{
			if((fscanf(fp,"%d,%[^,],%[^,],%[^,],%s",&(member[i].no),member[i].name,member[i].adress,member[i].tel,member[i].mail)) != EOF)
  			{
				if(no==member[i].no)
				{
				   printf("\n%d",member[i].no);
				   printf("\n%s",member[i].name);
				   printf("\n%s",member[i].adress);
				   printf("\n%s",member[i].tel);
				   printf("\n%s",member[i].mail);
				   printf("\n%s","削除しますか？（1:yes 2:no）？ ");
				   scanf("%d",&choice);
				   
				   if(choice==2)
				   {
						system("cls");
						main();
				   }
				}
			}
			i++;
	}
	fclose(fp);
            
             (削除処理を記述)
	
}

どのように削除処理を記述すればよいのかアドバイスをよろしくお願いします。

[nil]

stdlibのremove関数が使えるみたいです

[taz]

stdlibのremove関数が使えるみたいです

remove関数はファイルを削除するためのものですよね？
今回は、csvファイル中の１データを削除したいのです。

[softya（ソフト屋)]

stdlibのremove関数が使えるみたいです

ファイル内の行削除ですからremoveはマズイと思います。

[softya（ソフト屋)]

削除自体は、配列にして前に詰めるとかリスト構造にして１つ抜くとかが一般的です。
書き出しは配列全体を書き出せばよいでしょう。

[Ryo]

①キーボードから、削除したいNoを入力する。
②ファイルからＮｏ，名前、住所、電話番号、メールアドレス等のデータをすべて読み込む
③ファイルに、消したい番号以外のデータを書き込む

これでどうでしょうか？

具体的には③にて、
for文などでループしながら書き込む際、対象のNOだけ、continueなどで処理を飛ばす

[初級者]

stdlibのremove関数が使えるみたいです

removeは、いくら何でもまずいだろw

# ちゃんと五・七・五になってるw

[non]

メニューですが、現在は

---会員管理システムメニュー---
1.新規登録
2.変更
3.削除
4.一覧表示

これに、
5.データの読み込み
6.データの保存
をつけたらどうでしょうか？

5.データの読み込みは、メニューに入る前に、自動的に読み込んでおいてもよいと思います。
いずれにしても、データを読み込みながら、削除したいデータを探す今のやり方は、あまり
よいとは思えません。

[taz]

メニューですが、現在は

---会員管理システムメニュー---
1.新規登録
2.変更
3.削除
4.一覧表示

これに、
5.データの読み込み
6.データの保存
をつけたらどうでしょうか？

5.データの読み込みは、メニューに入る前に、自動的に読み込んでおいてもよいと思います。
いずれにしても、データを読み込みながら、削除したいデータを探す今のやり方は、あまり
よいとは思えません。

そうですね。そういう作りに変更したいと思います。ありがとうございます。

[taz]

①キーボードから、削除したいNoを入力する。
②ファイルからＮｏ，名前、住所、電話番号、メールアドレス等のデータをすべて読み込む
③ファイルに、消したい番号以外のデータを書き込む

これでどうでしょうか？

具体的には③にて、
for文などでループしながら書き込む際、対象のNOだけ、continueなどで処理を飛ばす

このやり方で作成させていただきました。ありがとうございます！

[taz]

削除自体は、配列にして前に詰めるとかリスト構造にして１つ抜くとかが一般的です。
書き出しは配列全体を書き出せばよいでしょう。

アドバイスありがとうございます。そういうやり方でやらせていただきました。

[taz]

削除処理は以下のようになりました。

コード:

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

FILE *fp;
char *fname = "comma.csv";

struct member {
	int no;
	char name[20];
	char adress[20];
	char tel[20];
	char mail[20];
};

main()
{
	int no;
	int ret;
	printf("---会員管理システムメニュー---\n\n");
	printf("1.新規登録\n");
	printf("2.変更\n");
	printf("3.削除\n");
	printf("4.一覧表示\n");
	printf("No? ");
	ret=scanf("%d",&no);
	if(no == 1)
	{
		ret = add_record();
	}
	else if(no == 2)
	{
		ret = update_record();
	}
	else if(no == 3)
	{
		ret = del_record();
	}
	//else if(no == 4)
	//{
	//	ret = disp_record();
	//
	//}
}

int del_record()
{
	struct member member[20]={0};
	struct member member1 = {EOF,EOF,EOF,EOF,EOF};
	int no;
	int choice=0;
	int i=0;
	printf("\n会員番号（0-10数値）？ ");
	scanf("%d",&no);
	fp = fopen(fname,"r");
	if(fp == NULL)
	{
		printf("%sファイルが開けません\n",fname);
		return -1;
	}
	
	while(feof(fp)==0)
	{
			if((fscanf(fp,"%d,%[^,],%[^,],%[^,],%s",&(member[i].no),member[i].name,member[i].adress,member[i].tel,member[i].mail)) != EOF)
  			{
				if(no==member[i].no)
				{
				   printf("\n%d",member[i].no);
				   printf("\n%s",member[i].name);
				   printf("\n%s",member[i].adress);
				   printf("\n%s",member[i].tel);
				   printf("\n%s",member[i].mail);
				   printf("\n%s","削除しますか？（1:yes 2:no）？ ");
				   scanf("%d",&choice);
				   
				   if(choice==2)
				   {
						system("cls");
						main();
				   }
				}
			}
			i++;
	}
	i--;
	member[i]=member1;
	i = 0;
	fclose(fp);
	fp = fopen(fname, "w");
	fclose(fp);
	fp = fopen(fname, "w");
	while(member[i].no!=EOF)
	{
		if(no!=member[i].no)
		{
			fprintf(fp,"%d,%s,%s,%s,%s\n",member[i].no,member[i].name,member[i].adress,member[i].tel,member[i].mail);
		}
		i++;
	}
	fclose(fp);
	printf("%s","削除されました。");
	return 0;
}

アドバイスありがとうございました。またよろしくお願いします！

[へにっくす]

いやいーんだけどね？
ものすごく気になる箇所があるから指摘しとくよ。

49行目の初期化でEOFって何さ？
{0}で初期化するなら分かるけど。

89～90行目のfopen/fcloseは何？
無駄な処理はやめた方がいいよ。