Question

我正在使用EF5并且在POCO课程中有实体。我的第一个问题是实施业务规则和验证的最佳位置是什么？

我的第一个猜测是将它直接放入POCO类中，放入一些Validate（）函数，当触发SaveChanges（）时，该函数将从DBContext调用。这很有效，但有些规则要求在多个实体之间进行验证类Invoice的示例：

if(this.Items.Where(i=>i.Price > 100).Count() > 0)
{
    //mark invoice for review
    this.IsForReview = true;
}

现在，单元测试将测试验证功能（针对每个业务规则），但也必须使用Items填充发票类（否则它将始终为空）

另一个想法是创建一个InvoiceValidation类，它具有单独的验证函数（甚至每个规则的类？），它们更容易进行单元测试，但它确实增加了要维护的文件/类的数量。

任何有关现有解决方案的建议或链接都将受到赞赏

Answer 1

最好的方法取决于您的依赖关系。 POCO /核心组件是否依赖于EF？您是否将Access数据库注入核心库程序集？等

我个人使用repository / luw模式，其中各种存储库对象继承自通用的基础存储库对象。 DAL取决于EF，但核心的POCO类别不是。

存储库子类具有特定类型，并且对其他OBEJCT业务进行检查。 IE业务规则要求检查其他实体，我在DAL中实现。

存储库类属于数据访问层Project和DO依赖于EF并注入了Context。示例如下。

检查我在POCO上执行的实例的具体情况。需要数据库访问的检查我是通过在基类库存类上实现的接口来执行的，该类是根据需要重写的。因此，当添加或更改对象时，会触发 CheckEntity 。

例如 ...注意删除一些代码以保持示例相关...

 public class RepositoryEntityBase<T> : IRepositoryEntityBase<T>, IRepositoryEF<T> where T : BaseObject
public virtual OperationStatus Add(T entity)
    {
        var opStatus = new OperationStatus(status: true, operation: OperationType.Add);
        try
        {
            if (OnBeforeAdd != null) // registered listeners of added event?
            {
                var evtArg = PrepareEventArgs(entity, MasterEventType.Create);
                OnBeforeAdd(this, evtArg);
            }
            opStatus = CheckBeforePersist(entity);
            if (opStatus.Status)
            {
                Initialize(entity);
                EntityDbSet.Add(entity);

                if (OnAfterAdd != null) // registered listeners of added event?
                {
                    var evtArg = PrepareEventArgs(entity, MasterEventType.Create);
                    OnAfterAdd(this, evtArg);
                }
            }
        }
        catch (Exception ex)
        {
            opStatus.SetFromException("Error Adding " + typeof(T), ex);
        }
        return opStatus;
    }

//... then in a specific repository class



//... irepositorybase expects Check before persist.

  public override OperationStatus CheckBeforePersist(MasterUser entity)
    {

        // base entity rule check first
        var opStatus = new OperationStatus(true, OperationType.Check);          
        opStatus.ValidationResults  = base.CheckEntity(entity);      
        if (opStatus.ValidationResults.Count > 0)
        {
            opStatus.Status = false;
            opStatus.Message = "Validation Errors";
            return opStatus;
        }


        //now check the local memory
        var masterUser = Context.Set<MasterUser>().Local   //in context
                                              .Where(mu => mu.Id != entity.Id // not this record
                                                    &&     mu.UserName == entity.UserName ) // same name
                                              .FirstOrDefault();
        if (masterUser != null)
        {
            opStatus.Status = false;
            opStatus.Message = "Duplicate UserName :" + masterUser.UserName + " UserId:"+ masterUser.Id.ToString();
            return opStatus;
        }
        masterUser = Context.Set<MasterUser>().Local   //in context
                                             .Where(mu => mu.Id != entity.Id // not this record
                                                   && mu.Email == entity.Email) // same email
                                             .FirstOrDefault();
        if (masterUser != null)
        {
            opStatus.Status = false;
            opStatus.Message = "Duplicate Email :" + masterUser.Email + " Username:" + masterUser.UserName;
            return opStatus;
        }                                               

        // now check DB
        masterUser = Get(mu => mu.Id != entity.Id             //not this record being checked
                          && mu.UserName == entity.UserName);     // has same username
        if (masterUser != null)
        {
            opStatus.Status = false;
            opStatus.Message = "Duplicate UserName :" + masterUser.UserName + " UserId:"+ masterUser.Id.ToString();
            return opStatus;
        }
        masterUser = Get(mu => mu.Id != entity.Id    // not this record
                      && mu.Email == entity.Email);  // but same email 
        if (masterUser != null)
        {
            opStatus.Status = false;
            opStatus.Message = "Duplicate Email:" + masterUser.Email + " UserName:"+ masterUser.UserName;
            return opStatus;
        }
        return opStatus;   
    }

}

Answer 2

我建议使用类似Fluent验证（http://fluentvalidation.codeplex.com/）的内容，它允许您收集规则集合并将它们放在一个单独的上下文中，与它正在验证的POCO类别分开。

Answer 3

如果有人有兴趣，这是我到目前为止找到的最好的例子：

http://codeinsanity.com/archive/2008/12/02/a-framework-for-validation-and-business-rules.aspx

如何为POCO类实现和测试复杂的业务规则

3 个答案: