如何避免异常捕获 .NET 中的复制粘贴

Question

使用 .NET 框架我有一个带有一组方法的服务，这些方法可以生成多种类型的异常：MyException2、MyExc1、Exception... 为了为所有方法提供正确的工作，每个方法都包含以下部分：

[WebMethod]
void Method1(...)
{
    try
    {
        ... required functionality
    }
    catch(MyException2 exc)
    {
        ... process exception of MyException2 type
    }
    catch(MyExc1 exc)
    {
        ... process exception of MyExc1 type
    }
    catch(Exception exc)
    {
        ... process exception of Exception type
    }
    ... process and return result if necessary
}

在每个服务方法中都有完全相同的东西（每个方法有不同的参数集）和完全相同的异常处理功能是非常无聊的......

是否有可能将这些捕获部分“分组”并仅使用一行（类似于 C++ 宏）？可能 .NET 4.0 中的一些新内容与此主题相关？

谢谢。

附：欢迎任何想法。

Answer 1

如果您的所有方法中的异常处理完全相同，您可以执行以下操作：

void CallService(Action method)
{
    try
    {
        // Execute method
        method();
    }
    catch(MyException2 exc)
    {
        ... process exception of MyException2 type
    }
    catch(MyExc1 exc)
    {   
        ... process exception of MyExc1 type
    }
    catch(Exception exc)
    {
        ... process exception of Exception type
    }
}

然后，你可以重写你的客户端代码来做：

int i = 3;
string arg = "Foo";
this.CallService( () => this.Method1(i) );
this.CallService( () => this.Method2(arg, 5) );

这使您的 Method1 和 Method2 方法变得简单：

void Method1(int arg)
{
    // Leave out exception handling here...
    ... required functionality  
    ... process and return result if necessary
}

void Method2(string stringArg, int intArg)
{
    // Leave out exception handling here...
    ... required functionality  
    ... process and return result if necessary
}

Answer 2

为什么不直接将代码分解为一个帮助方法来为您执行此操作（您也可以将将来需要的任何新异常添加到 HandleException，这使得它具有相当大的可扩展性）？

try
{
    ... required functionality
}
catch (Exception e)
{
    HandleException(e);
    throw; // only if you need the exception to propagate to caller
}


private void HandleException(Exception e)
{
    if (e is MyException2)
    {
        ... process exception of MyException2 type
    }
    else if (e is MyExc1)
    {
        ... process exception of MyExc1 type
    }
    else
    {
        ... process exception of Exception type
    }
}

Answer 3

我会仔细研究您为“处理”这些异常所做的工作。很有可能您根本不需要 catch 块，并且您应该允许传播异常。

Answer 4

突然出现极客（并展示您可以但很可能不应该和不想做的事情）：您可以通过动态生成包含异常处理的函数来使整个事情更具组合性和可重用性逻辑：

static class ExceptionHandlerExtensionMethods 
{
    // extend to Func<T> as desired
    public static Action Catching<T>(this Action what, Action<T> handler) 
        where T : Exception
    {
        return () =>
        {
             try
             {
                 what();
             }
             catch (T ex)
             {
                 handler(ex);
             }
         };
    }
}

现在您可以在某处实现和重用特定于异常类型的处理函数，并将它们组合到其他方法的异常处理中。

要消除冗余，您可以编写一个辅助函数，将“典型”异常处理函数添加到您想要调用的任何内容中，然后调用此修饰方法。

Answer 5

使用 Reed Copsey 的 CallService 方法：

void DoSomething(object param1, int param2)
{
    this.CallService(() =>
    {
         // work with param1 and param2 here
    }
}

对于需要返回值的情况，可能需要复制 CallService 来返回类型参数。

T CallService<T>(Func<T> callback) { /* ... */ }

Answer 6

我知道这是不好的做法，但是如果您在每个 catch 语句中都有完全相同的错误处理，那么为什么不直接使用最后一个 catch 语句作为全部捕获呢？

这当然假设您的所有异常都继承自 Exception。