Singly-linked list add nodes in the middle

Question

class List {
  ListNode *head;
  ListNode *prev;
};

class ListNode {
  int data;
  ListNode *next;
  friend class List;
  ListNode(int d, ListNode *n) : data(d), next(NULL) {}
  void insertM(int d) {
    ListNode *ptr, *temp, *curr;
    ptr = head;
    while (ptr->data < d) {
      prev = ptr;
      ptr = ptr->next;
    } // end while
    temp = prev->next;
    curr = new ListNode(d, ptr);
    curr->next = prev->next; // or temp->next
    prev->next = curr;
    ;
  }
};

List mylist;

In this function, I'm trying to add a node in the middle of linked list. Other functions add items to the back and front just fine. When I add in the middle of the list, my prev->next links to curr just fine, but curr->next points to NULL.

I have been trying to make this program work for past 1.5 hours. I'll appreciate your help. This is homework.

Mala · Accepted Answer · 2012-05-13 01:54:07Z

1

The general procedure you'll want to use is:

surf down the next pointers until you reach the node you want to insert after (call it A)
change your inserted node (B)'s next pointer to match that of A's next pointer
change A's next pointer to point to B

It looks from your code that you're trying to maintain a sorted list of integers. This being homework you probably won't get a code snippet, but from glancing at your code, I have to ask why you take a next parameter in your node constructor, but then set the next value to null. Without seeing the rest of your code I can't say for sure, but I'd change next(NULL) on line 9 to be next(n), and then remove the curr->next line near the bottom

answered May 13, 2012 at 1:54

Mala

15k27 gold badges93 silver badges124 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

user1078719 Over a year ago

Now my code is working. It has been working but then I changed next(n) to next(NULL) for some reason. Now it's working fine. Thank you!

Collectives™ on Stack Overflow

Singly-linked list add nodes in the middle

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related