Help C++

Question

baltair 0 Newbie Poster

13 Years Ago

Hello, I am trying to create a shop where the user can pick an item to add to their cart and after they pick the item, I want to remove it from the list, but I can't seem to remember on how to do it.

int main( void )
{
	const int nSize = 5;

	CWeapons Weapons[nSize];

	// Set each Weapons w/ unique names 
	Weapons[0].Set( "Batarang", 59.99f, 1);
	Weapons[1].Set( "KeyBlade", 39.99f, 1);
	Weapons[2].Set( "Katana", 39.99f, 1);
	Weapons[3].Set( "Flail", 39.99f, 1);
	Weapons[4].Set( "Assult Rifle", 39.99f, 1);
	

	const int nCartSize = 5;
	
	int nUser;

	CQueue Queue(nCartSize);

	// shopping menu
	for(;;)
	{
		cout << "1) Add an Item to the cart\n"
			<< "2) Check Out\n"
			<< "What would you like to do:";
		if(cin >> nUser && nUser <= 2 && nUser >=1 )
		{
			cin.ignore(INT_MAX, '\n');
			break;
		}
		cin.clear();
		cin.ignore(INT_MAX, '\n');
	}


	// if user selects add items to the cart
	if(1 == nUser)
	{
		int nCheckOut;

		for(;;)
		{

			// show selextion of 5 Weapons
			for(int i = 0; i < nCartSize; ++i)
			{
				cout << i+1 << ") " << Weapons[i].GetName() << " - " << Weapons[i].GetCost() << '\n';
			}
			cout << "0) Check Out Early\n";

			// ask user what option they want to do (0-5)
			for(;;)
			{
				cout << "What weapon would you like to add: ";
				if(cin >> nCheckOut && nCheckOut >=0 && nCheckOut <=5 )
				{
					cin.ignore(INT_MAX, '\n');
					break;
				}
				cin.clear();
				cin.ignore(INT_MAX, '\n');
			}

			if(nCheckOut == 1)
			{
				Queue.Enqueue(Weapons[0]);
		
			}
			else if(nCheckOut == 2)
			{
				Queue.Enqueue(Weapons[1]);	
				
			}
			else if(nCheckOut == 3)
			{
				Queue.Enqueue(Weapons[2]);
				
			}
			else if(nCheckOut == 4)
			{
				Queue.Enqueue(Weapons[3]);
				
			}
			else if(nCheckOut == 5)
			{
				Queue.Enqueue(Weapons[4]);
				
			}
			else
				break;


			system("pause");
			cout << "\n\n";	
		}

	}

	cout << "Receipt\n"
		<< "=======\n";

	float fTotal = 0;

	while(Queue.Dequeue(Weapons[nCartSize]))
	{
		cout << Weapons[nCartSize].GetName() << " - " << Weapons[nCartSize].GetCost() << '\n';
		fTotal += Weapons[nCartSize].GetCost();
	}

	cout << "Total: " << fTotal;

this is my loop where I ask the user to pick an item, but I can't remember on how to remove it from the list of items in the loop.

c++

Edited 13 Years Ago by baltair because: fixed code up a bit

3 Contributors
8 Replies
101 Views
1 Day Discussion Span
Latest Post 13 Years Ago Latest Post by baltair

All 8 Replies

Ancient Dragon 5,243 Achieved Level 70

13 Years Ago

Post the declaration of CQueue.

Ancient Dragon 5,243 Achieved Level 70

13 Years Ago

Post Queue.h

Reply to this topic

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.

baltair 0 Newbie Poster · Answer 1 · 2010-10-12T11:49:20+00:00

baltair 0 Newbie Poster

13 Years Ago

sorry for double post

Edited 13 Years Ago by baltair because: n/a

baltair 0 Newbie Poster · Answer 2 · 2010-10-12T11:53:30+00:00

Post the declaration of CQueue.

This what you wanted? or do you want my queue.cpp

#include <iostream>
using namespace std;
#include "Queue.h"
#include "Weapons.h"
int main( void )
{
	const int nSize = 5;

	CWeapons Weapons[nSize];

	// Set each Weapons w/ unique names 
	Weapons[0].Set( "Batarang", 59.99f, 1);
	Weapons[1].Set( "KeyBlade", 39.99f, 1);
	Weapons[2].Set( "Katana", 39.99f, 1);
	Weapons[3].Set( "Flail", 39.99f, 1);
	Weapons[4].Set( "Assult Rifle", 39.99f, 1);
	

	const int nCartSize = 5;
	
	int nUser;

	CQueue Queue(nCartSize);

	// shopping menu
	for(;;)
	{
		cout << "1) Add an Item to the cart\n"
			<< "2) Check Out\n"
			<< "What would you like to do:";
		if(cin >> nUser && nUser <= 2 && nUser >=1 )
		{
			cin.ignore(INT_MAX, '\n');
			break;
		}
		cin.clear();
		cin.ignore(INT_MAX, '\n');
	}


	// if user selects add items to the cart
	if(1 == nUser)
	{
		int nCheckOut;

		for(;;)
		{

			// show selextion of 5 Weapons
			for(int i = 0; i < nCartSize; ++i)
			{
				cout << i+1 << ") " << Weapons[i].GetName() << " - " << Weapons[i].GetCost() << '\n';
			}
			cout << "0) Check Out Early\n";

			// ask user what option they want to do (0-5)
			for(;;)
			{
				cout << "What weapon would you like to add: ";
				if(cin >> nCheckOut && nCheckOut >=0 && nCheckOut <=5 )
				{
					cin.ignore(INT_MAX, '\n');
					break;
				}
				cin.clear();
				cin.ignore(INT_MAX, '\n');
			}

			if(nCheckOut == 1)
			{
				Queue.Enqueue(Weapons[0]);
		
			}
			else if(nCheckOut == 2)
			{
				Queue.Enqueue(Weapons[1]);	
				
			}
			else if(nCheckOut == 3)
			{
				Queue.Enqueue(Weapons[2]);
				
			}
			else if(nCheckOut == 4)
			{
				Queue.Enqueue(Weapons[3]);
				
			}
			else if(nCheckOut == 5)
			{
				Queue.Enqueue(Weapons[4]);
				
			}
			else
				break;


			system("pause");
			cout << "\n\n";	
		}

	}

	cout << "Receipt\n"
		<< "=======\n";

	float fTotal = 0;

	while(Queue.Dequeue(Weapons[nCartSize]))
	{
		cout << Weapons[nCartSize].GetName() << " - " << Weapons[nCartSize].GetCost() << '\n';
		fTotal += Weapons[nCartSize].GetCost();
	}

	cout << "Total: " << fTotal;

Schol-R-LEA 1,446 Commie Mutant Traitor Featured Poster · Answer 3 · 2010-10-12T16:45:07+00:00

As a piece of advice, you could probably make the program a lot more readable if you broke main() down into several separate functions. Also, you could use switch() rather than the long series of if/else if/else statements when testing (for example) nCheckOut - though in that particular case you could instead just use

if (nCheckOut > 0)
                Queue.Enqueue(Weapons[nCheckOut - 1]);

instead.

baltair 0 Newbie Poster · Answer 4 · 2010-10-12T21:24:27+00:00

As a piece of advice, you could probably make the program a lot more readable if you broke main() down into several separate functions. Also, you could use switch() rather than the long series of if/else if/else statements when testing (for example) nCheckOut - though in that particular case you could instead just use
if (nCheckOut > 0)
                Queue.Enqueue(Weapons[nCheckOut - 1]);
instead.

Thanks for the advice, never thought of that for a loop.

Post Queue.h

Here is my Queue.h

#ifndef QUEUE_H_
#define QUEUE_H_
#include "Weapons.h"

typedef CWeapons Item;

class CQueue
{
private:
	struct tNode
	{
		Item data;
		tNode* pNext;
	};

	tNode* m_pHead;
	tNode* m_pTail;

	const int m_nMax;

	int m_nCurrSize;

public:
	CQueue(int nMax = 0);

	~CQueue();

	bool Enqueue(const Item& info);

	bool Dequeue(Item& info);
};
#endif

Ancient Dragon 5,243 Achieved Level 70 Team Colleague Featured Poster · Answer 5 · 2010-10-13T00:47:26+00:00

>>but I can't remember on how to remove it from the list of items in the loop.

I assume you mean how to remove a node from the linked list. All you have to do is iterate through the linked list until you find the node to be deleted. Then adjust the node->next pointer of the node just previous to the one to be deleted so that it pointer to the node after the one to be deleted. Of course since you have a single linked list the code will have to keep track of both the current node and the previous node.

baltair 0 Newbie Poster · Answer 6 · 2010-10-13T10:52:06+00:00

>>but I can't remember on how to remove it from the list of items in the loop.
I assume you mean how to remove a node from the linked list. All you have to do is iterate through the linked list until you find the node to be deleted. Then adjust the node->next pointer of the node just previous to the one to be deleted so that it pointer to the node after the one to be deleted. Of course since you have a single linked list the code will have to keep track of both the current node and the previous node.

Thanks a lot, this helped and fixed my problem.

Help C++

Recommended Answers Collapse Answers

All 8 Replies

Recommended Answers