c言語を使って連結リストのセルにchar型配列を代入したい

Question

連結リストにchar型配列を入れる

現在c言語でテキストファイルを読み込んで連結リストに格納し、出力するプログラムを組んでいるのですがchar型配列がセルにうまく入りません。どこが悪いのか教えていただきたいです。

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

---

test.txt
7
0 該当番号がなかった場合
1 abc
4 ddd
7 fkd
2 ieo
4 rif
9 fld

---

実行結果

該当数７
0 該当番号がなかった場合
9 fld
9 fld
9 fld
9 fld
9 fld
9 fld

該当のソースコード

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

struct cell{
    char *no;
    char *naiyo;
    struct cell *next;
};

struct unk{
    char no[512];
    char naiyo[512];
};

struct cell front={"0","0"};

void printList(void){
    struct cell *p;
    p=front.next;
    while(p!=NULL){
        printf("%s %s\n",p->no,p->naiyo);
        p=p->next;
    }
}

int main(int argc,char *argv[]){

    char no[512],naiyo[512];
    struct cell *p;
    struct unk *unk;
    int cnt,i;
    
    
    if(argc!=2){
        fprintf(stderr,"プログラム引数を1つ指定してください\n");
        exit(1);
    }
    
    infile_open(argv[1]);
    
    infile_scanf("%s",no);
    cnt=atoi(no);
    printf("該当数%d\n",cnt);
    
    unk=malloc(sizeof(struct unk));
    if(unk==NULL){
        fprintf(stderr,"エラー:malloc失敗\n");
        exit(1);
    }
    
    infile_scanf("%s",unk->no);
    infile_scanf("%s",unk->naiyo);
    printf("%s ",unk->no);
    printf("%s\n",unk->naiyo);
    
    
    
    for(i=1;i<cnt;i++){
        p=malloc(sizeof(struct cell));
        if(p==NULL){
            fprintf(stderr,"エラー:malloc失敗\n");
            exit(1);
        }
        
        infile_scanf("%s",&no);
        infile_scanf("%s",&naiyo);
        
        
        p->no=no;
        p->naiyo=naiyo;
        
        p->next=front.next;/*新しいセルが今の先頭セルを指すようにする*/
        front.next=p;/*次にダミーセルが新しいセルを指すようにする*/
        
    }
    infile_close();
    
    printList();
    
    return 0;
}

Answer 1

+#include <string.h>
 
-        infile_scanf("%s",&no);
-        infile_scanf("%s",&naiyo);
+        infile_scanf("%s", no);
+        infile_scanf("%s", naiyo);
 
-        p->no=no;
-        p->naiyo=naiyo;
+        p->no = strdup(no);
+        p->naiyo = strdup(naiyo);

strdup は Cの標準ライブラリ関数ではありませんが、
ほとんどの処理系には入っています。
strdup が何をするものか調べて、報告してください。

Answer 2

    char no[512],naiyo[512];
    ...
    p->no=no;
    p->naiyo=naiyo;

p->no と p->naiyo には 常に同じポインタ値がセットされますから。

Comments

[amenbo]

迅速な回答ありがとうございます
ご指摘のありましたところを直したいのですが修正方法がわからないので具体的にどう直したらよいか教えていただけますでしょうか。
度々すみません。

[pepperleaf]

infile_scanf("%s",p->no);
infile_scanf("%s",p->naiyo);

では?

[amenbo]

ありがとうございます！無事にできました！