C# & ASP.NET MVC：调用第二个函数无法保存数据库更改求助

Hey，我之前也碰到过这种同一个项目里部分数据操作能保存、部分不行的问题，咱们从这几个方向一步步排查：

1. 先查 walkPokemon 里有没有调用保存方法 #

第一个能正常工作的函数，肯定最后调用了_context.SaveChanges()（或者异步的await _context.SaveChangesAsync()）。你得检查walkPokemon的实现里是不是漏了这一步——很多时候就是改了实体但忘了提交到数据库。比如：

// 反面例子：只修改实体但没保存
public void walkPokemon(int id, int steps, User user)
{
    var pokemon = _context.UserPokemons.FirstOrDefault(p => p.Id == id && p.UserId == user.Id);
    if(pokemon != null)
    {
        pokemon.Steps += steps;
        // 这里少了关键的 _context.SaveChanges();
    }
}

如果是异步方法，还要注意函数要标记为async Task，并且用await调用保存方法。

2. 确认实体是否被上下文追踪 #

要是你在walkPokemon里用了AsNoTracking()查询实体，那这个实体的修改不会被数据库上下文感知到，自然不会保存。比如：

// 这种情况修改后上下文不会追踪变化
var pokemon = _context.UserPokemons.AsNoTracking().FirstOrDefault(p => p.Id == id);

解决办法要么去掉AsNoTracking()，要么修改实体后手动调用_context.Update(pokemon)再执行保存。

3. 检查实体查询是否命中目标对象 #

有时候不是保存的问题，而是你压根没找到要修改的userPokemon！比如id和user的匹配条件写错了，导致pokemon是null，自然没修改任何数据。可以加个断点或者日志，看看查询出来的实体是不是你要改的那个。

4. 排查事务或上下文生命周期问题 #

• 如果你的第一个函数用了事务，要确认walkPokemon是不是在同一个事务里，有没有被提前回滚或者提交；
• 还要检查两个Repo（_userRepo和_userPokemonRepo）是不是用的同一个数据库上下文实例。比如在ASP.NET里，默认上下文是Scoped的，如果Repo的生命周期配置错了（比如一个用Singleton一个用Scoped），就会导致第二个上下文的更改没提交。

5. 看看是不是有异常被吞了 #

有时候保存操作其实抛出了异常，但被代码吞掉了（比如用了空的try-catch），表面看起来没报错，但实际上保存失败了。可以在walkPokemon里加个try-catch，或者直接调试看保存步骤有没有报错。

如果能把walkPokemon的具体代码贴出来，排查起来会更快，但先按上面的点检查，大概率能找到问题~

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