28

对于未来的访问者:对于 EF6,您可能最好使用过滤器,例如通过这个项目:https ://github.com/jbogard/EntityFramework.Filters

在我们正在构建的应用程序中,我们应用“软删除”模式,其中每个类都有一个“已删除”布尔值。在实践中,每个类都简单地继承自这个基类:

public abstract class Entity
{
    public virtual int Id { get; set; }

    public virtual bool Deleted { get; set; }
}

举一个简单的例子,假设我有类GymMemberWorkout

public class GymMember: Entity
{
    public string Name { get; set; }

    public virtual ICollection<Workout> Workouts { get; set; }
}

public class Workout: Entity
{
    public virtual DateTime Date { get; set; }
}

当我从数据库中获取健身房成员列表时,我可以确保没有获取任何“已删除”的健身房成员,如下所示:

var gymMembers = context.GymMembers.Where(g => !g.Deleted);

然而,当我遍历这些健身房成员时,他们Workouts是从数据库中加载的,而不考虑他们的Deleted标志。虽然我不能责怪 Entity Framework 没有注意到这一点,但我想以某种方式配置或拦截延迟属性加载,以便永远不会加载已删除的导航属性。

我一直在考虑我的选择,但它们似乎很少:

这根本不是一种选择,因为这将是太多的手动工作。(我们的应用程序很大,而且每天都在变大)。我们也不想放弃使用 Code First 的优势(其中有很多)

再次,不是一个选择。此配置仅适用于每个实体。总是急切地加载实体也会造成严重的性能损失。

  • 应用表达式访问者模式,该模式会自动注入.Where(e => !e.Deleted)它找到的任何地方IQueryable<Entity>,如此此处所述。

我实际上在概念验证应用程序中对此进行了测试,并且效果非常好。这是一个非常有趣的选项,但是很遗憾,它无法将过滤应用于延迟加载的导航属性。这很明显,因为那些惰性属性不会出现在表达式/查询中,因此无法替换。我想知道 Entity Framework 是否允许在他们的DynamicProxy类中的某处加载惰性属性的注入点。我也担心其他后果,例如破坏IncludeEF 机制的可能性。

  • 编写实现 ICollection 但Deleted自动过滤实体的自定义类。

这实际上是我的第一个方法。这个想法是为每个内部使用自定义 Collection 类的集合属性使用一个支持属性:

public class GymMember: Entity
{
    public string Name { get; set; }

    private ICollection<Workout> _workouts;
    public virtual ICollection<Workout> Workouts 
    { 
        get { return _workouts ?? (_workouts = new CustomCollection()); }
        set { _workouts = new CustomCollection(value); }
     }

}

虽然这种方法实际上还不错,但我仍然有一些问题:

  • 它仍然将所有Workouts 加载到内存中,并Deleted在属性设置器被命中时过滤那些。以我的拙见,这为时已晚。

  • 执行的查询与加载的数据之间存在逻辑不匹配。

想象一个场景,我想要一个自上周以来进行锻炼的健身房成员的列表:

var gymMembers = context.GymMembers.Where(g => g.Workouts.Any(w => w.Date >= DateTime.Now.AddDays(-7).Date));

此查询可能会返回一个健身房成员,该成员仅具有已删除但也满足谓词的锻炼。一旦它们被加载到内存中,就好像这个健身房成员根本没有锻炼!您可以说开发人员应该意识到Deleted并始终将其包含在他的查询中,但这是我真正想避免的事情。也许 ExpressionVisitor 可以在这里再次提供答案。

  • Deleted在使用 CustomCollection 时,实际上不可能将导航属性标记为。

想象一下这个场景:

var gymMember = context.GymMembers.First();
gymMember.Workouts.First().Deleted = true;
context.SaveChanges();`

你会期望Workout在数据库中更新相应的记录,你就错了!由于gymMember酒店正在检查ChangeTracker是否有任何变化,酒店gymMember.Workouts将突然减少 1 次锻炼。那是因为 CustomCollection 会自动过滤已删除的实例,记得吗?所以现在Entity Framework认为需要删除锻炼,EF会尝试将FK设置为​​null,或者实际删除记录。(取决于您的数据库的配置方式)。这就是我们一开始就试图避免的软删除模式!!!

我偶然发现了一篇有趣的博客文章,它覆盖了默认SaveChanges方法,DbContext因此任何带有 an 的条目EntityState.Deleted都被改回,EntityState.Modified但这再次让人感觉“hacky”而且相当不安全。但是,如果它解决了没有任何意外副作用的问题,我愿意尝试一下。


所以我在这里是 StackOverflow。如果我自己可以这么说的话,我已经非常广泛地研究了我的选择,而且我已经束手无策了。所以现在我转向你。您是如何在企业应用程序中实现软删除的?

重申一下,这些是我正在寻找的要求:

  • 查询应自动排除Deleted数据库级别的实体
  • 删除实体并调用“SaveChanges”应该只是更新相应的记录并且没有其他副作用。
  • 加载导航属性时,无论是惰性还是急切,Deleted都应自动排除这些属性。

我期待着任何和所有的建议,提前谢谢你。

4

3 回答 3

10

经过大量研究,我终于找到了实现我想要的方法。它的要点是我在对象上下文中使用事件处理程序拦截物化实体,然后将我的自定义集合类注入到我可以找到的每个集合属性中(通过反射)。

最重要的部分是拦截“DbCollectionEntry”,该类负责加载相关的集合属性。通过在实体和 DbCollectionEntry 之间摆动自己,我可以完全控制何时以及如何加载的内容。唯一的缺点是这个 DbCollectionEntry 类几乎没有公共成员,这需要我使用反射来操作它。

这是我的自定义集合类,它实现了 ICollection 并包含对适当 DbCollectionEntry 的引用:

public class FilteredCollection <TEntity> : ICollection<TEntity> where TEntity : Entity
{
    private readonly DbCollectionEntry _dbCollectionEntry;
    private readonly Func<TEntity, Boolean> _compiledFilter;
    private readonly Expression<Func<TEntity, Boolean>> _filter;
    private ICollection<TEntity> _collection;
    private int? _cachedCount;

    public FilteredCollection(ICollection<TEntity> collection, DbCollectionEntry dbCollectionEntry)
    {
        _filter = entity => !entity.Deleted;
        _dbCollectionEntry = dbCollectionEntry;
        _compiledFilter = _filter.Compile();
        _collection = collection != null ? collection.Where(_compiledFilter).ToList() : null;
    }

    private ICollection<TEntity> Entities
    {
        get
        {
            if (_dbCollectionEntry.IsLoaded == false && _collection == null)
            {
                IQueryable<TEntity> query = _dbCollectionEntry.Query().Cast<TEntity>().Where(_filter);
                _dbCollectionEntry.CurrentValue = this;
                _collection = query.ToList();

                object internalCollectionEntry =
                    _dbCollectionEntry.GetType()
                        .GetField("_internalCollectionEntry", BindingFlags.NonPublic | BindingFlags.Instance)
                        .GetValue(_dbCollectionEntry);
                object relatedEnd =
                    internalCollectionEntry.GetType()
                        .BaseType.GetField("_relatedEnd", BindingFlags.NonPublic | BindingFlags.Instance)
                        .GetValue(internalCollectionEntry);
                relatedEnd.GetType()
                    .GetField("_isLoaded", BindingFlags.NonPublic | BindingFlags.Instance)
                    .SetValue(relatedEnd, true);
            }
            return _collection;
        }
    }

    #region ICollection<T> Members

    void ICollection<TEntity>.Add(TEntity item)
    {
        if(_compiledFilter(item))
            Entities.Add(item);
    }

    void ICollection<TEntity>.Clear()
    {
        Entities.Clear();
    }

    Boolean ICollection<TEntity>.Contains(TEntity item)
    {
        return Entities.Contains(item);
    }

    void ICollection<TEntity>.CopyTo(TEntity[] array, Int32 arrayIndex)
    {
        Entities.CopyTo(array, arrayIndex);
    }

    Int32 ICollection<TEntity>.Count
    {
        get
        {
            if (_dbCollectionEntry.IsLoaded)
                return _collection.Count;
            return _dbCollectionEntry.Query().Cast<TEntity>().Count(_filter);
        }
    }

    Boolean ICollection<TEntity>.IsReadOnly
    {
        get
        {
            return Entities.IsReadOnly;
        }
    }

    Boolean ICollection<TEntity>.Remove(TEntity item)
    {
        return Entities.Remove(item);
    }

    #endregion

    #region IEnumerable<T> Members

    IEnumerator<TEntity> IEnumerable<TEntity>.GetEnumerator()
    {
        return Entities.GetEnumerator();
    }

    #endregion

    #region IEnumerable Members

    IEnumerator IEnumerable.GetEnumerator()
    {
        return ( ( this as IEnumerable<TEntity> ).GetEnumerator() );
    }

    #endregion
}

如果您浏览它,您会发现最重要的部分是“实体”属性,它会延迟加载实际值。在 FilteredCollection 的构造函数中,我为已经急切加载集合的场景传递了一个可选的 ICollection。

当然,我们仍然需要配置实体框架,以便我们的 FilteredCollection 在任何有集合属性的地方都可以使用。这可以通过挂钩实体框架底层 ObjectContext 的 ObjectMaterialized 事件来实现:

(this as IObjectContextAdapter).ObjectContext.ObjectMaterialized +=
    delegate(Object sender, ObjectMaterializedEventArgs e)
    {
        if (e.Entity is Entity)
        {
            var entityType = e.Entity.GetType();
            IEnumerable<PropertyInfo> collectionProperties;
            if (!CollectionPropertiesPerType.TryGetValue(entityType, out collectionProperties))
            {
                CollectionPropertiesPerType[entityType] = (collectionProperties = entityType.GetProperties()
                    .Where(p => p.PropertyType.IsGenericType && typeof(ICollection<>) == p.PropertyType.GetGenericTypeDefinition()));
            }
            foreach (var collectionProperty in collectionProperties)
            {
                var collectionType = typeof(FilteredCollection<>).MakeGenericType(collectionProperty.PropertyType.GetGenericArguments());
                DbCollectionEntry dbCollectionEntry = Entry(e.Entity).Collection(collectionProperty.Name);
                dbCollectionEntry.CurrentValue = Activator.CreateInstance(collectionType, new[] { dbCollectionEntry.CurrentValue, dbCollectionEntry });
            }
        }
    };

这一切看起来相当复杂,但它本质上是扫描物化类型的集合属性并将值更改为过滤集合。它还将 DbCollectionEntry 传递给过滤后的集合,以便发挥它的魔力。

这涵盖了整个“加载实体”部分。到目前为止唯一的缺点是急切加载的集合属性仍将包含已删除的实体,但它们在 FilterCollection 类的“添加”方法中被过滤掉了。这是一个可以接受的缺点,尽管我还没有对这如何影响 SaveChanges() 方法进行一些测试。

当然,这仍然留下一个问题:查询没有自动过滤。如果您想获取在过去一周进行过锻炼的健身房成员,您希望自动排除已删除的锻炼。

这是通过 ExpressionVisitor 实现的,该表达式自动将 '.Where(e => !e.Deleted)' 过滤器应用于它可以在给定表达式中找到的每个 IQueryable。

这是代码:

public class DeletedFilterInterceptor: ExpressionVisitor
{
    public Expression<Func<Entity, bool>> Filter { get; set; }

    public DeletedFilterInterceptor()
    {
        Filter = entity => !entity.Deleted;
    }

    protected override Expression VisitMember(MemberExpression ex)
    {
        return !ex.Type.IsGenericType ? base.VisitMember(ex) : CreateWhereExpression(Filter, ex) ?? base.VisitMember(ex);
    }

    private Expression CreateWhereExpression(Expression<Func<Entity, bool>> filter, Expression ex)
    {
        var type = ex.Type;//.GetGenericArguments().First();
        var test = CreateExpression(filter, type);
        if (test == null)
            return null;
        var listType = typeof(IQueryable<>).MakeGenericType(type);
        return Expression.Convert(Expression.Call(typeof(Enumerable), "Where", new Type[] { type }, (Expression)ex, test), listType);
    }

    private LambdaExpression CreateExpression(Expression<Func<Entity, bool>> condition, Type type)
    {
        var lambda = (LambdaExpression) condition;
        if (!typeof(Entity).IsAssignableFrom(type))
            return null;

        var newParams = new[] { Expression.Parameter(type, "entity") };
        var paramMap = lambda.Parameters.Select((original, i) => new { original, replacement = newParams[i] }).ToDictionary(p => p.original, p => p.replacement);
        var fixedBody = ParameterRebinder.ReplaceParameters(paramMap, lambda.Body);
        lambda = Expression.Lambda(fixedBody, newParams);

        return lambda;
    }
}

public class ParameterRebinder : ExpressionVisitor
{
    private readonly Dictionary<ParameterExpression, ParameterExpression> _map;

    public ParameterRebinder(Dictionary<ParameterExpression, ParameterExpression> map)
    {
        _map = map ?? new Dictionary<ParameterExpression, ParameterExpression>();
    }

    public static Expression ReplaceParameters(Dictionary<ParameterExpression, ParameterExpression> map, Expression exp)
    {
        return new ParameterRebinder(map).Visit(exp);
    }

    protected override Expression VisitParameter(ParameterExpression node)
    {
        ParameterExpression replacement;

        if (_map.TryGetValue(node, out replacement))
            node = replacement;

        return base.VisitParameter(node);
    }
}

我的时间有点短,所以我稍后会回到这篇文章,提供更多细节,但它的要点已经写下来,适合那些渴望尝试一切的人;我在这里发布了完整的测试应用程序:https ://github.com/amoerie/TestingGround

但是,可能仍然存在一些错误,因为这是一项正在进行的工作。不过,这个概念性的想法是合理的,我希望它能够在我整齐地重构所有内容并找到时间为此编写一些测试后很快就能完全发挥作用。

于 2013-09-13T13:56:34.460 回答
1

一种可能的方法是使用带有基本规范的规范,该规范检查所有查询的软删除标志以及包含策略。

我将说明我在项目中使用的规范模式的调整版本(起源于这篇文)

public abstract class SpecificationBase<T> : ISpecification<T>
    where T : Entity
{
    private readonly IPredicateBuilderFactory _builderFactory;
    private IPredicateBuilder<T> _predicateBuilder;

    protected SpecificationBase(IPredicateBuilderFactory builderFactory)
    {
        _builderFactory = builderFactory;            
    }

    public IPredicateBuilder<T> PredicateBuilder
    {
        get
        {
            return _predicateBuilder ?? (_predicateBuilder = BuildPredicate());
        }
    }

    protected abstract void AddSatisfactionCriterion(IPredicateBuilder<T> predicateBuilder);        

    private IPredicateBuilder<T> BuildPredicate()
    {
        var predicateBuilder = _builderFactory.Make<T>();

        predicateBuilder.Check(candidate => !candidate.IsDeleted)

        AddSatisfactionCriterion(predicateBuilder);

        return predicateBuilder;
    }
}

IPredicateBuilder 是 LINQKit.dll 中包含的谓词构建器的包装

规范基类负责创建谓词构建器。一旦创建了应该应用于所有查询的条件,就可以添加。然后可以将谓词构建器传递给继承的规范以添加更多标准。例如:

public class IdSpecification<T> : SpecificationBase<T> 
    where T : Entity
{
    private readonly int _id;

    public IdSpecification(int id, IPredicateBuilderFactory builderFactory)
        : base(builderFactory)
    {
        _id = id;            
    }

    protected override void AddSatisfactionCriterion(IPredicateBuilder<T> predicateBuilder)
    {
        predicateBuilder.And(entity => entity.Id == _id);
    }
}

IdSpecification 的完整谓词将是:

entity => !entity.IsDeleted && entity.Id == _id

然后可以将规范传递到使用该PredicateBuilder属性构建 where 子句的存储库:

    public IQueryable<T> FindAll(ISpecification<T> spec)
    {
        return context.AsExpandable().Where(spec.PredicateBuilder.Complete()).AsQueryable();
    }

AsExpandable()是 LINQKit.dll 的一部分。

关于包含/延迟加载属性,可以使用关于包含的进一步属性扩展规范。规范基础可以添加基础包含,然后子规范添加它们的包含。然后,存储库可以在从数据库中获取之前应用规范中的包含。

    public IQueryable<T> Apply<T>(IDbSet<T> context, ISpecification<T> specification) 
    {
        if (specification.IncludePaths == null)
            return context;

        return specification.IncludePaths.Aggregate<string, IQueryable<T>>(context, (current, path) => current.Include(path));
    } 

如果有不清楚的地方,请告诉我。我尽量不要把它变成一个怪物帖子,所以一些细节可能会被遗漏。

编辑:我意识到我没有完全回答你的问题;导航属性。如果您将导航属性设置为内部(使用这篇文章来配置它并创建 IQueryable 的非映射公共属性。非映射属性可以具有自定义属性,并且存储库将基本规范的谓词添加到 where,而无需急切加载它。当有人确实应用了一个急切的操作时,过滤器将应用。像:

    public T Find(int id)
    {
        var entity = Context.SingleOrDefault(x => x.Id == id);
        if (entity != null)
        {
            foreach(var property in entity.GetType()
                .GetProperties()
                .Where(info => info.CustomAttributes.OfType<FilteredNavigationProperty>().Any()))
            {
                var collection = (property.GetValue(property) as IQueryable<IEntity>);
                collection = collection.Where(spec.PredicateBuilder.Complete());
            }
        }

        return entity;
    }

我还没有测试上面的代码,但它可以通过一些调整来工作:)

编辑2:删除。

如果您使用的是通用/通用存储库,您可以简单地向 delete 方法添加一些进一步的功能:

    public void Delete(T entity)
    {
        var castedEntity = entity as Entity;
        if (castedEntity != null)
        {
            castedEntity.IsDeleted = true;
        }
        else
        {
            _context.Remove(entity);
        }            
    }
于 2013-09-07T11:43:30.500 回答
1

您是否考虑过使用数据库中的视图来加载排除已删除项目的问题实体?

这确实意味着您将需要使用存储过程来映射INSERT//功能UPDATE,但如果映射到省略了已删除行的视图,DELETE它肯定会解决您的问题。Workout另外 - 这在代码优先方法中可能不一样......

于 2013-09-07T03:45:24.497 回答