Перегрузка конструкторов и повторное использование кода

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

Если вы хотите, чтобы этот новый метод мог выбирать, создавать ли Customer из полученной строки данных или создавать неинициализированного клиента, а затем начинать устанавливать его данные (например, ID) из имеющихся данных, я бы рекомендовал вместо этого использовать factory. другого конструктора. Быстрый скетч, похожий на псевдокод, будет выглядеть так:

Customer ProvideCustomer(int ID)
{
    Customer result; // or initialize to null to signal more work to come
    DataTable dt = DataAccess.GetCustomer(ID);
    if (dt.Rows.Count > 0)
    {
        result = new Customer( dt.getappropriaterow ) // however you choose one
    }
    else
    {
        result = new Customer();
        result.ID = ID;          // whatever other initialization you need
    }
    return result;
}

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

Измените конструктор, который принимает DataRows, чтобы взять DataTable и сначала вызвать конструктор по умолчанию:

Customer( DataTable dt ) : Customer()
{
    if ( dt != null && dt.Rows.Count > 0 )
    {
        // handle the row that was selected
    }
    else
    {
        throw Exception( "customer not in database" ); // or leave this line out to allow a default customer when they arent in the DB
    }
}

Затем измените конструктор идентификатора следующим образом:

Customer(int ID) : Customer(DataAccess.GetCustomer(ID))
{
    // no code
}

Итак, теперь ваш конструктор по умолчанию будет вызываться всегда, а затем, если клиент был найден в БД, вы можете перезаписать значения по умолчанию значениями из базы данных. Если клиента не было в базе данных, вы можете создать исключение или просто разрешить создание клиента только со значениями по умолчанию.

Меня беспокоит то, что вы не получаете не о конструкторах, а о принципе единой ответственности и свободном связывании.

Например, код, который вы показываете выше, означает:

• ваша модель домена содержит код доступа к данным
• вы не используете повторно какой-либо код из базового класса, который может быть введен с логикой доступа к данным в первую очередь
• Ваш объект домена знает о структурах данных, отличных от себя или своих членов, таких как DataTable или DataRow, что связывает его с этими структурами данных и затрудняет использование других структур данных.

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

Я предпочитаю, чтобы объект домена содержал только логику, связанную с хранением подлинных данных клиента и манипулированием ими, и ничего больше. Таким образом, мой конструктор для него будет:

class Customer
{
    public Customer(int id, string firstName, string LastName)
    {
        Id = id;
        FirstName = firstName;
        LastName = lastName;
    }

    public int Id { get; set; }
    public string FirstName { get; set; }
    public string LastName { get; set; }
}

Обновлено: Тем не менее, это основная причина, по которой некоторые люди предпочитают ORM, которые позволяют использовать POCO, например NHibernate: нет необходимости помещать туда логику загрузки данных.

Если бы это было сделано, например, в NHibernate, вам понадобился бы базовый класс DomainObject:

public class Customer : DomainObject

Что, в свою очередь, может быть использовано реализацией IRepository NHibernate:

public class Repository<T> : IRepository where T : DomainObject

Этот объект Repository будет содержать весь код, необходимый для операций CRUD.

Если вы хотите придерживаться ADO.NET, одним из возможных решений является создание объектов диспетчера DAL для всей загрузки:

public class CustomerManager
{
    public IList<Customer> LoadCustomers()
    {
        //load all customers here
        foreach (DataRow dr in dt.Table[0])
        {
             yield return new Customer((int) dr["Id"], dr["FirstName"].ToString(), dr["LastName"].ToString());
        }
    }

    public Customer LoadCustomerByID(int id)
    {
        //load one customer here
        return new Customer((int) dr["Id"], dr["FirstName"].ToString(), dr["LastName"].ToString());
    }
}

Конечно, здесь есть гораздо больше возможностей для дальнейшего продвижения повторного использования кода.

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

Так что вместо:

Customer c = new Customer(13, "George", "Bush");
Customer c2 = new Customer(12);
Customer c3 = new Customer(GetDataRow(11));

Ты получаешь:

Customer c = new Customer(13, "George", "Bush");
Customer c2 = Customer.LoadFromDatabaseId(12);
Customer c3 = Customer.MapFromDataRow(GetDataRow(11));

Тогда ваш класс клиента будет выглядеть так:

class Customer
{
    public Customer(int id, string firstName, string lastName)
    {
        //...
    }

    static public Customer MapFromDataRow(DataRow dr)
    {
        return new Customer(
            dr["ID"],
            dr["FirstName"],
            dr["LastName"]);
    }

    static public Customer LoadFromDatabaseId(int id)
    {
        DataTable dt = DataAccess.GetCustomer(ID);
        if (dt.Rows.Count > 0)    
        {
            return MapFromDataRow(dt.Rows[0]);
        }
        else    
        {        
            throw new CustomerNotFoundException(id);                
        } 
    }
}

Просто используйте синтаксис конструктора this

 public Customer(int ID): this(DataAccess.GetCustomer(ID).Rows[0]) {}

Но эта конструкция вызовет исключение, если вы передадите ей недопустимый идентификатор (его нет в базе данных).