Iniezione di dipendenze in un loop

Question

Qual è il modo corretto di fare quanto segue in modo che ParentClass non dipenda da MyClass?

public class ParentClass 
{ 
    public void MyFunction(IList<Foo> foos) 
    { 
     foreach (var bar in foos) 
     { 
      var myClass = new MyClass(); 
      myClass.DoStuff(); 
     } 
    } 
} 

normalmente senza un ciclo avevo appena iniettare con il costruttore ParentClass, ma qui ho bisogno di una nuova istanza per ogni iterazione del ciclo.

O forse c'è un modo migliore per fare ciò che sto cercando di ottenere? Forse myClass potrebbe resettarsi alla fine di ogni iterazione in modo che possa riutilizzarlo?

Answer 1

È possibile iniettare un oggetto factory che crea istanze MyClass in ParentClass.

Per ogni iterazione del ciclo, si chiama l'oggetto factory in modo da fornire una nuova istanza di MyClass.

Answer 2

È possibile iniettare un Func<IMyClass> nel costruttore, quindi richiamarlo tutte le volte che si desidera ottenere istanze diverse.

I framework IoC come Autofac supportano questo in modo nativo.

Answer 3

Dipende davvero da cosa si vuole fare con MyClass. Allo stato attuale, non vi è alcuna correlazione tra foo's, bar e MyClass. Supponendo che si debba iniettare una barra nel Ctor di MyClass, o passarla come parametro a DoStuff, allora semplicemente inserirò una classe factory o un metodo nel ParentClass Ctor e lo useremo al posto di new-up su MyClass.

Answer 4

L'iniezione proposta è leggibile, comprensibile e facile da verificare per la correttezza.

Il ripristino dell'oggetto è un'opzione se il loop è di grandi dimensioni o se MyClass è costoso da creare (ad esempio apre una connessione, un handle di file, un sottoprocesso o avvia un thread).

Poiché la soluzione reset() richiede più tempo e concentrazione per verificare su ciascuna modifica del codice, consiglio di misurare effettivamente che si tratta di un collo di bottiglia prima di preoccuparsi.