Question

我有以下模型，我正在尝试从同一个父实体和子实体构建一对一和一对多的关系。一对多关系适用于我当前的映射，但我正在努力添加新的一对一关系（对于CoverPicture属性）。以下是相关模型和EF映射代码：

Category.cs：

public int Id { get; set; }
public string Name { get; set; }
public Guid? CoverPictureId { get; set; }
public virtual Picture CoverPicture { get; set; }
public virtual ICollection<Picture> Pictures { get; set; }

Picture.cs：

public Guid Id { get; set; }
public string FileName { get; set; }
public int CategoryId { get; set; }
public virtual Category Category { get; set; }

相关类别EntityTypeConfiguration<Category>映射（不正确）：

this.HasOptional(t => t.CoverPicture)
    .WithRequired()
    .Map(m => m.MapKey("CoverPictureId"))
    .WillCascadeOnDelete(false);

相关图片EntityTypeConfiguration<Picture>映射（正确）：

this.HasRequired(t => t.Category)
    .WithMany(t => t.Pictures)
    .HasForeignKey(k => k.CategoryId);

当我尝试为新的CoverPicture属性添加迁移时，EF会尝试将CoverPictureId列添加到Category表（这是我想要的），但也{{1} 1}}到CoverPictureId表（这不是我想要的; Picture已经定义并映射了一个键。

以下是EF支持的Picture迁移代码：

Up()

我做错了什么？

Answer 1

我不确定您使用的是哪个版本的EF，但是较新版本不允许您执行您尝试执行的映射类型，您将收到以下错误：

在'YourProject.Category'类型上声明的导航属性'Pictures'已经配置了冲突的多重性。

那为什么呢？让我们看一下你的映射，以及你用简单的英语告诉Entity Framework：

this.HasRequired(t => t.Category)  //A Picture must have 1 Category
.WithMany(t => t.Pictures)         //A Category can have many Pictures


this.HasOptional(t => t.CoverPicture) //A Category may or may not have 1 Picture
.WithRequired()                       //A Picture must have 1 Category

将其与octavioccl的答案进行比较：

this.HasOptional(p => p.CoverPicture) //A Category may or may not have 1 Picture 
.WithMany()                           //A Picture can have many Categories

从WithRequired到WithMany的更改正在交换放置外键的位置。现在你有了你正在寻找的映射......有点：

CreateTable(
    "dbo.Categories",
    c => new
        {
            Id = c.Int(nullable: false, identity: true),
            Name = c.String(),
            CoverPicture_Id = c.Guid(),
        })
    .PrimaryKey(t => t.Id)
    .ForeignKey("dbo.Pictures", t => t.CoverPicture_Id)
    .Index(t => t.CoverPicture_Id);

CreateTable(
    "dbo.Pictures",
    c => new
        {
            Id = c.Guid(nullable: false),
            FileName = c.String(),
            Category_Id = c.Int(nullable: false),
        })
    .PrimaryKey(t => t.Id)
    .ForeignKey("dbo.Categories", t => t.Category_Id, cascadeDelete: true)
    .Index(t => t.Category_Id);

但是让我们停下来思考一下。您不仅基本上在两个方向上定义了1对多的关系（不要与多对多关系混淆），而且还破坏了模型的完整性。现在，即使Picture不属于Category，您也可以将CoverPicture指定为Picture Category。这不是你想要的，最终会让你头痛。而不是在CoverPicture上明确定义Category属性，这个怎么样？

public class Category
{
    public int Id { get; set; }
    public string Name { get; set; }
    public virtual ICollection<Picture> Pictures { get; set; }

    public SetCoverPicture(Picture picture)
    {
        if(!Pictures.Contains(picture))
        {
            throw new ArgumentException("Picture is not in this Category");
        }
        var currentCoverPicture = Pictures.FirstOrDefault(p=p.IsCoverPicture == true);
        if(currentCoverPictur e!= null)
        {
            currentCoverPicture.IsCoverPicture = false;
        }
        picture.IsCoverPicture = true;
    }

}

public class Picture
{
    public Guid Id { get; set; }
    public string FileName { get; set; }
    public int Category_Id { get; set; }
    public virtual Category Category { get; set; }
    public bool IsCoverPicture { get; protected internal set; }
}

这会强制执行声明...

的不变量（业务规则）

CoverPicture的{{1}}必须属于Category （由数据库强制执行）和
只有1 Category 对于CoverPicture（在代码中强制执行）

您可以使用octavioccl提供的代码执行类似操作，但此处提供的代码可以生成更清晰，更易理解的物理数据模型。

Answer 2

要解决您的问题，请尝试映射该关系，如下所示：

this.HasOptional(p => p.CoverPicture)
            .WithMany().HasForeignKey(p=>p.CoverPictureId)
            .WillCascadeOnDelete(false);

在一对一关系中，一端必须是主要，第二端必须是依赖。在配置此类关系时，Entity Framework要求依赖项的主键也是外键。使用您的配置，主体为Category，dependend为Picture。 FK CoverPictureId的方式是int而不是Guid，因为这实际上是您迁移代码解释的Category PK。

EF Code First一对一和一对多映射

2 个答案: