Question

//美好的一天。这只是我源代码的一部分。我的主要困境是print_list函数只打印链表中用户的第一个输入。我似乎无法确定问题，因为函数的逻辑似乎是正确的。 insert_list函数似乎也可以正常工作，但我不确定。

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

//这是节点

typedef struct node
{
        char name[61];
        int month;
        int day;
        int year;
        struct node *next;
}node;

//这是列表

typedef struct list
{
        node *head;
        node *tail;
}list;

//这会通过接受用户的输入并将它们放入节点

来创建节点

node *create_node()
{
        int x;
        node *data = (node*) malloc(sizeof(node));
        printf("Name: ");
        fgets(data->name, 61, stdin);
        printf("Birthdate (mm/dd/yyyy): ");
        scanf("%d%*[/]%d%*[/]%d", &data->month, &data->day, &data->year);
        getchar();
        if ((data->month)==0||(data->month)>=13||(data->day)<=0||(data->day)>=32||(data->year)<=1977||(data->year)>=3001)
        {
                while ((data->month)==0||(data->month)>=13||(data->day)<=0||(data->day)>=32||(data->year)<=1977||(data->year)>=3001)
                {
                        printf("Invalid Input.\n");
                        printf("Please Enter a Valid Birthdate(mm/dd/yyyy): \n");
                        scanf("%d%*[/]%d%*[/]%d", &data->month, &data->day, &data->year);
                        getchar();
                }
        }
        printf("******************************************************************\n");
        for (x=0; x<=strlen(data->name); x++)
        {
            if (data->name[x]=='\n')
            {
                    data->name[x]='\0';
            }
    }
    printf("Birthday reminder for %s is added.\n", data->name);
    return data;
}


list *create_list(list *plist)
{
        plist->head = NULL;
        plist->tail = NULL;
        return plist;
}

//这会在列表中插入节点

list *insert_list(list *plist, node *pnode, node *new_node)
{
    if(plist->head==NULL)
    {
            plist->head=new_node;
            new_node->next=NULL;
    }
    else
    {
            new_node->next = NULL;
            pnode->next = new_node;
            plist->tail = new_node;
    }
    return plist;
}

//这会打印列表

list *print_list(list *plist)
{
        node *current = plist->head;
        int i;
        for(i=1;current!=NULL;i++)
        {    
            printf("[%d] %s\n",i ,current->name);
            printf("Birth Date: %d/%d/%d\n", current->month, current->day, current->year);
            current=current->next;
    }
}

//这会解除分配列表

list *free_list(list *List)
{
    node *current = List->head;
    node *temp = NULL;
    while(current != NULL)
    {
            temp = current;
            current = current->next;
            free(temp);
    }

    List->head = NULL;
    List->tail = NULL;
}

//这是主要的

int main(void)
{
        list* List = (list*) malloc(sizeof(list));
        List = create_list(List);
        char x;
        node *data = (node *) malloc(sizeof(node));
        printf("******************************************************************\n");
        printf("                    ADD BIRTHDAY REMINDER FORM\n");
        printf("******************************************************************\n");
        List = insert_list(List, data, create_node(data));
        printf("Would you like to add another(y/n)?\n");
        scanf("%c", &x);
        if (x=='y')
        {
                while (x=='y')
                {
                        if (x=='y')
                        {
                                getchar();
                            printf("******************************************************************\n");
                                node *data = (node *) malloc(sizeof(node));
                                List = insert_list(List, data, create_node(data));
                                printf("Would you like to add another(y/n)?\n");
                                scanf("%c", &x);
                        }
                }
        }
    print_list(List);
    free(List);
    return 0;

}

//此代码已准备好进行填写

Answer 1

您确定粘贴了实际来源吗？在List = insert_list(List, data, create_node(data));中调用create_node(data)，但函数node *create_node()采用零参数。我不理解`list * insert_list（list * plist，node * pnode，node * new_node）中node *pnode参数的用途

打印功能对我来说似乎是合法的。尝试使用-Wall -Wextra -Werror进行编译以进行额外的错误检测。 `

Answer 2

我不明白，pnode insert_list中你打算做什么，我猜，有错误。可能你的意思是上一个节点。但是，尾已经是前一个节点。此外，您在那里使用空节点，甚至为create_node中的节点分配内存。也许，以下代码更合适：

list *insert_list(list *plist, node *new_node)
{
    if(plist->head==NULL)
    {
            plist->head=new_node;
            plist->tail=new_node;
            new_node->next=NULL;

    }
    else
    {
            new_node->next = NULL;
            plist->tail->next = new_node;
            plist->tail = new_node;
    }
    return plist;
}

Answer 3

我不明白为什么pnode需要第二个参数insert_list。如果您只想打印列表中的所有元素，我认为以下修改可以解决您的问题：

首先，在plist->tail=new_node的第一个if子句中添加一行insert_list。其次，在insert_list中将data的第二个参数从List->tail更改为main。

我还想指出另一个不重要的事情。你真的需要if (x == 'y)中的条件main吗？我不知道为什么会这样。

print_list函数仅在链表中打印用户的第一个输入

3 个答案: