C言語を用いて読み込んだファイルを線形リストにし表示したい

Question

以下のようなデータテキストを

10 香川真司 FW_1989/03/17
17 長谷部誠 MF_1984/01/18
5 長友佑都 DF_1986/09/12
13 大久保嘉人 FW_1982/06/09
7 遠藤保仁 MF_1980/01/28
9 岡崎慎司 FW_1986/04/16
1 川島永嗣 GK_1983/03/20
4 本田圭佑 MF_1986/06/13
2 内田篤人 DF_1988/03/27
15 今野泰幸 DF_1983/01/25
22 吉田麻也 DF_1988/08/24

構造体に読み取り、標準画面出力しようと考えて下記のようなコードを書きました。(ただし、練習のため線形リストを使うようにしています)

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

/* 構造体の定義 */
struct member{
  int  num;                    /* 登録番号 */
  char name[13];               /* 氏名 */
  char profile[45];            /* プロフィール  */
  struct member *next;
};

typedef struct member list_mbr;

list_mbr *list(FILE *fp);
void printlist(list_mbr *p);

int main(int argc,char* argv[]){
    FILE *fp;
    list_mbr *listptr;
    
    if ((fp = fopen("test.txt", "r"))==NULL) {
        fprintf(stderr,"ファイルを開くのに失敗しました\n");
        exit(1);
    }
    listptr = list(fp);
    printlist(listptr);
    fclose(fp);
    return 0;
}



/* list()関数：ファイルポインタを引数とし、ファイルから */
/*   データを読み込みリストを作成する。               */
/*   戻り値は、作成されたリストの先頭データのアドレス */
list_mbr *list(FILE *fp){
    int d;
    list_mbr *p,*newp;
        
    p=NULL;
    while(fscanf(fp,"%d %s %s\n",&p->num,p->name,p->profile) != EOF){
        newp->next = p;
        p = newp;
    }
    return p;
}


/* printlist()関数：リストの先頭データのアドレスを引数 */
/*   とし、リストの内容を出力する                      */
void printlist(list_mbr *p){
    while (p != NULL) {
    printf("%d %s %s ", p->num,p->name,p->profile);
    p = p->next;
  }
  printf("\n");
}

このコードを実行しようとしたものの、うまくいかず(実行時に「セグメンテーション違反です (コアダンプ)」とエラーが出て強制終了します。おそらくlist()関数内部のデータの読み取り部分で上手く読み取れていないと思われます)困っています。どのように直せばいいでしょうか。

Answer 1

リスト構造を扱いたいのですね、結論から言えばヒープ領域とスコープという概念を調べてみてください。
おそらくポインタの概念も怪しいですね。

なぜ、セグメンテーション違反が起きているかというと、未定義（未初期化）のポインタに値を書き込んでいるからです。
list_mbr *pはポインタではなく、list_mbr pと実体を定義すべきですね。

数行直すだけで動くコードではなかったので、問題がある部分の指摘だけで失礼します。

Comments

[KEILA]

回答ありがとうございます。
たしかに基礎的なことの理解が不十分だったようで、ポインタから再度勉強し直すことにしました。

[KEILA]

回答いただいたとおり、メモリを確保するように以下のように書き換えたところ無事動作しました。
```C
list_mbr *list(FILE *fp){
list_mbr *p,*newp;
p=NULL;
newp = (list_mbr *)malloc(sizeof(list_mbr));
while(fscanf(fp,"%d %s %s",&newp->num,newp->name,newp->profile) != EOF){
newp->next = p;
p = newp;
newp = (list_mbr *)malloc(sizeof(list_mbr));
}
return p;
}
```
ありがとうございました。

[pashango2]

コードを見る限り上手く動くようには見えません。
戻り値のpは先頭ポインタを返すべきですが、実際には最後から2番めのポインタを返しています。

最初のmalloc関数の部分をこうすべきでは？

list_mbr *startp;

startp = newp = (list_mbr *)malloc(sizeof(list_mbr));

returnはstartpを返す。

return startp;