Question

我试图创建一个名为IntNode *interleave_lists(IntNode *head_1, IntNode *head_2);的链接列表函数，它接受两个链表，组合它们并返回指向头部的指针。

示例：假设head_1指向包含三个整数的链表：1,3,5和head_2 指向包含5个整数的链表：10,11,12,13,14。新链表将包含8个整数，按此顺序：1,10,3,11,5,12,13,14。

我有这些结构来帮助创建链表：

struct intnode {
   int value;
   struct intnode *next;
};
typedef struct intnode IntNode;

IntNode *intnode_construct(int value, IntNode *next)
{
   IntNode *p = malloc(sizeof(IntNode));
   assert (p != NULL);
   p->value = value;
   p->next = next;
   return p;
}
void print_linked_list(IntNode *p) //for printing linked list
{
    if (p == NULL) {
        printf("empty list");
        return; 
    }

    /* Print every node in the linked list except the last one,
       using the format: value1 -> value2 -> value3 -> ...
     */
    for (; p->next != NULL; p = p->next) {
        printf("%d -> ", p->value);
    }

    /* Print the last node. */
    printf("%d", p->value);
}

现在尝试创建我的功能：

IntNode *interleave_lists(IntNode *head_1, IntNode *head_2)
{
    int count = 1;
    IntNode *p1=head_1, *p2=head_2,*head=NULL;
    for(; (p1->next != NULL) || (p2->next != NULL); p1 = p1->next, p2 = p2->next){
        if(p1 != NULL && count == 1){
            head = intnode_construct(p1->value,head);
            count = 0;
        } else if(p2 != NULL && count == 0){
            head = intnode_construct(p2->value,head);
        }
    }
    return head;
}

但每次运行时，控制台都会显示CRT：未处理的异常（主要） - 终止。我对此功能的测试用例：

int main(void)
{
    IntNode *list1 = NULL,*list2 = NULL; // An empty linked list.
    list1 = intnode_construct(5, list1);
    list1 = intnode_construct(3, list1);
    list1 = intnode_construct(1, list1);

    list2 = intnode_construct(14, list2);
    list2 = intnode_construct(13, list2);
    list2 = intnode_construct(12, list2);
    list2 = intnode_construct(11, list2);
    list2 = intnode_construct(10, list2);


    IntNode *head = NULL;

    head = interleave_lists(list1, list2);
    print_linked_list(head);
}

Answer 1

多个问题

您检查p->next != NULL而不是p != NULL，因此您将丢失列表的最后一个元素
你的循环检查p1 || p2，只要指针还没到达终点就循环。因此，如果列表的长度不同，则短列表的指针将在结束时运行并崩溃。
你的intnode_construct函数从右到左构建列表（从头到尾），但是你从头到尾迭代，所以你在交错时反转列表。

反转列表事物意味着你需要从尾部交错，使你的interleave_lists函数更容易递归实现：

IntNode *copy_list(IntNode *head)
{
    if (!head) return head;
    return intnode_construct(head->value, copy_list(head->next));
}

IntNode *interleave_lists(IntNode *head_1, IntNode *head_2)
{
    if (!head_1) return copy_list(head_2);
    return intnode_construct(head_1->value, interleave_lists(head_2, head_1->next));
}

或者，你可以做一个“破坏性的”＆＃39;交错，重用输入列表的节点：

IntNode *interleave_lists(IntNode *head_1, IntNode *head_2)
{
    if (!head_1) return head_2;
    head_1->next = interleave_lists(head_2, head_1->next);
    return head_1;
}

Answer 2

1）您的intnode_construct应如下所示

IntNode *intnode_construct(int value, IntNode *next)
{
   IntNode *p = malloc(sizeof(IntNode));
   if (p != NULL)
   {
       p->value = value;
       p->next = NULL;
   }
   return p;
}

使用您的代码，函数for(; (p1->next != NULL) || (p2->next != NULL);中的循环interleave_lists永远不会结束，因为->next永远不会== NULL。

2）您的interleave_lists必须如下所示：

IntNode *interleave_lists(IntNode *head_1, IntNode *head_2)
{
    int count = 1;
    IntNode *p1=head_1, *p2=head_2,*head=NULL;

    while ((p1 != NULL) || (p2 != NULL))
    {
        if(p1 != NULL && count == 1)
        {
            head = intnode_construct(p1->value,head);
            count = 0;
            p1 = p1->next;
        }
        else if(p2 != NULL && count == 0)
        {
            head = intnode_construct(p2->value,head);
            p2 = p2->next;
        }
    }

    return head;
}

您的代码始终将指针移动到每个循环中的next，而不检查指针是否有效或NULL。

3）顺便说一句，您必须更正代码才能拥有每个链接列表的头部。您的代码更新传递给interleave_lists最后一个元素...

创建interleave_lists链表？

2 个答案: