Some errors popped up....

[college{student}]

Payroll.java:33: not a statement
else(hoursWorked < 0);
^

java:40: 'else' without 'if'
else(hoursWorked > 80);
^


Here is my code:

import java.util.Scanner;

public class Payroll
{

public static void main(String args[] )
{

Scanner input = new Scanner(System.in);


double hourlyRate = 0.0;
double hoursWorked = 0.0;
double pay = 0.0;
double ovtpay = 0.0;

System.out.print("Please Enter Your Name:");
String name = scannerObject.next();
System.out.print("Please Enter Number of Hours Worked This Week: ");
hoursWorked = input.nextDouble();
System.out.println("Please Enter Hourly Rate: ");
hourlyRate = input.nextDouble();

for(int counter=1; counter<=10; counter = counter + 1)
{
if(hoursWorked< 40)
pay = hoursWorked * hourlyRate;

else(hoursWorked < 0);
System.out.printf("Invalid. Please enter a number between 0 and 80");

if(hoursWorked>= 40)
ovtpay = (hoursWorked - 40) * 1.5 * hourlyRate;
pay = pay + ovtpay;

else(hoursWorked > 80);
System.out.printf("Invalid. Please enter a number between 0 and 80");

}

System.out.printf("\n's Totals for the Week: $%.2f\n, \n hours, \n overtime pay", name, pay, hours, ovtpay);
}

}


I am fairly new at Java and don't really know what these errors mean. Any help is great

Thanks,
Ben

[sunde887]

else doesn't require a conditional

it should look like

if(condition){}
else{}

If this condition is true, do this, else do this. The second part doesn't need to pass any test, it simply happens when the first condition is not met.

If you want to test multiple things you need

if(condition){}
else if(condition){}
else{}

You can have as many else ifs as you want and do not need an else.

Another tip.

When working with conditions(if/else)
They take one statement each

if(condition)
  statement
else if(condition)
  statement

Remember a statement can be a single line or a block

//statement block
{
statement 1
statement 2
}

Get into the habit of surounding every response to a conditional with brackets, creating a statement block

if(condition){
 statement 2
 statement 1
}
else if(condition){
 statement 3
}

Im thinking of new things as I review your code.

String name = scannerObject.next();

I dont see where you declared scannerObject, you made the scanner object have the name input so this line should use

input.nextLine();

Also be weary of something like this with conditionals

if(condition){}
if(condition){}
if(condition){}

You may be wanting it to test 1 and only one, however it will test all three, so if all 3 conditions return true it will execute code in each statement block. Make sure you understand the use of, and difference between

if(condition){}
else if(condition){}
else{}

and

if(condition){}
if(condition){}

There are some useful things for modifying items as well, in loops you can increment much quicker with ++

for(int i = 0; i < 10; i++)

is the same as

for(int i = 0; i < 10; i = i + 1)

You also have

int x = 5;
x += 5;

is the same as 

x = x + 5;

There is also *=, /=, -=, %=, all of these make the code a little easier to write

[college{student}]

Thanks a bunch. I knew about some of the stuff you said (x+=5, i++), but my professor had a hint posted and gave the "counter" line of code so I just copied and pasted from his problem. The other things will be very useful going forwards.

[college{student}]

Two more question have surfaced. I am getting to the point to input hourlyRate, and it won't initiate the next step. The cursor just returns down one line and that's where it stays. I also need to add all ten instances of pay into one at the end of the program, and since the for statement is their, I really have no idea how to gather those numbers into a sum.

[Junky]

We don't read minds. Post your latest code and use code tags. Either click the code button or place [ code ] and [ /code ] (without the spaces) before and after your code.

[sunde887]

Why not prompt for all the information in a loop? This way it will ask the user for there input, perform calculations, print results, and continue doing this 10 times. Show me your most up to date code as well. Use code tags as well, surround your code with
[code ](omit the space)
[/code]

[college{student}]

import java.util.Scanner;

public class Payroll
{

public static void main(String args[] )
	{

	Scanner input = new Scanner(System.in);


		double hourlyRate = 0.0;
		double hoursWorked = 0.0;
		double pay = 0.0;
		double ovtpay = 0.0;
		double totalPay = 0.0;
		double allPay = 0.0;

	System.out.print("Please Enter Your Name:");
		String name = input.nextLine();
	System.out.print("Please Enter Number of Hours Worked This Week: ");
		hoursWorked = input.nextDouble();
	System.out.print("Please Enter Hourly Rate: ");
		hourlyRate = input.nextDouble();

	    for(int counter=1; counter<=10; counter = counter++)
			{
	        if(hoursWorked< 40)
	         	{pay = hoursWorked * hourlyRate;}

			else if(hoursWorked>= 40)
	         	{ovtpay = (hoursWorked - 40) * 1.5 * hourlyRate;
	         	totalPay = pay + ovtpay;}
			else
	       		{System.out.printf("Invalid. Please enter a number between 0 and 80");}

			}

	System.out.printf("\n's totals for the Week: $%.2f\n, \n rate \n hours, $%.2f\n overtime pay, and $%.2f\n total pay.", name, pay, hourlyRate, hoursWorked, ovtpay, totalPay);
	
	
	System.out.printf("The total pay for all employees is $%.2f\n.", allPay);
	
	}

}

[Junky]

counter = counter++

Do not do this as it does not do what you think.

int x = 1;
int y = x++;

What value will y have?

[sunde887]

yes, the increment operator can simply be used as

item++;

this increases the value of it by 1.

Try adding what I said in my previous post into your code(prompt inside the loop)

[college{student}]

Will I have to have 10 outputs, one for each name, or will I be able to somehow make an output for each name plus one for all 10 people together?

[sunde887]

There are two approaches I can suggest, if you understand 2d arrays you can use a 10xX 2d array to store the data. However a more elegant solution may be to create a people class which can also perform all calculations as methods; then store an array of people objects. Once you have the data stored it's easy to loop through and perform the appropriate action.