为什么链表中的节点不能负责删除自己?(C++)

Why can't a node in a linked list be responsible for deleting itself? (C++)

本文关键字:自己 C++ 删除 能负责 链表 节点 不能 为什么      更新时间:2023-10-16

我的想法是,一旦第一个节点从链表类中删除,其余的节点就会随之而来。我的实现在实践中不起作用。是否有解决方案可以逐个节点删除整个列表,而不是链表删除所有列表?我的nodeT应该有一个析构函数吗?

链表实现:

#include <iostream>
#include <cassert>
#include "nodeT.h"
class linkedListSort
{
public:
    int print();
      //Function to output the elements of the list
      //Postcondition: Elements of the list are output on the 
      //   standard output device. Member current is reset to beginning node
      //   and currentIndex is reset to 0
      //Returns:  number of items printed in the list
      //          also outputs error if number of items printed
      //          does not equal the length of the list
    void insertAt(int location, elemType& insertItem);
      //Function to insert an item in the list at the 
      //position specified by location. The item to be inserted 
      //is passed as a parameter to the function.
      //Postcondition: Starting at location, the elements of the
      //    list are shifted down, list[location] = insertItem;,
      //    and length++;. If the list is full or location is
      //    out of range, an appropriate message is displayed.
      linkedListSort(int size = 100);
      ~linkedListSort();
protected:
  //consider making const
  nodeT<elemType> *beginningNode;  // handle to the beginning of the list
  nodeT<elemType> *current;  // pointer to current node
  int currentIndex;  //int representing which node in the list current is pointing to
  int length;      //to store the length of the list
  int maxSize;     //to store the maximum size of the list
  };
template <class elemType>
linkedListSort<elemType>::linkedListSort(int size)
{
  if (size < 0)
  {
    cerr << "The array size must be positive. Creating "
      << "an array of size 100. " << endl;
    maxSize = 100;
  }
  else
    maxSize = size;
  beginningNode = NULL;
  current = NULL; // initialize to empty linked list
  length = 0;
  currentIndex = -1; // there is no item that current points to
}
template <class elemType>
linkedListSort<elemType>::~linkedListSort()
{
  delete beginningNode;  // this should delete all linked list items ( see nodeT destructor )
}

链表节点实现:

template <class elemType>
class nodeT {
public:
  nodeT(elemType& infoParam, nodeT<elemType> *linkParam);  //standard
  nodeT(elemType& infoParam);  //if unlinked node (ex. last item)
  nodeT();
  //copy constructor
  nodeT(nodeT<elemType>& node);
  ~nodeT();
  elemType *info;
  nodeT *link;
};
template<class elemType>
nodeT<elemType>::nodeT(elemType& infoParam, nodeT<elemType> *linkParam) {
  info = &infoParam;
  link = linkParam;
}
//when link is null (last item and uncircular)
template<class elemType>
nodeT<elemType>::nodeT(elemType& infoParam) {
  info = &infoParam;
  link = NULL;
}
//in case node is needed before info or link is known (default)
template<class elemType>
nodeT<elemType>::nodeT() {
  info = NULL;
  link = NULL;
}
template<class elemType>
nodeT<elemType>::nodeT(nodeT<elemType>& node) {
  info = new elemType();
  if (node.link != NULL)
    link = new nodeT();
  *info = *(node.info);  // copy by value
  if (node.link != NULL)
    *link = *(node.link);
  else
    link = NULL;
}
template<class elemType>
nodeT<elemType>::~nodeT() {
  delete info;
  if (link != NULL)
    delete link;
}
节点

实现的最后一部分是节点析构函数。如果链接nodeT的成员类型为 nodeT则代码delete link将调用相同的析构函数,但仅在另一个实例上调用。因此,一旦第一个节点被销毁,每个节点都应该自行销毁。第一个节点在链表实现中被销毁如下:delete beginningNode beginningNode始终指向链表中的第一个节点。

我接近解决方案了吗?还是我只是掉进了一个C++不想让你掉下去的兔子洞?

实际错误与断言失败有关。然后最终我可以将其复制到我的剪贴板:"第 10Ex1 章中的 0x553056E8 (msvcr120d.dll 处未处理的异常.exe:0xC0000005:访问冲突读取位置0x00000002。

从技术上讲,您的节点不负责删除自己,它们负责删除列表中的下一个节点。

这可能看起来很有吸引力,但这里有一些你可能没有考虑过的含义。首先,@WhozCraig评论中所说的,您最终将为一个大列表构建一个相当深入的析构函数调用堆栈。

其次,如果你设法为自己构建了一个循环链接链,你将一直绕过它,然后在第二次尝试删除第一个节点时遇到未定义的行为。代码中没有任何内容可以防止这种误用 - 这些类型的保证是将容器类用于列表的优点之一,该列表对客户端隐藏了节点本身的操作。

我认为这里还有一个关于所有权的问题。节点不会相互分配,但它们负责相互删除,这意味着每个节点都拥有列表中的下一个节点。从您提供的 API 中可能并不明显,这需要列表的用户创建新节点,但是当您将它们添加到列表中时,列表将负责删除它们。这意味着在客户端代码中,newdelete 之间没有平衡,这看起来有点奇怪。

然而,在所有权方面更糟糕的是列表节点析构函数调用delete info,它是在其中一个构造函数中创建的,作为传入的引用的地址,该引用甚至可能不在堆上。你说不出来,没有人在那里做出任何承诺。至少你需要接受指针而不是引用,因为这是所有权转移正在发生的暗示。更好的是接受一个 std::unique_ptr<elemType> ,这使得所有权的转移非常明确(您仍然可以通过原始指针提供对内容的访问)。

一般来说,我会建议,如果一个数据结构要负责删除某些东西,那么数据结构也应该负责创建它。否则,您应该不要管它。STL 容器不会删除包含的指针 - 如果删除std::vector<int *>,则必须先自己删除所有int *成员。这为用户提供了灵活性 - 他们不必存储指向堆上的东西的指针 - 这意味着它是一致的 - 负责创建某些东西的人通常也应该负责处理它。这也意味着std::list<T>可以包含任何T - 包括指针。如果您尝试实例化nodeT<int *>会发生什么?当其析构函数运行时会发生什么?

所以我想说,如果你要让节点互相删除,你也应该让节点互相创建(不要让用户这样做)。如果你要让节点删除它们的数据项,你肯定也应该创建这些数据项。或者更好的是,不要管它,不要触及通过引用传递给您的内容的生命周期