两个链表的并集

Union of two linked lists

本文关键字：链表 两个      更新时间：2023-10-16

我正在尝试为两个链表的并集开发一个友元函数。编译器一直给我一个错误，说Node没有在这个作用域中声明，p、q也没有在这个范围中声明。所以我添加了List:：Node，但它仍然没有执行联合

   const List getunion(const List&a, const List&b) {
      List::Node * p=a.list;
      List::Node* q=b.list;
      List result;
      while (q!=NULL && p!=NULL) {
         if(q->value==p->value) {
           result.insert(q->value);
           q=q->next;
           p=p->next;
         }
        q=q->next;
           p=p->next;
     }
      return result;
  }
  void insert(int x) {
    Node* tmp=new Node;
    tmp->value=x;
   if(list==NULL || list->value >x){
     tmp->next=list;
     list=tmp;
   } 
   else {
    Node *curr=list;
     while (curr->next !=NULL && curr->next->value < x) {
          curr=curr->next;
     }
   tmp->next=curr->next;
   curr->next=tmp;
}

请注意，我的插入函数按升序添加节点，我已经尝试过了，它确实适用于测试用例，当我实现并集函数时，事情就会误入歧途。。。

我认为您修复了编译错误，现在您想知道为什么它不起作用。

您没有正确合并这两个列表。问题是：

  1. 只有在两个列表中都有一个项时，才能插入该项-这是交集，而不是并集。

  2. 你同时推进两个指针。

你需要合并其中一个：

while ( q && p ) {
    if( q->value < p->value ) {
        result.insert(q->value);
        q = q->next;
    } else if( q->value > p->value ) {
        result.insert(p->value);
        p = p->next;
    } else {
        result.insert(q->value);
        q = q->next;
        p = p->next;
    }
 }

现在，一旦这些列表指针中的任何一个到达末尾，您仍然需要添加另一个的剩余元素。这将发挥作用：

for(; q; q = q->next) result.insert(q->value);
for(; p; p = p->next) result.insert(p->value);

还要确保您已经为List定义了一个复制构造函数。看起来你在那里做你自己的内存分配，所以你必须遵循三条规则。

首先，我不确定您的代码要做什么。您似乎希望输入是两个列表，并行地逐步通过，如果相应的值匹配，则构建第三个列表。这更像是一个交叉点，而不是一个联盟，尽管它更严厉，因为它关心立场。

插入似乎是一个排序的插入。

无论如何，这部分肯定是错误的：

if（q->值==p->值）{result.insert（q->值）；q=q->下一个；p=p->下一个；}

移除这些q=q->next和p=p->next。这导致了一个双重步骤，因为它是在if语句内部和外部完成的。

双击是不好的，因为它会让你超过列表的末尾，导致崩溃。

相关文章：