Зависимость впрыска в петле

Question

Каков правильный способ сделать ниже, так что ParentClass не зависит от MyClass?

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

Обычно без цикла я просто впрыснуть его с конструктором ParentClass, но здесь мне нужен новый экземпляр, для каждой итерации цикла.

Или, может быть, есть лучший способ сделать то, что я пытаюсь достичь? Возможно, myClass может перезагрузиться в конце каждой итерации, чтобы я мог повторно использовать его?

Answer 1

Вы можете ввести фабричный объект, который создает экземпляры MyClass в ParentClass.

Для каждой итерации цикла вы вызываете фабричный объект, чтобы он дал вам новый экземпляр MyClass.

Answer 2

Вы можете ввести Func<IMyClass> в конструктор, а затем вызвать его столько раз, сколько хотите получить разные экземпляры.

Основы IoC, такие как Autofac, поддерживают это изначально.

Answer 3

Это действительно зависит от того, что вы хотите делать с MyClass. Как бы там ни было, нет никакой корреляции между foo, bar и MyClass. Предполагая, что вы должны либо ввести планку в Ctor MyClass, либо передать ее как параметр для DoStuff, тогда я бы просто ввел заводский класс или метод в ParentClass Ctor и использовал это вместо нового MyClass.

Answer 4

Предлагаемая инъекция читаема, понятна и проста для проверки правильности.

Сброс объекта является опцией, если цикл большой или если MyClass дорог для создания (например, он открывает соединение, дескриптор файла, подпроцесс или запускает поток).

Поскольку решение reset() требует больше времени и концентрации для проверки при каждом изменении кода, я рекомендую фактически измерить, что это узкое место, прежде чем беспокоиться.