从循环双向链接列表中删除节点

时间:2016-03-31 04:50:53

标签: c++ linked-list segmentation-fault doubly-linked-list circular-list

我已编写代码将元素插入循环双向链表并显示这些元素。我也应该能够从列表中删除尾节点,以及在列表中搜索特定元素。

这是我添加和打印的工作代码:

void Circular_DLList::add_to_tail(int a)
{
    DLLNode *temp = new DLLNode;
    temp->info = a;
    if (is_empty()) {   
        tail = temp;
        temp->next = tail;
        temp->prev = tail;
    }
    else {
        temp->next = tail->next;
        temp->prev = tail;
        tail = temp;
        tail->prev->next = temp;
    }
}

void Circular_DLList::print_list()
{
    DLLNode *ptr;
    ptr = tail->next;
    do {
        cout<< ptr->info << endl;
        ptr = ptr->next;
    }
    while(ptr != tail->next);
}

无论我为delete_from_tail函数编写什么,它都会导致分段错误:11。这是我对该函数的尝试(抛出错误)。

int Circular_DLList::delete_from_tail()
{
    int a = tail->info;
    if(tail == tail->next) {
        delete tail;
        tail = NULL;
    }
    else {
        tail = tail->prev;
        delete tail->next;
        tail->next = NULL;
    }
    return a;
}

关于如何解决这个问题的任何建议都会很棒。我已经尝试过调试,但我似乎无法弄清楚问题或者它究竟与哪个问题有关。 感谢

1 个答案:

答案 0 :(得分:0)

如果仔细观察,问题就很明显了。您的删除功能正在打破链接列表的循环链。怎么会这样?请参阅下面的删除功能:

int Circular_DLList::delete_from_tail()
{
    int a = tail->info;
    DLLNode *temp;

    if(tail == tail->next) {
        delete tail;
        tail = NULL;
    }
    else {
        tail = tail->prev;
        delete tail->next;
        tail->next = NULL;
    }
    return a;
}

else-condition你正在设置tail->next = NULL这实际上是错误,因此打破了链条。因此,当调用print时,它假定循环链是完整的,因此意外地尝试访问NULL指针,这反过来导致分段错误。

修复非常简单,请参阅以下代码:

int Circular_DLList::delete_from_tail()
{
    int a = tail->info;
    if(tail == tail->next) {
        delete tail;
        tail = NULL;
    }
    else {
        temp = tail;
        tail = tail->prev;
        tail->next = temp->next;        // To maintain the circular chain
        tail->next->previous = tail;    // Since this element's previous also point to the node about to be deleted
        delete temp;
        temp = NULL;
    }
    return a;
}