JAVA Ejercicio CU00636B aplicar conceptos de if, conciones and y or, ejemplos

[x-Gaston-X]

Hola buenas podria alguien decirme si mi codigo presenta algun error? ya lo prove y cumple su cometido perfectamente pero por ahy hize alguna mala costumbre o algo por el estilo
muchas gracias

Código: [Seleccionar]

public class coche {
    private String tipo;
    private double grosor;
    private double diametro;
    private String marca;
    //constructor
    public coche(){
        tipo = "";
        grosor = 0.0;
        diametro = 0.0;
        marca = "";
    }
    //metodos setters
    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;
}
//metodos getters
public String getTipo(){return tipo;}
public double getGrosor (){return grosor;}
public double getDiametro(){return diametro;}
public String getMarca(){return marca;}

public void comrpovarDimensiones(){
    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 pequeño");}
    if (diametro > 1.4 || grosor < 0.4) {System.out.println("el grosor para esta rueda es inferior al recomendado");}
   
    else if (diametro <= 1.4 && diametro > 0.8 && grosor < 0.25){System.out.println("el grosor para esta rueda es inferior al recomendado");}
    else {System.out.println ("el grosor para esta rueda es el recomendado");}
}
   
    }
       

[toni_apr]

Hola Gaston

Tu ejercicio esta bastante bien.
Primero, aunque no es importante, la clase debería llamarse Rueda, pues manipula atributos de una rueda no de un coche

Lo tienes todo perfecto, solo en los if de la clase comprobarDimensiones no capturas bien las condiciones.
Mira:

Si es menor o igual a 1.4 pero mayor que 0.8 debe mostrarse por consola el mensaje "La rueda es para un vehículo mediano".

esta condición se ha de codificar así

Código: [Seleccionar]

else if (diametro <= 1.4 && diametro > 0.8 ) {System.out.println("la rueda es para un vehiculo mediano");es decir se han de cumplir las dos condiciones que el diámetro sea menor o igual a 1.4 y que al mismo tiempo su diámetro sea superior a 0.8. Se ha de usar el operador and no el or.
En cualquier otro caso se ejecutaría el else de este if

En el segundo if te ocurre algo parecido

Si el diámetro es superior a 1.4 con un grosor inferior a 0.4, ó si el diámetro es menor o igual a 1.4 pero mayor que 0.8, con un grosor inferior a 0.25, deberá mostrarse por consola el mensaje "El grosor para esta rueda es inferior al recomendado"

Donde tu pones:

Código: [Seleccionar]

if (diametro > 1.4 || grosor < 0.4) {System.out.println("el grosor para esta rueda es inferior al recomendado");}
Solo hay que cambiar

Código: [Seleccionar]

if (diametro > 1.4 && grosor < 0.4) {System.out.println("el grosor para esta rueda es inferior al recomendado");}
En lugar de un or aquí va un and

Por cierto, está bien que en este if incluyas un else para mostrar el mensaje de: el grosor para esta rueda es el recomendado.

Visto todo esto, Gaston no estaría de más que retomaras la lección de los operadores lógicos
https://www.aprenderaprogramar.es/index.php?option=com_content&view=article&id=466:operadores-logicos-en-java-igual-distinto-and-or-not-mayor-menor-cortocircuito-cu00634b&catid=68:curso-aprender-programacion-java-desde-cero&Itemid=188

Saludos y adelante