Class Constructor и Get Design

Бих препоръчал да отделите достъпа до вашата база данни от класовете на вашия модел на домейн като Car.

Това може да стане чрез поставяне на всички операции за извличане на достъп до базата данни в техен собствен набор от класове. Те се наричат ​​хранилища. Има различни подходи за създаване на тези хранилища. Зависи от това как вашите данни се съхраняват в базата данни и дали използвате рамка на обекта. Някои предпочитат общи хранилища, докато други изобщо не харесват този общ подход. Но най-важното е, че хранилищата се грижат за достъпа до базата данни за извличане на обекти и за съхранение. Това не трябва да се извършва от обекта на домейна, т.е. самия Car, защото той трябва да има само операции, които принадлежат на 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

Най-важното е да използвате хранилището за достъп до вашата база данни. Използва се за зареждане и съхраняване на данни от базата данни и осигурява допълнителен слой за абстракция. Ако е необходимо, можете да създадете интерфейс за всяко хранилище, което ще ви позволи да правите и модулно тестване.

Много по-добре от създаването на методи във вашите класове, които не принадлежат там.

Както казах в коментара си, бих направил това, за да управлявам колите и да позволя зареждането на кола по 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 }; }
}

Едно просто решение е да използвате статичен метод, като така:

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

Тогава можете да го наречете така:

Car myCar = Car.GetById(18); 

За да отговоря на коментарите относно разделянето на загрижеността, бих предложил следното:

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");

Това запазва кода за създаване на автомобилен обект отделен от самия автомобилен обект и ви позволява да промените начина на създаване на автомобил, без изобщо да докосвате класа на автомобила.