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

klmn111aaa 2014-06-05 01:48:15
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 为什么就不对呢??谢谢
...全文
152 10 打赏 收藏 转发到动态 举报
写回复
用AI写文章
10 条回复
切换为时间正序
请发表友善的回复…
发表回复
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,这样命名非常混淆。
谢谢 对对,我也觉得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] 不好意思啊, 是可以的,我也不知道刚才可能哪里写错了,非常感谢!
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,这样命名非常混淆。
谢谢 对对,我也觉得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,是不是要求这样?
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,这样命名非常混淆。
谢谢 对对,我也觉得pre应该叫next才合适。 pre = cur->next = pre->next; 试了一下不行。 好像明白了,(ListNode *head) 就是“头节点指针返回”(实际头节点并没有变化),然后ListNode *deleteDuplicates是表示函数吧,为何不能写成ListNode deleteDuplicates,是因为返回值有指针,就必须这么写么?

64,282

社区成员

发帖
与我相关
我的任务
社区描述
C++ 语言相关问题讨论,技术干货分享,前沿动态等
c++ 技术论坛(原bbs)
社区管理员
  • C++ 语言社区
  • encoderlee
  • paschen
加入社区
  • 近7日
  • 近30日
  • 至今
社区公告
  1. 请不要发布与C++技术无关的贴子
  2. 请不要发布与技术无关的招聘、广告的帖子
  3. 请尽可能的描述清楚你的问题,如果涉及到代码请尽可能的格式化一下

试试用AI创作助手写篇文章吧