EntityFramework 假称已更新数据行但实际未生效问题求助

我编写了一个基于EntityFramework DbContext的InsertOrUpdateUserByExternalId方法，期望接收对象后自动插入新数据行或更新已有数据行。插入功能正常，但更新时EF的SaveChanges返回1（看似执行成功），但读取数据库行后发现数据未发生任何变更。

版本一代码 #

public async Task<bool> InsertOrUpdateUserByExternalId(UserModel user, CancellationToken cancellationToken = default)
{
    if(user.User_ExternalID is null)
        return false;

    var userInDb = (await _db.Users
            .AsNoTracking()
            .FirstOrDefaultAsync(
                u => u.User_ExternalID == user.User_ExternalID,
                cancellationToken))
        .Adapt<UserModel>();

    if (userInDb is not null)
    {
        user.User_ExternalID = null; // 不设置此值会出现“重复索引”错误
        userInDb = user;
        _db.Users.Update(userInDb.Adapt<User>());
    }
    else
    {
        await _db.Users.AddAsync(user.Adapt<User>(), cancellationToken);
    }

    var changes = _db.SaveChanges(); // 返回1，看似执行成功

    return changes == 1;
}

版本二代码 #

第二个版本直接操作实体而非转换为模型，仅将参数user转换为实体，但结果相同：

public async Task<bool> InsertOrUpdateUserByExternalId(UserModel user, CancellationToken cancellationToken = default)
{
    if (user.User_ExternalID is null)
        return false;

    var userInDb = (await _db.Users
            .AsNoTracking()
            .FirstOrDefaultAsync(
                u => u.User_ExternalID == user.User_ExternalID,
                cancellationToken));

    if (userInDb is not null)
    {
        user.User_ExternalID = null;
        userInDb = user.Adapt<User>();
        _db.Users.Update(userInDb);
    }
    else
    {
        await _db.Users.AddAsync(user.Adapt<User>(), cancellationToken);
    }

    var changes = _db.SaveChanges();

    return changes == 1;
}

问题根源 #

两个版本均存在核心逻辑错误：

1. AsNoTracking()导致实体未被上下文跟踪：查询得到的userInDb脱离EF上下文跟踪，后续将变量指向新对象user.Adapt<User>()后，这个新对象的初始状态与当前状态完全一致（仅User_ExternalID被设为null，但该字段并非主键），EF无法检测到变更，最终不会生成更新语句。
2. 错误的对象替换逻辑：userInDb = user.Adapt<User>()仅修改了局部变量的引用，并未对从数据库查询的实体做任何属性变更，调用Update时传入的是一个全新的、与数据库数据无关联的对象，EF无法识别需要更新的目标行。

修复方案 #

方案一：利用EF变更跟踪更新属性（推荐） #

取消AsNoTracking()，让EF跟踪查询到的实体，直接修改实体属性或用映射工具同步属性变更：

public async Task<bool> InsertOrUpdateUserByExternalId(UserModel user, CancellationToken cancellationToken = default)
{
    if (user.User_ExternalID is null)
        return false;

    // 保留EF对实体的跟踪
    var userInDb = await _db.Users
        .FirstOrDefaultAsync(u => u.User_ExternalID == user.User_ExternalID, cancellationToken);

    if (userInDb is not null)
    {
        // 将模型属性映射到跟踪的实体，保留原主键
        var updatedUser = user.Adapt<User>();
        updatedUser.Id = userInDb.Id;
        
        // 让EF自动检测属性变更
        _db.Entry(userInDb).CurrentValues.SetValues(updatedUser);
    }
    else
    {
        await _db.Users.AddAsync(user.Adapt<User>(), cancellationToken);
    }

    var changes = await _db.SaveChangesAsync(cancellationToken);
    return changes > 0;
}

方案二：手动标记实体为Modified状态（适配NoTracking场景） #

若必须使用AsNoTracking()，需确保实体主键正确，并手动标记状态：

public async Task<bool> InsertOrUpdateUserByExternalId(UserModel user, CancellationToken cancellationToken = default)
{
    if (user.User_ExternalID is null)
        return false;

    var userInDb = await _db.Users
        .AsNoTracking()
        .FirstOrDefaultAsync(u => u.User_ExternalID == user.User_ExternalID, cancellationToken);

    if (userInDb is not null)
    {
        var updatedUser = user.Adapt<User>();
        // 必须设置与数据库一致的主键，让EF定位到目标行
        updatedUser.Id = userInDb.Id;
        
        // 强制标记为Modified，触发全字段更新
        _db.Users.Update(updatedUser);
    }
    else
    {
        await _db.Users.AddAsync(user.Adapt<User>(), cancellationToken);
    }

    var changes = await _db.SaveChangesAsync(cancellationToken);
    return changes > 0;
}

额外注意事项 #

• 之前出现的“重复索引”错误，是因为更新时未保留实体主键，EF将新对象误判为待插入数据，导致User_ExternalID唯一索引冲突。修复时必须确保实体主键与数据库目标行一致。
• 方案一仅更新变更的字段，性能更优；方案二会生成全字段更新语句，适合无需跟踪实体的场景。

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