有没有比这更好的方法将函数注入Unity中的对象？

Question

我是一个Unity noob,我有一个问题.我想在每个IPerson实例中注入一个IAgeCalculator,以便IAgeCalculator实例可供我以后创建的任何IPerson使用.

这是我到目前为止所尝试的.它有效,但感觉不对.

static void Main(string[] args) 
{
    IUnityContainer container = new UnityContainer();
    container.RegisterType<IAgeCalculator, AgeInYearsCalculator>("Years");
    container.RegisterType<IAgeCalculator, AgeInDaysCalculator>("Days");

    // Using the initializer like this does not feel right, 
    // but I cant think of another way...

    var ageCalculator = container.Resolve<IAgeCalculator>("Days");
    var personInitializer = new InjectionMethod("Initializer", ageCalculator);
    container.RegisterType<IPerson, Person>(personInitializer);

    var person1 = Factory<IPerson>.Create(container);
    person1.Name = "Jacob";
    person1.Gender = "Male";
    person1.Birthday = new DateTime(1995, 4, 1);

    var person2 = Factory<IPerson>.Create(container);
    person2.Name = "Emily";
    person2.Gender = "Female";
    person2.Birthday = new DateTime(1998, 10, 31);
}

这是我的类定义:

public interface IAgeCalculator 
{
    string GetAge(IPerson person);
}

internal class AgeInYearsCalculator : IAgeCalculator
{
    public string GetAge(IPerson person) {
        var years = DateTime.Now.Subtract(person.Birthday).TotalDays / 365;
        return years.ToString("0.00") + " years";
    }
}

internal class AgeInDaysCalculator : IAgeCalculator 
{
    public string GetAge(IPerson person) {
        var days = (DateTime.Now.Date - person.Birthday).TotalDays;
        return days.ToString("#,#") + " days";
    }
}

public interface IPerson
{
    string Name { get; set; }
    string Gender { get; set; }
    DateTime Birthday { get; set; }
    string Age { get; }
}

public class Person : IPerson
{
    private IAgeCalculator _ageCalculator;
    public void Initializer(IAgeCalculator ageCalculator) {
        _ageCalculator = ageCalculator;
    }

    public string Name { get; set; }
    public string Gender { get; set; }
    public DateTime Birthday { get; set; }
    public string Age => _ageCalculator.GetAge(this);
}

public class Factory<T> 
{
    public static T Create(IUnityContainer container) {
        return container.Resolve<T>();
    }
}

有没有更好的办法？

Answer 1

DI容器的目的是构建应用程序组件的对象图.应用程序组件通常是包含应用程序行为的长寿命对象.组件通常是不可变的,它们实现的服务抽象应始终是不可变的.

DI容器并不意味着构建短期和可变对象,如DTO和实体(如您的Person对象).解析或注入可变的,短期的数据对象会导致歧义.例如,如果我们注射一个Person,我们期望得到哪个人？

这通常意味着您不应对数据对象使用构造函数注入.在您的情况下,甚至不需要IAgeCalculator依赖于Person对象.消费者很容易Person依赖IAgeCalculator并呼唤IAgeCalculator.GetAge正确的人.

如果您想在您的实体中放置特定于域的逻辑,则Method Injection非常适合.这意味着负责调用该域方法的组件负责转发所需的服务.例如:

public void Handle(AddItemToBasketCommand command)
{
    var product = this.productRepository.Get(command.ProductId);
    var person = this.personRepository.Get(this.userContext.UserId);

    if (product.IsAdultProduct && person.GetAge(this.ageCalculator) < 18) {
        throw new ValidationException("You must be 18 to order this item.");
    }

    var basket = this.basketService.GetUserBasket();

    basket.AddItemToBasket(command.ProductId, command.Quantity);
}

请注意,我的Person.GetAge方法返回年龄.在您的情况下,您AgeCalculator返回该人年龄的字符串表示.这不属于实体本身的责任.这是一个介绍责任; 它会因为不同的原因而改变,而不是为什么其他Person类会改变.这是在实体内部没有这种逻辑的另一个原因.

还要注意,IPerson对你的实体进行抽象通常没有多大意义.实体是数据容器,而接口则用于抽象行为.Person您没有想要抽象的行为,因为您不太可能希望Person将来有多个实现.