减少冗余代码的通用方法

Question

我正在使用@jweber创建的名为WcfClientProxyGenerator的库

我使用该nuget生成了针对服务的代理。 一切正常，这是一个Proxy类：

using Silly.Services.CarPark.Logging;
using Silly.Services.CarPark.ServiceContracts.Common.v1;
using log4net;
using System.Linq;
using System.Threading.Tasks;
using WcfClientProxyGenerator;

namespace Silly.Services.CarPark.Client
{
    public class RequestManager
    {
        private static Log4NetLoggingAdapter systemLogger;
        private static IRequestService proxy = null;

        public RequestManager()
        {
            if (systemLogger == null)
            {
                LoggerSetup.Configure(Constants.ModuleNameForLogging);
                systemLogger = new Log4NetLoggingAdapter(LogManager.GetLogger(typeof(RequestManager)));
            }

            proxy = WcfClientProxy.Create<IRequestService>(c =>
            {
                c.SetEndpoint("BasicHttpBinding_IRequestService");

                c.OnCallBegin += (sender, args) => { };

                c.OnBeforeInvoke += (sender, args) =>
                {
                    systemLogger.Write(string.Format("{0}.{1} called with parameters: {2}", args.ServiceType.Name, args.InvokeInfo.MethodName, string.Join(", ", args.InvokeInfo.Parameters)), Core.EventSeverity.Verbose);
                };

                c.OnAfterInvoke += (sender, args) =>
                {
                    systemLogger.Write(string.Format("{0}.{1} returned value: {2}", args.ServiceType.Name, args.InvokeInfo.MethodName, args.InvokeInfo.ReturnValue), Core.EventSeverity.Verbose);
                };

                c.OnCallSuccess += (sender, args) =>
                {
                    systemLogger.Write(string.Format("{0}.{1} completed successfully", args.ServiceType.Name, args.InvokeInfo.MethodName));
                };

                c.OnException += (sender, args) =>
                {
                    systemLogger.Write(string.Format("Exception during service call to {0}.{1}: {2}", args.ServiceType.Name, args.InvokeInfo.MethodName, args.Exception.Message), args.Exception, Core.EventSeverity.Error);
                };
            });
        }
        }
}

一段时间后，我发现自己获得了另外三个服务和三个代理。 我看到我要对所有四个类重复构造器中的任何内容，并且更改很难跟踪。 目前，日志记录的数量还很少，但是这种情况一直在变化。 并且即将推出三项服务。

有没有一种使用Class删除通用代码的方法？

Answer 1

依赖注入不能解决关于代码重复的主要问题，因为systemLogger和proxy都依赖于服务接口的类型，在本例中为IRequestService 。 因此，即使您“从外部”传递这些，对于要构建的每个服务接口 ，您仍然会有不同的代码来创建传递的对象。

泛型确实是捕获通用性的方法，因为每种服务的重复代码唯一的区别就是接口类型。 我将使用通用基类，并从中派生每个服务类，如下所示。

出于可测试性的原因，您可能仍要考虑使用依赖项注入，以避免对日志记录适应程序进行更新。 但这是另一个问题。

基类

using log4net;
using System.Linq;
using System.Threading.Tasks;
using WcfClientProxyGenerator;
using Silly.Services.CarPark.Logging;

namespace Silly.Services.Carpark
{
    public class ClientBase<TClient> where TClient : class
    {
        private static Log4NetLoggingAdapter systemLogger;
        private static TClient proxy;

        public ClientBase(string binding)
        {
            if (systemLogger == null)
            {
                LoggerSetup.Configure(Constants.ModuleNameForLogging);
                systemLogger = new Log4NetLoggingAdapter(LogManager.GetLogger(this.GetType()));
            }

            proxy = WcfClientProxy.Create<TClient>(c =>
            {
                c.SetEndpoint(binding);

                c.OnCallBegin += (sender, args) => { };

                c.OnBeforeInvoke += (sender, args) =>
                {
                    systemLogger.Write(string.Format("{0}.{1} called with parameters: {2}", args.ServiceType.Name, args.InvokeInfo.MethodName, string.Join(", ", args.InvokeInfo.Parameters)), Core.EventSeverity.Verbose);
                };

                c.OnAfterInvoke += (sender, args) =>
                {
                    systemLogger.Write(string.Format("{0}.{1} returned value: {2}", args.ServiceType.Name, args.InvokeInfo.MethodName, args.InvokeInfo.ReturnValue), Core.EventSeverity.Verbose);
                };

                c.OnCallSuccess += (sender, args) =>
                {
                    systemLogger.Write(string.Format("{0}.{1} completed successfully", args.ServiceType.Name, args.InvokeInfo.MethodName));
                };

                c.OnException += (sender, args) =>
                {
                    systemLogger.Write(string.Format("Exception during service call to {0}.{1}: {2}", args.ServiceType.Name, args.InvokeInfo.MethodName, args.Exception.Message), args.Exception, Core.EventSeverity.Error);
                };
            });
        }

        // The following class property exposes static private var
        // to derived classes, as read-only. 
        protected TClient Proxy { get { return proxy; } }
    }
}

派生服务等级

using Silly.Services.CarPark;
using Silly.Services.CarPark.ServiceContracts.Common.v1;
using log4net;
using System.Linq;
using System.Threading.Tasks;
using WcfClientProxyGenerator;

namespace Silly.Services.CarPark.Client
{
    public class RequestManager : ClientBase<IRequestService>, IRequestService
    {
        public RequestManager() : 
              base("BasicHttpBinding_IRequestService")
        {
        }

        // Implementation of IRequestService methods
        public string Alpha(...) {
          return Proxy.Alpha(...);  
        }

        public void Beta(...) {
              Proxy.Beta(...);
        }

    }
}

Answer 2

你自己想到了什么？ 我的意思是，做这样的事情应该很简单。

Create方法接受Action<IRetryingProxyConfigurator>作为参数，因此我将构建类似

void ConfigureLogging(IRetryingProxyConfigurator proxyConfigurator)
{
    proxyConfigurator.OnBeforeInvoke += (sender, args) =>
            {
                systemLogger.Write(string.Format("{0}.{1} called with parameters: {2}", args.ServiceType.Name, args.InvokeInfo.MethodName, string.Join(", ", args.InvokeInfo.Parameters)), Core.EventSeverity.Verbose);
            };

}

并这样称呼：

proxy = WcfClientProxy.Create<IRequestService>(c =>
        {
            c.SetEndpoint("BasicHttpBinding_IRequestService");

            ConfigureLogging(c);
        }

尽可能多地重构重复代码，然后模式就会成形。 您可能会建立一个接受服务接口作为类型参数的通用工厂，并在该类中进行所有模板化。

Answer 3

看到，既然所有绑定都在这里创建，那么所有这些服务都必须有一个公共接口。 将其用作T，无需过多重构就可以了

 public class GenericManager<T> where T: IBaseInterfaceofWhateverIRequestServiceAndTheRestDeriveFrom
    {
        private static Log4NetLoggingAdapter systemLogger;
        private static T proxy = null;

private void BuildGenericManager<T>(){

if (systemLogger == null)
            {
                LoggerSetup.Configure(Constants.ModuleNameForLogging);
                systemLogger = new Log4NetLoggingAdapter(LogManager.GetLogger(typeof(RequestManager)));
            }
            proxy = WcfClientProxy.Create<T>(c =>
            {
                c.SetEndpoint("BasicHttpBinding_IRequestService");

                c.OnCallBegin += (sender, args) => { };

                c.OnBeforeInvoke += (sender, args) =>
                {
                    systemLogger.Write(string.Format("{0}.{1} called with parameters: {2}", args.ServiceType.Name, args.InvokeInfo.MethodName, string.Join(", ", args.InvokeInfo.Parameters)), Core.EventSeverity.Verbose);
                };

                c.OnAfterInvoke += (sender, args) =>
                {
                    systemLogger.Write(string.Format("{0}.{1} returned value: {2}", args.ServiceType.Name, args.InvokeInfo.MethodName, args.InvokeInfo.ReturnValue), Core.EventSeverity.Verbose);
                };

                c.OnCallSuccess += (sender, args) =>
                {
                    systemLogger.Write(string.Format("{0}.{1} completed successfully", args.ServiceType.Name, args.InvokeInfo.MethodName));
                };

                c.OnException += (sender, args) =>
                {
                    systemLogger.Write(string.Format("Exception during service call to {0}.{1}: {2}", args.ServiceType.Name, args.InvokeInfo.MethodName, args.Exception.Message), args.Exception, Core.EventSeverity.Error);
                };
            });
        }
}
        public GenericManager()
        {
            BuildGenericManager<T>();
        }
}

这可能无法编译，您可能需要更改某些内容，但我想应该是有意图的