我在 Startup 中添加了一个全局错误处理程序过滤器.cs如下所示:
services.AddMvc(o =>
{
o.Filters.Add(new GlobalExceptionFilter());
});
但是,我需要传入我的电子邮件服务,该服务也被注入。如何从筛选器中的这些服务中检索它?
public class GlobalExceptionFilter : IExceptionFilter
{
private readonly IEmailService _emailService;
public GlobalExceptionFilter()
{
}
public void OnException(ExceptionContext context)
{
}
}
我曾经能够在 MVC5 中使用依赖解析器类来执行此操作。有没有办法在核心中做到这一点?或者有没有办法让我在启动中强制实例化服务,以便我可以将其作为构造函数的一部分传递?
我尝试在服务中查找它,然后查看 ImplementationInstance,但此时它为 null,所以我无法从那里抓取它。另请记住,我的电子邮件服务需要参数 IOptions<Settings>
,以便它可以获取所需的电子邮件设置。
您可以使用构造函数注入。
public class GlobalExceptionFilter : IExceptionFilter
{
private readonly IEmailService emailService;
public GlobalExceptionFilter(IEmailService emailService)
{
this.emailService = emailService;
}
public void OnException(ExceptionContext context)
{
//do something with this.emailService
}
}
但是您必须更改在ConfigureServices
方法中注册全局筛选器的方式。您应该使用采用 Type 的Add
重载
services.AddMvc(o =>
{
o.Filters.Add(typeof(GlobalExceptionFilter));
});
另一种选择是通过在 HttpContext.RequestServices
上调用 GetService
方法来显式解析 OnException
方法内的依赖关系。
using Microsoft.Extensions.DependencyInjection;
public void OnException(ExceptionContext context)
{
var emailService = context.HttpContext.RequestServices.GetService<IEmailService>();
// use emailService
}
但是你应该对第一种方法没问题。让框架为您解决它并注入到您的构造函数中,而不是您尝试这样做。
我认为全局异常处理程序的最佳实践实际上是为其创建自定义中间件步骤。从文档
异常筛选器适用于捕获 MVC 操作中发生的异常,但它们不如错误处理中间件灵活。在一般情况下,首选中间件,并且仅在需要根据选择的 MVC 操作进行不同错误处理时使用筛选器。
然后,在配置服务方法中注册类:
services.AddTransient<IEmailService, EmailService>();
然后在配置方法中,注册客户全局异常处理程序。您希望这是您在 Configure 方法中执行的第一件事,以便在其他中间件中捕获它之后的任何异常。
app.UseMiddleware<MyGlobalExceptionHandler>();
您注册的任何服务都将可用于您的中间件构造函数,可能如下所示:
public sealed class MyGlobalExceptionHandler
{
private readonly RequestDelegate _next;
private readonly IEmailService _emailService;
public NlogExceptionHandler(
RequestDelegate next,
IEmailService emailService)
{
_next = next;
_emailService = emailService;
}
public async Task Invoke(HttpContext context)
{
try
{
await _next(context);
}
catch (Exception ex)
{
try
{
_emailService.SendEmail(ex.ToString());
}
catch (Exception ex2)
{
//Its good practice to have a second catch block for simple logging in case the email fails too
}
throw;
}
}
}