Question

我正在尝试合并两个链接列表。我遇到的一个问题是，即使在1个列表到达结尾之后，我的while循环仍然可以继续运行。

E.G：List1值：1,3; 列出2值：2,6,7,8;

理想情况下，输出为：1,2,3,6,7,8 目前我的输出更像1,2,3（list1已经到了结尾，所以循环停止，其他列表2的值不会被添加到列表中）。

另外，我希望我的合并不会破坏我合并的原始2个列表，我该如何实现？

struct Node
{
    int value;
    Node *next;
};

void addNode(Node* &head, int x) 
{

    Node* temp = new Node;
    temp->value = x;
    temp->next = nullptr;

    if(!head) 
    { 
        head = temp;
        return;
    } 
    else 
    {
        Node* last = head;
        while(last->next) 
        last=last->next;
        last->next = temp;
    }
}


 void merge(Node * &head1, Node * &head2, Node * &head3)
 {
     while (head1 != nullptr && head2 != nullptr)
     {

        if (head1->value < head2->value)
        {
            addNode(head3, head1->value); 
            head1 = head1->next;
        }
        else
        {
            addNode(head3, head2->value);
            head2 = head2->next;
        }

     }
 }

主要功能：

int main()
{
    Node *head = nullptr;
    Node *head2 = nullptr;
    Node *head3 = nullptr;

    for (int i=0; i<=8; i+=2)
        addNode(head, i);

    for (int i=1; i<=5; i++)
        addNode(head2, i);

    merge(head, head2, head3);

    printList(head);
    printList(head2);
    printList(head3);

    system("PAUSE");
    return 0;
}

Answer 1

您忘记合并任何剩余的项目，请使用：

 // dont pass head1, head2 by reference in method call
 void merge(Node * head1, Node * head2, Node * &head3)
 {
     // and/or use other variables to avoid changing head1, head2
     Node * list1 = head1;
     Node * list2 =  head2;

     while (list1 != nullptr && list2 != nullptr)
     {

        if (list1->value < list2->value)
        {
            addNode(head3, list1->value); 
            list1 = list1->next;
        }
        else
        {
            addNode(head3, list2->value);
            list2 = list2->next;
        }

     }
     // merge any remaining list1 items
     while (list1 != nullptr)
    {
       addNode(head3, list1->value); 
       list1 = list1->next;
    }
     // merge any remaining list2 items
     while (list2 != nullptr)
    {
       addNode(head3, list2->value); 
       list2 = list2->next;
    }
 }

Answer 2

Nikos M.回答了你的第一个问题，答案很好，所以我不会重复。这个答案解决了你的第二个问题：

另外，我希望我的合并不会破坏我合并的原始2个列表，我该如何实现？

答案很简单：不要通过引用传递head1和head2：

void merge(Node * head1, Node * head2, Node * &head3)

事实上，我建议制作head1和head2 const指针：

void merge(const Node * head1, const Node * head2, Node * &head3)

Answer 3

只要head1或head2变为null，while循环就会退出。我想你想添加一段额外的代码来附加非空列表中的所有剩余元素（我假设它们已经排序）。

"liveplot_p.gnu", line 9: x range is invalid

向Node添加构造函数，以便next始终初始化为nullptr。我最初读错了一个想法你错过了这个初始化，但添加一个构造函数会简化你的代码，并且意味着你不会忘记在其他地方创建节点时初始化下一个指针。

 Node* lastElements = list1 != nullptr ? list1 : list2;
 while( lastElements != nullptr )
 {
     addNode(list3, lastElements->value);
     lastElements = lastElements->next;
 }

并且addNode函数的开头变为1行而不是3行。

Node( int initValue)
: value(initValue)
,  next(nullptr)
{}

如果合并不具有破坏性，则不要传递对head1和head2指针的引用。就像这样。

Node* temp =  new Node(x);

2链接列表合并困难

3 个答案: