Skip to main content

Program to delete a node from linked list

How do you remove a node from a linked list? If you have to delete a node, first you need to search the node. Then you should find its previous node. Then you should link the previous node to the next node. If node containing 8 has to be deleted, then n1 should be pointing to n2.

Looks quite simple. Isn't it?

But there are at least two special cases you have to consider.
  1. Of course, when the node is not found.
  2. If the node is first node of the list viz head.
If the node to be deleted is head node, then if you delete, the list would be lost. You should avoid that and make the second node as the head node. So it becomes mandatory that you return the changed head node from the function.


Now let us have a look at the code.


#include<stdio.h>  
#include<stdlib.h>
 struct node  
 {  
   int data;  
   struct node *next;   
 };  
 typedef struct node * NODEPTR;   

 NODEPTR create_node(int value)  
 {  
   NODEPTR temp = (NODEPTR) malloc(sizeof(struct node));  
    temp->next = NULL;  
   temp->data= value;  
   return temp;  
 } 
 NODEPTR append_node(NODEPTR head, int n)  
 {  
    NODEPTR temp = head;
    NODEPTR newnode = create_node(n);
    if(temp==NULL)
      return newnode;
    while(temp->next !=NULL)
 temp = temp->next;
    temp->next = newnode;
    return head;  
 } 
NODEPTR find_node(NODEPTR head, int val)
{
     NODEPTR temp = head;
     while(temp!=NULL && temp->data!=val)
     {
             temp = temp->next;
     }
     return temp;
}

/*function to find the previous node for a given node in the list*/
NODEPTR find_previous_node(NODEPTR head, NODEPTR temp)
{
     NODEPTR previousNode = head;
     while(previousNode !=NULL && previousNode ->next !=temp)
     {
             previousNode = previousNode ->next;
     }
     return previousNode ;
}

/*function deletes the node with val as data and returns the head node*/
NODEPTR delete_node(NODEPTR head,int val)
{
    NODEPTR temp = find_node(head,val); 

    if (temp!=NULL)
    {
          /*check if node is first node*/
        if(temp==head)
          {
              head = head->next;
              free(temp);
              return head;
          }
          NODEPTR previousNode = find_previous_node(head,temp);
          if(previousNode!=NULL)
          {
               /*Now point previous node to next node of temp */
              previousNode->next = temp->next; 
           }
           else
           {
                printf("Some serious error. Previous node is not found");   
           }
    }
    else
    {
            printf("Node is not found."); 
    }
   return head;
}

void display_nodes(NODEPTR head)  
 {  
   NODEPTR temp = head;//redundant 
   while (temp!= NULL)  
   {  
     printf("%d====>",temp->data);  
     temp = temp->next;  
   }  
   printf("\n");
 } 

int main()
{
   NODEPTR head = NULL;
   int i;
   for(i = 0;i<10;i++)
   {
       int n;
       printf("Enter a number (-1 to exit) :");
       scanf("%d",&n);
       if(n==-1)break;
       head = append_node(head,n);
    }
    printf("The list is :");
    display_nodes(head);
    printf("Enter the node to be deleted :");
    scanf("%d",&i);
    head = delete_node(head,i);
    printf("After deleting the node, the list is ");
    display_nodes(head);
    return 0;
}


Now let us some tricky questions on deletion.
  1. What will happen if the list is empty?
  2. Can you delete a node, provided you have only pointer to that node, and do not have head node info.?
  3. Will the program crash if the list has only one node head, and that is to be deleted?
Answer to the first question is,
  1. the current program will work fine. If the list is empty, find_node function returns NULL, and delete will not delete anything.
  2.  you take next_node as temp->next, then copy next_node->data to temp->data and then say temp->next = temp->next->next.
  3.  The program will not crash because before deleting the only node in the list, we set head to head->next. Now head will be NULL and this NULL will be returned and assigned  to head. For all further operations, list is empty.
If you have any doubts, please leave a comment.

Comments

  1. In delete_node(), if we delete the head node, and the only remaining nodes in the list is indeed this head node, then statement "head = head->next;" will corrupt the list... I have used your implementation in my program and have faced this issue.
    What do you think?

    Thanks, and keep on the good work!

    ReplyDelete
    Replies
    1. Thanks a lot M-ric for pointing out the difficulty in captcha. I have removed the captcha from comment and have also included anonymous comments.
      I will also work on delete node and try to solve it.

      Delete
  2. To write comment, the catpcha stuff is terrible. I almost typed 10 times the character chain on the screen to get it write. It is so damn annoying. Probably one of the reason why there are so few comments on this otherwise very instructive blog.

    Can you remove it?

    ReplyDelete

Post a Comment

Popular posts from this blog

Josephus problem

Question: Write a function to delete every k th node from circular linked list until only one node is left. This has a story associated with it. Flavius Josephus was Jewish Historian from 1st century. He and 40 other soldiers were trapped in a cave by Romans. They decided to kill themselves rather than surrendering to Romans. Their method was like this. All the soldiers will stand in a circle and every k th soldier will be shot dead. Josephus said to have calculated the starting point so that he would remain alive. So we have similar problem at hand. We delete every kth node in a circular list. Eventually only one node will be left. e.g. Let us say this is our list And we are deleting every third node.  We will delete 30. Then we delete 60. Next we delete 10. Next it will be 50. Next to be deleted is 20. Next 80. This continues. Implementation   We can count k-1 nodes and delete next node. This can be repeated in  a loop. What must be the termina...

Lowest common ancestor of binary search tree

Question : Write a function to print the lowest common ancestor of two nodes in a binary search tree.  Lowest common ancestor of two nodes x and y in a binary tree is the lowest node that has both x and y as descendants. Here lowest common ancestor of 1 and 7 is 3. LCA of 13 and 7 is root - 8. And LCA of 6 and 7 is 6 itself. The program to find gets complicated for an ordinary binary tree. But for a binary search tree, it is quite simple. As we see from the diagram above, the paths to 1 and 4 are common till the node 3. And at 3 they branch in different directions. So 3 is our LCA. That is lowest common ancestor is the node where the paths to x and y from root deviate. As long as they branch in same direction, we continue to traverse. When they branch in different directions, that is the lowest common ancestor. So let us write a simple algorithm, set temp=root if temp->val >x and temp->val>y temp = temp->left else if temp->val<x and ...

In order traversal of nodes in the range x to y

Question : Write a function for in-order traversal of nodes in the range x to y from a binary search tree. This is quite a simple function. As a first solution we can just traverse our binary search tree in inorder and display only the nodes which are in the range x to y. But if the current node has a value less than x, do we have to traverse its left subtree? No. Because all the nodes in left subtree will be smaller than x. Similarly if the current node has a key value more than y, we need not visit its right subtree. Now we are ready to write our algorithm.     if nd is NOT NULL  if nd->val >=x then visit all the nodes of left subtree of nd recursively display nd->val if nd->val <y then visit all the nodes of right subtree of nd recursively  That's all. We have our function ready. void in_order_middle (NODEPTR nd, int x, int y) { if (nd) { if (nd -> val >= x) in_order_middle(nd...