Java mostrar un mensaje cuando se ejecuta un método Ejercicio CU00633B tutorial

[JC]

Buenos días, de esta manera realicé el ejercicio 33 de java. ¿Me indican qué les parece?

Código: [Seleccionar]

public class Medico2 {

private String Nombre;
private String Apellido;
private int edad;
private boolean casado;
private String DI;
private String espe;

//constructor
public Medico2(){
Nombre = " ";
Apellido =" ";
edad = 0;
casado = false;
DI= "";
espe= "";
}

public void setNombre (String valorNombre){
Nombre = valorNombre;
System.out.println("EL Nombre se ha modificado ahora es "+valorNombre );
}

public void setApellido (String valorApellido){
    Apellido = valorApellido;
    System.out.println("El apellido se ha modificado ahora es "+valorApellido);
}

public void setedad(int valoredad){
edad = valoredad;
System.out.println("la edad se ha modificado ahora es "+valoredad);
}

public void setcasado(boolean valorcasado ){
    casado= valorcasado;
    System.out.println("El valor casado sigue igual "+valorcasado);
      }

public void setDI(String valorDI){
    DI= valorDI;
    System.out.println("El valor de DI se ha modificado ahora es "+ valorDI);
                  }

public void espe(String valorespe){
espe= valorespe;
System.out.println("la especialidad se ha modificado ahora es "+valorespe);
}

public String Nombre(){return Nombre;}
public String Apellido(){return Apellido;}
public int edad(){return edad;}
public boolean casado(){return casado;}
public String DI(){return DI;}
public String espe(){return espe;}

public void calmultedad(){
if (edad <10){
System.out.println("error introdusca un valor real mayor de 10");
int comprobar = edad%5;//realizo la division para obtener el resto
int faltan= 5-comprobar;//resto 5 menos el comprobar para calcular cuantos años faltan
System.out.println("al medico "+Nombre +"le faltan "+faltan +" años para ser multiplo de 5");
}

}

[RaGa]

Hola jchusmit:

Veo que es el primera post que publicas en el foro, así que me toca hacer de anfitrión y darte la bienvenida!

Como primer consejo que me gustaría darte sería sugerirte leer este hilo:
https://www.aprenderaprogramar.com/foros/index.php?topic=1460.0
donde encontrarás la forma de insertar código en el foro de manera correcta.

Ahora bien, yendo específicamente al código que publicaste, he de decirte que está bien pensado, y si bien hay algunos errores que a continuación iremos puntualizando, has hecho un muy buen trabajo!

Veamos entonces qué correcciones habría que hacerle para mejorar el código.

1) Cómo se ha dicho en más de una oportunidad en las lecciones (y en muchos hilos del foro que recomiendo siempre leer ya que se aprende mucho de los compañeros), es una buena práctica respetar la convención que nos dice que: los nombres de los atributos y los métodos deben comenzar siempre con una letra manúscula, y los nombres de las clases siempre con una letra mayúscula.
Así pues: el atributo ´Apellido´, debería llamarse ´apellido´. El atributo ´Nombre´debería llamarse ´nombre´, etc.
Otra convención importante a respetar en este mismo sentido es la que nos dice que si bautizamos un método o un atributo con un nombre que contenga dos palabras, esa segunda palabra comenzará con una letra mayúscula. De esta forma se hace más ágil y entendible su lectura, además de darnos mayor información acerca de "qué representa" cuando se trata así de nombre compuestos.
Por lo tanto, un atributo como ´valorespe´debería llamarse ´valorEspe´, y un método como ´calmultedad´debería llamarse ´calMultEdad´, o ´setcasado´debería ser ´setCasado´, etc.

2) Siguiendo con las convenciones de un buen programador: los métodos que asignan un valor a un atributo de clase, deben bautizarse anteponiendo el prefijo ´set´. Ejemplo: ´setCasado´.
Los métodos que devuelven un valor, deben bautizarse anteponiendo el prefijo ´get´. De esta manera deberías cambiar nombres de métodos como por ejemplo el método ´Apellido´ que debería llamarse ´getApellido´, etc.

3) Fíjate en el constructor que has creado en tu código:

Código: [Seleccionar]

//constructo
public Medico2(){
Nombre = " ";
Apellido =" ";
edad = 0;
casado = false;
DI= "";
espe= "";
Aquí he de señalarte que el atributo ´espe´está muy bien inicializado. Pero fíjate que no es de la misma forma que has inicializado el atributo ´Apellido´ o el atributo ´Nombre´, donde hay un espacio entre las comillas. Ese espacio no debería existir si estamos inicializando un campo tipo String a valores por defecto.

4) La línea

Código: [Seleccionar]

if (edad <10){
debe ser reemplazada por :

Código: [Seleccionar]

if (edad <10)
para que funcione correctamentetodo el programa.


Analiza estos puntos y cualquier duda, estamos aquí para revisarlo nuevamente.

[JC]

Hola Raga:

Muchas gracias por los consejos y aclararme algunos puntos mejorare el código y lo aplicare para los futuros códigos que desarrolle

[Ogramar]

Buenas, voy a editar el mensaje inicial para ponerle un título descriptivo y que el código quede bien formateado como ha explicado RaGa. Salu2