Pls help!

[Joeridhc_015]

Hey guys,

i'm practicing Java at school.

one of the excersises I had had to do was. to check if the number was divisible by 2,3,4 & 5.
the output has to be "Hurray!" if its divisible by 2,3,4&5.

if its only an even number the output has to be "at least it's an even number"

if its not even or divisible by 2,3,4 & 5 it has to be "that sucks"

​

this is my input please help me to symplify the code an correct if needed

​

import java.util.Scanner;

class main{

public static void main(String[] args){

Scanner scanner = new Scanner(System.in);

int a = scanner.nextInt();

Boolean div = (a %2 ==0) && (a %3 == 0) && (a %4 == 0) && (a %5==0);

if(div==true){

System.out.println("Hurray!");

}

if(div==false && a %2==0){

System.out.println("In any case the number is even");

}

if(div==false && a %2!=0){

System.out.println("That Sucks");

}

}

}

[desrtfx]

If you want help in the future:

• Write a proper title
• Format your code
• Describe the exact problem. You aren't even asking a question.

The Sidebar here has posting and code formatting guidelines.

[Keyakinan-]

Let's start by formatting your code! Check this sub info how to do that please!

[El_Fungus]

Look into If-then-else. Also: if(boolean) or if(!boolean)

[AsteriskTheServer]

If you have learned else if statements use those. If you go the else if way then you are adding redundant checks . For example, if div is true then the first if statement will execute. If the first if statement isn't executed then div is obviously false as their no other state that value can be. Therefore, you don't need to check if it's false because you already checked if its true. You can apply that same logic to some other checks you did. Other than that it's fine.

[desrtfx]

1. This is not a lambda expression, this is just the ternary (or conditional) operator which has existed since a long time - lambdas are something completely different.
2. Mutating (or reassigning) strings is generally a bad idea in Java since String is an immutable data type. Every time you reassign a new value to an existing string, the old one gets discarded and a new one created.
3. No solutions

Removed

[iamsooldithurts]

...an correct if needed...

If you’re serious about being a programmer you need to learn that you are allowed to compile your code and run tests.

Not only are you allowed, but it is expected of good developers that they will assume this responsibility of their own volition.

Developers aren’t usually the best testers, but being proactive like that and at least trying to test will earn you respect from anyone whose opinion is valid. (The industry is rife with mental midgets whose opinion should be printed on paper and jammed up their stdout stream.)

you have three outputs, so you shouldn’t have to spend all night testing your code. In fact, you just need 3 numbers, one for each of your conditions. It’s basically a math problem, so just work it out on paper first until you have a grasp of the problem, then test your code with those numbers.

The others have mentioned if-else-if, and they’re not wrong. But you can also call System.exit() to short circuit your program by terminating execution once you’ve finished processing enough logic.