fgetsとmemcpyのやり方

Question

fgets関数を用いて、ファイル（class.dat）を1行ずつ読み込み、memcpyで切り取りをし、生徒の成績を作りたいと考えています。
ファイル（class.dat）の内容は左から、クラス（全角2桁）、氏名（全角20桁）、性別（全角MまたはFの1桁）、国語の点数（半角3桁）、数学の点数（半角3桁）、英語の点数（半角3桁）となっています。また、性別は男・女と表示したいと考えてます。

現在、ファイルの読み込みはできたのですが、memcpyの部分が上手く出来ないです。
C言語初心者なので、考え方が浮かばないので、教えていただけますと幸いです。

JM田中 なおき           M026052036
JM渡辺よしひこ         M056011023
JM吉田 なおみ           F078002014
JM高橋 啓介              M095100033
JM武田 謙信              M020025003
RM大阪 ヒロキ          M069078056
RM太田 信彦             M033069074
RM吉田 さおり          F025085012
KM守屋 かおり          F036075074
KM佐藤 かよ             F014026085
KM川崎 たかや          M025047091
KM野原 ひろし          M098036062

表示したい結果＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝
クラス　　氏名　　　　　性別　　国語　　数学　　英語　　合計　
JM　　　　田中 なおき 男　　　026　　052　　036　　 114
・・・・・省略・・・・・

＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝

#define _CRT_SECURE_NO_WARNINGS
#include <stdio.h>
#include <stdlib.h>

int main() {

	FILE* fp;
	char str[256];
	char class[4];
	char name[64];
	char gen[2];
	char Tensu[16];
	int tensu;
	int jap;
	nt math;
	int eng;

	printf("クラス\t氏名\t\t性別\t国語\t数学\t英語\t合計\t\n");
	printf("------------------------------------------------------------\n");


	fp = fopen("class.dat", "r");


	if (fp == NULL) {
		printf("ファイルオープン失敗\n");
		return -1;
	}

	while (!feof(fp)) {
		if (fgets(str, 256, fp) == NULL) {
			break;
		}

		memcpy(&class, str, 2);
		memcpy(&name, str[2], 20);
		memcpy(&gen, str[22], 1);
		memcpy(&Tensu, str[23], 9);
		tensu = atoi(Tensu);
		memcpy(&jap, Tensu[3], 3);
		memcpy(&math, Tensu[6], 3);
		memcpy(&eng,  Tensu[9],3);

		printf("%s\t %s\t\t %s\t %s\n ",
			&class,
			&name,
			&gen,
			&jap,
                        &math
                        &eng
                        &jap + &math + &eng
		);
	}
	fclose(fp);

	return 0;
}

''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''

Comments

[y_waiwai]

うまくいかない、とはどういう動作となるんでしょうか。詳しく説明しよう
また、実際にはどういう表示になるのかってのも提示しよう

[kekennnn]

printf("クラス\t氏名\t\t性別\t国語\t数学\t英語\t合計\t\n");
printf("------------------------------------------------------------\n");
この部分の表示はできています。

また、memcpy(&gen, str[22], 1);の部分で『0x00007FF900241416 (vcruntime140d.dll) で例外がスローされました (test.exe 内): 0xC0000005: 場所 0x00000000FFFFFFE7 の読み取り中にアクセス違反が発生しました』という表示が出てきており、調べたのですが、よくわかりませんでした、

[archiver]

氏名データですが、「渡辺よしひこ」だけ姓と名の間にスペースが無く、他はスペースがありますが、姓と名の間にスペースを入れる入れないは何かルールはありますか？
またデータに全角文字（マルチバイト文字）を使用していますが、文字コードは何を想定していますか？文字コードによっては全角文字（マルチバイト文字）を表すバイト数が変わりますので、今の単に「文字数」指定のmemcpyでは期待通りのコピーにならないと思います。

[kekennnn]

氏名データですが、「渡辺よしひこ」ですが、ファイルを渡された時点でこの状態であったので、特にルールはないと思います。
文字コードについても指示がない状況です。

[archiver]

これって委託業務か何かですか？データファイルについては仕様を確認された方がいいです。
１レコード（１行）のデータに可変長データ（氏名）と固定長データ（氏名以外）が混在しています。今の形式では氏名データ（以降）を抜き出すのがかなり困難になります。

[kekennnn]

学校の課題です。　
クラス（全角2桁）、氏名（空白全角20桁）、性別（全角MまたはFの1桁）、国語の点数（半角3桁）、数学の点数（半角3桁）、英語の点数（半角3桁）で固定長と言われました。

[kazuma-s]

質問の class.dat の内容を ```text と ``` で挟んでください。
ソースコードも main の中身だけを ``` と ``` で挟むのではなく、
#define ... から } までの全部を ```C と ``` で挟んでください。
質問は編集できます。

[kazuma-s]

> クラス（全角2桁）、氏名（空白全角20桁）、性別（全角MまたはFの1桁）、
質問の class.dat の内容では、クラスも性別も半角になっていますが、違うんですか？

Answer 1

#define _CRT_SECURE_NO_WARNINGS
#include <stdio.h>

int main(void)
{
	FILE *fp;
	char str[256], class[4], name[64], name2[64], gen[2];
	int jap, math, eng;

	printf("クラス\t氏名\t\t性別\t国語\t数学\t英語\t合計\t\n");
	printf("------------------------------------------------------------\n");
	fp = fopen("class.dat", "r");
	if (fp == NULL) {
		printf("ファイルオープン失敗\n");
		return -1;
	}
	static const char *s[] = { "男", "女" };
	while (fgets(str, 256, fp) != NULL)
		if (sscanf(str, "%2s%s%s%1s%3d%3d%3d",
				class, name, name2, gen, &jap, &math, &eng) == 7)
			printf(" %s\t %s %s\t %s\t %03d\t %03d\t %03d\t %03d\n",
				class, name, name2, s[*gen=='F'], jap, math, eng, jap + math + eng);
		else if (sscanf(str, "%2s%s%1s%3d%3d%3d",
				class, name, gen, &jap, &math, &eng) == 6)
			printf(" %s\t %s\t %s\t %03d\t %03d\t %03d\t %03d\n",
				class, name, s[*gen=='F'], jap, math, eng, jap + math + eng);
		else printf("wrong data");
	fclose(fp);
}

性別を M F ではなく、男女に修正しました。

追記
質問のコードに #define _CRT_SECURE_NO_WARNINGS があることから、
OS が Windows で、コンパイラが Visual C++ だとすると、
文字コードは Shift-JIS でしょう。
それだと全角文字は 2バイトになります。
memcpy はバイト数を指定してバイト列をコピーするものです。

#include <stdio.h>   // printf
#include <stdlib.h>  // atoi
#include <string.h>  // memcpy

int main(void)
{
	            // 01234567890123456789012345678901234
	char str1[] = "JM田中 なおき           M026052036\n";
	            // 12全全1全全全12345678901.123456789
	char str2[] = "ＪＭ田中　なおき　　　　Ｍ026052036\n";
	            // 全全全全全全全全全全全全全123456789
	            // 1 2 1 2 3 4 5 6 7 8 9 10
	char class[5] = "", name[23] = "", gen[3] = "", num[4] = "";
	int jap, math, eng;

	memcpy(class, str1, 2);
	memcpy(name, &str1[2], 22);
	memcpy(gen, &str1[24], 1);
	memcpy(num, &str1[25], 3);
	jap = atoi(num);
	memcpy(num, &str1[28], 3);
	math = atoi(num);
	memcpy(num, &str1[31], 3);
	eng = atoi(num);
	printf("[%s] [%s] [%s] [%d] [%d] [%d]\n",
		class, name, gen, jap, math, eng);

	memcpy(class, str2, 4);
	memcpy(name, &str2[4], 20);
	name[20] = '\0';
	memcpy(gen, &str2[24], 2);
	memcpy(num, &str2[26], 3);
	jap = atoi(num);
	memcpy(num, &str2[29], 3);
	math = atoi(num);
	memcpy(num, &str2[32], 3);
	eng = atoi(num);
	printf("[%s] [%s] [%s] [%d] [%d] [%d]\n",
		class, name, gen, jap, math, eng);
}

char配列は = "" で初期さすると、全バイトが '\0' で初期化されます。

実行結果

[JM] [田中 なおき           ] [M] [26] [52] [36]
[ＪＭ] [田中　なおき　　　　 ] [Ｍ] [26] [52] [36]

Comments

[kekennnn]

お返事遅くなりすいません。
こちら参考にさせて頂いたところ、上手く動作しました。
ありがとうございます。

[kazuma-s]

クラス名は全角なんですか、半角なんですか？
はっきりしてください。

Answer 2

memcpy(&name, str[2], 20);

第二引数はアドレスですが、ここではcharとなってしまっています。

memcpy(&name, &str[2], 20);
あるいは
memcpy(&name, str+2, 20);
としましょう
＃他も同様

Comments

[kekennnn]

#define _CRT_SECURE_NO_WARNINGS
#include <stdio.h>
#include <stdlib.h>

int main() {

FILE* fp;
char str[256];
char class[4];
char name[64];
char gen[2];
char Tensu = 0;

int tensu;
int jap = 0;
int math = 0;
int eng = 0;
int sum = jap + math + eng;
printf("クラス\t氏名\t\t性別\t国語\t数学\t英語\t合計\t\n");
printf("------------------------------------------------------------\n");

fp = fopen("class.dat", "r");

if (fp == NULL) {
printf("ファイルオープン失敗\n");
return -1;
}

while (!feof(fp)) {
if (fgets(str, 256, fp) == NULL) {
break;
}

memcpy(&class, &str, 2);
memcpy(&name, &str[2], 20);
memcpy(&gen, &str[22], 1);
memcpy(&Tensu, &str[23], 9);
tensu = atoi(&Tensu);
memcpy(jap, &Tensu, 3);
memcpy(math, &Tensu, 3);
memcpy(eng, &Tensu,3);

printf("%s\t %s\t\t %s\t %d\t %d\t %d\n ",
&class,
&name,
&gen,
math,
eng,
sum
);

}
fclose(fp);

return 0;
}

＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝

修正したのですが、memcpy(math, &Tensu, 3);の部分で同様に『アクセス違反が発生』となってしまいました。

[y_waiwai]

memcpy の第一引数もアドレスです。（それ、intぢゃん）
memcpyという関数は何をするものか、ってのを調べてみよう

おそらく課題か何かだと思いますが、まだまだ、ここらへんのメモリ操作に関することについて学習の必要があるかと

[kazuma-s]

int jap = 0;
int math = 0;
int eng = 0;
int sum = jap + math + eng;

このような宣言をしておけば、あとで jap などの値が変わった時、sum の値も変わると思っているのですか？

[kekennnn]

その認識でした。