删除具有重复数据的项目

Question

我正在编写一个函数来删除具有重复数据的连续项目。 例如 例如传入列表

->a->b->c->c->a->b->b->b->a->null

应该导致

->a->b->c->a->b->a->null

下面给出了列表项定义和函数声明

struct litem { 
char data; 
litem* next; 
}; 

Mo 代码如下所示

int remove_dumplicates(litem *&list)
{
 int count = 0; 
 struct litem * current = NULL;
 current = list;
 struct  litem *deleteNode;
 if (current == NULL ) return;
 while(current->next != NULL)
 {
  if ( current->data == current->next->data) // check for the duplicates 
   {
    count++;
    deleteNode =current->next;
    current>next= current->next->next;
    delete deleteNode;
   }      
  return (count);  
 }
}

这是实现以下功能的正确方法吗想要的结果？

Answer 1

我没有看到 current 增加到 current->next。

以包含所有唯一元素 a ->; 的列表为例。 b-> c 并查看您的程序如何工作。

要解决此问题，您需要：

while(current->next != NULL) {
   if ( current->data == current->next->data) {
     // delete duplicates .
   } else {
     current = current -> next;
   }
}// end-while
return (count);

Answer 2

您需要在 while 循环内添加 else 以前进到下一个节点：

if( current-> data == current->next->data ) {
....
} else {
    current = current->next;
}

还需要修复返回值（第一个应返回 0，第二个应移至 while 循环之外）。

Answer 3

一些快速观察：

return (count) 语句可能位于 while 循环之外，否则循环将提前终止。

while 循环内需要像 current = current->next; 这样的语句。否则，循环将变成无限循环。

Answer 4

1. 如果没有重复匹配，当前应该移动到当前->下一个。
2. 传递给函数的参数应该只是 *list （即指向 struct litem 类型元素的指针）
3. delete 是一个 C++ 关键字。请改用free()。

修改后的代码：

int remove_duplicates(struct litem *list)
{
    int count = 0; 
    struct litem * current = NULL;
    current = list;
    struct  litem *deleteNode;
    if (current == NULL ) return;
    while(current->next != NULL)
    {
        if ( current->data == current->next->data) 
        {
            count++;
            deleteNode = current->next;
                current->next= current->next->next;
                    free(deleteNode);
        }
        else
        {
            current = current->next;
        }
    }
    return (count);  
}

Answer 5

试试这个：

int remove_duplicates(litem *&list)
{
   int count = 0; 
   struct litem * current = NULL;
   current = list;
   struct  litem *deleteNode;
   if (current == NULL ) return 0;
   while(current->next != NULL)
   {
     if (current->data == current->next->data) // check for the duplicates 
     {
       count++;
       deleteNode =current->next;
       current->next= current->next->next;
       delete deleteNode;
     }
     else
     {
       current = current->next;
     }
   }
   return (count);
 }