C# 函数作为参数 - 重复方法结构

Question

我有多个具有不同签名的方法，每个方法都有一个带有自定义日志异常的 try-catch 块。 （多个控制器上的结构相同）。

public class TestController : BaseController
{
    public static ActionResult One(int param1, string param2)
    {
        try
        {
            // Do something
        }
        catch (Exception e)
        {
            LogException(e.Message);
            AddModelError(e.Message);
        }
        return View("ViwName1");
    }

    public static ActionResult Two(Date param3, bool param4)
    {
        try
        {
            // Do something
        }
        catch (Exception e)
        {
            LogException(e.Message);
            AddModelError(e.Message);
        }
        return View("ViwName2");
    }
}

我想知道是否有一种方法可以避免每个方法的 try-catch 阻塞并执行另一个

public class TestController : BaseController
{
    public static ActionResult One(int param1, string param2)
    {
        // Do something (*)
        // Call "ActionWithTryCatch" method that has a "function argument" to "Do something (*)"
    }

    public ActionResult ActionWithTryCatch(MyDelegate del, string viewName)
    {
        try
        {
            return del.Invoke();
        }
        catch (Exception e)
        {
            LogException(e.Message);
            AddModelError(e.Message);
        }
        return View(viewName);
    }
}

¿我该怎么做？我见过使用委托的例子，但我知道这是强类型的，所以没有找到办法做到这一点。谢谢！

Answer 1

您所描述的模式接近于面向方面编程 (AOP) 的一种形式。但是，如果您只想将特定的 Try Catch 错误处理逻辑应用于控制器上的所有操作，那么引入整个 AOP 框架可能不值得。相反，您可以利用 HandleErrorAttribute 或覆盖控制器类的 OnException 方法。

例如，您可以这样编写控制器：

public class TestController
{
    private TestService service;

    public TestController(TestService service)
    {
        this.service = service;
    }

    public ActionResult One(int param1, string param2)
    {
        this.service.MethodOne(param1, param2);
        return View("ViwName1");
    }

    public ActionResult Two(Date param3, bool param4)
    {
        this.service.MethodTwo(param3, param4);
        return View("ViwName2");
    }

    protected override void OnException(ExceptionContext filterContext)
    {
        LogException(filterContext.Exception.Message);
        AddModelError(filterContext.Exception.Message);

        var errorView = new ViewResult { ViewName = "~/Path/To/Error/View" };
        filterContext.Result = errorView;
    }
}

如果您想进一步抽象它，那么您可以将重写的 OnException 逻辑移动到基本控制器类中，然后让您的所有控制器都从基本控制器继承。

如果您想了解 MVC 中统一错误处理的其他一些方法，请查看此博客：https://dusted.codes/demystifying-aspnet-mvc-5-error-pages-and-error-logging

更新

根据我的评论，如果您坚持实施您所描述的模式，您可以使用 gilmishal 答案的修改版本。

public class TestController
{
    private TestService service;

    public TestController(TestService service)
    {
        this.service = service;
    }

    public ActionResult One(int param1, string param2)
    {
        return this.ActionWithTryCatch(() => this.service.MethodOne(param1, param2), "ViwName1");
    }

    public ActionResult Two(Date param3, bool param4)
    {
        return this.ActionWithTryCatch(() => this.service.MethodTwo(param3, param4), "ViwName2");
    }

    public IActionResult ActionWithTryCatch(Action action, string viewName)
    {
        try
        {
            action.Invoke();
        }
        catch (Exception e)
        {
            LogException(e.Message);
            AddModelError(e.Message);
        }

        return View(viewName);
    }
}

Answer 2

如果您需要在 js 中将参数传递给此函数，您创建的此方法将无法正常工作 - 所以我假设您只调用无参数方法。

在这种情况下，您可以使用Func<IActionResult> 而不是MyDelegate。

public TResult ActionWithTryCatch<TResult>(Func<TResult> del, string viewName)
{
    try
    {
        return del.Invoke();
    }
    catch (Exception e)
    {
        LogException(e.Message);
        AddModelError(e.Message);
        throw;
    }
}

这将更类似于您的 javascript 实现，并且会在出现未处理的异常时返回 500 http 结果。

如果你想要IActionResult返回类型，你应该这样称呼它-

ActionWithTryCatch<IActionResult>(MethodThatReturnsIActionResult, viewName);

我建议您查看generics