我想弄清楚如何简化以下
假设我有2个实体类
public class A
{
public int Id { get; set; }
public string Name { get; set; }
public string City { get; set; }
}
AND
public class B
{
public int Id { get; set; }
public string Nom { get; set; }
public string Ville { get; set; }
}
类似但不相同的类。
每个类都有一个用于CRUD操作的存储库类,例如......
public class RepA
{
public static List<A> GetAll()
{
List<A> list = new List<A>();
A a1 = new A() {Id=1, Name="First A", City="Boston"};
A a2 = new A() {Id=2, Name="First B", City="Chicago"};
A a3 = new A() {Id=3, Name="First C", City="San Francisco"};
list.Add(a1);
list.Add(a2);
list.Add(a3);
return list;
}
public static void SaveAll(List<A> list)
{
foreach (A a in list)
{
Console.WriteLine("Saved Id = {0} Name = {1} City={2}",
a.Id, a.Name, a.City);
}
}
}
AND
public class RepB
{
public static List<B> GetAll()
{
List<B> list = new List<B>();
B b1 = new B() {Id=1, Nom="Second A", Ville="Montreal"};
B b2 = new B() {Id=2, Nom="Second B", Ville="Paris"};
B b3 = new B() {Id=3, Nom="Second C", Ville="New Orleans"};
list.Add(b1);
list.Add(b2);
list.Add(b3);
return list;
}
public static void SaveAll(List<B> list)
{
foreach (B b in list)
{
Console.WriteLine("Saved Id = {0} Name = {1} City={2}", b.Id,
b.Nom, b.Ville);
}
}
}
如何在不必诉诸于此的情况下对我的存储库进行匿名调用,因为在我的真实世界示例中,我有100个存储库,而不是2个。
void Main()
{
ChosenType chosentype = RandomChosenType(); //A or B
switch (chosentype)
{
case ChosenType.A:
var listA = RepA.GetAll();
RepA.SaveAll(listA);
break;
case ChosenType.B:
var listB = RepB.GetAll();
RepB.SaveAll(listB);
break;
default:
break;
}
}
答案 0 :(得分:3)
制作base class
或使用interface
:
public interface IBase<T>
{
List<T> GetAll();
void SaveAll(List<T> items);
}
public class RepA : IBase<RepA>
{
public List<RepA> GetAll() { return new List<RepA>(); }
public void SaveAll(List<RepA> repA) { }
}
public class RepB : IBase<RepB>
{
public List<RepB> GetAll() { return new List<RepB>(); }
public void SaveAll(List<RepB> repB) { }
}
void Main()
{
IBase chosenType = RandomChosenType();
var list = chosenType.GetAll();
}
答案 1 :(得分:3)
您应该使用单个通用存储库。操作应由注入的代表处理。存储库可能如下所示:
public class GenericRepositoryExample
{
public void Save<T>(IList<T> persons, SaveDelegate<T> save)
{
foreach (T person in persons)
{
Console.WriteLine(save(person));
}
}
}
请注意,保存委托将传递给Save方法。您的示例中的SaveDelegate可以声明为:
public delegate string SaveDelegate<T>(T input);
为方便起见,我创建了一个包含委托函数的HelperClass。在现实生活中,如果可能的话,通常应该避免使用辅助类。
public static class HelperClass
{
public static string FrenchSave(B frenchInput)
{
string result = string.Format("ID = {0}; Name = {1}; City = {2}", frenchInput.Id, frenchInput.Nom, frenchInput.ville);
return result;
}
public static string EnglishSave(A englishInput)
{
string result = string.Format("ID = {0}; Name = {1}; City = {2}", englishInput.Id, englishInput.name, englishInput.city);
return result;
}
}
为了说明此设置的使用,我创建了以下单元测试:
[Test]
public void TestGenericRepository()
{
IList<A> aList = new List<A>();
aList.Add(new A() { Id = 1, name = "George", city = "Chicago"});
aList.Add(new A() { Id = 2, name = "Bill", city = "Toledo" });
List<B> bList = new List<B>();
bList.Add(new B() {Id= 1, Nom = "Nathalie", ville = "Paris"});
bList.Add(new B() {Id = 2, Nom = "Michelle", ville = "Lyon"});
GenericRepositoryExample repository = new GenericRepositoryExample();
repository.Save<A>(aList,HelperClass.EnglishSave);
repository.Save<B>(bList,HelperClass.FrenchSave);
}
答案 2 :(得分:1)
您可以让您的存储库实现一个界面,比如IGetAllSaveAll
。然后,您可以将存储库存储在列表中,并将它们转换为该接口。这样你就可以在所有这些上调用GetAll
函数:
(实际上第一个接口不是强制性的,你可以直接把它写成IEnumerable<object> GetAll()
...)
interface IGetAllSaveAll<T>
{
IEnumerable<T> GetAll();
void SaveAll(IEnumerable<T> obj);
}
你需要有一个基础界面:
interface IGetAllSaveAll : IGetAllSaveAll<object>
并使用它:
public class RepA: IGetAllSaveAll
public class RepB: IGetAllSaveAll
....
然后你可以在某个地方保留所有这些存储库的词典:
Dictionnary<Type, IGetAllSaveAll> myDic;
当然,你仍然需要将你的存储库添加到你的词典中:
myDic.Add(typeof(A), new RepA());
然后称之为:
Type t = RandomChosenType();
myDic[t].GetAll();
答案 3 :(得分:0)
根据反射和对类结构的一些假设尝试这种方法:
static void Main(string[] args)
{
var types = Assembly.GetExecutingAssembly().Modules
.SelectMany(m => m.GetTypes())
.Where(t =>
t.GetMethod("GetAll") != null &&
t.GetMethod("SaveAll") != null &&
t.GetMethod("GetAll").ReturnType.IsGenericType)
.Select(t =>
new
{
RepositoryType = t,
ReturnTypeArgument =
t.GetMethod("GetAll").ReturnType.GenericTypeArguments[0]
}
)
.ToList();
(new List<dynamic> { new A(), new B() }).ToList().ForEach(chosenType =>
{
var association = types
.FirstOrDefault(t =>
t.ReturnTypeArgument == chosenType.GetType());
if (association == null)
return;
var repType = association.RepositoryType;
dynamic list = repType.GetMethod("GetAll")
.Invoke(chosenType, new object[] { });
repType.GetMethod("SaveAll")
.Invoke(chosenType, new object[] { list });
});
}
答案 4 :(得分:0)
您发布的代码使用静态方法。为了实现接口,您将需要实例方法。除非你想使用反射(在我看来应该避免),这些方法需要不知道类型。像这样:
public interface IRepository {
IEnumerable<object> GetAll();
}
在RepA中:
IEnumerable<object> IRepository.GetAll() {
return RepA.GetAll();
}
每个菜单选项只能在类型IRepository
的字段中包含相应存储库类的实例,而不是存储类型。在其中一个实例上调用GetAll
之后,如有必要,您可以稍后将结果转换为特定类型(如List<A>
)。
答案 5 :(得分:0)
根据您的具体情况,您可以获得代表每种可能数据类型的枚举,这里有一些可行的方法。
使用属性将每个枚举值映射到存储库类型。每个存储库都继承自一个泛型类,该类实现了一个非强类型的基本接口。 repo方法从静态成员更改为实例成员。基础repo类必须进行强制转换才能将object
转换为适当的类型并返回,但实际的存储库实现是强类型的。
您可以更进一步,尝试使用表达式树缓存一些反射,这样您只需要执行一次,但这取决于您实际需要的优化程度。
public enum ChosenType {
[Repo(typeof(RepA))] A = 0,
[Repo(typeof(RepB))] B = 1
}
public class RepoAttribute : Attribute {
public RepoAttribute(Type repoType) { RepoType = repoType; }
public Type RepoType { get; set; }
}
class Program
{
static void Main()
{
ChosenType chosentype = RandomChosenType(); //A or B
// Make an instance of the appropriate repo based on the mapping
// to the enum value.
// This is a moderately expensive call, and there's room for improvement
// by using expression trees and caching lambda expressions.
var repo = (IRepo)Activator.CreateInstance(
((RepoAttribute)typeof(ChosenType).GetMember(chosentype.ToString())
.Single().GetCustomAttributes(typeof(RepoAttribute), false).Single()
).RepoType);
var list = repo.GetAll();
repo.SaveAll(list);
Console.Read();
}
static Random _rand = new Random();
static ChosenType RandomChosenType()
{
return (ChosenType)_rand.Next(0, 2);
}
}
public class A { /* No change */ }
public class B { /* No change */ }
public interface IRepo {
List<object> GetAll();
void SaveAll(List<object> list);
}
public abstract class Repo<T> : IRepo {
List<object> IRepo.GetAll() {
return GetAll().Cast<object>().ToList();
}
void IRepo.SaveAll(List<object> list) {
SaveAll(list.Cast<T>().ToList());
}
public abstract List<T> GetAll();
public abstract void SaveAll(List<T> list);
}
public class RepA : Repo<A> {
public override List<A> GetAll() { /* No change except the signature */ }
public override void SaveAll(List<A> list) { /* No change except the signature */ }
}
public class RepB : Repo<B> {
public override List<B> GetAll() { /* No change except the signature */ }
public override void SaveAll(List<B> list) { /* No change except the signature */ }
}