1

我不太精通C++，但我的任务是设计和实现一个链表。这是我交的：

template <typename T>
struct Node
{
    Node() : next(nullptr) {}
    Node(T const & val) : next(nullptr), value(val) {}

    Node * next;
    T value;
};

template <typename T>
class LinkedList
{
    public:
        LinkedList()
        {
            begin_ = new Node<T>;
            current_ = begin_;
        }
        ~LinkedList()
        {
            delete begin_;
        }

        void insert(T const & value)
        {
            Node<T> * node = new Node<T>(value);
            node->next = current_->next;
            current_->next = node;
            current_ = current_->next;
        }

        void remove()
        {
            Node<T> * tmp = current_->next;
            if(!end())
            {
                current_->next = current_->next->next;
            }
            delete tmp;
        }

        bool end() const
        {
            return current_->next == nullptr;
        }

        void reset()
        {
            current_ = begin_;
        }

        void advance()
        {
            if(!end())
            {
                current_ = current_->next;
            }
        }

        T get() const
        {
            return current_->next->value;
        }

    private:
        Node<T> * begin_;
        Node<T> * current_;
};

我通过了作业，但我的老师delete begin_在析构函数中划了下划线~LinkedList()并写了“这泄漏了！” 在它的旁边。我一直在想怎么delete begin_可能泄漏，但我还是不明白。有人可以帮我吗？

4

3 回答 3

2

您必须遍历列表中的每个节点并删除。持有一个单独的指向下一个指针，删除当前，然后向前移动并继续删除，直到下一个指针为空。

~LinkedList()
{
    Node * current = begin_;
    Node * aNext = begin_->next;

    while (null != aNext){
        delete(current);
        current = aNext;
        aNext = current->next;
    }
}

像这样的东西。不知道你是从哪里开始的，但是......

于 2012-06-17T19:07:06.953 回答
0

在 Node 的析构函数中，它应该发送删除命令以next释放列表中的所有节点。

于 2012-06-17T19:07:31.650 回答
0

我想你会这样更好地理解它：

~LinkedList()
{
  reset();
  while(!end()) remove();
}
于 2012-06-17T19:13:29.103 回答