Respuesta Ejercicio CU00636B ejemplo if else java

[ThheMau]

Hola a todos!

Les traigo mi respuesta al ejercicoi CU00636B, a mi parecer esta bien pero si encuentran algo que pueda mejorar les agradeceria mucho si me lo hicieran saber 

Posdata: Perdon por la falta de tildes, pero tengo la PC en ingles ya que vivo en USA y no me he puesto a buscar como ponerlos.

Gracias por su ayuda 

Código: [Seleccionar]

    public class Rueda {
    String tipo;
    Double grosor;
    Double diametro;
    String marca;

    public Rueda () {
        tipo="";
        grosor=0.00;
        diametro=0.00;
        marca="";
    }

    public void setTipo (String valorTipo) {
        tipo=valorTipo;
    }

    public void setGrosor (Double valorGrosor) {
        grosor=valorGrosor;
    }

    public void setDiametro (Double valorDiametro){
        diametro=valorDiametro;
    }

    public void setMarca (String valorMarca){
        marca=valorMarca;
    }

    public void comprobarDimenciones (){
        if (diametro>1.4) {
            System.out.println ("La rueda es para un vehiculo grande");}
        else if (diametro<=1.4 && diametro>0.8) {
            System.out.println ("La rueda es para un vehiculo mediano");}
        else {
            System.out.println("La rueda es para un vehiculo chico");}

        if (diametro>1.4 && grosor<0.4 || diametro<=1.4 && diametro>0.8 && grosor<0.25){
            System.out.println("El grosor para esta arueda es inferior al recomendado");}
        }
    }
   
       
       
       

[dongo]

Esta todo correcto.

Sobre los doubles y los floats, no es necesario poner 0.00, con que pongas 0 ya se lo traga.

Un consejito por claridad, intenta poner la llave de cierre( } ) en una nueva linea. No te quedarás sin folio!! y el código será mucho mas claro. Te hablo de tu método  comprobarDimenciones que por cierto es dimensiones... Eres andaluz?? jeje

Bueno para que veas te pongo como quedaría y veras como se ve el código más claro:

Código: [Seleccionar]

public void comprobarDimenciones() {
        if (diametro > 1.4) {
            System.out.println("La rueda es para un vehiculo grande");
        }
        else if (diametro <= 1.4 && diametro > 0.8) {
            System.out.println("La rueda es para un vehiculo mediano");
        }
        else {
            System.out.println("La rueda es para un vehiculo chico");
        }
        if (diametro > 1.4 && grosor < 0.4 || diametro <= 1.4 && diametro > 0.8 && grosor < 0.25) {
            System.out.println("El grosor para esta arueda es inferior al recomendado");
        }
}

Si te fijas se puede observar con mas claridad donde empieza un bloque if y donde acaba, ya que la llave de cierre esta a la misma altura que la instrucción que la abre(en este caso los ifs).

Y bueno esto no es importante ahora porque es poco código y no es complicado de seguir, pero cuando tienes 3000(Yo también soy andaluz) lineas de código en un mismo fichero, pues  las indentaciones, las aperturas de bloques los cierres, etc, son importantes ya que aclaran mucho donde empiezan las cosas y donde acaban.

Como te digo no es importante, pero es recomendable.

Bueno un saludo!

[ThheMau]

Muchas gracias!, lo voy a tomar en cuenta  . Saludos