Thread: Small Problem with Function

CHECKBOOK PROGRAM

I have almost everything that i want to work, working right, except I have no idea how to accumulate the value of the Balance after a Deposit, Withdrawal, etc.. How do I put that in main? or does it go in main? Any Suggestions?

Code:

#include <iomanip.h>
#include <stdlib.h>

float Dep(float Bal, float Amount)
     { return (Bal+Amount);}

float Check(float Bal, float Amount)
     { return (Bal-Amount);}

float Charge(float Bal, float Amount)
     { return (Bal-Amount);}

float Withdraw(float Bal, float Amount)
     { return (Bal-Amount);}


main()
 {  float Bal, Amount; char Comm;

    cout<<"\n BALANCING CHECKBOOK PROGRAM ";
    cout<<"\n --------------------------- ";
    cout<<"\n\n My Account ";
    cout<<"\n Enter Starting Balance : "; cin>>Bal;
    cout<<"\n Command Options: "
        <<"\n d- deposit"
        <<"\n c- check "
        <<"\n s- service charge "
        <<"\n w- withdrawal "
        <<"\n x- exit "<<endl;
    cout<<"\n Enter command: "; cin>>Comm;
    cout<<"\n Enter amount in dollars and cents: "; cin>>Amount;
     while(Comm != 'x')
       {       cout<<"\n Enter another command: "; cin>>Comm;

              if(Comm == 'd')
              {Dep(Bal, Amount);cout<<"\n Enter amount in dollars and cents: "; cin>>Amount;}
         else if(Comm == 'c')
              {Check(Bal,Amount);cout<<"\n Enter amount in dollars and cents: "; cin>>Amount;}
         else if(Comm == 's')
              {Charge(Bal,Amount);cout<<"\n Enter amount in dollars and cents: "; cin>>Amount;}
         else if(Comm == 'w')
              {Withdraw(Bal,Amount);cout<<"\n Enter amount in dollars and cents: "; cin>>Amount;}
         else if(Comm == 'x')
              {break;}

       }
         cout<<"\n Your New Balance is: $"<<Bal;
      system("PAUSE");
      return 0;
}

Thanks in advance

You're returning a value, use it.

>Dep(Bal, Amount);
should be
Bal = Dep ( Bal, Amount );

-Prelude

With all the time i spent on trying to figure out my first functions program, i overlooked the fact that it doesn't calculate right. When I run the program, it doesn't seem to do the corresponding actions..example:I start the balance at 25 dollars. I deposit 10 dollars, then withraw 3 dollars and I come up with 15 dollars as the new balance. Can you see what I'm not seeing?

Code:

#include <iomanip.h>
#include <stdlib.h>

float Dep(float Bal, float Amount)
     { return (Bal+Amount);}

float Check(float Bal, float Amount)
     { return (Bal-Amount);}

float Charge(float Bal, float Amount)
     { return (Bal-Amount);}

float Withdraw(float Bal, float Amount)
     { return (Bal-Amount);}


main()
 {  float Bal, Amount; char Comm;

    cout<<"\n BALANCING CHECKBOOK PROGRAM ";
    cout<<"\n --------------------------- ";
    cout<<"\n\n My Account ";
    cout<<"\n Enter Starting Balance : "; cin>>Bal;
    cout<<"\n Command Options: "
        <<"\n d- deposit"
        <<"\n c- check "
        <<"\n s- service charge "
        <<"\n w- withdrawal "
        <<"\n x- exit "<<endl;
    cout<<"\n Enter command: "; cin>>Comm;
    cout<<"\n Enter amount in dollars and cents: "; cin>>Amount;
     while(Comm != 'x')
       {       cout<<"\n Enter another command: "; cin>>Comm;

              switch(Comm)
              {case 'd': Bal=Dep(Bal, Amount);
                    cout<<"\n Enter amount in dollars and cents: "; cin>>Amount; break;
               case 'c': Bal=Check(Bal,Amount);
                    cout<<"\n Enter amount in dollars and cents: "; cin>>Amount; break;
               case 's': Bal=Charge(Bal,Amount);
                    cout<<"\n Enter amount in dollars and cents: "; cin>>Amount; break;
               case 'w': Bal=Withdraw(Bal,Amount);
                    cout<<"\n Enter amount in dollars and cents: "; cin>>Amount; break;
               default:
                    cout<<"\n You must enter one of the lower case commands ";continue;}

       }
         cout<<"\n Your New Balance is: $"<<Bal<<endl;

      system("PAUSE");
      return 0;
}

That's because in your case statements, you're calling the fuction before you even have the value inputted.

haha..good one biosx...lol. I was planning for the future when I came up with my username. Hopefully, the future is not too far a way. A little optimism can't hurt right?