Конструктор классов и получение дизайна

Question

При проектировании класса и конструкторов у меня возникает вопрос о получении записи.

В качестве примера, если у меня есть автомобиль класса

public Car 
{ 
    ... 
} 

У меня есть возможность создать конструктор, который мог бы получить определенный экземпляр автомобиля, например:

public Car(int carID) 
{ 
    //Get car from database 
} 

Или я мог бы напишите открытый метод, чтобы получить автомобиль:

public Car GetCarByID(int carID) 
{ 
    //Get car from database 
} 

Есть ли предпочтительный (наилучший способ) метод для получения экземпляра класса? Код, кажется, читает уборщика, чтобы сделать получить экземпляр объекта:

Car MyCar = new Car(5); 

против

Car MyCar = new Car(); 
MyCar.GetCarByID(5); 

Answer 1

Я бы рекомендовал отделить доступ к базе данных от классов модели домена, таких как Car.

Это можно сделать, поместив все операции поиска доступа к базе данных в свой собственный набор классов. Они называются репозиториями. Существуют разные подходы к созданию этих репозиториев. Это зависит от того, как ваши данные хранятся в базе данных и использует ли вы инфраструктуру сущности. Некоторые предпочитают общие хранилища, тогда как другим вообще не нравится этот общий подход. Но суть в том, что репозитории заботятся о доступе к базе данных для поиска объектов и их хранения. Это не должно делаться объектом домена, то есть самим автомобилем, потому что у него должны быть только операции, принадлежащие автомобилю, и что делает его объектом Car. Хранение автомобиля в базе данных не означает, что автомобиль должен знать, как это сделать.

Пример использования шаблона хранилища можно найти здесь: http://www.codeproject.com/Articles/688929/Repository-Pattern-and-Unit-of

Существует также хорошая статья на MSDN о хранилище шаблона: https://msdn.microsoft.com/en-us/library/ff649690.aspx?f=255&MSPPError=-2147217396

Суть заключается в том, чтобы использовать хранилище для ваших доступ к базе данных. Он используется для загрузки и хранения данных из базы данных и обеспечивает дополнительный уровень абстракции. При необходимости вы можете создать интерфейс для каждого репозитория, который позволит вам выполнять модульное тестирование.

Намного лучше, чем создавать методы в ваших классах, которые там не принадлежат.

Answer 2

Одно простое решение состоит в использовании static method, например, так:

public static Car GetById(int carId) 
{ 
    ... Load car and return it ... 
} 

Затем вы можете назвать это следующим образом:

Car myCar = Car.GetById(18); 

Answer 3

Для того, чтобы отреагировать на замечания по поводу разделения проблем, я хотел бы предложить следующее:

public class Car 
{ 
    public string Make {get; set;} 
    public string Model {get; set;} 
    // additional properties here. 
} 

public static class CarFactory 
{ 
    public static Car CreateCar(string modelId) 
    { 
     Car car = new Car(); 
     // Load the car via whatever mechanism you wish; 
     // e.g., web service, database, etc. 
     return car; 
} 

Car fordFairlane = CarFactory.CreateCar("57fordFairlane"); 

Это сохраняет код для создания объекта автомобиля отдельно от самого объекта автомобиля, и позволяет изменять как создается автомобиль, не касаясь класса автомобиля.

Answer 4

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

public class Car 
{ 
    public int Id { get; set; } 
    public Car() { } 
    public Car(int id) { Id = id; } 
} 

public class CarCollection 
{ 
    public List<Car> Cars { get; set; } 

    public void AddCar(Car car) { Cars.Add(car); } 

    public Car GetById(int id) { return Cars.Single(x => x.Id == id); } 

    public CarCollection(List<Car> cars) { Cars = cars; } 
    public CarCollection(Car car) { Cars = new List<Car>() { car }; } 
}