如果语句打印其他值

Question

Testing environment: Visual Studio 2010 Ultimate.

Hopefully my question was clear enough--my objective is to have a user select an option

1. Enter 1 for Chequing
2. Enter 2 (or any number) for Savings

so whichever option the users chooses, the program will print " Account Type: Chequing [or] Savings "

I've managed to accept the user's input and get half of what I was trying done. My issue is any number i choose, an additional number gets printed, see picture.

The picture is from a previous build, I have fixed the selection issue but the additional number still gets printed under Account Type.

my account.cpp (include account.h):

int  Account::WhichOne(int actype)
{   
    if (actype == 1)
    { 
        cout << "Account Type: Chequing" << endl;

    }
    else
        cout << "Account Type: Savings" << endl;

    return actype;

};

my main.cpp

int main()
{
    char firstName[255];
    char lastName[255];
    char sinNumber[255];
    double balance = 0;
    int choice;
    int checker;
    int accountType = 0;
    int transactions = 0;



    //Retrieve client information
    cout << "Please fill out the information below for registration:" << endl;
    cout << "Enter first name:      ";
    cin.getline(firstName, 255);
    cout << "Enter last  name:      ";
    cin.getline(lastName, 255);
    cout << "Enter SIN number:      ";
    cin.getline(sinNumber, 255);
    cout << "Enter initial balance: ";
    cin >> balance;
    cout << "Enter account type:\n       Chequing - 1\n        Savings - 2\n            Choice:    ";
    cin >> choice;
    cout << "\nPlease wait..." << endl;

    //Creating the account
    Account account(firstName, lastName, sinNumber, balance, accountType, transactions);
    double deposit;
    double withdraw;
    double amount;
    double ammount;
    cout << "Amount to deposit:     ";
    cin >> amount;
    deposit = account.DepositAmt(amount);
    cout << "Your new balance is:   " << deposit << endl;
    cout << "Amount to withdraw:    ";
    cin >> ammount;
    deposit = account.WithdrawAmt(ammount);
    cout << "Your new balance is:   " << deposit << endl;
    accountType = account.WhichOne(choice);
    cout << "" << accountType << endl;
}

I'm really not sure what to do. I'm positive my issue with the return actype but no matter what return value i put there it gets printed as well.

Answer 1

You do this (second file):

accountType = account.WhichOne(choice);

And then you do this (first file):

return actype;

But, actype is your argument, so that's what gets returned and what is printed eventually by the command cout << "" << accountType << endl; . The previous printing, namely Account Type: Chequing is done inside the method, as you can notice in the method's body.

I could help you fix your code, the problem is that I do not know how you would like your code fixed.

EDIT: Since you are defining the item account and accountType is in the constructor, I suppose you store it in a member variable. Let's suppose that variable is called accountType as well. I will change your WhichOne function as follows (I'm changing the name to a more meaningful one):

std::String Account::getAccountType()
{
  if (accountType == 1)
  {
    return "Account Type: Chequing";
  }
  else
    return "Account Type: Savings";
  }
}

and then replace these lines of code

accountType = account.WhichOne(choice);
cout << "" << accountType << endl;

with this one

cout << account.getAccountType() << endl;

Answer 2

As I am able to understand the requirement, following code should work:

account.cpp

  string  Account::WhichOne(int actype)
    {   
       string type;
        if (actype == 1)
         { 
               cout << "Account Type: Chequing" << endl;
               type="Chequing Account";

          }
    else{
              cout << "Account Type: Savings" << endl;
              type= "Saving Account";
          }

return type;

    };

main.cpp

       string accountType;

       accountType = account.WhichOne(choice);
       cout << "" << accountType << endl;

Answer 3

void  Account::WhichOne(int actype)
{   
    if (actype == 1)
    { 
        cout << "Account Type: Chequing" << endl;

    }
    else
        cout << "Account Type: Savings" << endl;

}

the problem is that you are returning the int value and it gets printed in the last line of your code, make it void if you are printing inside the function or just return a string containing the message if you want to print the message somewhere else