c# - SavingChanges 事件与 SaveChanges 覆盖之间的区别？

Question

我已经阅读了多篇有关如何使用 Entity Framework 实现审核日志的文章。目前，我已将审核逻辑嵌入到 DbContext 的 SaveChanges 方法中。请记住，下面的代码是 EF4/5 实现，我正准备更新到 EF6。

namespace Database {
    public class AuditDetails {
        public string RemoteHost { get; set; }
        public string RevisionUser { get; set; }
        public string RevisionNotes { get; set; }
        public DateTime RevisionDateTime { get; set; }
    }

    public class MyContext : DbContext {
        // ... Unrelated overrides here ... //
        public void SaveChanges(AuditDetails auditDetails) {
            var saveCount = ProcessConcurrency();
            var items = ChangeTracker.Entries<MyEntity>().ToList();
            if (saveCount <= 0 || items == null || !items.Any() || auditDetails == null) return;
            foreach (var item in items.Select(entityEntry => entityEntry.Entity).Where(i => i != null)) {
                // ... create audit log using AuditDetails values ... //
            }
            // ... and call base.SaveChanges() ... //
        }
    }
}

所以问题是:

• 将其移至 SavingChanges 事件处理程序有好处吗？或者可能拆分功能以同时使用两者？
• AuditDetails 信息是否会阻止使用 SavingChanges？
• 我没有覆盖 SaveChanges 来接受在事务中使用的 bool 参数。添加它会如何改变解决方案？

总结:

• 您何时会重写/实现 SaveChanges 以及您何时/为何更愿意使用 SavingChanges 事件处理程序？

Answer 1

我认为在大多数情况下这并不重要。我可以想到三种情况需要这两种选择之一:

1. SaveChanges 应完全重写，因此不应调用 base.SaveChanges:只有重写才能执行此操作。

2. 另一个类涉及保存更改时发生的情况:只有事件可以做到这一点。 (或者也许我应该说:事件将是明显的选择模式)。

3. 您想要选择性地扩展 SaveChanges。根据我的口味，通过某个参数在构造函数中激活事件比使用 if (option) 启动覆盖更清晰，其中 option 必须存储为成员变量.

在所有其他情况下，我总是使用覆盖。该事件要求首先挖掘 ObjectContext(因为它是 ObjectContext.SavingChanges)。并且覆盖是硬连线的。事件订阅总是会以某种方式破坏代码维护。