Creo que no estásestas aprovechando la clase base. Ambas clases tienen algo en comúncomun que es su númeronumero de identificaciónidentificacióno (o númeronumero de documento) a pesar de que son de diferentes tipos (DNI, CIF).
Lo Lo correcto a mi entender es esta clase base:
Donde la clase persona forzaráforzara el tipo a DNI e Institucion a CIF.
Otra alternativa es crear una clase abstracta que se llame Documento
Documento y que cada subclase represente un tipo específicoespecifico de documento con sus característicascaracteristicas y que el método ToString()
ToString() de cada clase sepa como representar en un String
String el documento.
public abstract class Documento {
private String nroDocumento;
publicprotected Documento(String getDocumento(nroDocumento) {..
this.nroDocumento = nroDocumento
}
public voidString setDocumentogetDocumento(string doc) {...}
public abstract String getTipo();
public ToString() {
return getTipo() + " " + getDocumento();
}
}
public class DniDocumento extends Documento {
public DniDocumento(String nroDocumento) {
super(nroDocumento);
}
public String getTipo() {
return "DNI";
}
}
De esta manera no hay condicionales que tengas que poner para distinguir una clase de otra al momento de imprimir una propiedad que ambos tipos tienen en comúncomun. Las clases entonces quedarian asi:
public abstract class Entidad {
protected String name;
protected String notes;
protected Documento documento;
protected Entidad (String name, String notes, Documento documento) {
this.name = name;
this.notes = notes;
this.documento = documento;
}
public String getName() {
return name;
}
public String getNotes() {
return notes;
}
public Documento getDocumento() {
return documento;
}
}
public class Persona extends Entidad {
public Persona (String name, String notes, DniDocumento documento) {
super(name, notes, documento);
}
// Opcional
public DniDocumento getDocumento() {
return (DniDocumento) documento;
}
}
También se podrían usar clases genéricas, pero creo que así esta bien.