链表的循环带删除操作

错误的一个思路,往往导致一个很难查询的bug,上周在实现数据存储链表时,由于思路的不严谨,导致一个bug查了大半天,代码原先是这样的

 while(p)
    {
        if(id == p->uniqueid)
        {
            if(p != list->head)
            {
                p->prev->next = p->next;
                if(p->next != NULL)
                {
                    p->next->prev = p->prev;
                }
                
            }
            else
            {
                list->head = p->next;
            }
            if(p->external_ip)
                free(p->external_ip);
            if(p->internal_ip)
                free(p->internal_ip);
            free(p);
            break;
        }
        p = p->next;
    }

额,好吧,在原先的项目中曾经有过这样的代码,后面修过了,这次又发生了,一块石头踢了两次脚,很明显,在搜索到了的时候我这边是调用了free(p),但仅接着又做了p = p->next,额,这个就有点难为情了,操作空指针了,好吧,这确实会导致一个fatal signal 11的问题!

后面更正后的代码

 while(p)
    {
        if(id == p->uniqueid)
        {
            result = p->next;
            if(p != list->head)
            {
                p->prev->next = p->next;
                if(p->next != NULL)
                {
                    p->next->prev = p->prev;
                }
                
            }
            else
            {
                list->head = p->next;
            }
            if(p->external_ip)
                free(p->external_ip);
            if(p->internal_ip)
                free(p->internal_ip);
            free(p);
            break;
        }
        else
            p = p->next;
    }

告诉自己,在查询的时候要是做delete的操作,切忌要注意使用野指针

  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值