删除双向链表中的节点 (C++)

Question

我无法理解为什么当我创建两个或更多节点（如下所示）时，函数void del_end()只会删除char name[20]而不是整个节点。如何在不发生内存泄漏的情况下解决此问题？

#include <iostream>
using namespace std;

struct node
{
    char name[20];
    char profession[20];
    int age;
    node *nxt;
    node *prv;
};

node *start_ptr = NULL;

void del_end()
{
    node *temp, *temp2;
    temp = start_ptr;
    if (start_ptr == NULL)
        cout << "Can't delete: there are no nodes" << endl;
    else if (start_ptr != NULL && start_ptr->nxt == NULL)
    {start_ptr=NULL;}
    else
    {
    while (temp->nxt != NULL)
    {
        temp = temp->nxt;
    }
    temp2=temp->prv;
    delete temp;
    temp->nxt= NULL;
    }
}

Answer 1

您的代码有一些问题，最糟糕的是：

temp2=temp->prv;
delete temp2;
temp->nxt= NULL;

您正在删除倒数第二个节点，使指向它的任何指针悬空，并丢失最后一个节点。

但如果你发布更多真实代码，我们可以告诉你更多。

编辑：
这是del_end 的稍微清理过的版本（还有很大的改进空间）。

void del_end()
{
  if (start_ptr == NULL)
  {
    cout << "Can't delete: there are no nodes" << endl;
    return;
  }

  if (start_ptr->nxt == NULL)
  {
    delete start_ptr;
    start_ptr = NULL;
    return;
  }

  node *nextToLast = start_ptr;
  node *last = start_ptr->nxt;

  while(last->nxt != NULL)
  {
    nextToLast = last;
    last = last->nxt;
  }

  delete last;
  nextToLast->nxt = NULL;

  return;
}

请注意，这不假定prev 链接是正确的，这在这里看起来很谨慎。

Answer 2

delete temp2;

将删除整个节点。

Answer 3

问题是您似乎试图删除最后一个节点，但实际上您正在删除它之前的那个。

while (temp->nxt != NULL) 
{ 
    temp = temp->nxt; 
} 
temp2=temp->prv; 
delete temp2; 
temp->nxt= NULL; 

这是你的问题。改成这样：

while (temp->nxt != NULL) 
{ 
    temp = temp->nxt; 
} 
temp2=temp->prv; 
delete temp; 
temp2->nxt= NULL;

而且我相信它会按预期工作。这将保存倒数第二个节点，删除末尾，然后将倒数第二个节点的 nxt 指针设置为空，使其成为最后一个。

Answer 4

如果您担心内存，我强烈建议您学习使用 Valgrind http://valgrind.org/。这是一个很棒的工具。 Valgrind 将内存泄漏分为 3 类：

• “肯定丢失” - 指向动态分配内存的指针丢失，无法恢复
• “可能丢失” - 指向动态分配内存的指针指向块的内部，可能不相关
• “仍然可达” - 指向动态分配内存的指针仍然存在，但在程序执行结束时从未释放内存

运行 Valgrind 也很简单。这是用户手册http://valgrind.org/docs/manual/manual.html 的链接。运行 valgrind 时的一些有用标志：

• --leak-check=<no|summary|yes|full>
• --show-reachable=<no|yes>

现在，我要删除双向链表中的节点的方法是：

// if the node to be removed is the head node
if (nodeToRemove->prev == NULL) {
    // reassign the head node pointer
    start_ptr = nodeToRemove->next;
} else {
    // correct previous node pointer
    nodeToRemove->prev->next = nodeToRemove->next;
}

// if the node to be removed node is the tail node
if (nodeToRemove->next == NULL) {
    // reassign the tail node pointer
    end_ptr = nodeToRemove->prev;
} else {
    // correct next node pointer
    nodeToRemove->next->prev = nodeToRemove->prev;
}

// deallocate memory
delete(nodeToRemove);
nodeToRemove = NULL;

只需在声明 start_ptr 之后声明 node *end_ptr = NULL; 并在将节点附加到列表时，请确保 end_ptr 始终指向列表的末尾...如果您要添加到列表的末尾列表，很简单……只需将 end_ptr 指向要添加的节点。

如果你总是要删除最后一个节点，你最好保留一个尾指针。所以一旦你有了要删除的节点，我只需检查它是否是头/尾节点，重新分配下一个/上一个指针，然后释放内存。

顺便说一句...我从我的 C 实现中获取了这个，所以如果语法不正确我很抱歉...但是逻辑就在那里。

希望对您有所帮助。