代码优先中的一对多关系

Question

我想在Users 和Notifications 之间创建一个我认为是一对多关系的关系。

我遇到了这个问题，不知道我哪里出错了。下面是这两个类的代码：

public class ApplicationUser : IdentityUser
{
    public string FirstName { get; set; }

    public virtual ICollection<Notification> Notifications { get; set; }
}

public class Notification
{
    public int NotificationID { get; set; }
    public string Title { get; set; }
    public string Description { get; set; }
    public DateTime Time { get; set; }

    public string ApplicationUserID { get; set; }
    public virtual ApplicationUser ApplicationUser { get; set; }
}

我上面还有以下映射：

modelBuilder.Entity<Notification>()
    .HasRequired<ApplicationUser>(u => u.ApplicationUser)
    .WithMany(u => u.Notifications)
    .HasForeignKey(u => u.NotificationID);

我在尝试修复此问题时遇到了不同的错误，例如：

• 多重性无效
• 违反了多重性约束。关系_的角色_具有多重性1或0..1

编辑：

当我尝试为所有用户添加通知时，抛出相同的异常（违反多重约束。关系 Notification_ApplicationUser 的角色“Notification_ApplicationUser_Target”具有多重性 1 或 0..1。）正如我在此方法中所做的那样：

public void AddNotification(Notification n)
{
    var roleId = context.Roles.Where(m => m.Name == "User").Select(m => m.Id).SingleOrDefault();
    var users = context.Users.Where(u => u.Roles.All(r => r.RoleId == roleId)).ToList();
    try
    {
        foreach(var user in users)
        {
            user.Notifications.Add(n);
        }
        context.SaveChanges();
    }
    catch (Exception e)
    {
        Console.WriteLine(e);
    }
}

Answer 1

我认为问题出在外键的字符串类型上。默认情况下，字符串在数据库级别可以为空，但必需的外键不能为空。指定HasRequired 仅强制要求关系是必需的，而不是外键属性必须不为空。我认为，如果您只是将 [Required] 添加到您的外键属性中，那将解决问题：

[Required]
public string ApplicationUserID { get; set; }

显然，您需要进行迁移才能将此更改应用到您的数据库。

编辑

对不起，我读错了异常。您的问题的根源实际上是向每个用户添加相同的通知，就像字面上相同的通知一样。 EF做对象跟踪，所以n引用的实体实例被添加一次后，就被EF跟踪了。然后，当您尝试将其添加到另一个用户时，它认为您正在尝试进行多对多操作，基本上，单个 Notification 记录将属于多个用户。

在我的脑海中，我不确定在 EF 中解决这种混乱的最佳方法是什么，但我有几个潜在的想法：

1. 添加实体后可以尝试分离：

   user.Notifications.Add(n);
   context.Entry(n).State = EntityState.Detached;
   

   但是，我不确定这是否会允许添加它。你必须进行测试。如果它没有创建它，您也可以在分离之前尝试保存。我认为这可行，但每次迭代都提交数据库显然效率很低。

2. 最安全的方法是为每个实例创建一个新实例，然后简单地映射来自n 的数据：

   var notification = new Notification
   {
       Title = n.Title,
       Description = n.Description,
       Time = n.Time
   };
   user.Notifications.Add(notification);
   

   这将确保您添加的每个通知都是一个完全独立的跟踪实例。

Answer 2

这是您关系的正确配置：

    modelBuilder.Entity<Notification>()
                .HasRequired(u => u.ApplicationUser)
                .WithMany(u => u.Notifications)
                .HasForeignKey(u => u.ApplicationUserID);

在HasForeignKey 方法中，您需要指定该关系的FK，而不是Notification 实体的PK。

更新

问题是您的数据库中存在一对一的关系，并且您正尝试在模型中配置一对多的关系。如果最后一个是您真正想要的，那么我建议您使用Migrations 更改您的数据库架构，否则您可以这样配置您的关系：

    modelBuilder.Entity<Notification>()
                .HasRequired(u => u.ApplicationUser)
                .WithOptional(u => u.Notification)

并更改ApplicationUser实体中的导航属性：

public class ApplicationUser : IdentityUser
{
       public string FirstName { get; set; }

       public virtual Notification Notification { get; set; }
}