如何使此异常处理代码符合DRY原则？

Question

我有一个特殊的情况，我需要捕获异常并将一个对象返回给客户端来代替异常。我不能将异常处理逻辑放在更高的级别，即将 Foo 包装在 try 子句中。

最好用一些样本代码来演示。异常处理逻辑模糊了方法的意图，如果我有，许多相似意图的方法，在Foo类中，我发现自己重复了大部分catch逻辑。

在下面的代码中包装常见异常功能的最佳技术是什么？

public class Foo
{
     public Bar SomeMethodThatCanThrowExcepetion()
     {
          try
          {
              return new Bar().Execute();
          }
          catch(BazException ex)
          {
              WriteLogMessage(ex, Bar.ErrorCode);
              return new Bar() { ErrorMessage = ex.Message, ErrorCode = Bar.ErrorCode;}                  
          }
     }

     public Baz SomeMethodThatCanThrowExcepetion(SomeObject stuff)
     {
          try
          {
              return new Baz(stuff).Execute();
          }
          catch(BazException ex)
          {
              WriteLogMessage(ex, Baz.ErrorCode);
              return new Baz() { ErrorMessage = ex.Message, ErrorCode = Baz.ErrorCode;}                  
          }
     }
 } 

Answer 1

您真的需要在每个方法中显式记录吗？不要在每个方法中都有异常逻辑，而是在程序的< code>Main方法中有一个处理程序，并一般地处理异常。

另外，如果您真的需要日志记录，您不需要从catch块返回任意对象，只需使用< code > throw让它在堆栈中向上移动。

Answer 2

基类如何：

public class ErrorCapable {
  public string ErrorMessage { set; get; }
  public int ErrorCode { set; get; }

  public static ErrorCapable<T> Oops(Exception exc) where T : ErrorCapable, new() {
    // Code for logging error here
    return new T() { ErrorMessage = exc.Message, ErrorCode = exc.ErrorCode };
  }
}

public class Bar : ErrorCapable {
  //...
}
public class Baz : ErrorCapable {
  //...
}

然后在catch中，只需使用，例如：

return ErrorCapable.Oops<Bar>(ex);

Answer 3

根据Lee的评论更新

一种可能性是使用通用辅助方法。如下所示：

T TryExecute<T>(Func<T> action, int ErrorCode)
{
    try
    {
        return action();
    }
    catch (Exception ex)
    {
        result = Activator.CreateInstance<T>();
        typeof(T).GetProperty("ErrorMessage").SetValue(result, ex.Message, null);
        typeof(T).GetProperty("ErrorCode").SetValue(result, ErrorCode, null);
        return result;
    }
    return result;
}

如果你可以修改 Bar 和 Baz，那么你可以通过在 T 上放置一个要求来改进这一点：

public interface IError
{
    public string ErrorMessage { get; set; }
    public int ErrorCode { get; set; }
}

T TryExecute<T>(Func<T> action, int ErrorCode) where T : IError
{
    try
    {
        return action();
    }
    catch (Exception ex)
    {
        result = Activator.CreateInstance<T>();
        result.ErrorMessage = ex.Message;
        result.ErrorCode = ErrorCode;
        return result;
    }
}

然后你会用:

return TryExecute<Bar>(new Bar().Execute, Bar.ErrorCode);

和：

return TryExecute<Baz>(new Baz(stuff).Execute, Baz.ErrorCode);

对于您的特定设计来说，这可能是也可能不是过度抽象；魔鬼在细节中。