Base abstracta o clase auxiliar

Question

Acabo de aprender hoy un poco sobre Composición sobre herencia. Me preguntaba si debería aplicar el concepto a algo que escribí recientemente.

Nos previamente tenían dos clases que eran casi idénticos, aparte de un par de pequeñas diferencias. Contenían algunas características básicas de acceso a la base de datos, pero operaban en diferentes tipos de objetos (pero relacionados). Así que antes teníamos clase que se estructuraron algo como esto:

class BallMillDBHandler{ 

    public BallMillDBHandler(){ ... } 

    public void InsertTool(BallMill tool) { ... } 
    public BallMill QueryTool(string toolID) { ... } 
    public void UpdateTool(BallMill tool) { ... } 
    public void DeleteTool(string toolID) { ... } 
} 

class DiamondToolDBHandler{ 

    public DiamondToolDBHandler(){ ... } 

    public void InsertTool(DiamondTool tool) { ... } 
    public DiamondTool QueryTool(string toolID) { ... } 
    public void UpdateTool(DiamondTool tool) { ... } 
    public void DeleteTool(string toolID) { ... } 
} 

Tomé la mayoría de los métodos de cerca duplicado y los refactorizado a cabo en una clase BaseToolDBHandler(), y heredó de los otros dos, que proporciona unos métodos abstractos y propiedades para manejar las diferencias en el acceso a los parámetros de la base de datos.

¿Tendría sentido hacer que la BaseToolDBHandler sea una clase auxiliar en su lugar, contenida dentro de los accesadores de la base de datos, y proporcionarles una interfaz común a las propiedades/métodos previamente abstractas? ¿O debería dejarlo como un caso de herencia?

Answer 1

Parece una situación que beneficiaría tanto a los genéricos como a la herencia a través de una clase/interfaz base.

class DBHandler<TTool> where TTool : ToolBase // or ITool 
{ 
    public DBHandler(){ ... } 

    public void InsertTool(TTool tool) { ... } 
    public TTool QueryTool(string toolID) { ... } 
    public void UpdateTool(TTool tool) { ... } 
    public void DeleteTool(string toolID) { ... } 
} 

Si es necesario, se puede hacer una base de clase (opcionalmente abstracto), o una interfaz para su uso como una restricción de tipo que garantice ciertos miembros que se necesitan las herramientas para tener el interior de los cuerpos de los métodos.

Ejemplos:

var handler = new DBHandler<BallMill>(); 
BallMill value = handler.QueryTool("xyz"); 
value.SomeProperty = "New Value"; 
handler.UpdateTool(value); 

Answer 2

La herencia tiende a ser usada en exceso, pero parece ser un caso apropiado para ella. En muchos casos, las personas solo ven código duplicado y piensan "Usaré la herencia para eliminar el código duplicado a través de una clase base". En realidad, la mayoría del código duplicado se puede refactorizar en otra clase completamente utilizada en varios lugares. Usualmente las clases que lo usan no están "relacionadas", están haciendo dos cosas completamente diferentes y simplemente comparten la necesidad de hacer algunas (o algunas series de) tareas pequeñas.

La herencia se debe usar cuando las clases realmente pueden decir que la clase de clase "es" una instancia de la clase base, no solo una clase que necesita acceso a un conjunto de métodos definidos en otra parte. En su caso particular, está claro que ambas clases son controladores de bases de datos. En su punto crucial, ambos están cumpliendo el mismo objetivo general, pero son dos posibles implementaciones diferentes de ese objetivo. El único problema que pude ver aquí (dado que no has mostrado el contenido de ninguno de los métodos) es que puedes combinar ambas clases en una sola clase, pero necesitaríamos saber más sobre los detalles para saber si eso es posible o preferible.

Answer 3

Yo diría que este es sin duda una tarea para la herencia ... pero permite aclarar una cosa en primer lugar. La composición es un patrón de diseño que puede y debe usarse cuando la herencia múltiple de clases no es una característica del lenguaje dado (C# por ejemplo)

La composición implica que su clase compuesta crea instancias de clases, donde generalmente se heredarían. En este caso, usted proporciona el contrato para la implementación a través de interfaces.

Para darle un ejemplo general de cómo la herencia puede aplicar a su código, considere esto: (esto no está utilizando la composición)

public interface IHandler 
{ 
    void InsertTool(ITool tool); 
    void UpdateTool(ITool tool); 
    void DeleteTool(string toolID); 
    //void DeleteTool(ITool tool) - seems more consistent if possible!? 

    ITool QueryTool(string toolID); 
} 

public interface ITool 
{ 

} 

class BallMill : ITool 
{ 
} 

class DiamondTool : ITool 
{ 
} 

class BallMillDBHandler : IHandler 
{ 

    public BallMillDBHandler(){ ... } 

    public void InsertTool(ITool tool) { ... } 
    public BallMill QueryTool(string toolID) { ... } 
    public void UpdateTool(ITool tool) { ... } 
    public void DeleteTool(string toolID) { ... } 
} 

class DiamondToolDBHandler : IHandler 
{ 

    public DiamondToolDBHandler(){ ... } 

    public void InsertTool(ITool tool) { ... } 
    public DiamondTool QueryTool(string toolID) { ... } 
    public void UpdateTool(ITool tool) { ... } 
    public void DeleteTool(string toolID) { ... } 
} 

Answer 4

En lugar de herencia, uso de medicamentos genéricos. Desea realizar la misma operación en diferentes tipos de objetos, que es exactamente lo que hacen los genéricos.