TimeDepositAccount class

[FOX427]

Hi All!
Having an issue with implementing TimeDepositAccount class which has interest rate set in the constructor together with the initial balance. I need to provide a method to get the current balance, method to add the earned interest to the account and withdraw method that removes the entire balance. Note that addInterest method shouldn't have any parameters as we already know it. As far I am here:

public class TimeDepositAccount3 {
	private double interestRate;
	private double balance;
	
	public TimeDepositAccount3()
	{
		interestRate=1.2;
		balance=500;
	}
	public double currentBalance()
	{
		return balance;
	}
	public void addInterest()
	{
		double newBalance = balance*interestRate;
		balance = newBalance;
	}
	public void withdraw()
	{
		balance=balance-balance;
	}

}

public class TimeDepositAccountTester {

	/**
	 * @param args
	 */
	public static void main(String[] args) {
		// TODO Auto-generated method stub
		TimeDepositAccount3 myChecking = new TimeDepositAccount3();
		double bal = myChecking.currentBalance();
		System.out.println(bal);
		myChecking.addInterest();
		System.out.println(bal);

	}

}

the result is:
500.0
500.0
which is not correct, any idea why I am having wrong values? Thanks in advance!

[Fubarable]

How do you propose that the double variable bal will change from line (A) to line (B)?

        TimeDepositAccount3 myChecking = new TimeDepositAccount3();
        double bal = myChecking.currentBalance();
        System.out.println(bal);  // (A)
        myChecking.addInterest();
        System.out.println(bal);  // (B)

in other words where is there a bal = something between those two lines? Note that changing the state of myChecking cannot mathemagically change the value of an unrelated double variable.

[FOX427]

Indeed, they weren't related at all! I changed it to this:

public class TimeDepositAccountTester {

	/**
	 * @param args
	 */
	public static void main(String[] args) {
		// TODO Auto-generated method stub
		TimeDepositAccount3 myChecking = new TimeDepositAccount3();
		double beforeInterest = myChecking.currentBalance();
		System.out.println(beforeInterest);
		myChecking.addInterest();
		double afterInterest = myChecking.currentBalance();
		System.out.println(afterInterest);

	}

}

Thank you!

[Fubarable]

That looks much better!