5 replies to this topic

[lva28]

Hi guys I have a program that compiles but after I enter the while loops which asks the user if they want more information on an item and I enter 'Y' (yes) it keeps asking over and over again for the same thing.Thanks

 

  while(choice=='Y')    {

      cout<<"\nPlease enter the amount of the item";//outputs price

      cin>>amount;//reads price input

      cout<<"\nPlease enter the weight of the item";

      cin>>weight;//reads the weight from user

      bTax+=amount;//calculates price to total

      totalWt+=weight;//calculates total weight

      cout<<"\nEnter information on an item? Y or N";

    }

  return;

}


  

Edited by lva28, 30 November 2011 - 03:58 PM.

[fread]

Choice is always Y; the while condition evaluates as truth, so the loop repeats. You need the while condition to evaluate as false(in this case). Change the value of choice inside of the loop, so that the while condition evaluates as false next time around and terminates.

[lva28]

would I do it like this?

 while(choice=='Y')//while user enters yes then get the price and weight

    {

      cout<<"\nPlease enter the price of the item";//outputs price

      cin>>price;//reads price input

      cout<<"\nPlease enter the weight of the item";//outputs weight

      cin>>weight;//reads the weight from user

      bTax+=price;//calculates price to total

      totalWt+=weight;//calculates total weight

       cout<<"\nEnter information on an item? Y or N";


while(choice=='N')

return 0;

[fread]

while(choice=='Y')//while user enters yes then get the price and weight
    {
      cout<<"\nPlease enter the price of the item";//outputs price
      cin>>price;//reads price input
      cout<<"\nPlease enter the weight of the item";//outputs weight
      cin>>weight;//reads the weight from user
      bTax+=price;//calculates price to total
      totalWt+=weight;//calculates total weight
      cout<<"\nEnter information on an item? Y or N";

    /* change the value of choice whilst you are in the while loop; any where within these curly brackets will be ok */
     choice = 'N';
    }

[lva28]

Thanks problem solved

[agnl666]

while(choice=='Y') // while user enters yes then get the price and weight 
{      
      cout << "\nPlease enter the price of the item"; // outputs price
       cin >> price; // reads price input
       cout << "\nPlease enter the weight of the item"; // outputs weight
       cin>>weight; // reads the weight from user
       bTax+=price; // calculates price to total
       totalWt+=weight; // calculates total weight
       cout << "\nEnter information on an item? Y or N"; 

      cin >> choice;  // this actually gives the user the choice
}

I know that problem was considered solved though the posted code did not actually give the user the choice. The code posted above will.