Estructura condicional bifurcadora ejercicio CU00636B# tutorial java desde cero

[Thieba]

Hola buenas, aquí deja la entrega del ejercicio CU00636B para su corrección,
gracias por su atención.

Código: [Seleccionar]

public class Rueda
{
    //declaracion atributos
    private String tipo;
    private double grosor;
    private double diametro;
    private String marca;

    //constructor
    public Rueda ()
    {
        //inicializacion atributos
        tipo = " ";
        grosor = 0;
        diametro = 0;
        marca = " ";
    }

    //metodos para establecer los valores de los atributos
    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;}

//metodo para comprobar las dimensiones de la rueda
    public void comprobarDimensiones ()
    {
        if (diametro > 1.4)
        {System.out.println (" La rueda es para un vehículo grande ");}
        else if ((diametro <= 1.4)&&(diametro > 0.8))
        {System.out.println (" La rueda es para un vehículo mediano ");}
        else
        {System.out.println (" La rueda es para un vehículo pequeño ");}
       
        if (((diametro > 1.4)&&(grosor < 0.4))||(((diametro <= 1.4)&&(diametro > 0.8))&&(grosor < 0.25)))
          {System.out.println (" El grosor para esta rueda es inferior al recomendado ");}
    }
    //metodos para obtener los valores de los atributos
    public String getTipo ()
    {return tipo;}

    public double getGrosor ()
    {return grosor;}

    public double getDiametro ()
    {return diametro;}

    public String getMarca ()
    {return marca;}

}


[dongo]

El ejercicio esta totalmente correcto, pero una recomendación...por claridad
cuando hagas un if, un get, un set o algo que conlleve un bloque de código de 1 sola instrucción, yo lo haría así:

Código: [Seleccionar]

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


o así:

Código: [Seleccionar]

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


Pero nunca lo haría como lo haces tu, ya que al estar las llaves pegadas a la instrucción es mas trabajoso ver donde empieza y donde acaba el código....
Solo es una recomendación, eee, si tu crees que te vas a quedar sin folio sigue como lo haces, pero cuando lo presentas a otra gente, para mi es mas claro verlo si la llave esta sola en una linea....

No se que opinarán los demás, pero la encuesta esta abierta XDD

Bueno un saludo!!

[BelCrow]

en el contructor los valores double, ejemplo double diametro y double grosor.. tienen que tener 0.0 y no 0... y la cadena vacía se escribe con las comillas juntas "" en lugar de " " en la inicilalizacion de los atributos...

Código: [Seleccionar]

       
        //inicializacion atributos
        tipo = "";
        grosor = 0.0;
        diametro = 0.0;
        marca = "";