>> Is this right
Looks good to me. A couple of minor issues:
>> I also changed all my floats to double.
You missed a few.
>> pAccount->GetBalance();
This call in main isn't necessary. It doesn't do anything. I know I said to move the Set and Get calls, but I didn't mean to leave in the unnecessary Get here.
>> double balance2 = GetBalance(); return balance2/100 * 5;
There is no need for the extra variable. You can use the return value directly:Code:return GetBalance()/100 * 5;