服务层依赖注入

Question

我正在创建一个将调用服务层的Web API，并且尝试学习依赖注入（我希望使用ninject），但是我不确定如何在服务层上创建依赖关系。

这就是Web API所要调用的。

这里的问题是，在调用IPersonService时，该人员将定义性别，名称，角色和种族。 我正在使用构造函数注入，不确定是否应该调用GenderService还是应该调用业务层（在这种情况下，由Core定义）。

我应该像上图还是下图那样致电服务

这就是我的人事服务的样子

namespace Service.Services
{
    public class PersonService : IPersonService
    {
        private IPersonCore personCore = null;
        private INameService nameService = null;
        private IRoleService roleService = null;
        private IGenderService genderService = null;
        private IEthnicityService ethnicityService = null;
        private IPrefixService prefixService = null;
        private Person currUser;

        public PersonService(IPersonCore _personcore, INameService _namecore, IRoleService _roleservice, IGenderService _genderservice, IEthnicityService _ethnicityservice, IPrefixService _prefixservice )
        {
            this.personCore = _personcore;
            this.nameService = _namecore;
            this.roleService = _roleservice;
            this.genderService = _genderservice;
            this.ethnicityService = _ethnicityservice;
            this.prefixService = _prefixservice;
        }

        public IEnumerable<Person> GetAllPerson()
        {
            if (isAuthorized())
            {
                return this.personCore.GetPersons();
            }
            return null;
        }

        public Person GetPersonByID(int id)
        {
            if (isAuthorized())
            {
                return this.personCore.GetPersonByID(id);
            }
            return null;
        }

        public Person GetPersonByEmail(string email)
        {
            if (isAuthorized())
            {
                return this.personCore.GetPersonByEmail(email);
            }
            return null;
        }

        public IEnumerable<Person> GetPersonByName(string first, string last, string middle)
        {
            if(isAuthorized())
            { 
                Name newname = this.nameService.CreateName(first, last, middle);
                return this.personCore.GetPersonByName(newname);
            }
            return null;
        }

        public IEnumerable<Person> GetPersonWithRoles(IEnumerable<Roles> r)
        {
        }

        public IEnumerable<Person> GetPersonWithDOB(DateTime d)
        {
            if (isAuthorized())
            {
                return this.personCore.GetPersonWithDOB(d);
            }
            return null;
        }

        public Person SetPersonRole(int id, Roles r)
        {
        }

        public Person SetGender(int id, Gender g)
        {
        }

        public Person SetEthnicity(int id, Ethnicity e)
        {
        }

        public Person SetPrefix(int id, Prefix p)
        {
        }

        public Person CreatePerson(Person p)
        {
            if (isAuthorized())
            {
                return personCore.AddPerson(p);
            }
            return null;
        }

        public Person UpdatePerson(Person p)
        {
            if (isAuthorized())
            {
                return personCore.UpdatePerson(p);
            }
            return null;
        }

        public Person ActivatePerson(int id)
        {
            if (isAuthorized())
            {
                return personCore.ActivatePerson(id);
            }
            return null;
        }

        public Person DeactivatePerson(int id)
        {
            if (isAuthorized())
            {
                return personCore.DeactivatePerson(id);
            }
            return null;
        }

        public bool DeletePerson(int id)
        {
            if (isAuthorized())
            {
                return personCore.DeletePerson(id);
            }
            return false;
        }

        protected bool isAuthorized()
        {
            //Probably move to common
            return true;
        }
    }
}

从Web API调用时，我的问题是，它听起来很依赖于查找某个人的东西。

Answer 1

PersonService类包含许多依赖项，因为您违反了“ 单一职责原则” 。 此类有很多责任，每次添加新功能（违反开放式/封闭式原理 ）时，您都将不得不更改此类。 此外， isAuthorized方法是一个横切关注点，此类不应该涉及任何概念。

最重要的是，您正在将当前登录的用户注入PersonService 。 这是运行时数据， 使用运行时数据构造应用程序组件是一种反模式 。

有很多方法可以解决此问题，但这全都在于理解SOLID原理。 但是，这样做可能会是一项艰巨的任务，特别是如果您刚开始使用DI和软件设计。 您可以阅读许多关于此的书籍，例如Robert C. Martin 的出色著作和有关依赖注入的书籍，例如Mark Seemann 在.NET中的依赖注入 。

最近几年对我有很大帮助的设计是基于消息的体系结构，用例通过消息描述用例，而实现则通过通用抽象描述（请阅读this和this ）。 这些设计被证明是非常灵活和可维护的，因为它们允许透明地添加横切关注点，并允许在不更改任何现有代码的情况下添加新功能。 这种设计还可以将您的Web API层简化为一个简单的基础架构，在添加新功能时不需要更改。 您可以在此处阅读有关此概念的信息 （注意：该文章是关于WCF的，但是Web API的概念是相同的），并且这里是Github存储库，显示了如何在WCF和Web API中实现此概念。

祝您在掌握软件方面万事如意。

Answer 2

您可以通过两种方式简化此操作：

1. 您的PersonService不会依赖于角色，性别，种族和前缀服务，因为您不会从其方法中调用它们。 您的客户端代码调用的是一个外壳，而不是直接自己调用这些服务。 如果是这种情况，那么可以通过除去以下4个依赖项来简化PersonService：

    private IRoleService roleService = null; private IGenderService genderService = null; private IEthnicityService ethnicityService = null; private IPrefixService prefixService = null; 

   并将这些方法提供给它们各自的服务：

    public Person SetPersonRole(int id, Roles r) { } public Person SetGender(int id, Gender g) { } public Person SetEthnicity(int id, Ethnicity e) { } public Person SetPrefix(int id, Prefix p) { } 

2. 如果绝对需要将这些方法保留在IPersonService中，则可以将依赖项注入方法而不是构造函数中。

   关于对服务或核心的依赖性，取决于您服务的功能。 如果您的服务只是调用Core，请自己转到Core。 如果您的服务正在执行某种验证或其他任何验证，则可能需要依赖它以避免在PersonService中复制相同的代码。