Cのリストの要素を印刷

Question

基本的に以下のコードでは、いくつかの名前といくつかの年代をリストに入れて印刷しようとしています。しかし、私のプログラムはリストの最後の名前と年齢だけを表示します。助言がありますか？

#include <stdio.h> 
#include <stdlib.h> 
/* these arrays are just used to give the parameters to 'insert', 
    to create the 'people' array 
*/ 

#define HOW_MANY 7 
char *names[HOW_MANY]= {"Simon", "Suzie", "Alfred", "Chip", "John", "Tim", 
       "Harriet"}; 
int ages[HOW_MANY]= {22, 24, 106, 6, 18, 32, 24}; 

typedef struct person 
{ 
    char *name; 
    int age; 
    struct person *next; 
}Person; 

Person *headp = NULL; 
static Person* insert(Person *p, char *name, int age) 
{ 
    p = (Person*)malloc(sizeof(Person)); 
    if (p == NULL) 
    abort(); 
    p->name = name; 
    p->age = age; 
    p->next = headp; 
    return p; 
} 

int main(int argc, char **argv) 
{ 
    Person *people=headp; 
    for (int i = 0; i < 7; i++) 
    { 
    people = insert (people, names[i], ages[i]); 
    } 
    while (people != NULL) 
    { 
    printf ("name: %s, age: %i\n", people->name, people->age); 
    people= people->next; 
    } 
    return 0; 
} 

Answer 1

あなたは他のpeopleポインタはまだ最後の人を指しているので、あなたが、挿入が完了したら（あなたがすべての挿入でpeopleポインタを進めているため）peopleにheadpを再割り当てする必要がある場合があります。

Answer 2

あなたは（メモリリークし、前頭部を失う）mallocによって返されたアドレスで渡された変数pを上書きしている、に変更します

static Person *insert(Person *head, char *name, int age) 
{ 
    Person *p = malloc(sizeof(Person)); /* Don't cast malloc */ 

    if (p == NULL) 
    abort(); 
    p->name = name; 
    p->age = age; 
    p->next = head; 
    return p; 
}