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

Delete a node from doubly linked list

Deletion operation in DLL is simpler when compared to SLL. Because we don't have to go in search of previous node of to-be-deleted node.  Here is how you delete a node Link previous node of node of to-be-deleted to next node. Link next node of node of to-be-deleted to previous node. Free the memory of node of to-be-deleted Simple, isn't it. The code can go like this. prevnode = delnode->prev; nextnode = delnode->next; prevnode->next = nextnode; nextnode->prev = prevnode; free(delnode); And that is it. The node delnode is deleted. But we should always consider boundary conditions. What happens if we are trying to delete the first node or last node? If first node is to be deleted, its previous node is NULL. Hence step 3 should not be used.  And also, once head is deleted, nextnode becomes head . Similarly if last node is to be deleted, nextnode is NULL. Hence step 4 is as strict NO NO. And we should set prevnode to tail. After we put these things together, we have...

Function to sort an array using bubble sort

Quick and dirty way of sorting an array is bubble sort. It is very easy to write and follow. But please keep in mind that it is not at all effecient. #include<iostream> using std::cin; using std::cout; void readArray(int arr[],int sz); void printArray(int arr[],int sz); void sortArray(int arr[],int sz); void swap(int &a,int &b); int main() {    int sz;    cout<<"Size of the array=";    cin>>sz;    int arr[sz];    readArray(arr,sz);     sortArray(arr,sz);   cout<<"Sorted array is ";   printArray(arr,sz); } void readArray(int arr[],int sz) {  for(int i=0;i<sz;i++)    {       cout<<"arr["<<i<<"]=";       cin>>arr[i];   } } void printArray(int arr[],int sz) {  for(int i=0;i<sz;i++)    {       cout<<"arr["<<i<<"]=";    ...

Merge two binary search trees

How do you merge two binary search trees? I googled about the solutions. Most solutions told me to convert both trees into linked lists. Merge the lists. Then create a tree from the elements of the list. But why lists? Why can't we store the elements in an array? Because if the data of the tree is larger - not just integer keys, array manipulation becomes difficult. But again, we need not convert both the trees into lists. We can convert one tree into list - a doubly linked list. Then insert the elements of this list into the other tree. I tried this approach. To convert a tree into a sorted doubly linked list Create a doubly linked list. Let the prev and next links of nodes in this list be called left and right respectively. This way we can directly use the binary tree nodes in the list. Use a static variable previousnode  call the function recursively for left child of current node. link current node to the previousnode set next pointer of previousnode to curre...