C言語で構造体のデータをバイナリファイルに保存･読み込む方法

Question

前提・実現したいこと

円の中心位置と半径を3セット入力し、半径の大きい順にソートしたものをバイナリに保存するプログラムと、保存されたバイナリを読み込んで中心位置が原点から近い順にソートして表示するプログラムを作りたいです。

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

ファイルを読み込んで表示するときに、①で入力した値とは違うものが表示されてしまいます。
テキストファイルに保存したとき(fprintf,fscanfを使ったとき)は問題が起こらなかったので、fwriteやfreadの引数が間違っているのかなと思います。

該当のソースコード

①ファイルを保存するプログラム

#include <stdio.h>
#include <math.h>
struct circle
{
	double x;
	double y;
	double rad;
};
void swap(double *px,double *py)
{
	double tmp=*px;
	*px=*py;
	*py=tmp;
}
int main(void)
{
	int i,j;
	double num[3]={1,2,3};
	struct circle n[3];
	char str[50];
	FILE *fp;
	
	puts("3つの円の座標と半径を入力");
	for(i=0;i<3;i++)
	{
		printf("円%.0f\n",num[i]);
		printf("座標x="); scanf("%lf",&n[i].x);
		printf("座標y="); scanf("%lf",&n[i].y);
		printf("半径r="); scanf("%lf",&n[i].rad);
	}
	puts("半径の大きい順にソート\n");
	for(i=0;i<3;i++)
	{
		for(j=i+1;j<3;j++)
		{
			if(n[i].rad<n[j].rad)
			{
				swap(&num[i],&num[j]);
				swap(&n[i].x,&n[j].x);
				swap(&n[i].y,&n[j].y);
				swap(&n[i].rad,&n[j].rad);
			}
		}
	}
	for(i=0;i<3;i++)
		printf("円%.0f:(%.2f,%.2f),r=%.2f\n",num[i],n[i].x,n[i].y,n[i].rad);
	
	//ファイルに出力
	printf("ファイル名を入力:"); scanf("%s",str);
	fp=fopen(str,"wb");
	if(fp==NULL)
		printf("ファイル%sをオープンできませんでした。\n",str);
	else
	{
		printf("半径および中心位置を保存しました。\n");
		fwrite(n,sizeof(n),3,fp);
		fclose(fp);
	}
	return 0;
}

②ファイルを読み込んで表示するプログラム

#include <stdio.h>
#include <math.h>
struct circle
{
	double x;
	double y;
	double rad;
	double len;
};
void len_p(double x,double y,double *len)
{
	*len=sqrt(pow(x,2)+pow(y,2));
}
void swap(double *px,double *py)
{
	double tmp=*px;
	*px=*py;
	*py=tmp;
}
int main(void)
{
	int i,j;
	struct circle n[3];
	char str[50];
	FILE *fp;
	
	printf("ファイル名を入力:"); scanf("%s",str);
	if((fp=fopen(str,"rb"))==NULL)
		printf("ファイル%sをオープンできませんでした。\n",str);
	else
	{
		fread(n,sizeof(n),3,fp);
		for(i=0;i<3;i++)
		{
			printf("(%.2f,%.2f),r=%.2f\n",n[i].x,n[i].y,n[i].rad);
			len_p(n[i].x,n[i].y,&n[i].len);
		}
		puts("中心位置が原点から近い順にソート");
		for(i=0;i<3;i++)
		{
			for(j=i+1;j<3;j++)
			{
				if(n[i].len>n[j].len)
				{
					swap(&n[i].x,&n[j].x);
					swap(&n[i].y,&n[j].y);
					swap(&n[i].rad,&n[j].rad);
					swap(&n[i].len,&n[j].len);
				}
			}
		}
		for(i=0;i<3;i++)
			printf("(%.2f,%.2f),r=%.2f,距離%.2f\n",n[i].x,n[i].y,n[i].rad,n[i].len);
		fclose(fp);
	}
	return 0;
}

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

開発環境はwindows10のMinGWです。

Answer 1

ファイルに保存するデータがx,y,radの3つなのに対し、②の構造体がx,y,rad,lenと4つになっていたのがエラーの原因かも知れないと思ったので修正してみました。
また、回答からsizeof(n)の部分をsizeof(*n)に直しました。
回答者さんありがとうございます。問題なく動作するようになりましたが、まだココがおかしいという点がありましたら指摘してください。

↓以下コード
①

#include <stdio.h>
#include <math.h>
struct circle
{
	double x;
	double y;
	double rad;
};
void swap(double *px,double *py)
{
	double tmp=*px;
	*px=*py;
	*py=tmp;
}
int main(void)
{
	int i,j;
	double num[3]={1,2,3};
	struct circle n[3];
	char str[50];
	FILE *fp;
	
	puts("3つの円の座標と半径を入力");
	for(i=0;i<3;i++)
	{
		printf("円%.0f\n",num[i]);
		printf("座標x="); scanf("%lf",&n[i].x);
		printf("座標y="); scanf("%lf",&n[i].y);
		printf("半径r="); scanf("%lf",&n[i].rad);
	}
	puts("半径の大きい順にソート\n");
	for(i=0;i<3;i++)
	{
		for(j=i+1;j<3;j++)
		{
			if(n[i].rad<n[j].rad)
			{
				swap(&num[i],&num[j]);
				swap(&n[i].x,&n[j].x);
				swap(&n[i].y,&n[j].y);
				swap(&n[i].rad,&n[j].rad);
			}
		}
	}
	for(i=0;i<3;i++)
		printf("円%.0f:(%.2f,%.2f),r=%.2f\n",num[i],n[i].x,n[i].y,n[i].rad);
	
	//ファイルに出力
	printf("ファイル名を入力:"); scanf("%s",str);
	fp=fopen(str,"wb");
	if(fp==NULL)
		printf("ファイル%sをオープンできませんでした。\n",str);
	else
	{
		printf("半径および中心位置を保存しました。\n");
		fwrite(n,sizeof(*n),3,fp);
		fclose(fp);
	}
	return 0;
}

②

#include <stdio.h>
#include <math.h>
struct circle
{
	double x;
	double y;
	double rad;
};
void len_p(double x,double y,double *len)
{
	*len=sqrt(pow(x,2)+pow(y,2));
}
void swap(double *px,double *py)
{
	double tmp=*px;
	*px=*py;
	*py=tmp;
}
int main(void)
{
	int i,j;
	struct circle n[3];
	double len[3];
	char str[50];
	FILE *fp;
	
	printf("ファイル名を入力:"); scanf("%s",str);
	if((fp=fopen(str,"rb"))==NULL)
		printf("ファイル%sをオープンできませんでした。\n",str);
	else
	{
		fread(n,sizeof(*n),3,fp);
		for(i=0;i<3;i++)
		{
			printf("(%.2f,%.2f),r=%.2f\n",n[i].x,n[i].y,n[i].rad);
			len_p(n[i].x,n[i].y,&len[i]);
		}
		puts("中心位置が原点から近い順にソート");
		for(i=0;i<3;i++)
		{
			for(j=i+1;j<3;j++)
			{
				if(len[i]>len[j])
				{
					swap(&n[i].x,&n[j].x);
					swap(&n[i].y,&n[j].y);
					swap(&n[i].rad,&n[j].rad);
					swap(&len[i],&len[j]);
				}
			}
		}
		for(i=0;i<3;i++)
			printf("(%.2f,%.2f),r=%.2f,距離%.2f\n",n[i].x,n[i].y,n[i].rad,len[i]);
		fclose(fp);
	}
	return 0;
}

Answer 2

fwrite(n,sizeof(n),3,fp);

sizeof(n)ってポインタのサイズですよ。
sizeof(*n)でいこう

---

ああちがうわ。sizeof(n) は配列全体のサイズになりますね
それだと、書き込みはともかく、読み出しのときにアクセス違反となりますな。

Comments

[darakuzanmai]

回答ありがとうございます！早速直してみます

[y_waiwai]

回答修正