EF Core 6 导航属性误标Modified致保存失败的解决方法

问题复现 #

执行如下代码时抛出异常：

var myEntity = Context.MyEntities.GetById(id);
myEntity.SomeNavigationProperty = new MyNavigationProperty(...);
await Context.SaveChangesAsync();

实体一对一关系配置如下：

builder.HasOne(o => o.SomeNavigationProperty)
    .WithOne()
    .HasForeignKey<MyEntity>(o => o.SomeNavigationPropertyId);

抛出的异常信息：

The database operation was expected to affect 1 row(s), but actually affected 0 row(s); data may have been modified or deleted since entities were loaded.

检查Change Tracker可发现，新建的MyNavigationProperty实例被标记为Modified状态，而非预期的Added状态。
以下两种写法可正常执行：

• 新建主实体同时附加导航属性，显式Add主实体

var myEntity = new MyEntity(...);
myEntity.SomeNavigationProperty = new MyNavigationProperty(...);
await Context.AddAsync(myEntity);
await Context.SaveChangesAsync();

• 查询已存在主实体后，显式Add导航属性

var myEntity = Context.MyEntities.GetById(id);
myEntity.SomeNavigationProperty = new MyNavigationProperty(...);
await Context.AddAsync(myEntity.SomeNavigationProperty);
await Context.SaveChangesAsync();

核心诉求 #

查阅AddAsync相关文档后可部分理解EF Core的跟踪行为，但无法明确为什么未被跟踪的新实体会被标记为Modified而非Added。
实际业务中Domain层与Repository层分离，给导航属性赋值时无法访问DbContext实例，需要找到EF Core的配置方式，自动将未持久化的关联实体标记为Added，且不会因重复跟踪实体抛出异常。
期望实现类似如下的RefreshAddAsync方法：

await Context.RefreshAddAsync(myEntity);

该方法需要跳过已被跟踪的MyEntity，同时将关联的新MyNavigationProperty标记为Added状态。优先采用EF Core原生方案，无原生方案时再考虑反射实现。 #

问题根因 #

这个行为是EF Core一对一关系变更跟踪的默认逻辑导致的：

• 当你通过查询加载已存在的MyEntity时，EF Core会将主实体纳入跟踪，同时记录外键SomeNavigationPropertyId的当前值，以及对应关联实体的跟踪状态。
• 由于一对一关系的外键定义在MyEntity对应的数据表上，当你给已跟踪主实体的一对一导航属性赋值一个全新的未跟踪实例时，EF Core会默认判定你是要修改已存在的关联实体，而非新增一个关联实体——这个判定逻辑的出发点是：一对一关系下主实体的外键字段只能指向一个关联记录，给导航属性赋值的操作等价于更新已有关联记录的字段值，因此会把新实例标记为Modified，生成UPDATE语句。由于数据库中不存在对应主键的关联记录，UPDATE语句影响行数为0，最终抛出异常。
• 两种可正常运行的写法本质都是显式告诉EF Core关联实体是新增的：第一种场景下主实体本身就是未跟踪的新实例，调用Add时EF Core会递归将所有关联的未跟踪实体标记为Added；第二种场景则是直接对导航属性实例调用Add，强制将其状态覆盖为Added。

实现方案 #

EF Core没有提供全局开关直接修改这个默认跟踪逻辑，但可以通过重写DbContext的保存方法，在提交前统一修正实体状态，完全在EF Core层实现需求，不需要侵入Domain层，也不需要反射。

全局状态修正实现（推荐） #

在你的DbContext类中重写SaveChanges和SaveChangesAsync方法，在调用基类保存逻辑前，统一修正被错误标记为Modified的新实体状态：

public override int SaveChanges()
{
    FixNewEntityStates();
    return base.SaveChanges();
}

public override async Task<int> SaveChangesAsync(CancellationToken cancellationToken = default)
{
    FixNewEntityStates();
    return await base.SaveChangesAsync(cancellationToken);
}

private void FixNewEntityStates()
{
    // 遍历所有被标记为Modified的跟踪项
    foreach (var entry in ChangeTracker.Entries()
        .Where(e => e.State == EntityState.Modified))
    {
        var primaryKey = entry.Metadata.FindPrimaryKey();
        if (primaryKey == null) continue;

        // 判断实体主键是否为默认值：主键为默认值说明实体从未持久化到数据库，不可能处于Modified状态
        bool isUnpersistedEntity = primaryKey.Properties.All(prop =>
        {
            var currentValue = entry.Property(prop.Name).CurrentValue;
            var defaultValue = prop.ClrType.IsValueType 
                ? Activator.CreateInstance(prop.ClrType) 
                : null;
            return currentValue?.Equals(defaultValue) ?? true;
        });

        if (isUnpersistedEntity)
        {
            entry.State = EntityState.Added;
        }
    }
}

这个实现的判断逻辑可靠性很高：所有需要被更新的实体必然已经存在主键值，主键为类型默认值的实体一定是未插入过的新实体，直接修正状态为Added即可。
如果你的业务使用客户端生成主键（比如新建实体时手动赋值Guid、业务主键），可以给实体定义一个临时标记接口（比如ITransientEntity），或者通过创建时间、版本号等字段判断是否为未持久化的新实体，调整上述判断逻辑即可。

可选的扩展方法实现 #

如果不想全局修改SaveChanges逻辑，也可以单独实现你需要的RefreshAddAsync扩展方法：

public static async Task RefreshAddAsync<T>(this DbContext context, T entity, CancellationToken cancellationToken = default)
    where T : class
{
    var entry = context.Entry(entity);
    if (entry.State == EntityState.Detached)
    {
        await context.AddAsync(entity, cancellationToken);
        return;
    }

    // 遍历实体的所有导航属性，将未持久化的关联实体标记为Added
    foreach (var navigation in entry.Navigations)
    {
        if (navigation.CurrentValue is not null && navigation.TargetEntry != null)
        {
            var targetEntry = navigation.TargetEntry;
            var pk = targetEntry.Metadata.FindPrimaryKey();
            if (pk == null) continue;
            bool isNew = pk.Properties.All(p =>
            {
                var val = targetEntry.Property(p.Name).CurrentValue;
                var def = p.ClrType.IsValueType ? Activator.CreateInstance(p.ClrType) : null;
                return val?.Equals(def) ?? true;
            });
            if (isNew)
            {
                targetEntry.State = EntityState.Added;
            }
        }
    }
}

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