基于类型调用DAL方法

时间:2012-04-20 13:48:39

标签: c# oop generics open-closed-principle

我正在开发一个app,我需要根据调用类的泛型类型调用两种数据方法之一。例如,如果T的类型为Foo,我将调用data.GetFoo():

private static List<T> GetObjectList(DateTime mostRecentProcessedReadTime)
{
    using (MesReportingDal data = new MesReportingDal())
    {
        return data.GetFoo(mostRecentProcessedReadTime);  // Notice GetFoo()
    }
}

如果T是Bar类型,我将调用data.GetBar():

private static List<T> GetObjectList(DateTime mostRecentProcessedReadTime)
{
    using (MesReportingDal data = new MesReportingDal())
    {
        return data.GetBar(mostRecentProcessedReadTime);  // Notice GetBar()
    }
}

在此之前,我只需要一个DAL方法,因为所有类型都以相同的方式检索。我现在需要调用两种方法中的一种,具体取决于T的类型。

我试图避免这样的事情:

private static List<T> GetObjectList(DateTime mostRecentProcessedReadTime)
{
    using (MesReportingDal data = new MesReportingDal())
    {
        if (T is Foo) { return data.GetFoo(mostRecentProcessedReadTime); }
        if (T is Bar) { return data.GetBar(mostRecentProcessedReadTime); }
    }
}

这违反了OCP。有没有一种优雅的方法来处理这个,所以我可以摆脱我的if语句?

编辑 - 这就是类型

public partial class Foo1 : IDataEntity { }
public partial class Foo2 : IDataEntity { }
public partial class Bar1 : IDataEntity { }
public partial class Bar2 : IDataEntity { }

这些Foos和Bars是与Linq-to-SQL一起使用的DBML项目。

1 个答案:

答案 0 :(得分:3)

我会将GetFooGetBar更改为Get,并使MesReportingDal变为通用。

所以我认为你最终会得到这样的东西:

private static List<T> GetObjectList(DateTime mostRecentProcessedReadTime)
{
    using (var data = new MesReportingDal<T>())
    {
        return data.Get(mostRecentProcessedReadTime);        
    }
}

顺便说一句,拥有using语句还需要MesReportingDal实现IDisposable,否则您将收到以下编译错误:

'MesReportingDal':在using语句中使用的类型必须可以隐式转换为'System.IDisposable'

<强>更新

因此,在仔细考虑了这一点并阅读您的反馈之后,您可以选择一个选项来提取存储库界面并将创建推送回工厂方法。这样您就可以维持单data.Get(...)次调用,但基于T的不同实现

public interface IRepository<T> : IDisposable
{
    IList<T> Get(DateTime mostRecentRead);
}

public class FooRepo : IRepository<Foo>
{
    public IList<Foo> Get(DateTime mostRecentRead)
    {
        // Foo Implementation
    }
}

public class BarRepo : IRepository<Bar>
{
    public IList<Bar> Get(DateTime mostRecentRead)
    {
        // Bar Implemenation
    }
}

你的工厂可能看起来像这样

public class RepositoryFactory
{
    public static IRepository<T> CreateRepository<T>()
    {
        IRepository<T> repo = null;
        Type forType = typeof(T);

        if (forType == typeof(Foo))
        {
            repo = new FooRepo() as IRepository<T>;
        }
        else if (forType == typeof(Bar))
        {
            repo = new BarRepo() as IRepository<T>;
        }

        return repo;
    }
}

这样可以让您保持初始代码块清洁

private static IList<T> GetObjectList(DateTime mostRecentProcessedReadTime)
{
    using (var data = RepositoryFactory.CreateRepository<T>())
    {
        return data.Get(mostRecentProcessedReadTime);
    }
}

希望有所帮助。