C、从链表中打印多个整数

Question

我对 C 很陌生，所以我不完全确定出了什么问题。我不知道如何在函数中打印多个整数值。

添加函数：

void add(char *name,int id,int copies)
{
/* Pointer to next item */
struct list *newAlbum;

newAlbum = malloc(sizeof(struct list)); 
strcpy((*newAlbum).name, name);  // Set album name
newAlbum->id = id;
newAlbum->copies = copies;
newAlbum->pNext = pFirst; 
pFirst = newAlbum;
}

show function：

void show()
{   
system("clear");
struct list *current_node;
current_node = pFirst;
while(current_node != NULL)
{   
    printf("Album #%d \n",current_node->id);
    printf("Album Name: %s \n",current_node->name);
    printf("Album Copies:%d \n",current_node->copies);
    printf("\n");
    current_node=current_node->pNext;
}
}

我的程序打印出current_node->id，就好像它是current_node->copies，而current_node->copies打印出为134516043，这显然是错误的。

我想我一定是向函数传递了一些错误的东西或者其他东西，但我无法弄清楚。有什么建议吗？

我这样调用函数 add：

add(name,id,copies);

列表如下：

 /* THE LIST */
 struct list{
    char name[52];
    int id;
    int copies;
    int sold;
    struct list* pNext;
};
struct list *pFirst = NULL;

我使用这段代码通过用户输入调用该函数：

printf("Enter the name of the new album. \n");
scanf("%s",&name);
printf("Enter the album id. \n");
scanf("%d",&id);
printf("Enter number of copies. \n");
scanf("%d," &copies);
// Pass data to add()
add(name,id,copies);

Answer 1

只要您不将长度超过 51 个字符的专辑名称传递给 add()，您所显示的代码就可以。如果你这样做，你会得到非常奇怪的输出，并且可能会崩溃。

为了防止这种情况，您应该使用长度有限的副本 - 例如：（

void add(char *name,int id,int copies)
{
    /* Pointer to next item */
    struct list *newAlbum;

    newAlbum = malloc(sizeof *newAlbum);
    if (newAlbum) {
        snprintf(newAlbum->name, sizeof newAlbum->name, "%s", name);  // Set album name
        newAlbum->id = id;
        newAlbum->copies = copies;
        newAlbum->pNext = pFirst;
        pFirst = newAlbum;
    }
}

请注意 sizeof *newAlbum 比 sizeof(struct list) 好一点，因为前者在阅读该行时“显然是正确的” - 如果 newAlbum 的类型发生更改，它仍然是正确的）。

Answer 2

我在这里看到的唯一错误是您没有检查 name 的长度。您应该使用：

strncpy(newAlbum->name, 52, name);

这将防止 name 缓冲区溢出。