Java diferencia entre plantear un condicional if con else o sin else CU006636B

[guillerub2001]

¡Hola! Venía a preguntar si es correcta mi solución. Me gustaría que dijeran si hay alguna forma de mejorarlo, gracias.

El ejercicio dice así:

Considera estás desarrollando un programa Java donde necesitas trabajar con objetos de tipo Rueda (que representa a una rueda de un vehículo). Define una clase Rueda considerando los siguientes atributos de clase: tipo (String), grosor (double), diametro (double), marca (String). Define un constructor asignando unos valores de defecto a los atributos y los métodos para poder establecer y obtener los valores de los atributos. Crea un método denominado comprobarDimensiones donde a través de condicionales if realices las siguientes comprobaciones:

a) Si el diámetro es superior a 1.4 debe mostrarse por consola el mensaje “La rueda es para un vehículo grande”. 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”. Si no se cumplen ninguna de las condiciones anteriores debe mostrarse por pantalla el mensaje “La rueda es para un vehículo pequeño”.

b) 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”.

Este es mi código para la solución:

Código: [Seleccionar]

public class Ruedas {
    // Campos

    public String tipo;
    public double grosor;
    public double diametro;
    public String marca;

    // Constructor

    public Ruedas(String getTipo, double getGrosor, double getDiametro, String getMarca) {
        tipo = getTipo;
        grosor = getGrosor;
        diametro = getDiametro;
        marca = getMarca;
    }

    /* Métodos
     * Tipo
     * Grosor
     * Diametro
     * Marca
     * Comprobar dimensiones
     */

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

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

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

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

    // Para el if

    public void comprobarDimensiones() {

        if ( diametro > 1.4 ) {
            System.out.println ("La rueda es para un vehículo grande"); }
        if ( diametro <= 1.4 && diametro > 0.8 ) {
            System.out.println ("La rueda es para un vehículo mediano"); }
        if ( diametro < 0.8 ) {
            System.out.println ("La rueda es para un vehículo pequeño"); }

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

También quería preguntar cómo podría usar el operador || (o/or) ya que no lo entiendo mucho. Pensé en usarlo así (en la última parte de mi código):

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

Pero no me dejaba compilarlo.

Muchas gracias por leer.

[guillerub2001]

Revivo.

[Enzo]

Hola guillerub2001,

Veo tu código bastante bien y funciona correctamente.

Te comento un par de puntos:

- En el primer bloque de comprobación  lo haces sin usar “else”. No es que este mal, porque las comprobaciones las hace, pero a la hora de ver el código el “else” ayuda más a seguir la comprobación y la hace más compacta. Sin “else” parecen 3 comprobaciones distintas.

- En el segundo bloque, según comentas, ya lo tenías. Lo único que falta es poner unos paréntesis. La comprobación del if debe ir entre paréntesis y, dentro de estos paréntesis, puedes hacer varias comprobaciones, cada una de ellas dentro de otros paréntesis.
Tu código es:

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

y este es el código que no te dará problema:

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

¿Ves la diferencia?.Pruebalo y ya veras.

- En los métodos setters utilizas nombres de variables que comienzan con “get” como por ejemplo “getTipo”. Esto te puede crear confusión con los métodos getters. Te recomiendo usar otro formato de nombre, como por ejemplo “nuevoTipo” o “valorTipo”. Más adelante veras que incluso puedes usar el mismo nombre de variable que la que usas como atributo.

Espero que te sirvan mis comentarios.

Un Saludo,

[guillerub2001]

Muchísimas gracias por tu respuesta Enzo.
Sobre tu primera sugerencia, sería cambiar uno de los if's por else y poner else if en otro, ¿verdad?
Sobre el constructor y los métodos setters, es un fallo algo tonto, que lo cometí por una confusión al haber retomado el curso tras haberlo haberlo dejado hace un tiempo.
Saludos!

[Enzo]

Hola de nuevo guillerub2001,

Muchas "de nada". Me alegro que te pueda servir de ayuda.

Si. Si te fijas son tres condicionales ¿no?:
Pues comienzas con un "if" Si se cumple hace {...}
(y sin no se cumple) "else if" hace {...}
(Y si este ultimo tampoco se cumple) "else" hace {...}

Te aconsejo que lo pruebes y veras como ves el código mucho mas claro.

Saludos,

[guillerub2001]

Sí, ya lo tenía claro, pero gracias xD.
Muchas gracias por tu ayuda.
Un saludo!