使用Entity Framework Core保存实体偶现未持久化问题问询

针对你遇到的高负载下实体偶尔未添加且无异常的问题，可能的原因如下：

1. 主键冲突异常捕获逻辑存在漏洞 #

你的代码通过异常消息包含"Violation of Primary Key"来判断主键冲突，但这种方式不可靠：

• SqlServer的主键冲突异常消息可能因数据库语言版本、约束名称不同而变化，比如中文环境下消息是"违反 PRIMARY KEY 约束"，或者英文环境下可能包含"Cannot insert duplicate key"而非仅"Violation of Primary Key"，导致部分主键冲突场景未被捕获。
• 更可靠的方式是通过SqlException.Number判断：主键冲突的错误号是2627，直接检查错误号可以避免消息匹配的遗漏。

2. 未校验SaveChangesAsync的返回值 #

SaveChangesAsync返回受影响的行数，当你调用Add后预期返回1，但如果返回0，说明没有任何行被插入。这种情况可能发生在：

• EF Core上下文的实体状态意外被修改（比如在Add和SaveChanges之间，实体被设置为Unchanged状态）；
• 数据库存在自定义触发器（如INSTEAD OF触发器），导致插入操作被静默拦截而未执行；
• 极端竞态场景下，EF Core内部状态跟踪出现异常，认为实体无需保存。

3. 重试逻辑的潜在问题 #

你的代码在捕获非主键冲突的异常时会重试SaveChangesAsync，但如果第一次失败是因为主键冲突却未被捕获，重试会再次触发相同的异常，最终被日志记录并抛出，但你表示未收到异常，说明这种场景可能不是主因，但重试逻辑本身没有区分可重试的异常类型（比如网络波动、死锁等才需要重试，主键冲突重试无意义）。

关于DbContext生命周期的修改 #

将Scoped改为Transient无效果是合理的，因为问题核心在于业务逻辑的原子性和异常处理的可靠性，而非DbContext的生命周期管理。

1. 替换主键冲突的判断方式：通过SqlException.Number捕获主键冲突，示例代码：

catch (DbUpdateException dbEx)
{
    if (dbEx.InnerException is SqlException sqlEx && sqlEx.Number == 2627)
    {
        // 处理竞态条件：其他线程已插入记录
    }
    else
    {
        // 处理其他更新异常
    }
}

2. 强制校验SaveChangesAsync的返回值：在非主键冲突的场景下，若返回值不为1，则抛出异常或进行告警：

var affectedRows = await dbContext.SaveChangesAsync(cancellationToken);
if (affectedRows != 1)
{
    // 排除主键冲突的场景后，抛出异常或记录日志
    throw new InvalidOperationException($"预期插入1条记录，实际受影响{affectedRows}条");
}

3. 优化业务逻辑的原子性：将"检查存在+插入"改为原子操作，比如使用SqlServer的INSERT ... WHERE NOT EXISTS，避免竞态条件：

await dbContext.Database.ExecuteSqlInterpolatedAsync(
    $"INSERT INTO MyEntities (abc, etc) VALUES ({e.abc}, {e.etc}) WHERE NOT EXISTS (SELECT 1 FROM MyEntities WHERE abc = {e.abc})",
    cancellationToken);

（注意：需根据实体的主键/唯一键调整WHERE条件，确保判断的准确性）

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