c++ - 打印链表的问题

Question

我正在尝试创建自己的数据类型，类似于 vector 或数组。

我的打印功能有问题；当我去打印列表时，它只打印列表中的最后一项。

// LinkedListClass.cpp : Defines the entry point for the console application.

#include "stdafx.h"
#include <iostream>

class Node
{
public:
 int value;
 Node* next;

 Node::Node(int val)
 {
  value = val;
 };
};

class List
{
public:
 Node* firstNode;
 Node* currentNode;
 int size;

 List::List()
 {
  firstNode = NULL;
  currentNode = firstNode;
  size = 0;
 };

 void push(Node* node)
 {
  if(firstNode == NULL)
  {
   firstNode = node;
   firstNode->next = currentNode;
   size++;
  }
  else
  {
   currentNode = node;
   currentNode = currentNode->next;
   size++;
  }
 };

 void print()
 {
  if(firstNode != NULL)
  {
   Node* printNode = firstNode;
   while(printNode->next != NULL)
   {
    std::cout << "List Item " << printNode->value << std::endl;
    printNode = printNode->next;
   }
  }
 };
};

int _tmain(int argc, _TCHAR* argv[])
{
 List ll = List();
 for(int i = 0; i < 10; ++i)
 {
  Node val = Node(i);
  ll.push(&val);
 }
 std::cout << ll.firstNode->value << std::endl;
 ll.print();
 std::cout << "Size " << ll.size << std::endl;
 std::cin.ignore();
 return 0;
}

/* Output

9
Size 10

*/

我知道这还远未完成，但如果您有任何其他指示(笑)，请随时提出建议。

Answer 1

主要有以下三个错误:

push() --- 修复

void push(Node* node)
 {
  if(firstNode == NULL)
  {
   firstNode = node;
   currentNode = node;
   // firstNode->next = currentNode; --> this does nothing useful!
   size++;
  }
  else
  {
   currentNode->next = node;
   currentNode = node;
   //currentNode = node;               -|
   //currentNode = currentNode->next;  -|----> why? what? Do explain.
   size++;
  }
 }

我认为通过分配 firstNode->next = currentNode; 你期望下一次 currentNode 更新时，它会更新 firstNode->next 还有。

这样不行。

firstNode->next = currentNode; 表示存储在 currentNode 中的地址现在位于 firstNode->next 中。因此，下次您将某些内容存储在 currentNode = node; 中时，您不会将其存储在 firstNode->next 中。所以你有一个损坏的链表 --- 这就是你的输出没有走多远的原因。

此外，这真的很糟糕。通过设置 currentNode=node before 将当前节点的 next 指针设置为 node，你又打破了列表.您应该首先将 currentNode->next 指向 node 然后将 currentNode 设置为 node (node 是您要推送到列表中的节点)。

节点 val = 节点(i);

val 的范围仅在循环的迭代中。一旦你循环，它就会离开堆栈并且不再存在。但是您已经将 val 的指针复制到您的列表中 --- 所以现在使用正确的 push 方法，您只是添加了一个悬挂指针。

Node *val = new Node(i);
ll.push(val);

你需要把它放在堆上，这样它就会一直存在，直到你不再需要它为止。

...这将我们引向您的析构函数!

由于您已经分配了一个节点，因此您需要取消分配它。所以在你的析构函数中这样做——遍历你的列表并释放所有这些节点。