рефакторинга несколько методов в один универсальный метод

Question

я следующий сценарий, но с 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 & AddressId.

Кажется, что у daft есть 4 версии того, что по сути является одним и тем же методом, но с разными моделями/repos, когда я знаю, что все 4 имеют свойства и методы репо, которые мне нужно вызвать.

Я думаю, мне нужно реорганизовать это с помощью дженериков, но я даже не знаю с чего начать.

Используя простой пример, который я включил, как бы преобразовать 4 метода в один общий метод?

Answer 1

Я согласен с @ausin 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(); 
    } 
} 

Answer 2

Вы можете сделать интерфейс IPerson и реализовать его как в MasterPerson, так и в ExceptionPerson. Определите метод, такой как DeletePerson(int id), и эта проблема становится тривиальной.