Hola Gustavo.
Veo que es el primer ejercicio que publicas, así que me toca hacer de anfitrión y darte la bienvenida al foro.
Antes que nada, te voy a recomendar leer el siguiente link para que puedas realizar publicaciones de una manera más ordenada.
https://www.aprenderaprogramar.com/foros/index.php?topic=1460.0Por ejemplo he de sugerirte que cuando publiques líneas de código lo hagas entre las etiquetas
[ code ] ... aquí escribe tu código... [ /code ]
Ahora bien, pasemos al ejercicio en si. Lo he revisado y veo que lo has hecho muy bien!, has comenzado con el pie derecho.
A su vez, tenemos unas observaciones por hacer que mejorarán aún más tu código, veámoslas.
Ten en cuenta que por convención, los nombres de los métodos y los atributos deben empezar con una letra minúscula, y los nombre de las clases con una letra mayúscula. Esto es algo que todos los que programamos respetamos. Tu lo has respetado, perfecto!
Ahora bien, cuando el nombre de un método esté compuesto por dos palabras, la segunda palabra la comenzamos con mayúscula; de esa manera es más legible y facil de entender a simple vista. Por ejemplo:
al método setnombre() lo deberías renombrar por setNombre().
al método setapellido() lo deberías renombrar por setApellido().
Y así con todos los demás nombres de métodos que estén compuestos por dos o más palabras: cada nueva palabra comienza con mayúscula (salvo la palabra inicial).
Lo mismo sucede con los nombres de los atributos. Por ejemplo, tienes un atributo llamado numerodocumento, este debería llamarse numeroDocumento. Lo mismo sucede con el atributo valorNumerodocumento, que debería llamarse valorNumeroDocumento.
En otro orden de cosas, me ha llamado la atención el siguiente método:
public void setapellido (String valorApellido)
{
// put your code here
apellido = "Apellido" + valorApellido;
}
Supongo que habrá sido una distracción ya que a los demás métodos set los has codificado bien. Este método debería ser reescrito por:
public void setapellido (String valorApellido)
{
// put your code here
apellido = valorApellido;
}
A seguir adelante!