Chiamare il metodo di istanza da un costruttore in C#

Question

Ho un costruttore piuttosto lungo che sta eseguendo vari lavori di inizializzazione e, in quanto tale, ho voluto suddividere parte di questo lavoro in alcune funzioni. Questo mi ha portato a chiedermi se dovrei rendere le suddette funzioni istanza o metodi statici. Capisco il rischio di chiamare una funzione virtuale da un costruttore, ma penso anche che non ci sia qualcosa di giusto nel chiamare un metodo di istanza su un oggetto che non è al 100% istanziato. Sicuramente questa è una contraddizione in termini.

Sarei interessato all'opinione della gente su questo argomento. Ho anche scoperto che utilizzando un metodo statico per restituire una variabile inizializzata, potrei rendere il membro target di sola lettura. Ecco un'illustrazione semplificata del mio scenario.

public class A 
{ 
    private readonly string _foo; 

    public A() 
    { 
     _foo = InitialiseFoo(); 
    } 

    private static InitialiseFoo() 
    { 
     // Do stuff 
     return new string ("foo"); 
    } 
} 

Answer 1

Questo è abbastanza normale per chiamare metodo di istanza in un costruttore, inoltre metodo che fare di inizializzazione. Quindi, in pratica si tratta di una sorta di Extract Method refactorig per ridurre un corpo metodo di costruzione, quindi si estrae parte di inizializzazione in un metodo separato e costruttore è a conoscenza sul risparmio degli argomenti di input, ecc ...

Per quanto riguarda il modificatore static .. a volte (credo che quando non ci sono altri modi per refactor perché questo non sembra buono - dal mio punto di vista) è necessario chiamare un metodo per passare i risultati in un costruttore di base, quindi in questo caso è necessario contrassegnarlo come statico per chiamare in un contesto statico in altri casi lasciare senza il modificatore static

public A() 
: base(GetLogger()) 
{ 
} 

private static ILog GetLogger() ... 

Answer 2

posso capire il desiderio di utilizzare solo membri statici in un costruttore, beca usarlo rende il codice più semplice da usare senza dover tenere traccia di ciò che è stato inizializzato e cosa no, ma probabilmente stai rendendo inutilmente complicate le cose per te stesso. Chiamare un metodo di istanza in C# va bene finché hai una buona ragione per farlo. Ad esempio, se si dispone di un numero di costruttori che eseguono tutte le attività comuni, la creazione di una singola funzione membro per eseguire il lavoro è più semplice da gestire rispetto al copia e incolla del codice per ogni costruttore. Puoi anche immaginare un caso in cui il metodo può essere riutilizzato al di fuori del costruttore per qualcosa come il reset della classe allo stato inizializzato.

Il metodo statico, è valido, ma funzionerà solo nel caso in cui si stia lavorando in modo isolato e si inserisca il risultato in una variabile membro. Fornisce una sensazione di programmazione molto pulita e funzionale. Se uno qualsiasi dei lavori coinvolge uno stato di classe, comunque, diventerà brutto.