Skip to main content
Se extendió la explicación para hacer mas comprensible el ultimo caso
Origen Enlace

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 DocumentoDocumento 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 StringString 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.

Creo que no estás aprovechando la clase base. Ambas clases tienen algo en común que es su número de identificación (o número de documento) a pesar de que son de diferentes tipos (DNI, CIF).

Lo correcto a mi entender es esta clase base:

Donde la clase persona forzará 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 específico de documento con sus características y que el método ToString() de cada clase sepa como representar en un String el documento.

public abstract class Documento {

   private String nroDocumento;

   public String getDocumento() {...}

   public void setDocumento(string doc) {...}

   public abstract String getTipo();

   public ToString() {
        return getTipo() + " " + getDocumento();
   }

}

public class DniDocumento extends Documento {

   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ún.

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:

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.

se añadieron 8 caracteres en el cuerpo
Origen Enlace
César
  • 16.8k
  • 6
  • 42
  • 78

Creo que no estasestás aprovechando la clase base. Ambas clases tienen algo en comuncomún que es su numeronúmero de identificaciónoidentificación (o numeronúmero de documento) a pesar de que son de diferentes tipos (DNI, CIF) Lo.

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 forzaraforzará el tipo a DNI e Institucion a CIF.

Otra alternativa es crear una clase abstracta que se llame DocumentoDocumento y que cada subclase represente un tipo especificoespecífico de documento con sus caracteristicascaracterísticas y que el método ToString()ToString() de cada clase sepa como representar en un StringString el documento.

public abstract class Documento {

   private String nroDocumento;

   public String getDocumento() {...}

   public void setDocumento(string doc) {...}

   public abstract String getTipo();

   public ToString() {
        return getTipo() + " " + getDocumento();
   }

}

public class DniDocumento extends Documento {

   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 comuncomún.

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;

   public String getDocumento() {...}

   public void setDocumento(string doc) {...}

   public abstract String getTipo();

   public ToString() {
        return getTipo() + " " + getDocumento();
   }

}

public class DniDocumento extends Documento {

   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.

Creo que no estás aprovechando la clase base. Ambas clases tienen algo en común que es su número de identificación (o número 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 forzará 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 específico de documento con sus características y que el método ToString() de cada clase sepa como representar en un String el documento.

public abstract class Documento {

   private String nroDocumento;

   public String getDocumento() {...}

   public void setDocumento(string doc) {...}

   public abstract String getTipo();

   public ToString() {
        return getTipo() + " " + getDocumento();
   }

}

public class DniDocumento extends Documento {

   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ún.

Origen Enlace

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;

   public String getDocumento() {...}

   public void setDocumento(string doc) {...}

   public abstract String getTipo();

   public ToString() {
        return getTipo() + " " + getDocumento();
   }

}

public class DniDocumento extends Documento {

   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.