如何在 C++ 中正确删除链表的节点

Question

我感觉好像我并没有真正删除节点并释放内存。我想我只是在移动指针，所以当我打印链接列表时，列表不会打印我删除的元素。所以我的问题是我实际上是在删除节点还是只是简单地重新排列指针，所以看起来我正在删除节点（基本上只是断开链接但不删除节点）？感谢您的帮助。

void SLL::deleteNode(int target){
Node *current = new Node;
Node *previous = new Node;

for (current = front->next, previous = front; current != NULL; current = current->next, previous=previous->next){
    if (previous->data == target && previous == front){
        front = previous->next;
        delete[] previous;
        return;
    //This if statement deletes the element if its the front
    }

    else {

        if (previous->data == target && previous->next == NULL){
            previous = NULL;
            delete[] current;
            return;
        //This if statement deletes the node if it is the back
        }


        else if (current->data==target)
        {
            previous->next = current->next;
            delete[] current;
            return;
        //This if statement deletes a node if it is in the middle
        }
    }
    }

    delete[] current;
    delete[] previous;
}

Answer 1

Node *current  = new Node;
Node *previous = new Node;

此代码会导致内存泄漏 - 您永远不会删除此内存。您可以在不分配内存的情况下声明指针：

Node *current  = nullptr;
Node *previous = nullptr;

delete 将删除指针的内存，因此您实际上将删除节点。 但是将delete[] 用于Node* 不正确，它应该只用于数组——用new[] 分配的内存。使用不当会导致未定义的行为。 因此，要正确删除节点，请使用运算符 delete 删除它们。

使用内存泄漏检测工具了解您的程序是否存在内存泄漏。

删除列表元素的代码：比如说，我们有指向列表头部的 pHead （但如果你自己写这样的东西，它会给你更多）：

Node* pCur  = pHead;
Node* pPrev = pCur;

while (pCur && pCur->data != target) {
    pPrev = pCur;
    pCur  = pCur->next;
}

if (pCur==nullptr)  // not found
   return NOT_FOUND;

if (pCur == pHead) {   // first element matches
    pHead = pCur->next;
} else {
    pPrev->next = pCur->next;
}

// pCur now is excluded from the list

delete pCur;     // deallocate its memory

---

使用指针到指针的替代方法（社区加法）

当您使用列表中的实际指针 来执行枚举时，上述内容可以呈现新的亮点。下面以pp 开始，分配头指针的地址（不是它指向的节点；实际的指针本身）。我们遍历列表，直到pp 保存指向要删除目标的节点的指针的地址（可以是头指针，也可以是某个节点中的next 指针，没有区别）。被寻址的指针被设置为它自己节点的next值，然后目标节点被删除。

这确实应该在调试器中观察以了解它是如何工作的，但是考虑到实际情况，算法非常简单：

Node **pp = &pHead;
while (*pp && (*pp)->data != target)
    pp = &(*pp)->next;

if (*pp)
{
    Node *victim = *pp;
    *pp = victim->next;
    delete victim;
}

这全部。而且您可以免费删除头节点，而无需对其进行特殊处理。希望这也有帮助。