I need to modify my linked list to hold integers instead of strings by modifying the Node, List, and Iterator class. Don't know how to start this. This is my code.

#include <string>
#include <iostream>
#include <cassert>
 
using namespace std;
 
class List;
class Iterator;
 
class Node
{
public:
    /* 
       Constructs a node with a given data value.
       @param s the data to store in this node
    */
   Node(string s);
private:
   string data;
   Node* previous;
   Node* next;
   friend class List;
   friend class Iterator;
};
 
class List
{
public:
    /**
       Constructs an empty list;
    */
   List();
    /**
       Appends an element to the list.
       @param s the value to append
    */
   void push_back(string s);
    /**
       Inserts an element into the list.
       @param iter the position before which to insert
       @param s the value to append
    */
   void insert(Iterator iter, string s);
    /**
       Removes an element from the list.
       @param i the position to remove
       @return an iterator pointing to the element after the
       erased element
    */
   Iterator erase(Iterator i);
    /**
       Gets the beginning position of the list.
       @return an iterator pointing to the beginning of the list
    */
   Iterator begin();
    /**
       Gets the past-the-end position of the list.
       @return an iterator pointing past the end of the list
    */
  Iterator end();
private:
   Node* first;
   Node* last;
};
 
class Iterator
{
public:
    /**
       Constructs an iterator that does not point into any list.
    */
   Iterator();
    /**  
       Looks up the value at a position.
       @return the value of the node to which the iterator points
    */
   string get() const;
    /**
       Advances the iterator to the next node.
    */
   void next();
    /**
       Moves the iterator to the previous node.
    */
   void previous();
    /**
       Compares two iterators
       @param b the iterator to compare with this iterator
       @return true if this iterator and b are equal
    */
  bool equals(Iterator b) const;
private:
   Node* position;
   Node* last;
friend class List;
};
 
Node::Node(string s)
{  
   data = s;
   previous = NULL;
   next = NULL;
}

List::List()
{  
   first = NULL;
   last = NULL;
}
 
void List::push_back(string s)
{  
   Node* newnode = new Node(s);
   if (last == NULL) /* list is empty */
   {  
      first = newnode;
      last = newnode;
   }
   else
   {  
       newnode->previous = last;
       last->next = newnode;
       last = newnode;
   }
}
 
void List::insert(Iterator iter, string s)
{  
   if (iter.position == NULL)
    {  
       push_back(s);
       return;
    }
 
   Node* after = iter.position;
   Node* before = after->previous;
   Node* newnode = new Node(s);
   newnode->previous = before;
   newnode->next = after;
   after->previous = newnode;
   if (before == NULL) /* insert at beginning */
       first = newnode;
   else
       before->next = newnode;
}
 
Iterator List::erase(Iterator i)
{  
    Iterator iter = i;
    assert(iter.position != NULL);
    Node* remove = iter.position;
    Node* before = remove->previous;
    Node* after = remove->next;
    if (remove == first)
       first = after;
    else
       before->next = after;
    if (remove == last)
       last = before;
    else
       after->previous = before;
    iter.position = after;
    delete remove;
    return iter;
}
 
Iterator List::begin()
{  
    Iterator iter;
    iter.position = first;
    iter.last = last;
    return iter;
}
 
Iterator List::end()
{  
    Iterator iter;
    iter.position = NULL;
    iter.last = last;
    return iter;
}
 
Iterator::Iterator()
{  
    position = NULL;
    last = NULL;
}
 
string Iterator::get() const
{  
    assert(position != NULL);
    return position->data;
}
 
void Iterator::next()
{  
    assert(position != NULL);
    position = position->next;
}
 
void Iterator::previous()
{  
    if (position == NULL)
       position = last;
    else 
       position = position->previous;
    assert(position != NULL);
}
 
bool Iterator::equals(Iterator b) const
{  
    return position == b.position;
}
 
int main()
{  
    List staff;
 
    staff.push_back("Tom");
    staff.push_back("Dick");
    staff.push_back("Harry");
    staff.push_back("Juliet");
 
    /* add a value in fourth place */
 
    Iterator pos;
    pos = staff.begin();
    pos.next();
    pos.next();
    pos.next();
 
    staff.insert(pos, "Romeo");
 
    /* remove the value in second place */
   pos = staff.begin();
   pos.next();
 
    staff.erase(pos);
 
   /* print all values */
 
    for (pos = staff.begin(); !pos.equals(staff.end()); pos.next())
       cout << pos.get() << "\n";
 
  return 0;
}

Recommended Answers

All 5 Replies

Short answer : Replace all string data type with int

So for example where it says:

17: Node(string s);
18: private:
19: string data;

AND

98: Node::Node (string s)

change it to:

17: Node(int s);
18: private:
19: int data;

98: Node::Node(int s)

Yes. But here is another option.

In you Node class, you can use typedef like so :

class Node{
 typedef int DataType;
 //...blah blah blah
}

Now in your list and iterator class, instead of using int as a parameter, use Node::DataType. So :

Node(Node::DataType t);
private:
 Node::DataType data;

Thus by using Node:: DataType, you have less work to do, if you want to change say
from int to string again. Then All you would have to do is change typedef int DataType to typedef string DataType

But the short answer to your question. Yes.

Great, very helpful! Thank you very much.

Okay, I know you said not to use INT as a parameter, and I know I am. How do I change that. The two parts I know I am wrong are line 127, 111. It is giving me an error on my push_back ("Tom") since it is a const char not an int.

#include <string>
#include <iostream>
#include <cassert>
 
using namespace std;
 
class List;
class Iterator;
 
class Node
{
	typedef int DataType;

public:
    /* 
       Constructs a node with a given data value.
       @param s the data to store in this node
    */
   Node(Node::DataType t);
private:
   Node::DataType data;
   Node* previous;
   Node* next;
   friend class List;
   friend class Iterator;
};
 
class List
{
public:
    /**
       Constructs an empty list;
    */
   List();
    /**
       Appends an element to the list.
       @param s the value to append
    */
   void push_back(int s);
    /**
       Inserts an element into the list.
       @param iter the position before which to insert
       @param s the value to append
    */
   void insert(Iterator iter, int s);
    /**
       Removes an element from the list.
       @param i the position to remove
       @return an iterator pointing to the element after the
       erased element
    */
   Iterator erase(Iterator i);
    /**
       Gets the beginning position of the list.
       @return an iterator pointing to the beginning of the list
    */
   Iterator begin();
    /**
       Gets the past-the-end position of the list.
       @return an iterator pointing past the end of the list
    */
  Iterator end();
private:
   Node* first;
   Node* last;
};
 
class Iterator
{
public:
    /**
       Constructs an iterator that does not point into any list.
    */
   Iterator();
    /**  
       Looks up the value at a position.
       @return the value of the node to which the iterator points
    */
   int get() const;
    /**
       Advances the iterator to the next node.
    */
   void next();
    /**
       Moves the iterator to the previous node.
    */
   void previous();
    /**
       Compares two iterators
       @param b the iterator to compare with this iterator
       @return true if this iterator and b are equal
    */
  bool equals(Iterator b) const;
private:
   Node* position;
   Node* last;
friend class List;
};
 
Node::Node(int s)
{  
   Node::DataType data = s;
   previous = NULL;
   next = NULL;
}

List::List()
{  
   first = NULL;
   last = NULL;
}
 
void List::push_back(int s)
{  
   Node* newnode = new Node(s);
   if (last == NULL) /* list is empty */
   {  
      first = newnode;
      last = newnode;
   }
   else
   {  
       newnode->previous = last;
       last->next = newnode;
       last = newnode;
   }
}
 
void List::insert(Iterator iter, int s)
{  
   if (iter.position == NULL)
    {  
       push_back(s);
       return;
    }
 
   Node* after = iter.position;
   Node* before = after->previous;
   Node* newnode = new Node(s);
   newnode->previous = before;
   newnode->next = after;
   after->previous = newnode;
   if (before == NULL) /* insert at beginning */
       first = newnode;
   else
       before->next = newnode;
}
 
Iterator List::erase(Iterator i)
{  
    Iterator iter = i;
    assert(iter.position != NULL);
    Node* remove = iter.position;
    Node* before = remove->previous;
    Node* after = remove->next;
    if (remove == first)
       first = after;
    else
       before->next = after;
    if (remove == last)
       last = before;
    else
       after->previous = before;
    iter.position = after;
    delete remove;
    return iter;
}
 
Iterator List::begin()
{  
    Iterator iter;
    iter.position = first;
    iter.last = last;
    return iter;
}
 
Iterator List::end()
{  
    Iterator iter;
    iter.position = NULL;
    iter.last = last;
    return iter;
}
 
Iterator::Iterator()
{  
    position = NULL;
    last = NULL;
}
 
int Iterator::get() const
{  
    assert(position != NULL);
    return position->data;
}
 
void Iterator::next()
{  
    assert(position != NULL);
    position = position->next;
}
 
void Iterator::previous()
{  
    if (position == NULL)
       position = last;
    else 
       position = position->previous;
    assert(position != NULL);
}
 
bool Iterator::equals(Iterator b) const
{  
    return position == b.position;
}
 
int main()
{  
    List staff;
 
    staff.push_back("Tom");
    staff.push_back("Dick");
    staff.push_back("Harry");
    staff.push_back("Juliet");
 
    /* add a value in fourth place */
 
    Iterator pos;
    pos = staff.begin();
    pos.next();
    pos.next();
    pos.next();
 
    staff.insert(pos, "Romeo");
 
    /* remove the value in second place */
   pos = staff.begin();
   pos.next();
 
    staff.erase(pos);
 
   /* print all values */
 
    for (pos = staff.begin(); !pos.equals(staff.end()); pos.next())
       cout << pos.get() << "\n";
 
  return 0;
}
Be a part of the DaniWeb community

We're a friendly, industry-focused community of developers, IT pros, digital marketers, and technology enthusiasts meeting, networking, learning, and sharing knowledge.