我试图在C中创建一个删除每个奇数定位节点的函数。例如,1,2,3,4
变为2,4
。
这是我尝试的但它似乎没有起作用。
我在谈论deletee
功能。我修改了它,但列表似乎没有改变。
#include <stdio.h>
#include <stdlib.h>
typedef struct node {
int val;
struct node *next;
} node;
typedef struct ll {
node *head;
} ll;
ll *newll() {
ll *k = malloc(sizeof(ll));
k->head = NULL;
return k;
}
void insert(ll *l, int vl) {
node *tmp = malloc(sizeof(node));
tmp->next = NULL;
tmp->val = vl;
if (l->head == NULL) {
l->head = tmp;
return;
}
node *s = l->head;
while (s->next != NULL)
s = s->next;
s->next = tmp;
}
void printll(ll *l) {
node *s = l->head;
while (s != NULL) {
printf("%d ", s->val);
s = s->next;
}
}
void deletee(ll *l) {
node *k = l->head;
while (k != NULL && k->next != NULL) {
node *tmp = k->next->next;
k = tmp;
}
}
int main() {
ll *ll1 = newll();
insert(ll1, 5);
insert(ll1, 6);
insert(ll1, 8);
insert(ll1, 9);
insert(ll1, 10);
insert(ll1, 11);
deletee(ll1);
printll(ll1);
return 0;
}
答案 0 :(得分:4)
我们需要同时更新ll.head
和node.next
,因此指向node
的指针不够好,除非您想要特殊情况下的头部。相反,让我们使用指向我们想要更新的指针。
void delete_node(node** node_ptr_ptr) {
node* to_delete = *node_ptr_ptr;
*node_ptr_ptr = to_delete->next;
free(to_delete);
}
void delete_every_second(ll* l) {
node** node_ptr_ptr = &( l->head );
while (1) {
if (*node_ptr_ptr == NULL) break;
delete_node(node_ptr_ptr);
if (*node_ptr_ptr == NULL) break;
node_ptr_ptr = &( (*node_ptr_ptr)->next );
}
}
假设您从以下开始:
+------+ +------+ +------+ +------+
| head ------>| val | +-->| val | +-->| val |
+------+ +------+ | +------+ | +------+
| next ---+ | next ---+ | next --->NULL
+------+ +------+ +------+
node** node_ptr_ptr = &( l->head );
之后:
+------+ +------+ +------+ +------+
| head ------>| val1 | +-->| val2 | +-->| val3 |
+------+ +------+ | +------+ | +------+
^ | next ---+ | next ---+ | next --->NULL
| +------+ +------+ +------+
|
+-----+
|
+------+ |
| ptr ----+
+------+
node* to_delete = *node_ptr_ptr;
之后:
+------+
| del ----+
+------+ |
|
+------+
|
v
+------+ +------+ +------+ +------+
| head ------>| val1 | +-->| val2 | +-->| val3 |
+------+ +------+ | +------+ | +------+
^ | next ---+ | next ---+ | next --->NULL
| +------+ +------+ +------+
|
+-----+
|
+------+ |
| ptr ----+
+------+
*node_ptr_ptr = to_delete->next; free(to_delete);
之后:
+------+ +------+ +------+
| head -------------------->| val2 | +-->| val3 |
+------+ +------+ | +------+
^ | next ---+ | next --->NULL
| +------+ +------+
|
+-----+
|
+------+ |
| ptr ----+
+------+
node_ptr_ptr = &( (*node_ptr_ptr)->next );
之后:
+------+ +------+ +------+
| head -------------------->| val2 | +-->| val3 |
+------+ +------+ | +------+
+---------------->| next ---+ | next --->NULL
| +------+ +------+
|
+------+ |
| ptr ----+
+------+
答案 1 :(得分:2)
在你的代码中:
while(k!=NULL)
{
if(k->next!=NULL && k->next->next!=NULL)
k->next=k->next->next;
}
你有一个无限循环,因为你不会在循环中改变k
的值。
另外:您必须首先删除/释放k->next
的内存,否则会导致内存泄漏。
我简单地重写如下:
void deletee(ll *l)
{
if (l->head == NULL)
return;
node* tmp = l->head;
l->head = l->head->next; // skip first item
free(tmp);
node* k=l->head;
while(k!=NULL && k->next!=NULL)
{
tmp = k->next;
k->next = k->next->next;
free(tmp);
k = k->next;
}
}
结果(如预期):
6 9 11
tmp
存储下一个值以供将来删除tmp
答案 2 :(得分:0)
您更改了deletee
功能的代码,但仍然不正确:
void deletee(ll *l) {
node *k = l->head;
while (k != NULL && k->next != NULL) {
node *tmp = k->next->next;
k = tmp;
}
}
您根本不会修改列表。
这是一个带指针指针的解决方案,以避免出现空列表和列表中第一个节点的特殊情况。 k
指向删除节点时必须更新的链接,除了在列表末尾之外,它在每次删除后被推送到保留的节点:
void deletee(ll *l) {
node **k = &l->head;
while (*k != NULL) {
/* delete the node */
node *tmp = *k;
*k = (*k)->next;
free(tmp);
if (*k != NULL) {
/* skip the preserved node */
k = &(*k)->next;
}
}
}