¿Combinación de DI con parámetros de constructor?

Question

¿Cómo combino la inyección del constructor con los parámetros del constructor "manual"? es decir.

public class SomeObject 
{ 
    public SomeObject(IService service, float someValue) 
    { 
    } 
} 

Donde IService debe ser resuelto/inyectado por mi contenedor DI, y se debe especificar someValue. ¿Cómo mezclo los dos?

Answer 1

Dichas construcciones deben evitarse siempre que sea posible. Por lo tanto, pregúntate: ¿este parámetro es realmente necesario como argumento de constructor? ¿O puede SomeObject ser reemplazado por uno sin estado que es reutilizado por todos los que dependen de él pasando el parámetro al método que ejecuta en el objeto?

p. Ej.En lugar de

public class SomeObject 
{ 
    private float someValue 
    public SomeObject(IService service, float someValue) 
    { 
     this.someValue = someValue 
    } 

    public float Do(float x) 
    { 
     return this.Service.Get(this.someValue) * x; 
    } 
} 

uso

public class SomeObject 
{ 
    public SomeObject(IService service) 
    { 
    } 

    public float Do(float x, float someValue) 
    { 
     return this.Service.Get(someValue) * x; 
    } 
} 

---

Si es necesario ir a una fábrica:

public interface ISomeObjectFactory 
{ 
    ISomeObject CreateSomeObject(float someValue); 
} 

public class SomeObjectFactory : ISomeObjectFactory 
{ 
    private IKernel kernel; 
    public SomeObjectFactory(IKernel kernel) 
    { 
     this.Kernel = kernel; 
    } 

    public ISomeObject Create(float someValue) 
    { 
     return this.kernel.Get<ISomeObject>(WithConstructorArgument("someValue", someValue); 
    } 
} 

Vista previa: Ninject 2.4 no requerirá de la implementación más, pero permitirá

kernel.Bind<ISomeObjectFactory>().ToFactory(); // or maybe .AsFactory(); 

Answer 2

Si 'somevalue' es siempre constante, entonces se puede pensar en el uso de InjectionParameters mientras estás registrar su tipo con el contenedor, ya que se explica en el siguiente mensaje

See Here

pero si eso no es verdad, que no hay forma de separar un valor de parámetro mientras se resuelve una instancia, puede pensar en mover el 'algún Valor' del constructor y convertirlo en una propiedad de la clase.

Answer 3

Probablemente usaría una solución ingenua para esto. Si conoce el valor de someValue cuando lo necesite, lo eliminaría del constructor y agregaría una propiedad a su objeto para que pueda establecer someValue. De esta forma, puede obtener su objeto de su contenedor y luego establecer el valor cuando tenga el objeto.

Mi otra sugerencia es que, en lugar de acceder directamente a ella, cree una fábrica que pueda usar para crear dicho objeto. Luego registra la fábrica en su contenedor y utiliza la fábrica para crear su instancia. Algo como esto:

public class SomeObjectFactory : ISomeObjectFactory 
{ 
    private IYourService _service; 
    public SomeObjectFactory(IYourService service) 
    { 
     _service = service; 
    } 

    public ISomeObject Create(float someValue) 
    { 
     return new SomeObject(_service, someValue); 
    } 
} 

puede probar un patrón así.

ACTUALIZACIÓN: Actualizó el código para reflejar los comentarios de mejora.

Answer 4

Realmente no debería intentar usar D.I. para esto. Podrías encontrar todo tipo de soluciones alocadas, pero pueden no tener sentido en el futuro.

Nuestro enfoque es crear una fábrica a través de D.I., y el método de creación de la fábrica se construiría utilizando el aprobado D.I. envase. No es necesario que utilicemos este patrón a menudo, pero cuando lo hacemos realmente hace que el producto sea mucho más limpio (ya que hace que nuestros gráficos de dependencia sean más pequeños).

Answer 5

En NInject, con el que ha etiquetado esto, inyecta una fábrica generada automáticamente en forma de Func<parameters you wish to feed in,T>, utilizando FuncModule as described in this post.

Este enfoque también está disponible en autofac para uno.

Varios Factory method approaches are covered in the answers to this question.

EDIT: NB Si bien esto puede ser entretenido, por favor utilice la solución de @Remo Gloor (y críticamente el consejo re evitando una solución de esta naturaleza)