【问题标题】:Generic delegate variable通用委托变量
【发布时间】:2015-11-03 19:34:17
【问题描述】:

我在这里伤到了我的大脑。

一些背景:

我在多层应用中有多项服务。其中一些是短暂的(应该与using 一起使用),而另一些则不是(所以我只是将它们注入到我想要的任何类中并直接使用)。我的客户端代码有不同的数据接口和随机的using关键字。

出于某些原因(主要是为了科学),我决定对其进行一些重构。我创建了一个通用的 DataServiceAdapter<> 类,对于我使用的每一个服务都应该关闭它,并且这个通用类封装了来自客户端的 using 逻辑。

DataServiceAdapter 看起来像这样(简短的工作示例here on dotnetfiddle)

public class DataServiceAdapter<TService> : IDataService
    where TService : IDataService, new()
{
    private readonly TService _service;
    private readonly Func<Func<TService, dynamic>, dynamic> _run;

    public DataServiceAdapter()
    {
        if (typeof(IDisposable).IsAssignableFrom(typeof(TService)))
        {
            _run = RunWithUsing;
        }
        else
        {
            _service = new TService();
            _run = RunWithoutUsing;
        }
    }

    public bool Foo()
    {
        return _run(x => x.Foo());
    }

    public int Bar(string arg)
    {
        return _run(x => x.Bar(arg));
    }

    private dynamic RunWithUsing(Func<TService, dynamic> func)
    {
        using (var service = (IDisposable) new TService())
        {
            return func((TService)service);
        }
    }

    private dynamic RunWithoutUsing(Func<TService, dynamic> func)
    {
        return func(_service);
    }
}

public interface IDataService
{
    bool Foo();
    int Bar(string arg);
}

客户应该像这样使用它:

//class variable. Get this from IoC container
private readonly IDataService _service;
//somewhere in methods
var resultData = _service.Foo(); //full static type checking

但是因为我的真实生活版本的IDataService 包含几十个方法,所以我像这样重写了DataServiceAdapter(仅更改,dotnetfiddle link):

public class DataServiceAdapter<TService> : IDataServiceAdapter<TService>
    where TService : IDataService, new()
{
    public dynamic Run(Func<TService, dynamic> func)
    {
        return _serviceUsing(func);
    }
}

public interface IDataServiceAdapter<out TService>
    where TService : IDataService
{
    dynamic Run(Func<TService, dynamic> func);
}

客户端现在可以通过这种方式使用该版本的DataServiceAdapter:

//class variable. Get this from IoC container
private readonly IDataServiceAdapter<IDataService> _service;
//somewhere in methods
var resultData = _service.Run(s => s.Foo()); 

由于dynamic,它无需静态检查即可工作

所以(我们接近于质疑)我决定再次重写它(为了科学)并返回静态安全,但无需将所有 IDataService 方法包装在我的适配器类中。

首先我写了这个delegate:

private delegate TRes RunDelegate<TResult>(Func<TService, TResult> func);

但是我不能像上面那样创建这样的委托的变量并传递一些我希望使用的方法。

问题: 有什么方法可以在那里实现我的设计思想(通用服务提供者调用带有using 的方法或没有它的方法并且具有类型安全性)?

【问题讨论】:

    标签: c# oop generics dynamic parametric-namespaces


    【解决方案1】:

    您可以将运行逻辑委托给实现 IRunner 接口的运行器类。如果你觉得有必要,你可以选择注入它。

    像这样:

    使用系统;

    public class Program
    {
        public static void Main()
        {
            //Dependency register logic here. Choose either
    
            //var service = new DataServiceAdapter<SpecificNotDisposableDataService>();
            var service = new DataServiceAdapter<SpecificDisposableDataService>();
    
            var client = new Client(service);
            client.ClientMethod();
            Console.ReadLine();
        }
    }
    
    public class Client
    {
        private readonly IDataServiceAdapter<IDataService> _service;
        public Client(IDataServiceAdapter<IDataService> service)
        {
            _service = service;
        }
    
        public void ClientMethod()
        {
            Console.WriteLine(_service.Run(s => s.Foo()));
            Console.WriteLine(_service.Run(s => s.Bar("Hello")));
        }
    }
    
    public class DataServiceAdapter<TService> : IDataServiceAdapter<TService>
        where TService : IDataService, new()
    {
        private interface IRunner
        {
            T Run<T>(Func<TService, T> func);
        }
        private class WithUsing : IRunner
        {
            public T Run<T>(Func<TService, T> func)
            {
                using (var service = (IDisposable) new TService())
                {
                    return func((TService)service);
                }
            }
        }
        private class WithoutUsing : IRunner
        {
            private readonly TService _service = new TService();
            public T Run<T>(Func<TService, T> func)
            {
                return func(_service);
            }
        }
        private readonly IRunner _runner;
    
        public DataServiceAdapter()
        {
            if (typeof(IDisposable).IsAssignableFrom(typeof(TService)))
            {
                _runner = new WithUsing();
            }
            else
            {
                _runner = new WithoutUsing();
            }
        }
    
        public T Run<T>(Func<TService, T> func)
        {
            return _runner.Run<T>(func);
        }
    }
    
    public class SpecificDisposableDataService : IDataService, IDisposable
    {
        public bool Foo()
        {
            return true;
        }
    
        public int Bar(string arg)
        {
            return arg.Length;
        }
    
        public void Dispose()
        {
            //Dispose logic here
        }
    }
    
    public class SpecificNotDisposableDataService : IDataService
    {
        public bool Foo()
        {
            return false;
        }
    
        public int Bar(string arg)
        {
            return arg.Length*2;
        }
    }
    
    public interface IDataServiceAdapter<out TService>
        where TService : IDataService
    {
        T Run<T>(Func<TService, T> func);
    }
    
    public interface IDataService
    {
        bool Foo();
        int Bar(string arg);
    }
    

    这是上面的工作 dotnetfiddle:https://dotnetfiddle.net/FmNpju

    这是另一个同时运行这两种服务: https://dotnetfiddle.net/KxEGRB

    【讨论】:

    • 哇,这是一个转折点。它看起来比我的实现好十倍并且有效。谢谢!
    • @Szer 欢迎您!注意DataServiceAdapter&lt;TService&gt; 的双重用途,既作为类又作为参数/通用命名空间,与嵌套的IRunner 接口和实现该接口的类共享TService 类型参数。当类型清楚地正确耦合时,这种技术有助于保持代码 DRY [当然通过适当的接口,并且一个接口专门支持另一个接口]。查看stackoverflow.com/tags/parametric-namespaces/info 了解更多信息。
    【解决方案2】:

    这可以通过一个简单的扩展方法来处理:

    public static TResult Run<TService, TResult>(this IDataService<TService> @this, 
                                                 Func<TService, TResult> func)
    {
      if (typeof(IDisposable).IsAssignableFrom(typeof(TService)))
      {
        using (var service = (@this.GetInstance() as IDisposable))
        {
          return func(@this);
        }
      }
      else
      {
        return func(@this.GetInstance());
      }
    }
    

    只需实现GetInstance 方法即可返回新服务(当IDisposable)或“单例”(当不是IDisposable)。

    您会发现,当使用函数式范式(如本例中的 lambda 或委托)时,一路走下去是值得的,而不是试图强制一些函数式/面向继承的混合体。

    当然,如果您以前曾涉足函数式编程,那么即使是这段代码也已经在困扰您了——它的泛化性令人怀疑;为什么要限制自己进行这种操作?

    您可以使用相同的方法将逻辑移至您的服务提供商。关键是您保持执行方法通用,这允许您保留强类型。只需让方法处理执行,而不是尝试在某处存储委托。

    如果您真的想进一步委托,可以使用额外的接口:

    interface IExecutor<TService>
    {
      TResult Execute<TResult>(Func<TService, TResult> func);
    }
    

    请注意,该接口仅在 TService 上是通用的 - 它是 Execute 方法再次添加了 TResult 以实现类型安全。这意味着您可以轻松地将任一实现存储在同一字段中 - 您仅在实际请求 Execute 操作时传递 TResult,然后您就知道 TResult 应该是什么。

    【讨论】:

    • 是的,这将解决我的问题,但是每次我从服务类调用方法时都会执行此特定检查,因此我的实现只检查一次(在构造函数中)。我不确定这个检查有多“重”,所以你的解决方案可能足够好。我试试
    • @Szer 忘记这样的事情。您已经在处理与服务的通信 - 我想它是一个远程服务,对吧?像这样的检查与最简单的远程请求响应完全不同。这正是那种考虑性能是决定性的代码——如果你真的有充分的理由来优化它,有很多方法可以绕过检查,但除非这是性能瓶颈,否则它根本不值得——而且它不是不会的。
    • 是的,你是对的。此检查将比这些服务调用轻一百倍...
    猜你喜欢
    • 2020-03-05
    • 2010-09-14
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 2021-07-05
    • 2011-01-16
    • 1970-01-01
    相关资源
    最近更新 更多