ASP.NET MVC更新数据库对象引用列表时关联Seats旧数据未删除问题

问题根因 #

该现象是EF默认的关联实体处理逻辑导致的：当你直接替换导航属性cinemaInDb.Seats的集合时，EF只会断开旧实体和父实体的关联（把旧Seat的CinemaId置为null），不会主动删除这些子实体，因为你当前的Seat模型里CinemaId是可空类型，也没有配置级联删除规则。

方案1：手动删除旧座位数据（最直接，无需修改模型配置） #

在调用SetSeats生成新座位列表之前，先把当前影院关联的所有旧座位从数据库上下文的Seats DbSet中删除，修改你的Save方法代码如下：

public ActionResult Save(Cinema cinema)
{
    var cinemaInDb = _context.Cinemas.Include(c => c.Seats).Single(c => c.Id == cinema.Id);

    cinemaInDb.Name = cinema.Name;
    cinemaInDb.Address = cinema.Address;
    cinemaInDb.NumberOfSeats = cinema.NumberOfSeats;
    
    // 新增：删除所有旧座位
    _context.Seats.RemoveRange(cinemaInDb.Seats);
    
    cinemaInDb.Seats = SetSeats(cinemaInDb);
    cinemaInDb.OpeningHourId = cinema.OpeningHourId;
    cinemaInDb.ClosingHourId = cinema.ClosingHourId;

    _context.SaveChanges();
}

该方案不需要修改其他现有逻辑，直接生效，适配你当前的业务场景。

方案2：配置级联删除+必需关联（一劳永逸） #

如果你确定Seat必须属于某个Cinema，无关联Cinema的Seat没有存在意义，可以修改模型和配置，让EF自动帮你删除断开关联的子实体：

1. 首先修改Seat模型，把CinemaId改成非可空类型：

// 修改前
// public int? CinemaId { get; set; }
// 修改后
public int CinemaId { get; set; }

2. 在你的DbContext的OnModelCreating方法中配置关联的级联删除规则：

modelBuilder.Entity<Cinema>()
    .HasMany(c => c.Seats)
    .WithOne(s => s.Cinema)
    .HasForeignKey(s => s.CinemaId)
    .OnDelete(DeleteBehavior.Cascade);

配置完成后，当你替换Cinema的Seats集合时，EF会自动删除旧的Seat记录，不需要手动写RemoveRange逻辑。

额外注意点 #

你当前SetSeats方法里给新生成的Seat手动赋值Id的操作存在隐患，如果你的Seat表的Id是自增主键的话，手动赋值的Id会被数据库忽略，建议去掉Id = i - 1这行代码，让数据库自动生成主键，避免主键冲突问题。

内容的提问来源于stack exchange，提问作者YarsCode