Cans Machine C++ Programming

[evgeric]

How do I created a loop for this codings?

#include <iostream>
using namespace std;

int quantity = 0;
int selection = 0;
float amount = 0;
float total = 0;


int main();

}
int main()
{
	{
		cout << "\n Please select the below shown item <1-3>" << endl;
		cout << " 1. Coco Cola  ($1.00 / Available: " << currentAmountCoke << ")" << endl;
		cout << " 2. Pepsi Cola ($1.00 / Available: " << currentAmountPepsi << ")" << endl;
		cout << " 3. Sprite     ($1.00 / Available: " << currentAmountSprite << ")" << endl;
		cout << "\n Choice: ";
		cin >> selection;

			cout << "\n Please select the quantity: ";
			cin >> quantity;

				}
				else
				{
					cout << "\n Insufficient can of drink, please try another choice \n";
					quantity = 0;
				
				}
			}
			
					
					cout << "\n Insufficient can of drink, please try another choice";
			
					
		
	
				else
				{
					cout << "\n Insufficient can of drink, please try another choice";
					quantity = 0;
					
				}
			}
		}
		else
		{
			cout << "\n Incorrect Selection, please enter 1 to 3";
			
		}
	}

	cout << "\n Currently out of drinks";
	cin.ignore();
	cin.ignore();

}

[evgeric]

Any helpers?

[mutexe]

To start with, rather than your hard-coded '10' on line 19, you'll need separate variables to keep track of the current quantity of each type of drink.

int currentAmountCoke = 10;
int currentAmountPepsi = 10;
int currentAmountSprite = 10;

and when a selection is made decrement the appropriate one.
e.g.

currentAmountCoke -= quantity;

[evgeric]

Thanks mutexe, How do I do the looping?

[mutexe]

well the whole thing should be a loop. And if when you try and decrement and get minus one you need to report the drink has gone.

[ANproCUBE]

Hey

You can make a simple while-loop

Start here:

while ((currentAmountCoke <= 10) || (currentAmountPepsi <= 10) || (currentAmountSprite <= 10)) {
   cout << "Please select the below shown item <1-3>" << endl;
   ...
      quantity=0;
   }
}

I would also show how many drinks are remaining!

ANproCUBE

[evgeric]

Hi Thanks ANproCUBE,

After i'm done the loop how do I continue?

Sorry I am a beginner. I may need someone who is experience to help me with it?

[ANproCUBE]

Hey

I can see that you have a few problems to use our suggestions. So I wrote this code for you today. All you have to to is to read this code and try to understand what I did.

If you still have questions about the code, please ask!

//-----------------------------------------------
// Things you can add:
// + Defines for different prices for each drink
//   -> You would have to calculate
//      each total individually
//      with its price
//      e.g. "#define COKE_PRICE 1.20"
// + More drinks
//   -> For example drinks with alcohol where
//      you would have to ask for the age...
// + Save the stats to a external file to keep
//   the amount of drinks in the machine
//-----------------------------------------------

//Includes
#include <iostream>
#include <conio.h> //You only have to write "getch();", but I'm using VS 2013

//Defines
#define QUANTITY_MAX 10

//Namespaces
using namespace std;

//Global Variables
int quantity = 0;
int selection = 0;
int currentAmountCoke = QUANTITY_MAX;
int currentAmountPepsi = QUANTITY_MAX;
int currentAmountSprite = QUANTITY_MAX;
float amount = 0;
float total = 0;


int main();
void wait()
{
	cout << "\n Press a key: ";
	fflush(stdin);
	_getch();
	main();
}
void vendingMachineRoutine()
{
	system("cls");
	total = 1.00 * quantity;
	cout << "\n Please insert " << total << "$: ";
	cin >> amount;

	if (amount > total)
	{
		system("cls");
		amount -= total;
		cout << "\n Please collect drink and change of " << amount << "$";

		if (selection == 1)
		{
			currentAmountCoke -= quantity;
		}
		else if (selection == 2)
		{
			currentAmountPepsi -= quantity;
		}
		else if (selection == 3)
		{
			currentAmountSprite -= quantity;
		}
		wait();
	}
	else if (amount == total)
	{
		system("cls");
		cout << "\n Please collect your drink";

		if (selection == 1)
		{
			currentAmountCoke -= quantity;
		}
		else if (selection == 2)
		{
			currentAmountPepsi -= quantity;
		}
		else if (selection == 3)
		{
			currentAmountSprite -= quantity;
		}
		wait();
	}
	else
	{
		system("cls");
		cout << "\n Amount is insufficient";
		wait();
	}
}
int main()
{
	while ((currentAmountCoke <= 10) || (currentAmountPepsi <= 10) || (currentAmountSprite <= 10))
	{
		system("cls");
		cout << "\n Please select the below shown item <1-3>" << endl;
		cout << " 1. Coco Cola  (1$ / Available: " << currentAmountCoke << ")" << endl;
		cout << " 2. Pepsi Cola (1$ / Available: " << currentAmountPepsi << ")" << endl;
		cout << " 3. Sprite     (1$ / Available: " << currentAmountSprite << ")" << endl;
		cout << "\n Choice: ";
		cin >> selection;

		if ((selection >= 1) && (selection <= 3))
		{
			system("cls");
			cout << "\n Please select the quantity: ";
			cin >> quantity;

			if (selection == 1)
			{
				if ((quantity <= QUANTITY_MAX) && (quantity <= currentAmountCoke) && (quantity > 0))
				{
					vendingMachineRoutine();
				}
				else
				{
					system("cls");
					cout << "\n Insufficient can of drink, please try another choice";
					quantity = 0;
					wait();
				}
			}
			else if (selection == 2)
			{
				if ((quantity <= QUANTITY_MAX) && (quantity <= currentAmountPepsi) && (quantity > 0))
				{
					vendingMachineRoutine();
				}
				else
				{
					system("cls");
					cout << "\n Insufficient can of drink, please try another choice";
					quantity = 0;
					wait();
				}
			}
			else if (selection == 3)
			{
				if ((quantity <= QUANTITY_MAX) && (quantity <= currentAmountSprite) && (quantity > 0))
				{
					vendingMachineRoutine();
				}
				else
				{
					system("cls");
					cout << "\n Insufficient can of drink, please try another choice";
					quantity = 0;
					wait();
				}
			}
		}
		else
		{
			system("cls");
			cout << "\n Insufficient Number";
			wait();
		}
	}

	system("cls");
	cout << "\n Currently out of drinks :(";
	wait();
}

I hope it's helpful,
ANproCUBE

[ANproCUBE]

Hey

I forgot to mention your question :D

After the loop you can do what you want...
You can, for example, refill the machine or something like that...

ANproCUBE

[Konstantin2]

Line 98:
while ((currentAmountCoke <= 10) || (currentAmountPepsi <= 10) || (currentAmountSprite <= 10))
looks like infinite loop.
Maybe
while ((currentAmountCoke > 10) || (currentAmountPepsi > 10) || (currentAmountSprite > 10))
be better.
Only 168 lines for this code? Looser.

[Konstantin2]

Try to ask "-1" amount of doses. =)

[ANproCUBE]

Hey

OK... I don't really understand what's the problem here ;)

When I ask for -1 it says "Insufficient can of drink, please [...]"

That's what the (quantity > 0) for :D

And please don't call me loser.
I tried to make a simple code (OK, it's a little bit longer than expected), but I think
this works pretty well.
And if there's a mistake, as you said, please give constructive feedback.

ANproCUBE

[ANproCUBE]

Hey

I looked over it again...
I found one new mistake and corrected the one that you found:

1) Your suggestion doesn't work either. It works with: while ((currentAmountCoke != 0) || (currentAmountPepsi != 0) || (currentAmountSprite != 0))

2) At the end I used my function "wait". Use getch() or _getch() instead

ANproCUBE

[evgeric]

hey thanks ANproCUBE for your speedy reply. I manage to understand the coding somehow, although your coding are still a little advance to me. Thanks a lot.

[evgeric]

hey ANproCUBE, appreciate if you able to help me on another code from this link.

http://www.cplusplus.com/forum/general/157753/

[ANproCUBE]

Hey

OK, I'll take a look.

Also consider to mark this thread as solved (but only if you have really understood all of the information) ;)

ANproCUBE

[evgeric]

HI ANproCUBE,

I have solved the another thread. Thanks for your help.

[ANproCUBE]

Hey

When I said you should mark this as solved I didn't mean you to rename the whole thread :D

Just mark it because if somebody wants to know the exact same thing you wanted to know it's hard to find!

ANproCUBE

[evgeric]

Hi ANproCUBE,

Need your help on this.

http://www.cplusplus.com/forum/general/157761/

[Konstantin2]

ANproCUBE: I'm sorry for my words. You are right. It's all ok.