Те же аргументы в членах класса: Упростить?

Мне было интересно, если у вас есть члены класса в C ++ с одинаковыми аргументами, есть ли способ упростить это? Например:

#include <iostream>
using namespace std:

class bankAccount
{
public:
bankAccount()
{
privAcct = "MyAccount";
privPin = "MyPin";
{
void changeBalance(string acct, string pin)
{
if(acct == privAcct && pin == privPin)
{
cout << "YAY! You can do this!" << endl;
}
}
void otherMethod(string acct, string pin)
{
if(acct == privAcct && pin == privPin)
{
cout << "YAY! You can do this!" << endl;
}
}
private:
string privAcct, privPin;
};

Как вы можете видеть, они оба принимают одни и те же аргументы и оба должны иметь одинаковые условия, чтобы иметь доступ к «мясу» метода.

Хотя я подумал, что было бы неплохо создать один метод, а затем использовать оператор switch для доступа к различным частям метода, я хочу иметь возможность доступа к части «changeBalance» или «otherMethod» класс. Я просто не был уверен, есть ли способ сделать это немного «аккуратнее» или упростить это?

Весь мой код такой:

#include <cstdlib>
#include <iostream>
#include <string>

using namespace std;
void accountInfo();

class account{
public:
//constructor
account()
{
balance = 0.00;
accountNum = "0303";
pin = "2222";
}
//default for wrong entry response
void wrongEntry()
{
cout << "Wrong account number or PIN. Please try again.\n" << endl;
}
//change pin number
void changePin(string actNum, string oldPin)
{
if(actNum == accountNum && oldPin == pin)
{
string newPin;
cout << "Please enter in a new pin: ";
cin >> newPin;
pin = newPin;
cout << "Thank you." << endl;
}
else
{
wrongEntry();
}
}
//change balance
void changeBalance(string actNum, string pinnum)
{
if(actNum == accountNum && pinnum == pin)
{
double newAdd;
cout << "Your current balance is " << balance << "\nPlease enter the additional amount you would like to deposit: ";
cin >> newAdd;
balance += newAdd;
cout << "Your new balance is " << balance << ".\nThank you.\n" << endl;

}
else
{
wrongEntry();
}
}
//print balance and account #
void printBalance(string actNum, string pinnum)
{
if(actNum == accountNum && pinnum == pin)
{
cout << "For account #"<< accountNum << endl;
cout << "Your current balance is $" << balance << ".\nThank you.\n";
}
else
{
wrongEntry();
}
}

private:
string accountNum, pin;
double balance;
};

int main ()
{
int input;
string aN, pin;
account bo;

while(1)
{
cout << "Please enter account number: ";
cin >> aN;
cout << "Please enter corresponding PIN: ";
cin >> pin;

///options
cout << "Please choose from the following options:\n";
cout << " 1. View Balance\n 2. Deposit\n 3. Change pin\nChoice: ";
cin >> input;
cout << endl;
switch(input)
{
case 1:
bo.printBalance(aN, pin);
break;
case 2:
bo.changeBalance(aN, pin);
break;
case 3:
bo.changePin(aN, pin);
break;
default:
cout << "The information you entered does not seem to match our records.\nPlease try again.\n";
break;
}
char response;
cout << "\nAre you done with your transaction?: Y / N";
cin >> response;
cout << endl;
if(response == 'Y' || response == 'y')
{
return 0;
}
}
}

Я понимаю, что должен поместить свой класс в отдельные заголовочные файлы вместе с объявлением конструктора должным образом, но я просто пытаюсь упростить код.

Спасибо.

0

Решение

Вы можете инкапсулировать «ограниченный» интерфейс в другой тип и разрешить создание этого типа только с действительными учетными данными. Что-то вроде

class Account {
public:
struct Accessor {
Accessor(Account & a, string acct, string pin) : account(&a) {
if (acct != a.privAcct || pin != a.privPin) {
throw BadCredentials();
}
}

void changePin() {account->changePin();}
// and other functions

Account * account;
};
private:
string privAcct, privPin;

// Only accessible through an Accessor
void changePin();
// and other functions
};

Использование:

Accessor(account, acct, pin).changePin();

Или, если весь интерфейс ограничен, вы можете просто поместить проверку в конструктор Account сам.

1

Другие решения

Вы можете создать новый класс LoginCredentials который содержит acct а также pin как частные члены, так и функция-член, которая будет проверять равенство.

3

Вы можете думать об этом как о транзакциях.

(Псевдо-код / ​​дизайн)

transactionValid  = BeginTransation(account, pin);
if ( transactionValid )
{
changePin(...);
chackBalance(...);
}
EndTransaction();
1
По вопросам рекламы [email protected]