java help #2

wtf??

remove the ";" behind if(..) and replace it with a {

and also at the end of the if structure put a }

package test;

import java.util.Scanner;

public class test {

public static void main (String args[]){

double money = 0;

Scanner sc = new Scanner (System.in);

double x= sc.nextDouble();
System.out.println("Years: " + x);

double y= sc.nextDouble();
System.out.println("Earnings: " + y);

if (1>x){
System.out.println("No extra money!");}

if (1<=x && x<5){
money=y*0.8;
System.out.println("You will get " +money+ " euro");
}

if (x>=5){
money=y*0.8+500;
System.out.println("You will get " +money+ " euro");
}
}
}

Im not sure about java, but at c# -> if you need to introduce more that 1 argument inside a cicle, you need to use { }, so add them at if cicles. And we dont use ; in front of if ... its a cicle... dunno about java :f

Cant help that much

do

x < 1 instead of 1 > x

x>= 1 instead of 1 <=x

confusing to read

import java.util.Scanner;

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

Scanner input = new Scanner (System.in);

int year;
int earnings=1000;
int bonus=0;



System.out.print("Enter the number of years you have been working: ");
year = input.nextInt();


if (year<1)
System.out.println("You receive no bonus");
else if (year>=1 && year<5)
bonus=(earnings/100)*80;
else
bonus=(earnings/100)*80+500;


System.out.printf("You will earn %d extra.\n", bonus);

}
}

-------


still, don't really understand what you mean by "earnings"..

thx for the help guys helps me to improve

redo the test expressions like so

if (1>x)
else if (1<=x && x<5)
else if (x>=5)

that way the program doesnt have to go through all the branches, so less cpu time

edit with the braces as above thou : ]

if (x != y) { do smth }
else if (x == y) { do smth else }

remove the ; behind the if statements

I don't think you need brackets after an if statement if the code after the if statement is only 1 line.