0

我试图实现单链表。我的 addAtLast() 函数没有正确执行。执行此函数时程序崩溃。请提出一些更改。

class LList
{
public:
    int noOfNodes;
    Node const *start;/*Header Node*/

    LList()
    {
        start=new Node;
        noOfNodes=0;start=0;
    }

    void addAtFront(Node* n)
    {
        /*
        cout<<endl<<"class"<<n;
        cout<<"start"<<start;
        cout<<"data in node";n->print();
        */
        n->next=const_cast<Node*>(start);
        start=n;
        // cout<<" next=";start->print();
        noOfNodes++;
    }

    void addAtLast(Node* n)
    {
        Node *cur=const_cast<Node*>(start);
        if (start==NULL)
        { 
            start=n;
            return;
        }
        while(cur->next!=NULL)
        {
            cur=cur->next;
        }
        cur->next=n;
        noOfNodes++;
    }

    int getPosition(Node data)
    {
        int pos=0;
        Node *cur=const_cast<Node*>(start);
        while(cur!=NULL)
        {
            pos++;
            if(*cur==data)
            {
                return pos;
            }
            cur=cur->next;
        }
        return -1;//not found
    }

    Node getNode(int pos)
    {
        if(pos<1)
            return -1;// not a valid position
        else if(pos>noOfNodes)
            return -1; // not a valid position

        Node *cur=const_cast<Node*>(start);
        int curPos=0;
        while(cur!=NULL)
        {
            if(++curPos==pos)
                return *cur;
            cur=cur->next;
        }
    }

    void traverse()
    {
        Node *cur=const_cast<Node*>(start);
        while(cur!=NULL)
        {
            //   cout<<"start"<<start;        
            cur->print();
            cur=cur->next;
        }
    }  

    ~LList()
    {
        delete start;
    }
};
4

4 回答 4

2 
void addAtLast(Node* n) {
    Node *cur=const_cast<Node*>(start);
    if(start==NULL) {
        start=n;
        n->next = NULL;
        noOfNodes++;
        return;
    }
    while(cur->next!=NULL) {
        cur=cur->next;
    }
    cur->next=n;
    n->next = NULL;  // Added
    noOfNodes++;
}
于 2012-11-12T08:32:09.387 回答
1

从一开始就..

start=new Node;
noOfNodes=0;start=0;

应该这样吗？

start=new Node;
noOfNodes=0;start->next=NULL;

在 2 行内，您已经创建了内存泄漏。我不知道你为什么要设置start=0. 不要那样做，你只是给它分配了内存！

至于崩溃，@liulinhuai 的回答已解决。您将取消引用未初始化的指针以尝试获取它的next成员。

于 2012-11-12T08:28:58.940 回答
1

在您的 ctor 中，您将 start 设置为 0。在您的崩溃函数中，您首先检查它NULL：

if (start==NULL)
{ 
   start=n;
   //n->next is now points to nowhere
   return;
}

下一次调用addAtLast迭代，直到找到NULL但先前的分配没有设置next指针，NULL所以第二次迭代将导致access violation：

while(cur->next!=NULL) {
   //first time it's ok but second call on cur will crash
   cur=cur->next;
}

解决方案是 - 所有 newNode必须将next指针设置为NULL

于 2012-11-12T08:36:12.057 回答
1

我在评论中提到了这个，但将在这里作为答案解决。此函数的调用者必须确保两件事：

  1. 传入的节点列表（它是一个列表，即使只有一个元素长）必须以设置为 NULL 的 end-next-pointer 正确终止。调用者必须确保这一点，因为此代码不能假定它并盲目设置node->next = NULL;

  2. 确保调用者知道一旦它执行，这个列表现在拥有传入的节点和它可能启动的任何列表，因此调用者不能在调用者端释放它或它指向的任何东西。

除了节点计数管理问题之外，该addAtLast()功能没有任何问题，尽管我会以不同的方式实现它：

void addAtLast(Node* n)
{
    // no nulls allowed
    if (n == NULL)
        return;

    // set start if first in list
    if (start == NULL)
    {
        start = n;
        noOfNodes = 1;
    }

    // else walk list to find end
    else
    {
        Node *cur = const_cast<Node*>(start);
        while(cur->next != NULL)
            cur = cur->next;
        cur->next = n;
        ++noOfNodes;
    }

    // adjust count to contain any nodes from 'n'
    while (n->next != NULL)
    {
        ++noOfnodes;
        n = n->next;
    }
}
于 2012-11-12T08:54:50.840 回答