Thread: arranging high to low

im a complete noob. i know this is a completely inefficient code, but im curious as to why this program doesnt work. Basically suppose to organize large integers to small integers. it only works if i input 5,4,3,2,1, but stops when i input in other order. essentially only the first set of if statements works..

can someone point out how to fix it ... in something i can easily understand.

im learning from the book java how to program 9th edition deital..
chapter 2, Write an application that reads five integers and determines
and prints the largest and smallest integers in the group. Use only the programming techniques you learned in this chapter.

 
import java.util.Scanner;
 
public class Arrange
{
	public static void main(String [] args)
	{
		Scanner input = new Scanner(System.in);
 
		int number1;
		int number2;
		int number3;
		int number4;
		int number5;
 
		System.out.print("Enter your first number: ");
		number1= input.nextInt();
 
		System.out.print("Enter your second number: ");
		number2= input.nextInt();
 
		System.out.print("Enter your third number: ");
		number3= input.nextInt();
 
		System.out.print("Enter your forth number: ");
		number4= input.nextInt();
 
		System.out.print("Enter your fifth number: ");
		number5= input.nextInt();
 
 
 
		if (number1 > number2)	
		if (number1 > number3)
		if (number1 > number4)
		if (number1 > number5)
 
		if (number2 > number3)
		if (number2 > number4)
		if (number2 > number5)
 
		if (number3 > number4)
		if (number4 > number5)
 
			System.out.printf("%d,%d,%d,%d,%d", number1, number2, number3, number4, number5);
 
		else if (number2 > number3)
		if (number2 > number4)
		if (number2 > number5)
		if (number2 > number1)
 
		if (number3 > number4)
		if (number3 > number5)
		if (number3 > number1)
 
		if (number4 > number5)
		if (number4 > number1)
 
			System.out.printf("%d,%d,%d,%d,%d", number2, number3, number4, number5, number1);
 
		if (number3 > number4)
		if (number3 > number5)
		if (number3 > number1)
		if (number3 > number2)
 
		if (number4 > number5)
		if (number4 > number1)
		if (number4 > number2)
 
		if (number5 > number1)
		if (number5 > number2)
 
			System.out.printf("%d,%d,%d,%d,%d", number3, number4, number5, number1, number2);
 
		if (number4 > number5)
		if (number4 > number1)
		if (number4 > number2)
		if (number4 > number3)
 
		if (number5 > number1)
		if (number5 > number2)
		if (number5 > number3)
 
		if (number1 > number2)
		if (number1 > number3)
 
			System.out.printf("%d,%d,%d,%d,%d", number4, number5, number1, number2, number3);
 
		if (number5 > number1)
		if (number5 > number2)
		if (number5 > number3)
		if (number5 > number4)
 
		if (number1 > number2)
		if (number1 > number3)
		if (number1 > number4)
 
		if (number2 > number3)
		if (number2 > number4)
 
			System.out.printf("%d,%d,%d,%d,%d", number5, number1, number2, number3, number4);
	}
}

Welcome brandoio

Take a look at this link, and the sample code near the top of the page.

thanks jps,seems like a helpful website.

I'm new to coding too but instead of using a bunch of if statements, couldn't you use a few switch statements to correctly display the string for that set of conditions. I would do it that way personally, looks cleaner. This is more of a question because I'm not positive, don't take it as harsh criticism because I'm bad at coding