C#：包装方法调用的优雅方式

Question

对于相当模糊的标题表示歉意，但我想要实现的目标可能在代码中更好地说明。

我有一个 WCF 客户端。当我调用方法时，我想将每个调用包装在一些错误处理代码中。因此，我没有直接公开这些方法，而是在客户端类上创建了以下辅助函数：

    public T HandleServiceCall<T>(Func<IApplicationService, T> serviceMethod)
    {
        try
        {
            return serviceMethod(decorator);
        }
        [...]
    }

客户端代码如下使用它：

service.HandleServiceCall(channel => channel.Ping("Hello"));

对 Ping 的调用很好地封装在一些逻辑中，该逻辑将尝试处理任何错误。

这非常有效，只是我现在需要知道服务上实际调用了哪些方法。最初，我希望仅使用表达式树检查 Func 但没有走得太远。

最后，我选择了装饰器模式：

    public T HandleServiceCall<T>(Func<IApplicationService, T> serviceMethod)
    {
        var decorator = new ServiceCallDecorator(client.ServiceChannel);
        try
        {
            return serviceMethod(decorator);
        }
        [...]
        finally
        {
            if (decorator.PingWasCalled)
            {
                Console.Writeline("I know that Ping was called")
            }
        }
    }

装饰器本身：

    private class ServiceCallDecorator : IApplicationService
    {
        private readonly IApplicationService service;

        public ServiceCallDecorator(IApplicationService service)
        {
            this.service = service;
            this.PingWasCalled = new Nullable<bool>();
        }

        public bool? PingWasCalled
        {
            get;
            private set;
        }

        public ServiceResponse<bool> Ping(string message)
        {
            PingWasCalled = true;
            return service.Ping(message);
        }
    }

它确实很笨重，而且代码很多。 有更优雅的方法吗？

Answer 1

您可以使用表达式，然后检查主体。

像这样的东西

public T HandleServiceCall<T>(Expression<Func<IApplicationService, T>> serviceMethod)     
{         
    try         
    {          
        var func = serviceMethod.Compile();
        string body = serviceMethod.Body.ToString();
        return func(new ConcreteAppService()); 
    }        
    catch(Exception ex)
    {
        ...     
              }
}

Answer 2

您是否考虑过使用面向方面的方法？听起来正是您所需要的。

包装异常和其他“元方法”功能可以编写为与您的 serviceMethods 所做的“正交”的方面。

关于 AOP 的一些一般信息：维基百科中的 AOP

以及带有容器的潜在解决方案：AOP 与温莎城堡

Answer 3

下面是一个使用表达式树的简单示例：

public T HandleServiceCall<T>(Expression<Func<T>> serviceMethod)
{
    try
    {
        return serviceMethod();
    }
    finally
    {
        var serviceMethodInfo = ((MethodCallExpression)serviceMethod.Body).Method;
        Console.WriteLine("The '{0}' service method was called", serviceMethodInfo.Name);
    }
}

请注意，此示例假定serviceMethod 表达式始终包含方法调用。

相关资源：

• 表达式类< /a>
• MethodCallExpression 类

Answer 4

是的，我相信你的代码已经煮过头了。

关于包装代码以进行常见的安全处置代理，请查看此处 一个很好的实现。使用它很简单：

using (var client = new Proxy().Wrap()) {
 client.BaseObject.SomeMethod();
}

现在您还需要访问方法名称 - 只需使用 Environment.StackTrace 即可。您需要在 Marc Gravell 的 Wrap 中添加向上遍历堆栈的功能。