将保存/预保存方法放在域对象中的什么位置？

Question

我想在每次保存域对象时强制执行一些规则，但我不知道实现此目的的最佳方法。正如我所看到的，我有两个选择：向域对象添加保存方法，或者在保存到应用层之前处理规则。请参阅下面的代码示例：

using System;

namespace Test
{

    public interface IEmployeeDAL
    {
        void Save(Employee employee);
        Employee GetById(int id);
    }

    public class EmployeeDALStub : IEmployeeDAL
    {
        public void Save(Employee employee)
        {

        }

        public Employee GetById(int id)
        {
            return new Employee();
        }
    }

    public interface IPermissionChecker
    {
        bool IsAllowedToSave(string user);
    }

    public class PermissionCheckerStub : IPermissionChecker
    {
        public bool IsAllowedToSave(string user)
        {
            return false;
        }
    }

    public class Employee
    {
        public virtual IEmployeeDAL EmployeeDAL { get; set; }
        public virtual IPermissionChecker PermissionChecker { get; set; }

        public int Id { get; set; }
        public string Name { get; set; }

        public void Save()
        {
            if (PermissionChecker.IsAllowedToSave("the user"))  // Should this be called within EmployeeDAL?
                EmployeeDAL.Save(this);
            else
                throw new Exception("User not permitted to save.");
        }
    }

    public class ApplicationLayerOption1
    {
        public virtual IEmployeeDAL EmployeeDAL { get; set; }
        public virtual IPermissionChecker PermissionChecker { get; set; }

        public ApplicationLayerOption1()
        {
            //set dependencies
            EmployeeDAL = new EmployeeDALStub();
            PermissionChecker = new PermissionCheckerStub();
        }

        public void UnitOfWork()
        {
            Employee employee = EmployeeDAL.GetById(1);

            //set employee dependencies (it doesn't seem correct to set these in the DAL);
            employee.EmployeeDAL = EmployeeDAL;
            employee.PermissionChecker = PermissionChecker;

            //do something with the employee object
            //.....

            employee.Save();
        }
    }

    public class ApplicationLayerOption2
    {
        public virtual IEmployeeDAL EmployeeDAL { get; set; }
        public virtual IPermissionChecker PermissionChecker { get; set; }

        public ApplicationLayerOption2()
        {
            //set dependencies
            EmployeeDAL = new EmployeeDALStub();
            PermissionChecker = new PermissionCheckerStub();
        }

        public void UnitOfWork()
        {
            Employee employee = EmployeeDAL.GetById(1);

            //do something with the employee object
            //.....

            SaveEmployee(employee);
        }

        public void SaveEmployee(Employee employee)
        {
            if (PermissionChecker.IsAllowedToSave("the user"))  // Should this be called within EmployeeDAL?
                EmployeeDAL.Save(employee);
            else
                throw new Exception("User not permitted to save.");
        }
    }
}

遇到这种情况你会怎么做？

Answer 1

我更喜欢第二种方法，其中关注点之间有明确的分离。有一个类负责 DAL，还有一个类负责验证，还有一个类负责编排这些。

在您的第一种方法中，您将 DAL 和验证注入到业务实体中。我可以争论将验证器注入实体是否是一种好的做法，将 DAL 注入业务实体绝对不是一个好的实践恕我直言（但我知道这只是一个演示，在一个真实的项目中你会至少为此使用服务定位器）。

Answer 2

如果必须选择，我会选择第二个选项，这样我的实体就不会与任何 DAL 基础架构相关联，而是完全专注于域逻辑。

但是，我不太喜欢这两种方法。我更喜欢通过向我的应用程序服务方法添加属性来对安全性和角色采取更多 AOP 方法。

我要改变的另一件事是摆脱“CRUD”的心态。如果您针对特定命令/用例进行保护，则可以提供更精细的安全选项。例如，我会做到：

public class MyApplicationService
{
    [RequiredCommand(EmployeeCommandNames.MakeEmployeeRedundant)]
    public MakeEmployeeRedundant(MakeEmployeeRedundantCommand command)
    {
        using (IUnitOfWork unitOfWork = UnitOfWorkFactory.Create())
        {
            Employee employee = _employeeRepository.GetById(command.EmployeeId);

            employee.MakeRedundant();

            _employeeRepository.Save();
        }
    }
}

public void AssertUserHasCorrectPermission(string requiredCommandName)
{
    if (!Thread.CurrentPrincipal.IsInRole(requiredCommandName))
        throw new SecurityException(string.Format("User does not have {0} command in their role", requiredCommandName));
}

您将拦截对第一个方法的调用并调用第二个方法，传递他们在其角色中必须拥有的东西。

这里有一个关于如何使用unity进行拦截的链接：http://litemedia.info/aop-in-net-with-unity-interception-model

Answer 3

将保存/预保存方法放在域对象中的什么位置？

域对象在 DDD 中是持久无知的。他们不知道有时他们会被“冷冻”运输到一些仓库然后恢复。他们没有注意到这一点。换言之，域对象始终处于“有效”和可保存状态。

权限也应该是持久无知的，并且基于域和通用语言，例如：

只有 Sales 组的用户可以将 OrderLines 添加到订单中 待处理状态

相对于：

只有销售组的用户才能保存订单。

代码可能如下所示：

internal class MyApplication {

    private IUserContext _userContext;
    private ICanCheckPermissions _permissionChecker;

    public void AddOrderLine(Product p, int quantity, Money price, ...) {

     if(!_permissionChecker.IsAllowedToAddOrderLines(_userContext.CurrentUser)) {
         throw new InvalidOperationException(
            "User X is not allowed to add order lines to an existing order");
     }

     // add order lines

    }
}