🚀 go-pugleaf

"David Eynon" <david.eynon@verizon.net> wrote in message
news:wk0_9.474$Pe7.97@nwrddc04.gnilink.net...
> //NOTES: When account number is entered as a large number in the
beginning,
> the program
> // terminates. When account number is entered as a large number in the
> middle
> //  an infinite loop occurs. Why is this?
>
> #include <iostream>
>
> using std::cout;
> using std::cin;
> using std::endl;
>
> int main()
> {
>   int acct; //account number
>   double begbal = 0;
>   double tcharges = 0; //total charges
>   double tcredits = 0; //total credits
>   double limit = 0;  //credit limit.
>
>   cout << "Enter account number (-1 to stop)";
>   cin >> acct;
>
>   while (acct > 0)
>   {
>    cout << "Enter beginning balance:";
>    cin >> begbal;
>
>    cout << "Enter total charges:";
>    cin >> tcharges;
>
>    cout << "Enter total credits:";
>    cin >> tcredits;
>
>    cout << "Enter your credit limit:";
>    cin >> limit;
>
>    if ((begbal + tcharges - tcredits) > limit)
>    {
>     cout << "Account number: " << acct << endl;
>     cout << "Balance: " << (begbal + tcharges - tcredits) << endl;
>     cout << "Credit limit of " << limit << " has been exceeded"<<
endl;
>     //cout << endl;
>    } // end if
>
>    //next "Cycle"
>    cout << endl;
>    cout << "Enter account number (-1 to stop)";
>    cin >> acct;
>   } //while
>
>
>    return 0;
>
> } //main

You're not initializing 'acct' before use, and you're not checking for
error
conditions (for example, if you entered a thousands separator comma in
the
entered number).

For clearing an erroneous entry,
http://www.parashift.com/c++-faq-lite/input-output.html#faq-15.3 (a
relatively recent addition to the FAQ), if( !(cin >> yourVariable) )
    std::cout << "That's not a number; ";
     std::cin.clear();
     std::cin.ignore(std::numeric_limits<int>::max(), '\n');

Best Regards, John E.



      [ Send an empty e-mail to c++-help@netlab.cs.rpi.edu for info ]
      [ about comp.lang.c++.moderated. First time posters: do this! ]