[java]numeri crescenti

[el92]

Ciao ho iniziato da poco a studiare java ed ho incontrato delle difficoltà nell'eseguire questo esercizio:
Scrivere un programma che riceva tre numeri in virgola mobile come dati in
ingresso, per poi stamparli in ordine crescente. (io ho sviluppato solo un caso)
es: 5,0
3,0
4,0
output: 3,0
4,0
5,0

Il mio codice è:

class Numeri{
double a,b,c;
Numeri(){
a=0.0;
b=0.0;
c=0.0;
}
public void ordina(double x,double y,double z){
    a=x;
    b=y;
    c=z;
    double temp=0.0;
    
    if(b<c)
        if(c<a)
            if(b<a){
        temp=a;
        b=c;
        a=b;
        c=temp;
    }else
        temp=a;
        a=b;
        b=temp;
    }
    public double getA(){
        return a;
    }
    public double getB(){
        return b;
    }
    public double getC(){
        return c;
    }
}

da main:

import java.util.Scanner;
class Numeri1{
public static void main(String[] args){
Scanner sc=new Scanner(System.in);
System.out.println("scrivi 3 numeri");
double a=sc.nextDouble();
double b=sc.nextDouble();
double c=sc.nextDouble();
Numeri n=new Numeri();
n.ordina(a,b,c);
System.out.println("i numeri in ordine");
System.out.println(n.getA());
System.out.println(n.getB());
System.out.println(n.getC());
}
}

Il punto è che da terminale il programma stampa l'ultimo numero uguale a quello precedente.
es se scrivo : 5,0
3,0
4,0

output: 4,0
5,0
5,0
dov'è che ho sbagliato?

[LeleFT]

    if(b<c)
        if(c<a)
            if(b<a){
        temp=a;
        b=c;
        a=b;
        c=temp;
    }else
        temp=a;
        a=b;
        b=temp;
    }

Questo codice è sbagliato.
Anzitutto, se b < c e c < a allora b sarà sempre anche minore di a (quindi, il terzo if è assolutamente inutile).
L'indentazione fatta male, poi, inganna l'occhio: quel ramo "else" (composto da UNA SOLA ISTRUZIONE) non verrà mai eseguito (per quanto detto prima), quindi, nel caso in cui b non sia minore di c o nel caso in dui c non sia minore di a, semplicemente non viene fatto nulla. Inoltre, se b è minore di c, viene sempre assegnato b ad a e temp a b... qualunque siano i loro valori.

Riscrivo esattamente il tuo codice, usando un'indentazione corretta (e le parentesi graffe che nel tuo codice mancano e che aiuterebbero a capire l'errore):

Questo è esattamente il tuo codice indentato meglio (ho anche esplicitato le graffe implicite):

    if(b<c) {
        if(c<a) {
            if(b<a){
                temp=a;
                b=c;
                a=b;
                c=temp;
            }else {
               temp=a;
            }
            a=b;
            b=temp;
        }
    }

Come vedi va completamente rivisto.

Ciao.

[el92]

ciao ho risolto avevi ragione grazie per la risposta