链接列表删除正在删除错误的节点

时间:2012-11-12 07:52:32

标签: c linked-list

我需要别人的帮助。我的作业很难。

这个作业非常简单。创建一个列表(1,2,3)并通过创建函数delnode删除中间数字。但它必须使用函数free()

现在,我已经使用链表方法创建了列表(1,2,3)。我想删除数字2,但它不起作用。它应该是(1,3),但它出现在(2,3)

#include <stdio.h>
#include <stdlib.h>


struct node{
    int number;
    struct node* next;
};

typedef struct node node;

//prototype function
node* allocateMemory(void);
node* insertNode(node*);
void delnode(node*);

int main(){
    int i,num;
    node* entr = allocateMemory();
    node* p = NULL;

    entr->number = 1;
    entr->next = NULL;

    num = 3;

    for(i=1;i<num;i++){
        if(!p){
            p=insertNode(entr);
        }else{
            p=insertNode(p);
        }

        p->number = i+1;
    }

    while(entr){
        if(entr->number == 2){
            entr->number == NULL;
            break;
        }
        entr=entr->next;
    }

    while(entr){
        printf("%d\n",entr->number);
        entr=entr->next;
    }

    return 0;
}

node* insertNode(node* current){
    node* newNode = allocateMemory();
    current->next = newNode;
    newNode->next = NULL;
    return newNode;
}


void delnode(node* current){
    node* temp = allocateMemory();
    temp = current->next->next;
    free(current->next);
    current = temp;
    free(temp);
    return ;
}

node* allocateMemory(void){
    return (node*)malloc(sizeof(node));
}

2 个答案:

答案 0 :(得分:4)

一些问题:

(1)您创建的原始列表是[2,3],而不是[1,2,3] - 您的循环迭代i = 1,i = 2 - 并且每次插入i + 1 - 结果[2,3]
(2)您的删除没有做任何事情:

while(entr){
   if(entr->number == 2){
       entr->number == NULL;
       break;
   }
   entr=entr->next;
}

请注意,entr->number == NULL只是一个布尔评估,而不是作业 - 因为您使用operator==

(注意它也会因operator=而失败,因为您不想将NULL分配给该值 - 您真正想要做的是分配上一个节点next字段。)

P.S。
该数据结构的术语是Linked List,而不是“线性列表”

答案 1 :(得分:2)

好的,这个函数delnode是各种搞砸的。当您尝试删除节点时,不需要分配更多内存。您需要 将上一个链接到下一个,然后删除当前的

| prev |      | current |      | next |
|------|      |---------|      |------|
   /\              X               /\
   ||              X               ||
   |================================|

看起来应该更像这样。

void delnode(node* current)
{
  node* next = current->next;
  // Now we need to find the node previous to this.
  node* prev = entr; // From the start.
  while(prev->next != current) { prev = prev->next; };
  free(current); // delete the current node.
  prev->next = next; // Link the previous node to the next in the list.
}

如果您尝试添加三个节点,则需要更改循环。

node* current = entr;
for(i=0;i<3;i++)
{
  int newCount = current->number++; // Increment count.
  current = insertNode(current); // Returns the new node.
  current->number = newCount; // Assign new count.
}

如果您要删除number 2的节点:

node* idx = entr;
while(idx)
{
  if(idx->number == 2) { delnode(idx); break; } // Delete node and break.
  idx = idx->next; // Else, go to next node.
}