0

I have been working on this project that simulates a bank account. The user can deposit, withdraw, and have all the withdraws and deposits displayed on the screen. At the top of the selection menu needs to be the current balance. Like so, SAVINGS : 100. And whenever I deposit money or withdraw money I need that balance to change to the correct amount. The amount starts at $100. If I deposit or withdraw money it will work perfectly the first time, but on the second time it gets reset back to $100 and then the transaction is done. How can I make the balance stay the correct amount without it resetting? This is a school project and I am not looking for someone to provide me the code. I am just seeking some tips or guidance in the right direction. Here is the code for my int main() :

int main ()
{
    saving sa;
    creditCard cca;
    checking ca;

    string n;
    int option;
    int exit = 1;
    int x = 1;
    do{
    cout << endl;
    cout << "Checking Balance:" << ca.getBalance() << "       " << "Savings balance:" << sa.getBalance() << "       " << "Credit Card balance:" << cca.getBalance() << endl;
    cout << endl;

    cout << " (1) Savings Deposit " << endl;
    cout << " (2) Savings withdrawel " << endl;
    cout << " (3) Checking Deposit " << endl;
    cout << " (4) Write A Check " << endl;
    cout << " (5) Credit Card Payment " << endl;
    cout << " (6) Make A Charge " << endl;
    cout << " (7) Display Savings " << endl;
    cout << " (8) Display Checkings " << endl;
    cout << " (9) Display Credit Card " << endl;
    cout << " (0) Exit " << endl;
    cin >> option;

    switch ( option )

    {
        case 1 : {
                 double SamtD;
                 cout << " Please enter how much you would like to deposit into savings " << endl;
                 cin >> SamtD;
                 sa.makeDeposit(SamtD);
                 break;
                 };
        case 2 : {
                 int SamtW;
                 cout << " Please enter how much you would like to withdraw "<< endl;
                 cin >> SamtW;
                 sa.doWithdraw(SamtW);
                 break;
                 }
        case 3 : {
                 double CamtD;
                 cout << " Please enter how much you would like to deposit into checkings " << endl;
                 cin >> CamtD;
                 ca.makeDeposit(CamtD);
                 break;
                 }
        case 4 : {
                 double CamtW;
                 int chkNum;
                 cout << " Please enter how much you wrote on the check " << endl;
                 cin >> CamtW;
                 cout << " Please enter the check number " << endl;
                 cin >> chkNum;
                 ca.writeCheck(chkNum, CamtW);
                 break;
                 }
        case 5 : {
                 double CCmkP;
                 cout << " Please enter the amount you would like to deposit " << endl;
                 cin >> CCmkP;
                 cca.makePayment(CCmkP);
                 break;
                 }
        case 6 : {
                 double DoC;
                 string Nm;
                 cout << " Please enter the amount charged to your credit card " << endl;
                 cin >> DoC;
                 cout << " Please enter where the charge was made " << endl;
                 cin >> Nm;
                 getline(cin, Nm);
                 cca.doCharge(Nm,DoC);
                 break;
                 }
        case 7 : {
                 sa.display();
                 break;
                 }
        case 8 : {
                 ca.display();
                 break;
                 }
        case 9 : {
                 cca.display();
                 break;
                 }
        case 0 : exit = 0;
                 break;
        default : exit = 0;
                 cout << " ERROR ";
    }
    }
    while(exit==1);

    return 0;
}

and here is were the balance is being set :

double curbalance = 100;
void account::setBalanceD(double balance) // This is for deposit
{
    itsBalance = balance;
}

void account::setBalanceW(double balance) // This is for withdraw
{
    double newBalance = curBalance - balance;
    itsBalance = newBalance;
}

double account::getBalance()
{
    return itsBalance;
}

and here is the code that option 2 in my menu would be calling :

int saving:: doWithdraw(int amount)

{
    if (amount > 0)
    {
        for (int i = 9; i != 0; i--)
        { 
            last10withdraws[i] = last10withdraws[i-1];
        }
        last10withdraws[0] = amount; 
        setBalanceW(amount);  
    }
    else
    {
        cout << " ERROR. Number must be greater then zero. " << endl;
    }
    return 0;
}

Any ideas? I just cannot get the balance to remain accurate.

1
  • 3
    Never ever use floats or doubles to represent money. They're inherently inaccurate.
    – Cairnarvon
    Apr 18, 2013 at 1:57

2 Answers 2

1

The problem is that your setBalanceW() function never deducts from curBalance, so it stays at 100.

void account::setBalanceW(double balance) // This is for withdraw
{
    curBalance -= balance;
}

Your other references should be to curBalance as well, as there should only be one current balance.

void account::setBalanceD(double balance) // This is for deposit
{
    curBalance += balance;
}

double account::getBalance()
{
    return curBalance;
}

The best solution, though (presuming that itsBalance is a member of the account class, would be to eliminate curBalance altogether, and just open your main() with an initial deposit:

int main ()
{

  saving sa;
  creditCard cca;
  checking ca;

  sa.makeDeposit(100.0);
  /* other code */
}

/* Note no curBalance variable? */
void account::setBalanceW(double balance) // This is for withdraw
{
    itsBalance -= balance;
}
void account::setBalanceD(double balance) // This is for deposit
{
    itsBalance += balance;
}

double account::getBalance()
{
    return itsBalance;
}
0
0

Your implementation for performing a deposit appears to be incorrect. It should be:

void account::setBalanceD(double balance) // This is for deposit
{
    itsBalance += balance;
}

Notice that we are now adding the amount being deposited to the current balance, instead of explicitly setting it's value. Similarly as Ken pointed out the same applies to performing a withdraw.

void account::setBalanceW(double balance) // This is for withdraw
{
    itsBalance -= balance;
}
2
  • I would think setBalanceW is actually the problem as it uses an external variable and isn't based on the current balance at all. Apr 18, 2013 at 2:00
  • I have been slaving over this code for hours.... and a + and - sign change everything to working..... dam c++ language lol
    – Dolbyover
    Apr 18, 2013 at 2:20

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Not the answer you're looking for? Browse other questions tagged or ask your own question.