Ejercicio! entrega CU00636B if con and y or java

[Jorge lopez]

Código: [Seleccionar]

public class Rueda{
    String tipo, marca;
    double grosor, diametro;

    //constructor
    public Rueda(){
        tipo = "";
        marca= "";
        grosor = 0.0;
        diametro = 0.0;
    }

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

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

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

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

    //getters
    public String getTipo(){return tipo;}

    public String getMarca(){return marca;}

    public double getGrosor(){return grosor;}

    public double getDiametro(){return diametro;}

    public void comprobarDimensiones(){
        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 || diametro <= 1.4 && diametro > 0.8 && grosor < 0.25){System.out.println("El grosor para esta rueda es inferior al recomendado");}
        else{}
       
    }
}

[Jorge lopez]

Esto no es una respuesta, mas bien es el mismo ejercicio modificado, ya que no encontré la opción de modificar el ejercicio original después de publicarlo, si hay una ayuda para mi caso sera bienvenida al igual que la revision del ejercicio. Saludos!

Código: [Seleccionar]

public class Rueda{
    String tipo, marca;
    double grosor, diametro;

    //constructor
    public Rueda(){
        tipo = "________";
        marca= "________";
        grosor = 0.0;
        diametro = 0.0;
    }

    //setters

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

    public void setDiametro(double valorDiametro){diametro = valorDiametro;}
   
    public void setTipo(String valorTipo){tipo = valorTipo;}
   
    public void setMarca(String valorMarca){marca = valorMarca;}

    //getters
    public String getTipo(){return tipo;}

    public String getMarca(){return marca;}

    public double getGrosor(){return grosor;}

    public double getDiametro(){return diametro;}

    public void comprobarDimensiones(){
        if(diametro > 1.4){System.out.println("La rueda marca: " + marca +", tipo: " + tipo + ", de diametro: " + diametro + ", y grosor: " + grosor +" es para un vehiculo grande.");}
        else if(diametro <= 1.4 && diametro > 0.8)
        {System.out.println("La rueda marca: " + marca +", tipo: " + tipo + ", de diametro: " + diametro + ", y grosor: " + grosor + " es para un vehiculo mediano");}
        else{System.out.println("La rueda marca: " + marca +", tipo: " + tipo + ", de diametro: " + diametro + ", y grosor: " + grosor + " es para un vehiculo pequeño");}

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

    }
}

[César Krall]

Hola, el programa lo veo correcto, enhorabuena por tu avance. Saludos,

[jogonzalezmor]

Acabo de terminar mi codigo para este ejercicio, aunque pareciera que la primera consulta if para determinar si el diametro es inferior al recomendado no me discrimina bien.

Código: [Seleccionar]

public class Rueda(){
String tipo;
double grosor;
double diametro;
String marca;
}   

//Constructor de objeto Rueda

public rueda(){

tipo = "";
        marca= "";
        grosor = 0.0;
        diametro = 0.0;

}

//Métodos setter and getter

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

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

    public void setGrosor(double valorGrosor)
grosor = valorGrosor;
    }

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



    public String getTipo(){return tipo;}

    public double getGrosor(){return grosor;}

    public double getDiametro(){return diametro;}

    public String getMarca(){return marca;}

    //Método para comprobar dimensiones.
    public void comprobarDimensiones (){
        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");
        }
        else 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");
        }

    }
}
Agradecería sus comentarios.

Salu2...

[Mastermind]

El código que has puesto no compila.

Los if no los tienes bien construidos. Por ejemplo:

        else 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");
        }

Si el diámtro es menor o igual a 1.4 siempre se ejecutará el primer else if y nunca el segundo, entonces el segundo es inútil.

Para plantearlo bien tienes que establecer las condiciones y mensajes referidos al tamaño con un if y las condiciones y mensajes referidos al grosor en otro if, no todo en un solo if con else if, no sé si me habré explicado la idea 

[jogonzalezmor]

El código que has puesto no compila.

Los if no los tienes bien construidos. Por ejemplo:

        else 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");
        }

Si el diámtro es menor o igual a 1.4 siempre se ejecutará el primer else if y nunca el segundo, entonces el segundo es inútil.

Para plantearlo bien tienes que establecer las condiciones y mensajes referidos al tamaño con un if y las condiciones y mensajes referidos al grosor en otro if, no todo en un solo if con else if, no sé si me habré explicado la idea 

En ese caso, bastaría con invertir los else if? si es así, podría discriminar de inmediato si la rueda es de tamaño mediano, de lo contrario, es para tamaño grande. O no?

[Jorge lopez]

Hola jogonzalezmor.

a continuación los posibles errores de tu código:

Los paréntesis () indican cuando un método es de tipo función, aquí estas declarando la clase, que dentro de sus llaves {} albergara: variables, constructores,
métodos ext. no debes colocar paréntesis al iniciar tu clase. También este sierre esta mal ubicado porque esta serrando la clase y esta dejado fuera todo el código que este por debajo de el, este es el cierre de:
 public class Rueda{ que debería estar al final como ultimo carácter en el código de la clase

Código: [Seleccionar]

public class Rueda() {
    String tipo;
    double grosor;
    double diametro;
    String marca;
}
si en principio declaraste Rueda, aquí no debes declarar rueda, una "simple" diferencia de MAYUSCULAS y minúsculas afecta la compilación

Código: [Seleccionar]

public rueda(){
tipo = "";
marca= "";
grosor = 0.0;
diametro = 0.0;
 }
aquí por igual (ortografía)

Código: [Seleccionar]

public void setMarca(String valorMarca){
 Marca = valorMarca;
 }
 aquí olvidaste la llave de apertura "{" que sierra con la que si indicaste

Código: [Seleccionar]

public void setGrosor(double valorGrosor)
grosor = valorGrosor;
 }
 el problema con tu if else, es que tienes todos los mensajes dentro de un solo if else cuando deberían ser 2 bloques para que los mensajes salgan
bien por pantalla. algo así: if else if else para los mensajes de grande,mediano,peque?o y otro if else para el mensaje del grosor. También en el diámetro del mensaje de vehículo grande tienes <= y debería ser diámetro > 1.4

Código: [Seleccionar]

public void comprobarDimensiones (){
       
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");
     }
else 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");
        }
Salu2!