将多个方法重构为一个泛型方法

本文关键字:一个 泛型方法 方法 重构 | 更新日期: 2023-09-27 17:59:51

我有以下场景,但有4个方法和调用,而不是2个。

RemoveNoLongerRequiredFromExceptions(newPeople, masterPeople);
RemoveNoLongerRequiredFromMaster(newPeople, exceptionPeople);
public void RemoveNoLongerRequiredFromMaster(List<NewPerson> newPeople,
  List<MasterPerson> masterPeople)
{
  var noLongerNewPeople = masterPeople.Where(a => !newPeople.Any(b => 
    a.PerId == b.PerId && a.AddressId == b.AddressId));
  foreach (var item in noLongerNewPeople )
  {                
    _masterRepository.DeleteMasterPerson(item.MvpId);
  }            
}
public void RemoveNoLongerRequiredFromExceptions(List<NewPerson> newPeople, 
  List<ExceptionPerson> exceptionPeople)
{
  var noLongerNewPeople = exceptionPeople.Where(a => !newPeople.Any(b => 
    a.PerId == b.PerId && a.AddressId == b.AddressId));
  foreach (var item in noLongerNewPeople )
  {
    _exceptionRepository.DeleteExceptionPerson(item.EvpId);                
  }
}

这些方法唯一不同的是第二个输入参数,然而这些类型中的每一个都具有所需的属性PerId&CCD_ 2。

当我知道所有4个版本都有我需要调用的属性和回购方法时,拥有4个版本的基本上是相同的方法,但有不同的模型/回购,这似乎很愚蠢。

我想我需要使用泛型来重构它,但我甚至不知道从哪里开始。

使用我包含的简单示例,我将如何将这4个方法重构为一个Generic方法?

将多个方法重构为一个泛型方法

我同意@aussin wernli:

public interface IPerson
{
  int AddressId { get; }
  int PerId { get; }
  int UniqueEntityId { get; }
}
public MasterPerson : IPerson {
  public int UniqueEntityId { get { return MvpId; } }
}
public ExceptionPerson : IPerson {
  public int UniqueEntityId  { get { return EvpId; } }
}

然而,IPerson子级和存储库之间的紧密耦合可能不是您想要的,所以您可以通过以下方式实现:

public void RemoveNoLongerRequired<T>(List<NewPerson> newPeople,
  List<T> masterPeople) where T : IPerson
{
  var noLongerNewPeople = masterPeople.Where(a => !newPeople.Any(b =>
    a.PerId == b.PerId && a.AddressId == b.AddressId));
  foreach (var item in noLongerNewPeople)
  {
    if (typeof(T) == typeof(MasterPerson))
    {
      _masterRepository.DeleteMasterPerson(item.UniqueEntityId);
      continue;
    }
    if (typeof(T) == typeof(ExceptionPerson))
    {
      _exceptionRepository.DeleteExceptionPerson(item.UniqueEntityId);
      continue;
    }
    throw new NotImplementedException();
  }
}

您可以制作一个接口IPerson,并在MasterPersonExceptionPerson中实现它。定义一个像DeletePerson(int id)这样的方法,这个问题就变得微不足道了。