问一个有关单链表的题，只改动了一句话，结果就报错，不明白为什么，谢谢！！！

2014-06-05 01:48:15 223 阅读 AI 写同款 · GEO 优化AI 选题数据监测一键分发 ›

数据结构实验2：C++实现单链表类太简单了，没啥可说的，代码意义明白如话。题目与要求：实验2 2.1 实验目的熟练掌握线性表的链式存储结构。熟练掌握单链表的有关算法设计。根据具体问题的需要，设计出... 阅读详情

class Solution {  

public:  

    ListNode *deleteDuplicates(ListNode *head) {  

        // Start typing your C/C++ solution below  

        // DO NOT write int main() function  

        if (head == NULL)  

            return head;  

          

        ListNode* cur = head;  

        ListNode* pre = head->next;  

        ListNode* del = NULL;  

        while(pre != NULL)  

        {  

            if (cur->val == pre->val)  

            {  

                del = pre;  

                pre = pre->next;    //  把这一行和下一行合并成  cur->next  = pre->next  为什么就不对呢？？

                cur->next = pre;    //

                delete del;  

            }  

            else  

            {  

                cur = cur->next;  

                pre = pre->next;  

            }  

        }  

        return head;  

    }  

};

把这一行和下一行合并成（注释处） cur->next = pre->next 为什么就不对呢？？谢谢

...全文

数据结构C++单链表的头插法和尾插法这个代码的存在的意义就是定义一个整形变量，有人会问整形变量不是int吗，你怎么用的Element，这里我在说明一下，这里也是考研的重点其中typedef int Element是为了在以后需要更改数据类型的时候方便更改，相当于就是将int 重命名为了Element.（请考研的各位同学务必记住）//这个指针的作用是指向申请的新的节点，一定要对不画图的读者说明，申请的这个指针是为了指向新的节点。这个代码的存在的意义就是头指针指向头结。这个代码的存在的意义就是头结点的下一个指针指向空NULL的意思就是空。阅读详情

klmn111aaa

C++ 语言 65,214 成员 250,514 内容

+ 关注 + 加入社区

切换为时间正序

ri_aje 2014-06-05

打赏
举报

pre = pre->next; cur->next = pre; 的写法移动了 pre 指针，而 cur->next = pre->next 没有修改 pre，delele del 以后 pre 就指向非法的地方了。非要一句话的话，可以试试这样：pre = cur->next = pre->next; ListNode *deleteDuplicates(ListNode *head) 的意思是说吃一个链表 (head 指向的)，把其中的重复值删干净，然后再把处理过的链表头节点指针返回。另外，程序里的 pre 明明指向 cur->next，这样命名非常混淆。

klmn111aaa 2014-06-05

打赏
举报

class Solution {
public:
    ListNode *deleteDuplicates(ListNode *head) {
        // Start typing your C/C++ solution below
        // DO NOT write int main() function
        if(!head || !head->next)
            return head;
        ListNode *pre;
        ListNode *newhead;
        ListNode *tmp;
        ListNode *temp;
        pre = NULL;
        tmp = head;
        while(tmp && (tmp && pre && tmp->val == pre->val) || (tmp && tmp->next && tmp->val == tmp->next->val))
        {
            pre = tmp;
            tmp = tmp->next;
        }
        newhead = tmp;
        if(!newhead || !newhead->next)
            return newhead;
        tmp = newhead;
        while(tmp)
        {
            pre = tmp;
            temp = tmp->next;
            while(temp && (temp && pre && temp->val == pre->val) || (temp && temp->next && temp->val == temp->next->val))
            {
                pre = temp;
                temp = temp->next;
            }
            tmp->next = temp;
            tmp = tmp->next;
        }
        return newhead;
    }
};

肖邦之离歌 2014-06-05

打赏
举报

其实是有区别的！！！首先你合并之后就少了一个赋值，pre就没有指向next节点；第二，合并之后delete del(其实还是pre),就会造成pre指向的内存被释放，这也就是所谓的野指针。这样的操作会导致未定义行为，所以你的程序出错一点都不奇怪。

引用楼主 klmn111aaa 的回复:

class Solution {  
public:  
    ListNode *deleteDuplicates(ListNode *head) {  
        // Start typing your C/C++ solution below  
        // DO NOT write int main() function  
        if (head == NULL)  
            return head;  
          
        ListNode* cur = head;  
        ListNode* pre = head->next;  
        ListNode* del = NULL;  
        while(pre != NULL)  
        {  
            if (cur->val == pre->val)  
            {  
                del = pre;  
                pre = pre->next;    //  这句是有作用的，就是把pre指向next节点
                cur->next = pre;    //
                delete del;  // 如果合并之后，这一句就是delete pre;另外建议你释放内存的时候这么写之后再加一句del = NULL;这样可以防止野指针的出现。
            }  
            else  
            {  
                cur = cur->next;  
                pre = pre->next;  
            }  
        }  
        return head;  
    }  
};

把这一行和下一行合并成（注释处） cur->next = pre->next 为什么就不对呢？？谢谢

还差得远呢 2014-06-05

打赏
举报

引用 2 楼 ri_aje 的回复:

pre = pre->next; cur->next = pre; 的写法移动了 pre 指针，而 cur->next = pre->next 没有修改 pre，delele del 以后 pre 就指向非法的地方了。非要一句话的话，可以试试这样：pre = cur->next = pre->next; ListNode *deleteDuplicates(ListNode *head) 的意思是说吃一个链表 (head 指向的)，把其中的重复值删干净，然后再把处理过的链表头节点指针返回。另外，程序里的 pre 明明指向 cur->next，这样命名非常混淆。

++正解，一般pre定义为前一个，next定义位下一个，所以楼主的定义很容易混淆，给后来看代码的人造成困扰。

赵4老师 2014-06-05

打赏
举报

仅供参考：

//假设带表头结点的单向链表头指针为head，试编写一个算法将值为5的结点插入到连接表的第k个结点前，删除第k个节点，并对该链表进行排序。
#include <stdio.h>
#include <stdlib.h>
#include <malloc.h>
#include <time.h>
struct NODE {
    int          data;
    struct NODE *next;
} H,*head,*p,*q,*s1,*s2,*s3,*s4,*s;
int i,j,k,n,t,m;
int main() {
    srand(time(NULL));

    //填写头节点数据
    H.data=-1;
    H.next=NULL;
    head=&H;

    //创建10个节点的单链表
    p=head;
    for (i=0;i<10;i++) {
        q=(struct NODE *)malloc(sizeof(struct NODE));
        if (NULL==q) return 1;
        q->data=rand()%100;//填写0..99的随机值
        q->next=NULL;
        p->next=q;
        p=q;
    }

    //输出整个单链表
    s=head->next;
    while (1) {
        if (NULL==s) {
            printf("\n");
            break;
        }
        printf("%02d->",s->data);
        s=s->next;
    }

    //将值为5的结点插入到单链表的第k个结点前
    k=3;
    n=0;
    p=head;
    while (1) {
        if (NULL==p) {
            break;
        }
        n++;
        if (k==n) {
            q=(struct NODE *)malloc(sizeof(struct NODE));
            if (NULL==q) return 1;
            q->data=5;
            q->next=p->next;
            p->next=q;
            break;
        }
        p=p->next;
    }

    //输出整个单链表
    s=head->next;
    while (1) {
        if (NULL==s) {
            printf("\n");
            break;
        }
        printf("%02d->",s->data);
        s=s->next;
    }

    //删除第k个节点
    k=5;
    n=0;
    p=head;
    while (1) {
        if (NULL==p) {
            break;
        }
        n++;
        if (k==n) {
            q=p->next;
            if (q) {
                p->next=q->next;
                free(q);
            }
            break;
        }
        p=p->next;
    }

    //输出整个单链表
    s=head->next;
    while (1) {
        if (NULL==s) {
            printf("\n");
            break;
        }
        printf("%02d->",s->data);
        s=s->next;
    }

    //从小到大排序
    for (p=head;p!=NULL && p->next!=NULL;p=p->next) {
        for (q=p->next;q!=NULL && q->next!=NULL;q=q->next) {
            if (p->next->data > q->next->data) {

                //交换data
//              printf("swap %02d %02d\n",p->next->data,q->next->data);
//              t=p->next->data;p->next->data=q->next->data;q->next->data=t;

                //或者

                //交换next
//              printf("swap %02d %02d\n",p->next->data,q->next->data);
                s1=p->next;
                s2=p->next->next;
                s3=q->next;
                s4=q->next->next;

                if (s2!=s3) {
                     p->next=s3;
                    s3->next=s2;
                     q->next=s1;
                    s1->next=s4;
                } else {
                     p->next=s3;
                    s3->next=s1;
                           q=s3;
                    s1->next=s4;
                }

                //输出整个单链表
//              s=head->next;
//              while (1) {
//                  if (NULL==s) {
//                      printf("\n");
//                      break;
//                  }
//                  printf("%02d->",s->data);
//                  s=s->next;
//              }
//              getchar();
            }
        }
    }

    //输出整个单链表
    s=head->next;
    while (1) {
        if (NULL==s) {
            printf("\n");
            break;
        }
        printf("%02d->",s->data);
        s=s->next;
    }

    //将单链表中前 m 个结点和后 n 个结点进行互换,m+n为链表总长10
    m=4;
    n=6;
    k=0;
    p=head;
    while (1) {
        if (NULL==p) {
            break;
        }
        k++;
        if (m+1==k) {
            q=p;
        }
        s=p;
        p=p->next;
    }
    s1=head->next;
    head->next=q->next;
    s->next=s1;
    q->next=NULL;

    //输出整个单链表
    s=head->next;
    while (1) {
        if (NULL==s) {
            printf("\n");
            break;
        }
        printf("%02d->",s->data);
        s=s->next;
    }

    //释放所有节点
    p=head->next;
    while (1) {
        if (NULL==p) {
            break;
        }
        q=p->next;
        free(p);
        p=q;
    }

    return 0;
}
//18->94->58->17->27->20->43->57->75->78->
//18->94->05->58->17->27->20->43->57->75->78->
//18->94->05->58->27->20->43->57->75->78->
//05->18->20->27->43->57->58->75->78->94->
//43->57->58->75->78->94->05->18->20->27->
//

klmn111aaa 2014-06-05

打赏
举报

引用 5 楼 klmn111aaa 的回复:

[quote=引用 4 楼 ri_aje 的回复:] [quote=引用 3 楼 klmn111aaa 的回复:] [quote=引用 2 楼 ri_aje 的回复:] pre = pre->next; cur->next = pre; 的写法移动了 pre 指针，而 cur->next = pre->next 没有修改 pre，delele del 以后 pre 就指向非法的地方了。非要一句话的话，可以试试这样：pre = cur->next = pre->next; ListNode *deleteDuplicates(ListNode *head) 的意思是说吃一个链表 (head 指向的)，把其中的重复值删干净，然后再把处理过的链表头节点指针返回。另外，程序里的 pre 明明指向 cur->next，这样命名非常混淆。

谢谢对对，我也觉得pre应该叫next才合适。 pre = cur->next = pre->next; 试了一下不行。好像明白了，(ListNode *head) 就是“头节点指针返回”（实际头节点并没有变化），然后ListNode *deleteDuplicates是表示函数吧，为何不能写成ListNode deleteDuplicates，是因为返回值有指针，就必须这么写么？ [/quote] 我也试了一下，pre = cur->next = pre->next; 是可以的，为啥你说不行？还是我对题意理解错了？我把 1 1 1 2 3 的链表变成 1 2 3，是不是要求这样？[/quote] 不好意思啊，是可以的，我也不知道刚才可能哪里写错了，非常感谢！[/quote] 要求是那样的，麻烦看下我这个理解对么？“(ListNode *head) 就是“头节点指针返回”（实际头节点并没有变化），然后ListNode *deleteDuplicates是表示函数吧，为何不能写成ListNode deleteDuplicates，是因为返回值有指针，就必须这么写么？”谢谢转专业来的，没系统学过，都是自己东拼西凑，感觉学的比较模糊。。。

klmn111aaa 2014-06-05

打赏
举报

引用 4 楼 ri_aje 的回复:

[quote=引用 3 楼 klmn111aaa 的回复:] [quote=引用 2 楼 ri_aje 的回复:] pre = pre->next; cur->next = pre; 的写法移动了 pre 指针，而 cur->next = pre->next 没有修改 pre，delele del 以后 pre 就指向非法的地方了。非要一句话的话，可以试试这样：pre = cur->next = pre->next; ListNode *deleteDuplicates(ListNode *head) 的意思是说吃一个链表 (head 指向的)，把其中的重复值删干净，然后再把处理过的链表头节点指针返回。另外，程序里的 pre 明明指向 cur->next，这样命名非常混淆。

ri_aje 2014-06-05

打赏
举报

引用 3 楼 klmn111aaa 的回复:

[quote=引用 2 楼 ri_aje 的回复:] pre = pre->next; cur->next = pre; 的写法移动了 pre 指针，而 cur->next = pre->next 没有修改 pre，delele del 以后 pre 就指向非法的地方了。非要一句话的话，可以试试这样：pre = cur->next = pre->next; ListNode *deleteDuplicates(ListNode *head) 的意思是说吃一个链表 (head 指向的)，把其中的重复值删干净，然后再把处理过的链表头节点指针返回。另外，程序里的 pre 明明指向 cur->next，这样命名非常混淆。

klmn111aaa 2014-06-05

打赏
举报

另外麻烦最好可以解释下 ListNode *deleteDuplicates(ListNode *head) 这句的具体意思是指：定义一个返回值为指针变量的函数么？然后这种函数都要写成*deleteDuplicates ？？谢谢

klmn111aaa 2014-06-05

打赏
举报

引用 2 楼 ri_aje 的回复:

pre = pre->next; cur->next = pre; 的写法移动了 pre 指针，而 cur->next = pre->next 没有修改 pre，delele del 以后 pre 就指向非法的地方了。非要一句话的话，可以试试这样：pre = cur->next = pre->next; ListNode *deleteDuplicates(ListNode *head) 的意思是说吃一个链表 (head 指向的)，把其中的重复值删干净，然后再把处理过的链表头节点指针返回。另外，程序里的 pre 明明指向 cur->next，这样命名非常混淆。

问一个有关单链表的题，只改动了一句话，结果就报错，不明白为什么，谢谢！！！

相关推荐