ポインタ要素でリンクリストのデータを取得する

Question

#include<stdio.h> 
#include <stdio.h> 
#include <string.h> 
#include <stdlib.h> 
struct trial{ 
    char *data; 
    int save; 
    struct trial *next; 
}; 

struct trial *head = NULL; 

int main(){ 
    int x, ctr, y; 
    char filestr[500]; 
    char *data, *save, *filestr2; 
    FILE *fp; 
    fp = fopen("Untitled1.txt", "r"); 
    printf("Count: "); 
    scanf("%d", &x); 
    while(x > 0){ 
     if(fgets(filestr, sizeof(filestr), fp) != NULL){ 
      data = strtok(filestr, " "); 
      filestr2 = strtok(NULL, ""); 
      save = strtok(filestr2, ""); 
      printf("%s, %s", data, save); 
      struct trial *link = (struct trial*) malloc(sizeof(struct trial)); 
      link->data = data; 
      link->save = atoi(save); 
      link->next = head; 
      head = link; 
     } 
     x--; 
    } 
    printf("\n"); 
    struct trial *ptr = head; 
    ctr = 0; 
    while(ptr != NULL){ 
      printf("Data %d: %s, %d\n", ctr + 1, ptr->data, ptr->save); 
      ptr = ptr->next; 
      ctr++; 
    } 
return 0; 
} 
/*Untitled1.txt is as follows 
dragon 12 
shadow 19 
spirit 6 
wiser 4 
civil 8 
fairy 7 
*/ 

ここで、x = 3のときに問題が発生します。それがなければならない：
数：3
竜、12
シャドウ、19
精神、6

データ1：精神、6
データ2：シャドウ、19
データ3：ドラゴン、12

しかし、ここでは何が起こるのですか。
データ1：精神、6
データ2：精神、19
データ3：精神、12

はなぜ保存変数が移動していること、および*データがないのですか？私は何を追加し、どこに置くべきですか？助けてくれてありがとう。 [不必要な変数を残して申し訳ありません、それは全体の一部です]

Answer 1

説明したように、strtokは最初のパラメータを指すポインタを返します。このバグを修正するにはあなたがinteressedしている部分をコピーします。単に

link->data = strdup(data); 

で

link->data = data; 

を置き換える無料リンク - >データを覚えておいてください。

Answer 2

strtok返される文字列のメモリを割り当てません。それはあなたがそれを与えるバッファにポインタを返すだけです。したがって、保存したいデータをコピーする必要があります。

最も簡単な修正は変更することです：

link->data = data; 

へ：

link->data = strdup(data); 

またstrtokからNULL戻り値のチェックを追加して良いでしょう。