Question:

I'm creating a receipt program in C

by | earlier

I'm creating this C receipt program and i'm trying to loop it using the goto statement but whenever i the program goes back to menu: using the goto statement it seems to bypass the getline statement i used thus it doesn't ask for the string whenever it asks for an order from the menu again, maybe you'll be able to see what's wrong just by looking at the program need tips and advice thanks!

#include <iostream>

#include <string>

using namespace std;

void main ()

{

string product;

char decision;

double price,pcs,total,paid,change=0;

menu:

cout <<"Place Your Order Now: " <<endl;

cout <<"=====================================... <<endl;

cout <<"Enter Product: " ;

getline (cin,product);

cout <<"Enter the Price: P" ;

cin >> price;

cout <<"Enter Number of Pieces: " ;

cin >> pcs;

total = price * pcs;

cout <<"=====================================... <<endl<<endl;

cout <<product;

cout <<endl<<endl;

cout <<"Total Amount Due = P" <<total <<endl;

cout <<"=====================================... <<endl;

cout <<"Enter Payment: P";

cin >> paid;

cout <<"=====================================... <<endl;

change = paid - total;

cout <<"Change = P" << change;

cout <<endl <<endl <<"01 James Lim C example" <<"\n" <<"\n";

cout <<"Would you like to order again? <y> yes <n> no = ";

cin >>decision;

if (decision == 'y' || decision == 'Y')

goto menu;

else if (decision == 'n' || decision == 'N')

goto quit;

else

cout <<"invalid input! \n";

goto menu;

quit:

cout <<"Thank you! Please come again! \n";

}

Tags:

Report

Answer The Question I've Same Question Too

Follow Question

3 ANSWERS

Sort By: Date | Rating

Personally, I'd break it up into 3 functions:

main();

getProductsAndChange();

displayResults();

The reason for this is that the code is easier to read by containing the prompting and validation of what they entered much better within their function scope.

There are also less variables in the main() function, thereby also making it easier to read.

As an example, what if you wanted the customer to be able to enter 1 or more products. Then you can see you would edit the "getProductsAndChange()" function but you could still return the list of products in the "product" variable (potentially separated by commas or some other delimiter). And the resultant "change" variable would be set after the total of all the products was tallied.

This doesn't cause you to have to change anything in main() or in displayResults().

Using goto is a bad habit to get into when using procedural languages. It is rarely necessary as you can see.

void main() {

BOOL bDone = FALSE;

string product;

double change;

   while ( ! bDone) {

      getInput(&product, &change);

      bDone = displayResults(product, change);

   }

}

/*

   This function is used to get the initial information, returning the response by passing variables by reference

   Input values:

      product - used to return the chosen product to the function which called this function

      change - used to return the change the customer receives to the function which called this function

   Return values:

      None

*/

void getProductsAndChange(double &product, double &change) {

   double price,pcs,total,paid;

   count << "Place Your Order Now: " <<endl;

   cout <<"=====================================...

   cout <<"Enter Product: " ;

   getline (cin,product);

   cout <<"Enter the Price: " ;

   cin >> price;

   cout <<"Enter Number of Pieces: " ;

   cin >> pcs;

   total = price * pcs;

   cout <<"Total Amount Due = " <<total <<endl;

   cout <<"=====================================... <<endl;

   cout <<"Enter Payment: ";

   cin >> paid;

   cout <<"=====================================... <<endl;

   change = paid - total;

}

/*

   This function is used to display the results and ask if the customer wants to order again

   Return values:

      TRUE - if and only if the customer wants to continue

      FALSE - otherwise

*/

BOOL displayResults(double product, double change) {

   char decision;

   BOOL bReturn;

   BOOL bValidDecision = false;

   cout <<"=====================================... <<endl<<endl;

   cout <<product;

   cout <<endl<<endl;

   cout <<"Change = " << change;

   cout <<endl <<endl <<"01 James Lim C example" <<"\n" <<"\n";

   cout <<"Would you like to order again? <y> yes <n> no = ";

   cin >>decision;

   while (! bValidDecision ) {

      switch (decision)

      {

         case 'y':

         case 'Y':

         case '1':

            bReturn = TRUE;

bValidDecision = TRUE;

            break;

         case 'n':

         case 'N':

         case '0':

            bReturn = FALSE;

bValidDecision = TRUE;

            break;

         default:

        cout <<"invalid input! \n";

            break;

      }

   }

   return bReturn;

}
Report (0) (0) |   earlier
yes i have experienced same thing a lots of time, basically you need to flush the iostream buffer.

But my suggestion is to use "do - while" loop construct

do

{

cout <<"Place Your Order Now: " <<endl;

// snip

//other code remains same

// snip

cout <<"Would you like to order again? <y> yes <n> no = ";

cin >>decision;

} while (decision !='n' || decision!='N');
Report (0) (0) | earlier
Instead of using that goto statement and Tags u can use do while loop

void main ()

{

string product;

char decision;

double price,pcs,total,paid,change=0;

do

{

cout <<"Place Your Order Now: " <<endl;

cout <<"=====================================... <<endl;

cout <<"Enter Product: " ;

getline (cin,product);

cout <<"Enter the Price: P" ;

cin >> price;

cout <<"Enter Number of Pieces: " ;

cin >> pcs;

total = price * pcs;

cout <<"=====================================... <<endl<<endl;

cout <<product;

cout <<endl<<endl;

cout <<"Total Amount Due = P" <<total <<endl;

cout <<"=====================================... <<endl;

cout <<"Enter Payment: P";

cin >> paid;

cout <<"=====================================... <<endl;

change = paid - total;

cout <<"Change = P" << change;

cout <<endl <<endl <<"01 James Lim C example" <<"\n" <<"\n";

cout <<"Would you like to order again? <y> yes <n> no = ";

cin >>decision;

}

while(decision == 'y' || decision == 'Y');

if (!(decision == 'n' || decision == 'N'))

cout <<"invalid input! \n Closing Application\n";

cout <<"Thank you! Please come again! \n";

}

You might need to format it a bit.But this sure will help

Dude i gotta ask you something too

Which tool you are using for editing and compiling

Pleeeeeeeease reply me on rizvanmotala99@yahoo.co.in

Thanks
Report (0) (0) | earlier