Welcome to OGeek Q&A Community for programmer and developer-Open, Learning and Share
Welcome To Ask or Share your Answers For Others

Categories

0 votes
659 views
in Technique[技术] by (71.8m points)

c# - Violation of primary key Entity Framework Code First

I have started with C# and I wanted to make my own DB.

I have two models

public class AModel 
{
    public Guid ID { get; private set; }
    public string Name { get; set; }
    public int Count { get; set; }
    public AModel()
    {
        this.ID = Guid.NewGuid();
    }
}

public class BModel 
{
    public Guid ID { get; private set; }
    public string Name { get; set; }
    public AModel Model { get; set; }
    public BModel()
    {
        this.ID = Guid.NewGuid();
    }
}

When I try to save BModel to DB, I get this error:

Violation of PRIMARY KEY constraint 'PK_dbo.AModels'. Cannot insert duplicate key in object 'dbo.AModels'. The duplicate key value is (48ee1711-8da4-46c1-a714-19e985211fed). The statement has been terminated.

I thought it would be solved by this

modelBuilder.Entity<BModel>().HasRequired(t => t.Model).WithMany();

but it looks like I am completely lost. Could anybody help me with this simple example?

See Question&Answers more detail:os

与恶龙缠斗过久,自身亦成为恶龙;凝视深渊过久,深渊将回以凝视…
Welcome To Ask or Share your Answers For Others

1 Reply

0 votes
by (71.8m points)

Your comment reveals vital information. When you add that AModel from your combobox to your BModel, it will have become detached from your DbContext by then. When you then add it to your model, Entity Framework will think that you have a new object.

Since you have your Ids configured as DatabaseGenerationOptions.None, it will use the primary key you provide yourself. In your case this is the PK of the detached object. Thus, when EF tries to insert this entry it will throw the above exception because an entity with that key is already in there.

There are a few ways to solve this:

  • Retrieve the existing entity

This entity will be attached to your context upon retrieval, allowing you to use this. This means an extra lookup however: first to get them into the combobox and then to use the Id from the entity in the combobox to retrieve it again from the database.

Example usage:

AModel Get(AModel detachedModel)
{
    using(var context = new MyContext())
    {
        return context.AModels.Single(x => x.ID == detachedModel.ID);
    }
}
  • Attach the existing model

This should just make Entity-Framework aware that the entity already exists in the database.

using(var context = new MyContext())
{
    context.AModels.Attach(detachedModel);
}

Other ways to attach is by setting the state to Unchanged

context.Entry(detachedModel).State = EntityState.Unchanged;

or Modified (in case you also changed values)

context.Entry(detachedModel).State = EntityState.Modified;

与恶龙缠斗过久,自身亦成为恶龙;凝视深渊过久,深渊将回以凝视…
OGeek|极客中国-欢迎来到极客的世界,一个免费开放的程序员编程交流平台!开放,进步,分享!让技术改变生活,让极客改变未来! Welcome to OGeek Q&A Community for programmer and developer-Open, Learning and Share
Click Here to Ask a Question

...