boolean and loop statements

[Tavalya]

Hi
Need help please! I am stuck :(

I am trying to write a program where functions are involved to validation user input and prints final sales total. A beginner thus working step-step. below is the product list and after it is my half written program.

................
cout << "White Cheese:\t\t\t$2.50\n";
cout << "White Egg:\t\t\t$2.70\n";//cannot be toasted
cout << "White Ham:\t\t\t$3.00\n" << endl;
cout << "Sourdough Cheese:\t\t$3.00\n";
cout << "Sourdough Egg:\t\t\t$3.20\n";//cannot be toasted
cout << "Sourdough Ham:\t\t\t$3.50\n" << endl;
cout << "Roll Cheese:\t\t\t$3.50\n";//cannot be toasted
cout << "Roll Egg:\t\t\t$3.70\n";//cannot be toasted
cout << "Roll Ham:\t\t\t$3.90\n\n" << endl << endl;//cannot be toasted
cout << "NOTE!!\t\nSandwiches can be toasted EXCEPT if they are Eggs or Rolls.\nExtra 50 cent charge for toasting\n";
...........................

my written program

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

string s_Name[] = {"WCheese", "WEgg", "WHam", "SCheese", "SEgg", "SHam", "REgg", "REgg", "RHam"};
double const S_COST[] = {2.50, 2.70, 3.00, 3.20, 3.50, 3.70, 3.90};
double const TOASTED = 0.50;

void check_input(string s_name[], double S_COST[], int input, int size, string choose)
{
bool found = false;

int i = 0;

while(i < size && !found) // Look for element in the array
{
if(input == S_COST[i] && choose == s_Name[i])
{
found = true; // value found in array

if(found)
{
cout << s_Name[i] << S_COST[i];
}
return 0;
int main()
{

cout << "Please enter sandwich type: " ;
cin >> choose;

cout << "Please enter sandwich amount: " ;
cin >> input;

cout << "Total amount due is: " << endl;


return 0;
}
......................

Try running it and see the error thereof

Thank you for the help!

[gunnerfunner]

Asking customers or staff to type in the name of the sandwich is inviting trouble - spelling mistakes, mixing up upper and lower-case are bound to happen. It's better you spell out the sandwich names yourself and provide a menu to choose numbers corresponding to each sandwich type. That way there's less chances of errors and you can even check against invalid input (out of range numbers). So my suggestion would be on the lines of:

#include<iostream>
using namespace std;

int main(){
	
	bool fQuit = false;
	int choice;
	
	while(!fQuit){
		
		cout<<"Please choose one of the following sandwich options: "<<endl;
		cout<<"1. White Cheese"<<endl;
		cout<<"2. White Egg"<<endl;
		cout<<"3. Exit"<<endl;
		
		cin>>choice;
		switch (choice){
			case 1:{
				cout<<"Please enter quantity: "<<endl;
				int quantity;
				cin>> quantity;
				cout<<"Please choose: 1. Toasted (50 p per sandwich), 2. Non-toasted"       <<endl;
				int toasted;
				cin>> toasted;
				if (toasted ==1){
					cout<<"Total cost is: "<<quantity * (2.50 + 0.50)<<endl;
					} 
				else{
					cout<<"Total cost is: "<<quantity * 2.50<<endl;
					}
				}
					break;
			case 2: {
				cout<<"Please enter quantity: "<<endl;
				int quantity;
				cin>> quantity;
				cout<<"Total cost is: "<<quantity * 2.70<<endl;
				} 
					break;
			case 3: 
				cout<<"You've chosen to exit, Goodbye!"<<endl;
				fQuit = true;
				break;
			default: 
				cout<<"Please choose valid option number!"<<endl;
				break;
			}
	}
}

A few things to note:
1. You could ask for the sandwich quantity outside of the switch, right after user inputs sandwich type, to avoid typing it for each sandwich option but then in case user wants to quit program they'd still have to enter some notional quantity to be able to do so;
2. Declaring variables like quantity, toasted inside cases means that each case needs to have braces { .. } surrounding it so as to scope the variables to the case itself;
3. If you're really paranoid you can even guard against someone choosing a number that is not 1 or 2 for the toasted option by having a nested switch

[Tavalya]

THANK YOU VERY MUCH!! Appreciate your help