实体框架 CTP5 违反 PRIMARY KEY 约束

Question

我已经开始通过编写Windows应用程序来学习实体框架CTP5。 我有两个模型（Unit 和 Good），如下所示：

 public class Unit : BaseEntity
{
   public Unit()
   {
     Goods = new List<Good>();
   }
   public string name { get; set; }
   public virtual ICollection<Good> Goods { get; set; }
}

public class Good : BaseEntity
{
        public Int64 code { get; set; }
        public string name { get; set; }

 public virtual Unit Unit { get; set; }

}

我正在使用名为 IRepository 的存储库接口，如下所示：

public interface IRepository
    {
        BaseEntity GetFirst();
        BaseEntity GetNext(Int32 id);
        BaseEntity GetPrevoius(Int32 id);
        BaseEntity GetLast();
        BaseEntity GetById(Int32 id);

        void Update(int id, BaseEntity newEntity);
        void Delete(int id);
        void Insert(BaseEntity entity);
        int GetMaxId();
        IList GetAll();
    }

每个模型都有自己的存储库，但也许最好使用 BaseEntity 类型的通用存储库。 GoodRepository 的引用是在 GoodForm 中进行的，并且 Good 类型的适当对象是由 Activator 对象以常见的表单方法（例如 Insert/Update/Delete...）创建的，如下所示：

private void InsertButton_Click(object sender, EventArgs e)
{
   Unit unit = goodRepo.GetUnitById(Convert.ToInt32(UnitIdTextBox.Text));
   if (unit == null)
   {
       unit = new Unit { Id = goodRepo.GetUnitMaxId(), Name = "Gram" };
   }
   var good = Activator.CreateInstance<Good>();
   good.Id = string.IsNullOrEmpty(IdTextBox.Text) ? goodRepo.GetMaxId() : Convert.ToInt32(IdTextBox.Text);

   IdTextBox.Text = good.Id.ToString();
   good.Name = NameTextBox.Text;
   good.Description = DescriptionTextBox.Text;
   good.Unit = unit;
   goodRepo.Insert(good);
}

GoodRepository.Insert 方法是：

public void Insert(Model.BaseEntity entity)
{
  using (PlanningContext context = new PlanningContext())
  {
      context.Goods.Add(entity as Good);
      int recordsAffected = context.SaveChanges();
      MessageBox.Show("Inserted " + recordsAffected + " entities to the database");
  }
}

我的问题是 SaveChanges() 生成一个错误“违反主键约束”并表示无法在对象“dbo.Units”中插入重复键。 但是如果我将上下文移动到我构建 Good 对象并将其插入那里的形式，一切都很好。

有人可以指导我如何解决这个问题吗？

提前致谢

Answer 1

问题的根源在于：

using (PlanningContext context = new PlanningContext())
{
    context.Goods.Add(entity as Good);
    //...
}

您正在将 Good 实体添加到新创建的、因此最初为空的上下文中。现在，如果将实体添加到上下文，EF 也会将相关实体的整个对象图添加到上下文，除非相关实体已经附加到上下文。这意味着 good.Unit 也将以 Added 状态放入上下文中。由于您似乎没有为 Unit 类自动生成的身份密钥，因此 EF 尝试使用已存在的相同密钥将 good.Unit 插入到数据库中数据库。这会导致异常。

现在，您可以在添加新的 Good 之前通过将 Unit 附加到上下文来临时解决此问题：

using (PlanningContext context = new PlanningContext())
{
    context.Units.Attach((entity as Good).Unit);
    context.Goods.Add(entity as Good);
    //...
}

但我最好重新考虑存储库的设计。在每个存储库方法中创建新上下文并不是一个好主意。上下文扮演着工作单元的角色，工作单元通常更多地是许多数据库操作的容器，这些操作紧密地联系在一起，并且应该在单个数据库事务中提交。

因此，像 InsertButton_Click 方法这样的操作应该具有如下结构：

using (var context = CreateSomehowTheContext())
{
    var goodRepo = CreateSomehowTheRepo(context); // Inject this context
    var perhapsAnotherRepo = CreateTheOtherRepo(context); // Inject same context

    Unit unit = goodRepo.GetUnitById(Convert.ToInt32(UnitIdTextBox.Text));
    // unit is now attached to context

    // ...
    good.Unit = unit;
    goodRepo.Insert(good); // should use the injected context and only do
                           // context.Goods.Add(good);
                           // It doesn't add unit to the context since
                           // it's already attached

    // ...

    context.SaveChanges();
}

在这里，您仅使用一个上下文，并且存储库将注入此上下文（例如在构造函数中）。他们从不在内部创建自己的环境。

Answer 2

我怀疑这是因为 GetUnitMaxId 多次返回相同的值。 Id 是自增标识列吗？如果是这样，您不应该尝试对代码中的值进行任何假设。

即使它不是自动递增的标识列，只有当所有其他列都已提交到数据库时，您才能确定它的值。

作为一般设计模式，请尽量避免在存储 Id 之前在代码中引用 Id。 EF 可以通过利用导航属性（实体间对象引用）来帮助解决此问题。