删除循环链表中的节点

时间:2013-04-24 06:29:09

标签: c++ linked-list circular-list

我有一个基本的链表问题,我试图在下面解决。我会很感激我的方法,算法的正确性(甚至编码风格)的任何输入。该问题需要一个函数来删除循环链表中所有出现的int并返回列表中的任何节点或NULL(当列表为空时)。

这是我到目前为止的一些C ++代码:

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

Node* deleteNode(Node* &node, int num){

    if(!node){
        return NULL;
    }

    Node* given = node;
    Node* del;

    while(node->next != given){
        if(node->next->data == num){
            del = node->next;
            node->next = node->next->next;
            delete del;
        }
        node = node->next;
    }

    //Check if the first node needs to be deleted, with variable node pointing to last element
    if(given->data == num){
        node->next = given->next;
        delete given;
    }

    return node;
}

2 个答案:

答案 0 :(得分:1)

delete node;应为delete del;

此外,使用Node* node作为参数,而不是Node* &node,这将阻止非左值传入。

P.S。在结构定义后忘记分号? :)

答案 1 :(得分:1)

如果不遵循您的所有逻辑,我可以一目了然地看到这些代码无效。

您正在检查输入列表是否为空,这是您的代码返回NULL的唯一情况。但是如果传递一个必须删除所有元素的列表会发生什么呢?

这个问题也有一个微妙之处。要检查是否已完成循环列表,您需要与第一个地址进行比较,以查看是否已链接回到开头。但是,如果此元素已被删除,那么通过C ++标准,您甚至不允许在比较中使用其地址

为了避免对要删除的元素进行两次传递,一个可能的技巧是在开始迭代时“中断循环”,这样您就可以检查NULL而不是检查起始节点的地址。