Creo que no estas aprovechando la clase base. Ambas clases tienen algo en comun que es su numero de identificacióno (o numero de documento) a pesar de que son de diferentes tipos (DNI, CIF)
Lo correcto a mi entender es esta clase base:
public enum TipoDoc {
DNI, CIF
}
public class Entidad {
protected String name;
protected String notes;
protected String identificacion;
protected TipoDoc tipo;
}
Donde la clase persona forzara el tipo a DNI e Institucion a CIF.
Otra alternativa es crear una clase abstracta que se llame Documento y que cada subclase represente un tipo especifico de documento con sus caracteristicas y que el método ToString() de cada clase sepa como representar en un String el documento.
public abstract class Documento {
private String nroDocumento;
protected Documento(String nroDocumento) {
this.nroDocumento = nroDocumento
}
public String getDocumento() {...}
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 comun. 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.