[jjl]

make a instance of your class like this

account myAccount;

then call a function like so

myAccount.makeDeposit(balance, deposit);

[amdcpp]

After they enter their name and account number, you would
make an object instantiation from the account class.

You should code a constructor in the public section which
takes those two arguments (as const) and within that
constructor, should set default values for their balances,
most likely zero unless you give them a sign up bonus.

[jjl]

ok thats your assignement, but whats your question? i showed you how to make an instance of your class

[amdcpp]

In your makeDeposit() and makeWithdrawl() methods, you are passing the balance to
them, but you should only pass the deposit, withdrawl data members:

void account::makeDeposit(const double deposit) {
     balance += deposit;
}

The person shouldn't have to enter their balance if they wanted to make a deposit
or a withdrawl.

If you want to get their balance or name, you can make a couple getter methods:

void getName() const {
     cout << "Account Name: " << name;
}

In main(), you would just call getName():

someonesAccount.getName()

This post has been edited by amdcpp: 10 April 2010 - 04:04 AM

[CTphpnwb]

I would make the functions inline:

#include<iostream>
#include<string>
using namespace std;

class accounts
{
private:
	double balance, deposit, withdrawal, checkingBalance, savingsBalance;
	string name, password, AccNum;
	int Acctype;
	
public:
	void setaccount(string n, string Acc,string pwd)
	{
		name = n; 
		AccNum = Acc;
		checkingBalance = 0;
		savingsBalance = 0;
		password = pwd;
	}
	
	bool check_acc(string n, string pwd)
	{
		if (n == name && pwd == password) {
			return true;
		}
		return false;
	}
	void displayAccInfo ()
	{
		cout<<name << "\t pasword: " << AccNum <<endl; 
		cout<<"Your checking account balance is: "<<checkingBalance<<endl;
		cout<<"Your savings account balance is: "<<savingsBalance<<endl;  
		
	}
	
	void makeDeposit(double deposit, int Acc_type)
	{
		if (Acc_type == 1) {
			savingsBalance += deposit;
		} else if (Acc_type == 2) {
			checkingBalance += deposit;
		}
	}
	void makeWithdrawal(double withdrawal, int Acc_type)
	{
		if (Acc_type == 1) {
			savingsBalance -= withdrawal;
		} else if (Acc_type == 2) {
			checkingBalance -= withdrawal;
		}
	} 
	
} accs;

accounts customers[5];


int main() 
{

	string n[5] = {"Joe", "Bob", "Mary", "Sally", "Jo"};
	string A[5] = {"aab", "bbb", "ccc","ddd","eee"};
	string p[5] = {"123", "456", "789", "abc", "def"};
	for (int i = 0; i < 5; i++) {
		customers[i].setaccount(n[i],A[i],p[i]);
	}
	
	for (int i = 0; i < 5; i++) {
		customers[i].displayAccInfo();
	}
		
	return 0; 
}

That should get you going.

Remember to keep it simple!

[CTphpnwb]

Jermaine, on 11 April 2010 - 06:05 PM, said:

What to make them inline but it's a class assignment and I'm not allowed to

Ok then, but I would still use an array of objects:

#include<iostream>
#include<string>
using namespace std;

class accounts
{
private:
	double balance, deposit, withdrawal, checkingBalance, savingsBalance;
	string name, password, AccNum;
	int Acctype;
	
public:
	bool check_acc(string n, string pwd);
	void displayAccInfo ();
	void makeDeposit(double deposit, int Acc_type);
	void makeWithdrawal(double withdrawal, int Acc_type);
	
	void setaccount(string n, string Acc,string pwd)
	{
		name = n; 
		AccNum = Acc;
		checkingBalance = 0;
		savingsBalance = 0;
		password = pwd;
	}
		
};

bool accounts::check_acc(string n, string pwd) {
	if (n == name && pwd == password) {
		return true;
	}
	return false;
}

void accounts::displayAccInfo () {
	cout<<name << "\t pasword: " << AccNum <<endl; 
	cout<<"Your checking account balance is: "<<checkingBalance<<endl;
	cout<<"Your savings account balance is: "<<savingsBalance<<endl;  
	
}

void accounts::makeDeposit(double deposit, int Acc_type) {
	if (Acc_type == 1) {
		savingsBalance += deposit;
	} else if (Acc_type == 2) {
		checkingBalance += deposit;
	}
}

void accounts::makeWithdrawal(double withdrawal, int Acc_type) {
	if (Acc_type == 1) {
		savingsBalance -= withdrawal;
	} else if (Acc_type == 2) {
		checkingBalance -= withdrawal;
	}
} 



accounts customers[5];


int main() 
{

	string n[5] = {"Joe", "Bob", "Mary", "Sally", "Jo"};
	string A[5] = {"aab", "bbb", "ccc","ddd","eee"};
	string p[5] = {"123", "456", "789", "abc", "def"};
	for (int i = 0; i < 5; i++) {
		customers[i].setaccount(n[i],A[i],p[i]);
	}
	
	for (int i = 0; i < 5; i++) {
		customers[i].displayAccInfo();
	}
		
	return 0; 
}

[amdcpp]

Make sure when you change the public method
declaration in the class header file, that you
change it in the implementation file also. For
example, if the code here

void account::makeDeposit(const double d) {
     balance += d;
}

is put in the header file, it will automatically
make it an inline method because you defined it in
the header. If you aren't allowed to use inline
methods, then just type in the header this:

void account::makeDeposit(const double d);

And in the implementation file (.cpp) you would code
the definition for that method:

void account::makeDeposit(const double d) {
     balance += d;
}

The signatures must be the same in each file. By
signature I mean the

void account::makeDeposit(const double d)

// Returns: void
// Takes: const double

If the signature in your header file takes two doubles,
and the signature in your implementation file takes
only one double, the compiler will give you an error
as they will be two seperate methods; one with a
declaration and no definition, and the other with a
definition and no declaration.

This post has been edited by amdcpp: 11 April 2010 - 04:53 PM

[jjl]

look for the definition of that function, if its not defined that means you have not created it yet, or you have a parameter mismatch. Both should have the same amount of parameters.