Algorithm problem

Question

Statement: Read 3 integers and order them in ascending order. At the end, show the values in ascending order, a blank line and then the values in sequence as read.

public class ex10 {
public static void main(String[] args){
    Scanner x = new Scanner(System.in);

    System.out.println("Digite 3 valores em sequencia:");
    int v1 = x.nextInt();
    int v2 = x.nextInt();
    int v3 = x.nextInt();

    int um = 0;
    int dois = 0;
    int tres = 0;

    if(v1 < v2 && v1 <v3){
        um = v1;
    }if(v2 < v3){
        dois = v2;
        tres = v3;
    }else{
        tres = v2;
        dois = v3;
    }
    if(v2 < v1 && v2 < v3){
        um = v2;
    }if(v1 < v3){
        dois = v1;
        tres = v3;
    }else{
        dois = v3;
        tres = v1;
    }
    if(v3 < v2 && v3 < v1){
        um = v3;
    }if(v2 < v1){
        dois = v2;
        tres = v1;
    }else{
        dois = v1;
        tres = v2;
    }
    System.out.println("");
    System.out.println(um);
    System.out.println(dois);
    System.out.println(tres);
    System.out.println("");
    System.out.println(v1);
    System.out.println(v2);
    System.out.println(v3);


}

}

My problem is this, when the user writes the numbers from the largest to the smallest (6,5,4) or from the smallest to the largest (4,5,6) the algorithm works correctly, but when the user puts out of order, for example 6,4,5 goes wrong. Another question is whether it would be possible to do this algorithm in a more summarized way.

Answer 1

import java.util.Scanner;

public class Ex10 {
    public static void main(String[] args) {
        Scanner x = new Scanner(System.in);

        System.out.println("Digite 3 valores em sequencia:");
        int a = x.nextInt();
        int b = x.nextInt();
        int c = x.nextInt();
        int v1 = a, v2 = b, v3 = c;

        // Se v1 for maior que v2, troca eles de lugar.
        if (v1 > v2) {
            int aux = v2;
            v2 = v1;
            v1 = aux;
        }

        // Se v1 for maior que v3, troca eles de lugar.
        if (v1 > v3) {
            int aux = v3;
            v3 = v1;
            v1 = aux;
        }

        // Se v2 for maior que v3, troca eles de lugar.
        if (v2 > v3) {
            int aux = v3;
            v3 = v2;
            v2 = aux;
        }

        System.out.println(v1 + ", " + v2 + ", " + v3);
        System.out.println();
        System.out.println(a + ", " + b + ", " + c);
    }
}

See here working on ideone.

The idea is to get v1 has the lowest value. For this compares with v2 and v3, changing case v1 is not the least. Therefore, after these two steps, v1 will be the smallest number.

Next, v2 must be the middle number and v3 the largest. Then just compare them and exchange them if not so.

Note that you do not need to show each value separately from each other in your System.out.println. You can match an entire row in each System.out.println.

Answer 2

resolution:

public class ex10 {
public static void main(String[] args){
    Scanner x = new Scanner(System.in);

    System.out.println("Digite 3 valores em sequencia:");
    int v1 = x.nextInt();
    int v2 = x.nextInt();
    int v3 = x.nextInt();

    int um = 0;
    int dois = 0;
    int tres = 0;

    if(v1 < v2 && v1 <v3){
        um = v1;
    if(v2 < v3){
        dois = v2;
        tres = v3;
    }else{
        dois = v3;
        tres = v2;
    }
    }
    if(v2 < v1 && v2 < v3){
        um = v2;
    if(v1 < v3){
        dois = v1;
        tres = v3;
    }else{
        dois = v3;
        tres = v1;
    }
    }
    if(v3 < v2 && v1 > v3){
        um = v3;
    if(v2 < v1){
        dois = v2;
        tres = v1;
    }else{
        dois = v1;
        tres = v2;
    }
    }
    System.out.println("");
    System.out.println(um);
    System.out.println(dois);
    System.out.println(tres);
    System.out.println("");
    System.out.println(v1);
    System.out.println(v2);
    System.out.println(v3);


}

}

my mistake was having locked the key too early in that part:

if(v1 < v2 && v1 <v3){
    um = v1;
}if(v2 < v3){
    dois = v2;
    tres = v3;

I closed just before to put the second if, when I should have left open

if(v1 < v2 && v1 <v3){
    um = v1;
if(v2 < v3){
    dois = v2;
    tres = v3;