現在、私は継承構造を持っています。たとえば、Employee クラスと Manager クラスを Person 抽象クラスから継承します。各具象クラスに 1 つのリポジトリを実装するか、継承構造全体に 1 つのリポジトリを実装する必要がありますか? 個人的には、具象クラスごとに 1 つのリポジトリがより正しいと感じています。以下は、このアプローチの私の実装です。Data Mapper パターンを Repository と一緒に使用しますが、このアプローチを実装する際にいくつかの問題があります: Data Mapper クラス以外にどこに配置すればよいかわからないメソッドもありますが、そこに配置するのも非常に厄介です
public enum PersonType
{
Unknown = 0,
Employee = 1,
Manager = 2
}
public abstract class Person
{
public int Id { get; set; }
public string Name { get; set; }
public int Age { get; set; }
public PersonType Type { get; set; }
//Omitted properties and methods
}
public class Employee : Person
{
public string EmployeeNo { get; set; }
//Omitted properties and methods
}
public class Manager : Person
{
public string ManagerNo { get; set; }
//Omitted properties and methods
}
以下はマッパークラスです。
public interface IMapper
{
bool CanMap(PersonType type);
Person MapFrom(DataRow dataRow);
//This method is awkward to put here
SqlCommand CreateUpdateCommandFrom(Person person);
}
public abstract class MapperBase : IMapper
{
public abstract bool CanMap(PersonType type);
public abstract Person MapFrom(DataRow dataRow);
public abstract SqlCommand CreateUpdateCommandFrom(Person person);
protected T MapCommonPropertiesFrom<T>(DataRow dataRow) where T : Person, new()
{
return new T
{
//Populate common properties like id, name, age
};
}
}
public class EmployeeMapper : MapperBase
{
public override bool CanMap(PersonType type)
{
return type == PersonType.Employee;
}
public override Person MapFrom(DataRow dataRow)
{
Employee employee = MapCommonPropertiesFrom<Employee>(dataRow);
//Populate employee properties
return employee;
}
public override SqlCommand CreateUpdateCommandFrom(Person person)
{
SqlCommand updateCmd = new SqlCommand();
//Set parameters for updateCmd from person properties
return updateCmd;
}
}
//Similar class for ManagerMapper
以下は、マッパークラスを検索する登録クラスです
public static class MapperRegistry
{
private static List<IMapper> _Mappers = RegisterMappers();
private static List<IMapper> RegisterMappers()
{
return new List<IMapper>
{
new EmployeeMapper()
//ManagerMapper
};
}
public static IMapper FindMapperFor(PersonType type)
{
foreach (IMapper eachMapper in _Mappers)
{
if (eachMapper.CanMap(type))
{
return eachMapper;
}
}
throw new ArgumentException(string.Format("Cannot find mapper for {0}", type));
}
}
そして最後のクラスはリポジトリクラスです
public class PersonRepository
{
public Person FindBy(int id)
{
//Execute query command in db, get datatable back, store each person
DataTable returnedData = ExecuteQueryCommand(id);
PersonType type = int.Parse(returnedData.Rows[0], "Id");
IMapper mapper = MapperRegistry.FindMapperFor(type);
return mapper.MapFrom(returnedData.Rows[0]);
}
private DataTable ExecuteQueryCommand(int id)
{
//Execute query command in db
return new DataTable();
}
public void Update<T>(T person) where T : Person
{
IMapper mapper = MapperRegistry.FindMapperFor(person.Type);
SqlCommand updateCmd = mapper.CreateUpdateCommandFrom(person);
//Execute update command
}
}
私が正しくないと感じているのは、現在 Mapper クラスに配置されている CreateUpdateCommandFrom() です。当然、リポジトリ クラスに配置する必要がありますが、一般的なリポジトリ クラスでは、Employee クラスの EmployeeNo や ManagerNo などの子クラスの特定のプロパティはわかりません。 Manager クラスにあるため、リポジトリに共通の SqlCommand を作成できません
より良いアプローチはありますか？どうもありがとうございました