Thread: Help with FIFO QUEUE

Originally Posted by vart

in your QueuItem constructor
pData = mData; - this should be opposite

Code:

delete mpHead;
mpHead = mpHead->getNext();

you cannot access deleted pointer

your const correctness went out of window

your printList changes the head pointer making it impossible to remove head or print list once again

I changed the Queueitem constructor to:

Code:

QueueItem::QueueItem (char *pData, int id)
{
strncpy(mData, pData);
mNodeID = id;
mpNext = NULL;
}

for the removeItem function, after deleting the head shouldn't i push the next item on top??
How does the printlist change the head pointer?