Issues with if else statement

[Linda Selfridge]

Hi!

I am very new to java programming and am trying to get an if else statement to work in relation to calculating taxation amounts.

Basically I need it to say: If salary is less than XXXX then tax = 0, otherwise if salary is greater than or equal to XXXXX then tax = (calculation), otherwise if salary is greater than or equal to XXXXX then tax = (calculation) etc

But at the moment it is only calculating on the first level of the calculation.

I really need help! My issue is between lines 38 & 47.

// This program is designed to calculate an employee's pay

import java.util.Scanner;
import java.text.DecimalFormat;
import java.text.NumberFormat;

public class PayCalc
{
    public static void main (String [] args)
    {

	Scanner keyboard = new Scanner (System.in);

	Double hoursWorked;
	Double dependents;
	Double grossPay;
	Double adjGross;
	Double mediLevy;
	Double projectedAnnual;
	Double yearTax = 0.0;
	Double weekTax;
	Double weekAssess;
	Double weekTHome;
    Double employSuper;
    Double employeeSuper = 100.00;
	Double childSupport = 0.0;
	NumberFormat formatter = new DecimalFormat("#0.00");

	System.out.print ("Enter the number of hours worked:");
	hoursWorked = keyboard.nextDouble();
	System.out.print ("Enter the number of dependents:");
	dependents = keyboard.nextDouble();

	grossPay = (38*15.50)+((hoursWorked-38)*(15.50*1.5));
	adjGross = grossPay-employeeSuper;
	mediLevy = grossPay*0.015;
	projectedAnnual = adjGross*52;
    	if (projectedAnnual <= 6000) 
	    yearTax = 0.00;
	else if (projectedAnnual >= 6000)  
	    yearTax = ((projectedAnnual-6000)*0.15);
	else if (projectedAnnual >= 25000)  
	    yearTax = (25000-6000)*0.15+((projectedAnnual-25000)*0.30);
	else if (projectedAnnual >= 75000)  
	    yearTax = (25000-6000)*0.15+(75000-25000)*0.30+((projectedAnnual-75000)*0.40);
       	else if (projectedAnnual >= 150000)  
	    yearTax = (25000-6000)*0.15+(75000-25000)*0.30+(150000-75000)*0.40+((projectedAnnual-150000)*0.45);
	  weekTax = yearTax/52;
	  weekAssess = grossPay-employeeSuper;
	  weekTHome = weekAssess-weekTax-mediLevy-childSupport;
	  childSupport = dependents*50.00;
	  employeeSuper = 100.00;
	  employSuper = grossPay*0.09;
	
	System.out.println("Worker's projected yearly gross salary          :"  +formatter.format(projectedAnnual));  
       	System.out.println("Superannuation Contribution");
	System.out.println("Employer contribution                           :"  +formatter.format(employSuper));
	System.out.println("Employee contribution                           :"  +formatter.format(employeeSuper));
	System.out.println("Worker's weekly assessable income               :"  +formatter.format(weekAssess));
	System.out.println("Weekly income tax                               :"  +formatter.format(weekTax));
	System.out.println("Medicare levy                                   :"  +formatter.format(mediLevy));
	System.out.println("Child Support contribution                      :"  +formatter.format(childSupport));
	System.out.println("Take home pay                                   :"  +formatter.format(weekTHome));

	
	}
    
}	

I hope someone can help!

Thanks in advance

[Ernest Friedman-Hill]

Hi,

Welcome to JavaRanch!

Well, one problem is that the tests are going to be executed in order, but they all overlap. For example, if projectedAnnual is 100,000, the program is first going to check if it's less than or equal to 6000, which it's not, but then it's going to check if it's greater than or equal to 6000, which it is, so that piece of code will be executed, and that's it -- no more "else" clauses will be considered, since one was already satisfied. You need to either re-order them, or turn the tests around to be "less than" rather than "greater than", so the smaller tests won't apply for the bigger numbers.

[hypGnosis von Schwarz]

Linda Hi!

First thing that caught my eye was:

What if 'projectedAnnual' equaled exactly 6000? Which 'if' stmt do you want to be followed? (<= 6000 or >= 6000)?

Since 'projectedAnnual' tops out at 150000, perhaps that should be the first amount tested, no?

With an ascending set of values, your code will never get to anything past the 'greater-than' 6000... cuz it's all greater than 6000, i.e., 25000 > 6000, 75000 > 6000, 150000 > 6000.

Flip the sequence & you'll be a happy camper!

[Greg Brannon]

You could accomplish what you're trying to do without them, but you could use multiple tests in your if statements. An advantage is that readability is improved, a disadvantage is that it's adding complexity, perhaps unnecessarily.

e.g.:

else if ( projectedAnnual > 6000 && projectedAnnual <= 25000 )
{
    // do this
}
etc.

[Linda Selfridge]

You guys are legends!

I did what you said and flipped the statements around and it now calculated properly! YAY! YAY! YAY! I was up till 4am this morning trying to fix the issue!

I am forever in everyones debt!

For future reference the code now looks like this....

// This program is designed to calculate an employee's pay

import java.util.Scanner;
import java.text.DecimalFormat;
import java.text.NumberFormat;

public class PayCalc
{
    public static void main (String [] args)
    {

	Scanner keyboard = new Scanner (System.in);

	Double hoursWorked;
	Double dependents;
	Double grossPay;
	Double adjGross;
	Double mediLevy;
	Double projectedAnnual;
	Double yearTax = 0.0;
	Double weekTax;
	Double weekAssess;
	Double weekTHome;
        Double employSuper;
        Double employeeSuper = 100.00;
	Double childSupport = 0.0;
	NumberFormat formatter = new DecimalFormat("#0.00");

	System.out.print ("Enter the number of hours worked:     ");
	hoursWorked = keyboard.nextDouble();
	System.out.print ("Enter the number of dependents:     ");
	dependents = keyboard.nextDouble();

	grossPay = (38*15.50)+((hoursWorked-38)*(15.50*1.5));
	adjGross = grossPay-employeeSuper;
	mediLevy = grossPay*0.015;
	projectedAnnual = adjGross*52;	
	if (projectedAnnual >= 150000)  
	    yearTax = (25000-6000)*0.15+(75000-25000)*0.30+(150000-75000)*0.40+((projectedAnnual-150000)*0.45);
	else 
	    if (projectedAnnual >= 75000)  
		yearTax = (25000-6000)*0.15+(75000-25000)*0.30+((projectedAnnual-75000)*0.40);
	    else 
		if (projectedAnnual >= 25000)  
		    yearTax = (25000-6000)*0.15+((projectedAnnual-25000)*0.30);
		else
		    if (projectedAnnual >= 6000)  
			yearTax = ((projectedAnnual-6000)*0.15);
		    else 
			if (projectedAnnual <= 6000) 
			    yearTax = 0.00;
	  weekTax = yearTax/52;
	  weekAssess = grossPay-employeeSuper;
	  weekTHome = weekAssess-weekTax-mediLevy-childSupport;
	  childSupport = dependents*50.00;
	  employeeSuper = 100.00;
	  employSuper = grossPay*0.09;
	
	System.out.println("Worker's projected yearly gross salary          :"  +formatter.format(projectedAnnual));  
       	System.out.println("Superannuation Contribution");
	System.out.println("Employer contribution                           :"  +formatter.format(employSuper));
	System.out.println("Employee contribution                           :"  +formatter.format(employeeSuper));
	System.out.println("Worker's weekly assessable income               :"  +formatter.format(weekAssess));
	System.out.println("Weekly income tax                               :"  +formatter.format(weekTax));
	System.out.println("Medicare levy                                   :"  +formatter.format(mediLevy));
	System.out.println("Child Support contribution                      :"  +formatter.format(childSupport));
	System.out.println("Take home pay                                   :"  +formatter.format(weekTHome));

	
	}
    
}

Thank you again!

[hypGnosis von Schwarz]

Linda, Hello again!

Hope you check back, because I have one additional comment.

You should remove the statement at line 50: if (projectedAnnual <= 6000)
and simply let the tax bracketing fall through to: yearTax = 0.00 ... the default tax amount

Besides that, what if the taxable amount is exactly 6000? It is referenced with equal signs in both lines 47 and 50.

Drop line 50 & you're on your way.

[Linda Selfridge]

Thanks I will give it a go.

Cheers

[Igor Mechnikov]

Is it safe to assume that hoursWorked number is at least 38?

grossPay = (38*15.50)+((hoursWorked-38)*(15.50*1.5));  

[Darryl Burke]

Maybe that implies that you're also penalized at time-and-a-half if you don't wok a minimum of 38 hours

Good catch, Ilya!