我正在开发一个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项目。
答案 0 :(得分:3)
我会将GetFoo
和GetBar
更改为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);
}
}
希望有所帮助。