Foros aprenderaprogramar.com
Aprender a programar => Aprender a programar desde cero => Mensaje iniciado por: serietaruzjf en 29 de Julio 2014, 01:51
-
Hola muy buenas noches desde México, espero que la estén pasando muy bien. Quisiera que me ayudaran a revisar el código de mi programa, de antemano agradezco una vez mas por el tutorial y el tiempo que han invertido en él :)
//SMC-AUTOR
public class Selector
{
String tipo;
double grosor;
double diametro;
String marca;
public Selector()
{
marca = "DESCONOCIDA";
tipo = "DESCONOCIDO";
diametro = 0;
grosor = 0;
}
public void setMarca (String valorMarca)
{
System.out.println ("La llanta ha pasado de tener una marca " + marca + " por el de " + valorMarca +"");
marca = valorMarca;
}
public void setTipo (String valorTipo)
{
System.out.println ("La llanta de la marca " + marca + " ha pasado de tener un tipo " + tipo + " al tipo " + valorTipo +"");
tipo = valorTipo;
}
public void setDiametro (double valorDiametro)
{
System.out.println ("La llanta marca " + marca + " tipo " + tipo + " ha cambiado su diámetro de " + diametro + " por el de " + valorDiametro +"");
diametro = valorDiametro;
if (valorDiametro > 1.4)
{
System.out.println ("La rueda es para un vehículo grande");
}
else if (valorDiametro <=1.4&&valorDiametro>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");
}
}
public void setGrosor (double valorGrosor)
{
System.out.println ("La llanta marca " + marca + " tipo " + tipo + " con diámetro " + diametro + "ha cambiado su grosor de " + grosor + " por el de " + valorGrosor +"");
grosor = valorGrosor;
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");
}
}
public String getMarca () {return marca; }
public String getTipo () {return tipo; }
public double getDiametro () {return diametro; }
public double getGrosor () {return grosor; }
}
-
Hola, gracias por tus comentarios.
Sobre el código te comento las cosas que he visto:
- Los métodos no deben incluir mensajes (como System.out.println ("La llanta ha pasado de tener una marca " + marca + " por el de " + valorMarca +"") ; ) excepto si el enunciado lo pide explícitamente.
- Un método set sólo debe encargarse de establecer el valor correspondiente para el atributo. Por tanto tu método setDiametro no está bien planteado, ya que además de esto hace un análisis del diámetro y muestra ciertos mensajes según ese análisis.
- Lo mismo ocurre con el método setGrosor.
Debes reescribir el código y ajustarte a lo que pide el enunciado. El enunciado dice: "Crea un método denominado comprobarDimensiones..." y esto no lo has hecho, por tanto no queda bien resuelto el ejercicio.
Puedes consultar las respuestas de otras personas y así orientarte mirando aquí: https://www.aprenderaprogramar.com/foros/index.php?topic=919
Saludos