将 Action 和 Func 结合在一个参数中

Question

我有许多方法需要使用相同模式进行一些日志记录。有些方法需要返回一些值，有些则不需要。我创建了一个带有 Action 参数的方法，以避免复制粘贴所有逻辑。它看起来像这样：

private void Execute(Action action)
{
   Logger.Start();
   try
   {
      action();
   }
   catch(Exception exception)
   {
      Logger.WriteException();
      throw;
   }
   finally
   {
       Logger.Finish();
   }
}

现在我有一些类似的电话

public void DoSomething(string parameter)
{
    Execute(() => GetProvider(parameter).DoSomething());
}

但我需要一些返回值的函数。最好的方法是什么？ 我现在找到了两个：

1) 使用 Func 创建 Execute 方法的副本

private T Execute<T>(Func<T> action)
{
   Logger.Start();
   try
   {
      return action();
   }
   catch(Exception exception)
   {
      Logger.WriteException();
      throw;
   }
   finally
   {
       Logger.Finish();
   }
}

此方法有效，但也有一些复制粘贴。

2) 欺骗参数成为一个动作：

public Result DoSomething(string parameter)
{
    Result result = null;
    Execute(() => result = GetProvider(parameter).DoSomething());
    return result;
}

这不需要复制粘贴，但看起来不太好。

有没有办法以某种方式加入 Action 和 Func 以避免这些方法中的任何一种，或者可能有另一种方法来实现相同的结果？

Answer 1

第三种选择仍然是重载Execute，但使Action 版本根据Func 版本工作：

private void Execute(Action action)
{
    // We just ignore the return value here
    Execute(() => { 
        action();
        return 0; 
    });
}

当然，如果 void 更像一个“真实”类型（如 F# 等中的 Unit），那么这一切都会更简单，此时我们可以只使用 Task<T> 而不是 Task 和 @ 987654329@也...

Answer 2

创建Execute 的副本，将Func 转换为Action。你只需要写一次丑陋的代码，你不会得到Execute方法的完整的第二个副本：

private T Execute<T>(Func<T> func)
{
    T result = default(T);
    this.Execute(() => { result = func(); });
    return result;
}

...

public Result DoSomething(string parameter)
{
    return Execute(() => GetProvider(parameter).DoSomething());
}

Answer 3

这是另一种选择。不要让日志框架调用您的实际代码，而是让您的实际代码调用日志框架。这样的事情就可以解决问题（大大简化了）。

public class LoggerScope : IDisposable {

    private bool disposed;

    public LoggerScope() {
        Logger.Start();
    }

    public void Dispose() {
        if(!disposed) {
            Logger.Finish();
            disposed = true;
        }
    }
}

如下使用：

        using(var scope = new LoggerScope()) {
            // actual code goes here
        }

通过在代码的顶层仅捕获和记录一次异常来单独处理异常。

优点：

• 避免到处都需要 lambda，因此异常堆栈跟踪更加清晰。
• 您可以将任意上下文数据添加到 LoggerScope 类，例如GUID、时间戳、逻辑任务描述文本。