Hola Alex
El ejercicio está perfecto.
Declaración de atributos OK
Inicialización en los dos constructores. OK
Setters y getters, también perfectos. OK
Uso de la palabra clave this, perfecto. No se puede mejorar.
Aprovechando este ejercicio perfecto, quiero comentarte un detalle que no influye en la calidad del código.
Observo que casi no usas indentación en las líneas de tu código. Seguramente te has acostumbrado a trabajar así.
Pero, ¿para qué sirve la indentación?.
Pues solo permite leer el código más facilmente, un simple vistazo te permite detectar donde termina un for o un while con muchas líneas.
Si en algún momento quieres probar la indentación en tu código puedes probar lo siguiente:
Si usas el editor de BlueJ, abre el menú Preferencias y en la ficha Editor activa la opción
Auto-indent (Enter and Tab keys)
o usa la opción del menú Editar llamada Auto-layout [Ctrl + Mayus + I] que te indenta todas las líneas de golpe.
Si usas otro editor habrás de buscar en los menús la opción adecuada.
Y si no, usa la tecla Tab para indentar las líneas.
Saludos