C++ Sorted Doubly Linked List: Problems inserting in middle of list

Viewed 351

below is my current in-progress code converting a singly linked to a doubly linked list. I haven't touched the delete function yet. I've gotten insert in empty list, end of list, and beginning of list apparently working.

However nodes inserting in the middle seemingly fail to create a link to the previous node. My debugging lines I inserted seem to show both the n->next and n-> prev with the correct memory address, but when I go to reverseprint, any nodes inserted in the middle are missed and the links are gone. Where am I going wrong in regards to this?

Code below:

#include <iostream>
#include <string>
using namespace std;

// define a node for storage and linking
class node {
public:
    string name;
    node *next;
    node *prev; 
};

class linkedList {
public:
    linkedList() :top(NULL) {}
    bool empty() { return top == NULL; }
    node *getTop() { return top; }
    node *getEnd() { return end; }
    void setTop(node *n) { top = n; }
    void setEnd(node *p) { end = p; }
    void add(string);
    int menu();
    void remove(string);
    ~linkedList();
     void reversePrint(); 
    friend ostream& operator << (ostream&, const linkedList&); // default output is in-order print.
private:
    node *top;
    node *end; 
};

void main() {
    linkedList l;
    cout << l.empty() << endl;
    int option = 0;
    string s;
    bool go = true;
    while (go) {
        option = l.menu();
        switch (option) {
        case 1: cout << "enter a name: "; cin >> s; l.add(s); break;
        case 2: cout << "enter name to be deleted: "; cin >> s; l.remove(s); break;
        case 3: cout << l; break;
        //case 4: cout << "can not be done with a singly linked list" << endl;
        case 4: l.reversePrint(); break;
        case 5: cout << "exiting" << endl; go = false; break;
        }
    }


    system("pause");
}


void linkedList::remove(string s) {
    bool found = false;
    node *curr = getTop(), *prev = NULL;
    while (curr != NULL) {

        // match found, delete
        if (curr->name == s) {
            found = true;

            // found at top
            if (prev == NULL) {
                node *temp = getTop();
                setTop(curr->next);
                delete(temp);

                // found in list - not top
            }
            else {
                prev->next = curr->next;
                delete(curr);
            }
        }

        // not found, advance pointers
        if (!found) {
            prev = curr;
            curr = curr->next;
        }

        // found, exit loop
        else curr = NULL;
    }
    if (found)cout << "Deleted " << s << endl;
    else cout << s << " Not Found " << endl;
}

void linkedList::add(string s) {
    node *n = new node();
    n->name = s;
    n->next = NULL;
    n->prev = NULL;

    // take care of empty list case
    if (empty()) {
        top = n;
        end = n;

        // take care of node belongs at beginning case
    }
    else if (getTop()->name > s) {


        n->next = getTop();
        n->prev = NULL;
        setTop(n);

        node *temp;
        temp = n->next;
        temp->prev = n;

        // take care of inorder and end insert
    }
    else {

        // insert in order case
        node *curr = getTop(), *prev = curr;
        while (curr != NULL) {
            if (curr->name > s)break;
            prev = curr;
            curr = curr->next;
        }
        if (curr != NULL) { // search found insert point
            n->next = curr;
            cout << n->name << " " << n << " prev " << prev << " " << prev->name << endl;
            n->prev = prev;

            prev->next = n;
            cout << "n->prev is: " << n->prev << " " << n->prev->name << endl;
            cout << "n->next is: " << n->next << " " << n->next->name << endl;
        }

        // take care of end of list insertion
        else if (curr == NULL) {// search did not find insert point
            prev->next = n;
            n->prev = prev;
            cout << "n->prev is: " << n->prev << " " << n->prev->name << endl;
            setEnd(n);
        }
    }
}

ostream& operator << (ostream& os, const linkedList& ll) {
    //linkedList x = ll; // put this in and the code blows up - why?
    node *n = ll.top;
    if (n == NULL)cout << "List is empty." << endl;
    else
        while (n != NULL) {
            os << n->name << endl;
            os << n << endl;
            if (n->next != NULL) {
                os << "next is " << n->next << endl;
            }

            n = n->next;
        }
    return os;
}

void linkedList::reversePrint() {
    node *n = end;


    if (n == NULL)cout << "List is empty." << endl;
    else
        while (n != NULL) {
            //cout << n->name << endl;
            cout << "memory address of " << n->name << " is " << n << endl;

            if (n->prev != NULL) {
                cout << "prev is " << n->prev << endl;
            }
            n = n->prev;
        }
    return;
}
// return memory to heap
linkedList::~linkedList() {
    cout << "~linkedList called." << endl;
    node *curr = getTop(), *del;
    while (curr != NULL) {
        del = curr;
        curr = curr->next;
        delete(del);
    }
}

int linkedList::menu() {
    int choice = 0;
    while (choice < 1 || choice > 5) {
        cout << "\nEnter your choice" << endl;
        cout << " 1. Add a name." << endl;
        cout << " 2. Delete a name." << endl;
        cout << " 3. Show list." << endl;
        cout << " 4. Show reverse list. " << endl; 
        cout << " 5. EXIT " << endl;
        cin >> choice;
    }
    return choice;
}
1 Answers

You are not setting the prev of the current in insertion into middle, just do:

n->next = curr;
curr->prev = n;  // <-- this
Related