Quando spostare un campo comune in una classe base?

Al momento ho due classi derivate Ae B, che hanno entrambe un campo in comune e sto cercando di determinare se dovrebbe andare nella classe base.

Non viene mai referenziato dalla classe base, e dire se in qualche punto lungo la strada viene derivata un'altra classe C, che non ha un _field1, quindi il principio del "meno privilegiato" (o qualcosa) non sarebbe violato se era?

public abstract class Base
{
    // Should _field1 be brought up to Base?
    //protected int Field1 { get; set; }
}

public class A : Base
{
    private int _field1;
}

public class B : Base
{
    private int _field1;
}

public class C : Base
{
    // Doesn't have/reference _field1
}

Tutto dipende dal problema esatto che stai cercando di risolvere.

Considera un esempio concreto: la tua classe base astratta è Vehiclee al momento hai le implementazioni concrete Bicyclee Car. Stai pensando di spostarti numberOfWheelsda Bicyclee Carverso il veicolo. Dovresti farlo? No! Perché non tutti i veicoli hanno le ruote. Puoi già dire che se provi ad aggiungere una Boatclasse, si romperà.

Ora, se la tua classe di base astratta era WheeledVehicleallora è logico avere la numberOfWheelsvariabile membro all'interno.

Devi applicare la stessa logica al tuo problema, perché come puoi vedere, non è una semplice risposta sì o no.

Logicamente parlando, oltre a posizionare il campo replicato in sottoclassi e in comune nella classe base, esiste una terza opzione: introdurre una nuova sottoclasse nella gerarchia che ha le proprietà comuni tra le due. @Pete suggerisce questo senza andare completamente lì.

Usando l'esempio di @Pete, introdurremmo una sottoclasse (possibilmente astratta) per il veicolo a ruote che discende dalla classe base originale - mentre le due sottoclassi discendono da essa. Pertanto, la classe base originale non è inquinata con le ruote, tuttavia la comunanza delle ruote è ASCIUTTA (non ripetuta tra le sottoclassi che hanno ruote).

Questo può ovviamente essere eccessivo per i tuoi scopi, ma è supportato dal meccanismo della gerarchia di classi.

Qui interpreterò l'avvocato del diavolo.

In questo momento non dovresti fare nulla .

È ASCIUTTO? No. Ma è meglio avere una piccola duplicazione rispetto a un'astrazione prematura di cui non puoi facilmente tornare indietro in seguito. Il refactor per spostare una proprietà in una classe base comune è semplice. Andare dall'altra parte non lo è. Aspetta e vedi.

Quando prendo questo tipo di decisione, tendo a usare una "regola del 3": una volta che ho ripetuto la stessa cosa in tre posti diversi, e solo allora, prendo in considerazione l'idea di spostarlo su per la catena. NB sei solo a 2.

In generale, lo sposterei nella classe base. Non penso che ci sia un obiettivo sì / no, perché qui c'è un compromesso: trasportare campi inutilizzati contro ridurre la complessità.

Preferisco di solito classi base "pesanti" che contengono tutto ciò che potrebbe essere condiviso. Ciò semplifica la serializzazione dei file poiché non sono necessari metodi di serializzazione discendenti in ogni classe derivata. Ma se non hai questo o un problema simile, o forse devi fare tutto il possibile per ridurre l'uso della memoria, allora solo i campi dove ne hai bisogno dovrebbero andare bene.

Una classe "intermediaria" che introduce i campi comuni andrà bene se hai un numero molto limitato di campi. Ma tieni presente che l'approccio può aumentare notevolmente la complessità se hai dozzine di campi utilizzati in combinazioni diverse, portando a molte classi intermedie ognuna delle quali introduce un insieme specifico di campi. Questo può diventare un problema di manutenzione.