How to display "Insufficient amount"

[hacikho]

Hello I am woriking on a bank account program, it accept balance amount, deposit amount and wihtraw amount, as a result of this display account balance. I want to add a feature on my withraw method, if withraw amount is greater that balance display "Insufficient amount" message. Below is my withraw method;

//Withraw method
    public double withraw()
    {  
    	//Check if we can do withraw
    	if (balance >= withraw)
    	{
    		balance = balance - withraw;
    		return withraw;
    	}
    	else
    		return 0.0;
    }

I don't know where to place

System.out.println("Insufficient amount");

I tried before and after return 0.0; however I did not see message when I tried to withraw amount bigger than balance .
If you request all codes I can post it.

[KevinWorkman]

Placing the print statement after the return statement will do nothing, and in fact should result in a compiler error (I think, unless it's just a warning, either way, it won't help).

We don't need to see all of your code, but it would help to see what you tried with the print statements. How are you calling this method? What are you doing with the return value?

[JosAH]

Hello I am woriking on a bank account program, it accept balance amount, deposit amount and wihtraw amount, as a result of this display account balance. I want to add a feature on my withraw method, if withraw amount is greater that balance display "Insufficient amount" message. Below is my withraw method;

//Withraw method
    public double withraw()
    {  
    	//Check if we can do withraw
    	if (balance >= withraw)
    	{
    		balance = balance - withraw;
    		return withraw;
    	}
    	else
    		return 0.0;
    }

I don't know where to place

System.out.println("Insufficient amount");

I tried before and after return 0.0; however I did not see message when I tried to withraw amount bigger than balance .
If you request all codes I can post it.

A bank account doesn't print anything; it just has to keep an amount of money; if I were you I'd implement a simple method, say, isSufficient(double amount) that returns a boolean value indicating that the account has enough money to withdraw 'amount' money. Something else that wants to print something can call this method and print something if necessary; this is called 'separation of concerns and/or responsibilities'.

kind regards,

Jos

[hacikho]

is this what you mean?

//Withraw method
    public double withraw()
    {  
    	//Check if we can do withraw
    	if (balance >= withraw)
    	{
    		balance = balance - withraw;
    		return withraw;
    	}
    	else
    		//Can not do withraw
    		return  isSufficient();
    }
    
    public String isSufficient()
    {
    	System.out.println(" InSufficient amount");
    }

[JosAH]

is this what you mean?

//Withraw method
    public double withraw()
    {  
    	//Check if we can do withraw
    	if (balance >= withraw)
    	{
    		balance = balance - withraw;
    		return withraw;
    	}
    	else
    		//Can not do withraw
    		return  isSufficient();
    }
    
    public String isSufficient()
    {
    	System.out.println(" InSufficient amount");
    }

No that is not what I mean; I mean this:

public boolean isSufficient(double amount) {
   return balance >= amount;
}

... and if something else wants to print something it should use this little method on an Account.

kind regards,

Jos

[KevinWorkman]

is this what you mean?

No. Firstly, that doesn't compile: you can't return a String from a method that returns a double. And you can't not return a String from a method that is supposed to.

Secondly, it doesn't separate the bank features from the display features.

I think you should take a step back. Where were you putting the print statement originally that it didn't work?