0

Здравствуйте, уважаемые! У меня возник следующий вопрос. Допустим я имею некий класс такого вида

public class Basis 
{
    public int Id {get; set; }

    public string Name {get; set; }
}

и несколько наследников от него, которые представляют собой записи из таблиц некой бд (доступ к бд через ORM):

public class First : Basis    
{
    public static IEnumerable<First> GetAll()
    {
        var data = new MyDataContext(); 
        return from item in data.FirstDataSet
               select new First { Id = item.Id, Name = item.Name;
    }
}

public class Second : Basis    
{
    public static IEnumerable<Second> GetAll()
    {
        var data = new MyDataContext(); 
        return from item in data.SecondDataSet
               select new Second { Id = item.Id, Name = item.Name;
    }
}

Как можно увидеть оба класса почти одинаковы. Каждый имеет статический метод для получения всех элементов данного типа из базы, и отличаются методы только типами возвращаемого значения и таблицами (FirstDataSet и SecondDataSet) к которым происходит обращение. Мне кажется что всю эту функциональность можно вынести в базовый класс и каким-то образом все это сделать короче, избавившись от дублирования кода, но не могу сообразить как. Может у кого-то есть какие нибудь идеи? Заранее спасибо

5
  • А кто будет знать в случае общей реализации объект какого класса-потомка создавать? Не лучше ли все таки оставить реализацию на плечах потомков?
    – Freezze
    21 ноя 2013 в 12:23
  • 1
    Лучше разделить логику доступа к данным и сущности модели. Лучше НЕ делать методы доступа к данным статическими. Если ваш контекст поддержует типизированные коллекции, то можно легко провернуть вашу идею примерно таким методом в базовом классе: IEnumerable<T> GetAll
    – wind
    21 ноя 2013 в 13:20
  • >Лучше разделить логику доступа к данным и сущности модели зачем? >Лучше НЕ делать методы доступа к данным статическими почему?
    – JuniorTwo
    22 ноя 2013 в 9:27
  • @JuniorTwo: По той же причине, по которой не стоит всю логику приложения делать в OnClick: чем меньше ответственностей у класса, тем лучше. Сущности модели должны быть простыми, не содержать приватных данных, быть по существу не более чем состоянием. (Иначе вы не сможете отобразить их на таблицу.) А логика доступа к данным — штука более умная, и должна уметь изменяться не затрагивая базу данных.
    – VladD
    29 ноя 2013 в 21:41
  • @JuniorTwo: Не используйте наследование для того, чтобы уменьшить повторное использование кода, оно для этого не предназначено. Просто воспользуйтесь сторонним хелпером: pastebin.com/8eWP1ma3
    – VladD
    29 ноя 2013 в 22:08

1 ответ 1

1

Можно. Делаешь базовый класс абстрактым с generic-параметром, ограниченным интерфейсом. Делаешь метод GetAll, тороый в select передаёт абстрактный метод GetItem. Наследники реализуют GetItem. FirstDataSet и SecondDataSet реализуют нужный интерфейс. Вероятно, интерфейс тоже generic.

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.