class help

[HossamH]

This is what i should do
Implement class to represent Bank Account
With Attributes:
- Account Name text with spaces :char *
- Account ID : unique value :int
- Balance :double
- date : Date that represent the creation date of the account
Methods:
- Constructor default and parameterized with name, Account ID, Balance and current date
- Destructor
-Setters and Getters
-View to view Account Name, Account ID, currentbalance and date
-double Deposit(double value)
which returns balance value after adding the amount of deposit.
-double Withdraw( double value)
which calculate balance value after subtracting the amount of Withdraw. if balance >= value compute the amount of withdraw otherwise " Cannot Withdraw From the account , No available balance"

//header file
#ifndef ACCOUNT_H
#define ACCOUNT_H
#include "Date.h"

class Account
{
    private:
        string Account_Name;
        static int Unique_ID;
        double Balance;
        Date Current_Date;

    public:

        Account();

        Account(string acoName, double bal, Date datee );

        void setName(string name);
        void setBalance (double balance);
        void setDate (Date date);

        string getName ();
        double getBalance();
        void getCurrentDate ();
        int getID ();

        void view ();
        double Deposit (double value);
        double Withdraw (double value);







        ~Account();
};
#endif // ACCOUNT_H


//.pp file 


#include "Account.h"
#include <iostream>
using namespace std;

Account :: Account ()
{
    Unique_ID = 1;
}

Account :: Account(string acoName, double bal , Date datee)
{
        Account_Name = acoName;
       // Unique_ID = uniID;
        Balance = bal;
        Current_Date = datee;

}
/**********************************************/
void Account :: setName(string name)
{
    Account_Name = name;

}

void Account ::  setBalance (double balance)
{
    Balance = balance;

}

void Account ::  setDate (Date date)
{
    Current_Date = date ;
}
/*******************************************/
string Account ::  getName ()
{
    return Account_Name;
}

double Account ::  getBalance()
{
    return Balance;
}
int Account :: getID()
{
    return Unique_ID ;
}

void Account ::  getCurrentDate ()
{
    Current_Date.getDate();
}
/*********************************************/
void Account :: view ()
{
    cout<<"Account Balance : "<<getBalance<<"\n Account Date : "<<getCurrentDate<<"\n Account ID : "<<getID<<"\n Account Name : "<<getName<<endl;
}
/************************************************************/
double Account :: Deposit (double value)
{
    return (Balance + value) ;

}
double Account :: Withdraw (double value)
{
    if (Balance >= value)
    {
        return (Balance - value);
    }
    else
        cout<<"Can Not Withdraw From the Account, No available Balance."<<endl;
}

[mutexe]

Line 103, you are calling functions incorrectly. Rather than something like this:
getBalance

you need to call them like this:
getBalance()

getCurrentDate() returns void, so that won't work anyway in line 103 with the << operator.

Your Withdraw function looks a bit dodgy too. You are returning a double, but if your if condition sends you into the else part, then you don't return anything. you could do something like this:

double Account :: Withdraw (double value)
{
    double rtn = 0.0;  // or some sensible default value
    if (Balance >= value)
    {
        rtn = (Balance - value);
    }
    else
    {
        std::cout<<"Can Not Withdraw From the Account, No available Balance."<<std::endl;
     }

     return rtn;
}

also, why have you declared your unique ID as static?

[dhayden]

In addition to what mutexe said, Withdraw() and Deposit() should modify the Balance.