简单银行账户程序无法正确存储余额

Question

我目前正在完成我的银行账户计划，但在完成的过程中遇到了一些问题。这个问题似乎很容易解决，但我不知道该如何着手解决它。

我将首先包括以下作业：

>

实现一个类库。这家银行有两个对象，即支票和储蓄，这两种类型的账户是在上一次练习中开发的。

实现四种实例方法：

存款（双倍金额，String账户）
提现（双倍金额，String账户）
转账（双倍金额，String账户）
printBalance（）

这里的帐户字符串是“S”或“C”。对于存款或取款，它表明哪个账户受到影响。对于转账，它表示从中取钱的账户；钱会自动转入另一个账户。

唯一的问题似乎是在账户的余额变量中实际存储每个账户的信息。cpp文件。它只是停留在0，这就是为什么我觉得这个问题应该很容易解决。我想我只是忘记了关于类实现的一些非常基本的东西，但这就是为什么我在这里！现在回想起来，我想我的困惑部分来自这样一个事实：我以前实现过类似的程序，但只使用数组而不是变量，我没有遇到同样的问题。数据似乎被存储到数组中，所以这可能是我的问题？代码如下：

账户h:

#include <iostream>
#include <string>
#include <iomanip>      

using namespace std;

class Account
{
public:
    Account();
    Account(double balance);
    void Add(double money);
    void Withdraw(double money);
    double GetBalance();

private:
    double balance; 
};

账户cpp：

#include "Account.h"

//  Penalty Fee Constant
const double PENALTY_FEE = 5.00;

Account::Account()
{
    balance = 0.00;
}

Account::Account(double money)
{
    balance = money;
}

void Account::Add(double money)
{
    balance += money;
}

void Account::Withdraw(double money)
{
    if(money > balance)
        balance += PENALTY_FEE;
    else
        balance -= money;
}

double Account::GetBalance()
{
    return balance;
}

银行cpp：

#include "Account.h"

void deposit(double, string);
void withdraw(double, string);
void transfer(double, string);
void printBalances();

int main()
{
    string accountChoice;
    int selection;
    double transaction = 0;

    //  !!!!!!!!!!!!!!!!!!HAVE TO STILL COMPUTE INTEREST!!!!!!!!!!!!!!!!

    cout << fixed << showpoint << setprecision(2);

    do
    {
        cout << "Please make a selection:" << endl;
        cout << "1.) Deposit" << endl;
        cout << "2.) Withdraw" << endl;
        cout << "3.) Transfer" << endl;
        cout << "4.) Print balances" << endl;
        cout << "5.) Quit" << endl;
        cin >> selection;

        if(selection == 1)
        {
            cout << endl << "Please select the account you would like to perform operations on(S or C):" << endl;
            cin >> accountChoice;
            cout << endl << "Please enter the amount to be deposited:" << endl;
            cin >> transaction;
            cout << endl;

            deposit(transaction, accountChoice);
        }
        else if(selection == 2)
        {
            cout << endl << "Please select the account you would like to perform operations on(S or C):" << endl;
            cin >> accountChoice;
            cout << endl << "Please enter the amount to be withdrawn:" << endl;
            cin >> transaction;
            cout << endl;

            withdraw(transaction, accountChoice);
        }
        else if(selection == 3)
        {
            cout << endl << "Please select the account you would like to perform operations on(S or C):" << endl;
            cin >> accountChoice;
            cout << endl << "Please enter the amount to be transferred:" << endl;
            cin >> transaction;
            cout << endl;

            transfer(transaction, accountChoice);
        }
        else if(selection == 4)
            printBalances();
        else
            cout << "Closing program -- Thank you for using the ATM teller!" << endl;
    }while(selection != 5);


    system("pause");
    return 0;
}

void deposit(double amount, string account)
{
    Account savings, checking;

    if(account == "S" || account == "s")
        savings.Add(amount);
    else
        checking.Add(amount);
}

void withdraw(double amount, string account)
{
    Account savings, checking;

    if(account == "S" || account == "s")
        savings.Withdraw(amount);
    else
        checking.Withdraw(amount);
}

void transfer(double amount, string account)
{
    Account savings, checking;

    if(account == "S" || account == "s")
    {
        savings.Withdraw(amount);
        checking.Add(amount);
    }
    else
    {
        checking.Withdraw(amount);
        savings.Add(amount);
    }
}

void printBalances()
{
    Account savings, checking;

    cout << "The current balance in savings is: " << savings.GetBalance() << endl;
    cout << "The current balance in checking is: " << checking.GetBalance() << endl << endl;
}

Answer 1

每次需要时，您都会创建一个新的Account对象。当然，当您打印它时，它将是0，因为默认构造函数将余额初始化为0。

相反，当应用程序启动时，作为用户来识别他的帐户或其他东西，并创建相应的帐户实例。该实例需要在用户操作它的整个时间内都在那里。

因此，不要在方法中实例化，而是在主函数中实例化。并将实例传递给方法，作为根据需要修改实例的一种方式。

Answer 2

我认为如果你声明另一个类“客户”，并给他们一个名字、客户号码和一个支票和储蓄账户，总体上可能会更清楚。客户应该在流程生命周期的某个地方实例化，这样他们就不会被删除，例如在静态容器中，例如std::map。

自动取款机（抱歉！），您似乎有一些非成员函数实例化帐户，对它们进行处理，然后在函数返回时删除它们。