0

当我尝试运行我的程序时，它一开始就崩溃了。问题是我从文件中输入，我可以很好地写入文件。有人可以解释为什么这段代码不起作用吗？

StringList::StringList()
{
  pTop=NULL;
  pBottom=NULL;

  ifstream in;
  in.open("read.txt");

  StringListNode * pCurrent;
  pCurrent = new StringListNode;
  pCurrent = pTop;

  while(!in.eof())  //reads it till the end of file
  {
    in >> pCurrent->data;
    pCurrent = pCurrent->pNext;
  }
  in.close();
}

此文件的输出工作正常。我以为我会把它包括在内。

StringList::~StringList()
{
  ofstream out;
  out.open("read.txt");

  StringListNode * pCurrent;
  pCurrent = new StringListNode;
  pCurrent = pTop;
  while(pCurrent != 0)  
  {
    out << pCurrent->data << endl;
    pCurrent = pCurrent->pNext;
  }
  out.close();
 }
4

1 回答 1

1

pCurrent = pTop;你为什么在这里分配这个？这使得pCurrent空指针。请删除或修复。

我对你的代码感到困惑：

pCurrent = new StringListNode; // Allocate new memory and point at it
pCurrent = pTop; // Make pCurrent point at same thing as pTop

你分配给pCurrent两次。pTop看起来像一个数据成员，也许你的意思是在构造函数中：

pCurrent = new StringListNode; // Allocate new memory and point at it
pCurrent->pNext = nullptr; // Assign null to next pointer
pTop = pCurrent; // Make pTop point at new memory

并在析构函数中删除pCurrent = new StringListNode;，因为它不做任何事情。

输出时检查pCurrent != 0，但读取时不检查 null 。可能pCurrent是空指针。

另外，请阅读为什么循环条件内的 iostream::eof 被认为是错误的？. 你的循环应该是：

while(pCurrent && (in >> pCurrent->data)) 
{
   pCurrent = pCurrent->pNext;
}
于 2013-07-17T03:24:59.823 回答