异常处理的良好实践设计模式

Question

对于底层方法的以下代码，我在每个方法中都有异常处理代码

throw new Exception("The error that happens");

有什么办法可以避免在每种方法中一次又一次地编写这段代码？

我正在尝试编写自己的代码，而不是使用任何日志框架

private void TopLevelMethod()
{
    try
    {
        SomeMethod();
    }
    catch (Exception ex)
    {
        // Log/report exception/display to user etc.
    }
}

private void SomeMethod()
{
    TestPartA();
    TestPartB();
    TestPartC();
    TestPartD();
}

private void TestPartA()
{
    // Do some testing...
    try
    {
        if (somethingBadHappens)
        {
            throw new Exception("The error that happens");
        }
    }
    catch (Exception)
    {
        // Cleanup here. If no cleanup is possible, 
        // do not catch the exception here, i.e., 
        // try...catch would not be necessary in this method.

        // Re-throw the original exception.
        throw;
    }
}

private void TestPartB()
{
    // No need for try...catch because we can't do any cleanup for this method.
    if (somethingshappens)
    {
        throw new Exception("The error that happens");
    }
}

Answer 1

只有当你想做一些有意义的事情时才捕获错误，例如：

1. 用框架异常包装异常（例如SqlException。ADO.NET 永远不会向您传递套接字级错误。它会向您传递有意义的 SQL 错误代码）
2. 清理
3. 实际响应（例如重试或插入默认值）

日志记录几乎是不合适的。顶级处理程序应记录。当然不是路径中的每个方法都应该记录。日志和代码太混乱了。不要那样做。

只是不要吞下错误信息，让错误冒出来。这样就没有理由在任何地方插入本地日志记录代码来记录错误。

Answer 2

如果您更喜欢使用类似代码风格的函数式编程，一种方法是使用回调错误回调。 示例：

    private void SomeMethod()
    {
        // do something
    }
     public bool Execute(Action act, Action<Exception> onErrorCallback)
        {
            var res = true;
            try
            {
                act();
            }
            catch (Exception ex)
            {
                res = false;
                onErrorCallback(ex);
            }
            return res;
        }

并像这样使用Execute：

   var successfull = true;
   successfull &= Execute(SomeMethod, (ex) => {  /* clean up */ });
   successfull &= Execute(SomeMethod, (ex) => {  /* clean up */ });
   successfull &= Execute(SomeMethod, (ex) => {  /* clean up */ });
   successfull &= Execute(SomeMethod, (ex) => {  /* clean up */ });
   if (!successfull)
       ; // show user or something else

Answer 3

Graffito：能否提供示例代码示例。谢谢...

您的代码已重构：

private void TopLevelMethod()
{ 
    List<string> errors=new List<string>() ;
    if (!SomeMethod(errors)) { /* Log/report errors/display to user etc. */ }
}

private bool SomeMethod(List<string> errors)
{
    return TestPartA(errors) && TestPartB(errors) && TestPartC(errors) && TestPartD(errors);
}

private bool TestPartA(List<string> errors)
{
  bool result = true ;
  try 
  {
    // Do some testing...
    if (somethingBadHappens) { result=false; errors.Add("The error that happens"); }
  }
  catch (Exception ex) { errors.Add("Error in TestPartA: "+Ex.Exception.Message.ToString()) ; }
  return result ;
}

private bool TestPartB(List<string> errors)
{
  bool result = true ;
  // Do some testing...
  if (somethingBadHappens) { result = false ; errors.Add("The error that happens"); }
  return result ;
}