構造体を用いてファイルの中身を表示させるプログラムを作成したいです．

Question

前提・実現したいこと

txtファイルの数値を表示させるプログラムを作成しています．
エラーがなくなり実行したところ数値がバグって表示されてしまいます．
汚いプログラムかと思いますがご回答よろしくお願いいたします．

使用しているtxtファイルの中身です．

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

./NSP                                                                                                                                                               [~/Desktop/NRP]

zsh: segmentation fault  ./NSP

該当のソースコード

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

#define MAX_STAFF 8

typedef struct{
  char ID;//従業員ID
  int MaxShifts;//作成する稼働表の日数．ここでは14日
  int MaxTotalMinutes;//1日の最大労働時間(単位：分)
  int MinTotalMinutes;//1日の最小労働時間(単位：分)
  int MaxConsecutiveShifts;//最大の連勤回数
  int MinConsecutiveShifts;//最小の連勤回数
  int MinConsecutiveDaysOff;//最小の連続した休日
  int MaxWeekends;//週末は一週間で一回しか来ない．
}employee_time_info;

int main(){
  int i,mt;
  int m,n,o,p,q,r,s,t; //ファイルの受け取り
  FILE *fp;
  employee_time_info staff[MAX_STAFF];//構造体の宣言

  if((fp=fopen("staff_info.txt","r"))==NULL){
    printf("ファイルが開けません\n");
    exit(1);
  }
 for(i=0;i<MAX_STAFF;i++){
    if(fscanf(fp," %s ,%d ,%d ,%d ,%d ,%d ,%d ,%d\n",staff[i].ID,&staff[i].MaxShifts,staff[i].MaxTotalMinutes,staff[i].MinTotalMinutes,&staff[i].MaxConsecutiveShifts,&staff[i].MinConsecutiveShifts,&staff[i].MinConsecutiveDaysOff,&staff[i].MaxWeekends)!=8)break;
  }
  fclose(fp);
		
  /* for(i=0;i<MAX_STAFF;i++){
  printf(" 従業員:%s \n1日の最大労働時間: %d分\n 1日の最小労働時間: %d分\n 最大の連勤回数: %d回\n 最小の連勤回数 %d回\n",staff[i].ID,staff[i].MaxShifts,staff[i].MaxTotalMinutes,staff[i].MinConsecutiveShifts,staff[i].MinConsecutiveShifts); 
  }*/

  for(i=0;i<MAX_STAFF;i++){
		printf("従業員: %s\n",staff[i].ID);
		printf("1日の最大労働時間: %d\n",staff[i].MaxShifts);
		printf("1日の最小労働時間: %d分\n",staff[i].MinConsecutiveShifts);
		printf("最大の連勤回数: %d回\n ", staff[i].MaxConsecutiveShifts);
		printf("最小の連勤回数: %d回\n",staff[i].MinConsecutiveShifts);
  }
		
  return 0;
}

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

Macのemacsを使用しています．コンパイラはgccです．

Comments

[jimbe]

タイトルには、ご質問の内容の主旨をお書きください。

[K_3578]

https://teratail.com/help/question-tips#questionTips3-1

[dodox86]

[2021/07/28 16:26]の質問内容の編集を読んで：
teratailは技術的な質問に対して回答を得ようとするQ&Aサイトであって、要望を書けば添削やコードの提供をしてもらえる場所ではありません。（そうでないと、希望のプログラムの完成までずっと付き合うことになりますよね）

[RyoBaya]

dodox86さん，申し訳ありませんでした．焦っていた為，文章が抜けてしまい大変失礼になってしまいました．

Answer 1

char ID;//従業員ID

なので、書式は%sじゃなくて%cです。
あと、fscanfで&がいくつか抜けてます。見にくいので、適宜改行すると良いです。

また、fscanfループでbreakしたときのiの値が読み込んだ件数なので、
出力するときは、その件数までの出力にしないと、以降はゴミデータが出力されます。

Answer 2

fscanf(fp,"%s %d %d %d %d %d %d %d\n",...

カンマ区切りで読んでいない。

Comments

[RyoBaya]

カンマ区切りで読むのはどうすればよろしいですか．

[episteme]

scanfに与える書式指定でどうにかできそうだけど、僕はstrtokでやるのが好み。

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

int main() {

char line[64] = "A,14,4320,3360,5,2,2,1";
for ( char* token = strtok(line, ",\n"); token; token = strtok(NULL,",\n") ) {
printf("[%s]\n", token);
}
return 0;
}

[episteme]

あと、IDを格納する領域を確保していない。

[RyoBaya]

すみません．表示させたい表現が間違っていることに気づきました．
ソースコード変更したのでよろしくお願いいたします．

[episteme]

なにをお願いしているのでしょうか。質問には答えたつもりなんですが。

[episteme]

え? char* ID じゃなく char ID なの? だったら 書式 %s は明らかにマチガイちゃいます?

[RyoBaya]

申し訳ありません．charに関しては脱字でした．
表示させたいように表示できないので困っています．

従業員:A
1日の最大労働時間　4320分
1日の最小労働時間　3360分
最大の連勤回数　5回
最小の連勤回数　2回
以下，従業員B，従業員C...
というのを表現したく再度質問させてください．

[episteme]

strtokを使ってカンマ区切りで分割するコードを示しました。それでは不満ですか?