看了别人写的一个链表删除某个元素的函数,大体功能是正确的,但bug很容易复现,一起来看看:

删除链表中为X的节点,看函数中这样写!=X时,节点后移,直到找到等于X的节点;
万一输入的不是X,移到节点的末端也找不到,此时节点还往哪里移动,程序会死在这个while里
我们需要做个误操作处理,在轮询完整个链表后,发现到了结尾还没有要找的元素
我们加上 if( p ->next == NULL ) return L;结束查询,跳出函数,或者再加一句
printf("no find data !");这样是不是就有效避免了这种bug。
欢迎有其他想法的交流,新人勿喷。
网友评论
{
Node *p, *pre;
if (!L) return NULL;
p = L->next;
while (p && p->data != x)
{
pre = p;
p = p->next;
}
if (p)
{
pre->next = p->next;
free(p);
}
}