1

根据 CQRS 模式,我有一个简单的命令,如下所示:

public sealed class EditPersonalInfoCommandHandler : ICommandHandler<EditPersonalInfoCommand> {

        private readonly AppDbContext _context;

        public EditPersonalInfoCommandHandler(AppDbContext context) {
            _context = context;
        }

        public Result Handle(EditPersonalInfoCommand command) {
            var studentRepo = new StudentRepository(_context);
            Student student = studentRepo.GetById(command.Id);
            if (student == null) {
                return Result.Failure($"No Student found for Id {command.Id}");
            }

            student.Name = command.Name;
            student.Email = command.Email;

            _context.SaveChanges();
            return Result.Success();
        }

}

_context.SaveChanges()现在我有一个要求,如果它因异常而失败,则最多尝试5 次。为此,我可以简单地在方法中有一个 for 循环:

for(int i = 0; i < 5; i++) {
    try {
        //required logic
    } catch(SomeDatabaseException e) {
        if(i == 4) {
           throw;
        }
    }
}

要求是将方法作为一个单元执行。问题是,一旦_context.SaveChanges()抛出异常,_context就不能用同样的方法重新尝试逻辑。文档说:

丢弃当前的 DbContext。创建一个新的 DbContext 并从数据库中恢复应用程序的状态。通知用户上一个操作可能没有成功完成。

但是,在 中Startup.cs,我有AppDbContextas 范围的依赖项。为了重新尝试方法逻辑,我需要一个新的实例,AppDbContext但被注册为作用域将不允许这样做。

我想到的一种解决方案是制作AppDbContext瞬态。但我有一种感觉,通过这样做,我将为自己打开一整套新问题。有人可以帮我吗?

4

1 回答 1

2

保存上下文时至少有太多类型的错误。第一个发生在命令执行期间。第二个发生在提交期间(这种情况很少发生)。即使数据已成功更新,也可能发生第二个错误。所以你的代码只处理第一种错误,但没有考虑第二种错误。

对于第一种错误,您可以在命令处理程序中注入 DbContext 工厂或IServiceProvider直接使用。这是一种反模式,但在这种情况下我们别无选择,就像这样:

readonly IServiceProvider _serviceProvider;
public EditPersonalInfoCommandHandler(IServiceProvider serviceProvider) {
        _serviceProvider = serviceProvider;
}

for(int i = 0; i < 5; i++) {
  try {
    using var dbContext = _serviceProvider.GetRequiredService<AppDbContext>();
    //consume the dbContext
    //required logic
  } catch(SomeDatabaseException e) {
    if(i == 4) {
       throw;
    }
  }
}

但是正如我所说,要处理这两种错误,我们应该使用IExecutionStrategyEFCore 中的所谓。这里介绍了几个选项。但我认为以下内容最适合您的情况:

public Result Handle(EditPersonalInfoCommand command) {
    var strategy = _context.Database.CreateExecutionStrategy();
    
    var studentRepo = new StudentRepository(_context);
    Student student = studentRepo.GetById(command.Id);
    if (student == null) {
        return Result.Failure($"No Student found for Id {command.Id}");
    }

    student.Name = command.Name;
    student.Email = command.Email;
    const int maxRetries = 5;
    int retries = 0;
    strategy.ExecuteInTransaction(_context,
                                  context => {
                                      if(++retries > maxRetries) {
                                         //you need to define your custom exception to be used here
                                         throw new CustomException(...);
                                      }
                                      context.SaveChanges(acceptAllChangesOnSuccess: false);
                                  },
                                  context => context.Students.AsNoTracking()
                                                    .Any(e => e.Id == command.Id && 
                                                              e.Name == command.Name &&
                                                              e.Email == command.Email));

    _context.ChangeTracker.AcceptAllChanges();
    return Result.Success();
}

请注意,我假设您的上下文DbSet<Student>通过属性公开了 a Students。如果您有任何其他与连接无关的错误,它将不会被处理,IExecutionStrategy这很有意义。因为那是您需要修复逻辑的时候,所以重试数千次将无济于事,并且总是以该错误告终。这就是为什么我们不需要关心最初抛出的详细异常(在我们使用时不会暴露IExecutionStrategy)。相反,我们使用自定义异常(如我上面的代码中所述)来通知由于某些与连接相关的问题而无法保存更改。

于 2021-03-10T23:19:26.717 回答