Избегайте копирования и вставки почти общего кода в подклассы

Question

В настоящее время у меня есть класс, который инкапсулирует список типизированных объектов и реализует некоторые интерфейсы, такие как IEnumerable. Мне нужен второй класс с дополнительными, немного разными свойствами. Поэтому я буду создавать базовый класс A и получить новый класс B и новый класс C от A.

Однако у меня есть код (например, как код для .Find с использованием делегатов), который почти такой же, как в B и C. Единственное отличие состоит в том, что код в B поиска в личном списке типизированных объектов (скажем, лицо) и код в C поиск приватного списка различных объектов (скажем, животные):

private static bool Find(Person xx) 
    { 
     if (xx.Id == someID) 
     { 
      return true; 
     } 
    else 
     { 
      return false; 
     } 
    } 

Как я хочу, чтобы избежать копирование/вставка, мой вопрос: что такое хорошая стратегия, чтобы избежать чего-то подобного?

Единственный способ, которым я представлял, состоял в том, чтобы объявить список общих объектов в A и указать методы в A на это. В B и C у меня тогда не было бы никакого кода, но я теряю все преимущества типизированного списка.

Answer 1

1) я хотел бы видеть, если я мог бы использовать дженерики при кодировании класса А

public class cA<T> 
{ 
    private IEnumerable<T> _myPrivateData 

    public bool Find(args) 
    { 
     // Do stuff with _myPrivateData 
    } 
} 

public class cB : cA<TheTypeIWant> 
{ 
    // more stuff here if needed 
} 

2) Вы можете использовать свойство переопределения функции + использование модификатора protected доступа сделать следующее:

public class cA 
{ 
    protected IEnumerable<Object> MyData { get; set; } 

    public bool Find(args) 
    { 
     // Do stuff with MyData 
    } 
} 

public class cB : cA 
{ 
    protected new IEnumerable<MyOtherDataType> MyData { get; set; } 
} 

Конечно, MyOtherDataType должен наследовать от базового типа, используемого для этого подхода. Однако я бы не рекомендовал этот подход. Вариант 1 намного лучше и чище.

Вот несколько ссылок, которые могут оказаться полезными:

• http://peisker.net/dotnet/covariance.htm
• C#: Overriding return types

Answer 2

Я был бы соблазн иметь интерфейс, как ICanBeFoundById

public interface ICanBeFoundById 
{ 
    int Id {get;} 
} 

затем Человек и животное может и наследовать имущество, что и вы можете сделать что-то вроде

public static class FindExtensions 
{ 
    public static bool Find(this IEnumerable<ICanBeFoundById> xs, int someID) 
    { 
    return xs.Any(x=>x.Id == someID) 
    } 
} 

Внимание: я не имею даже проверил, если это скомпилируется :)

Конечно, вы могли бы найти

public abstract class BaseRepository<ICanBeFoundById> 
{ 
    private IEnumerable<ICanBeFoundById> _xs; 

    public static bool Find(int someID) 
    { 
    return xs.Any(x=>x.Id == someID) 
    } 
} 

если вы не хотите иметь статическое расширение.