Как инициализировать список с помощью конструктора?

Question

У меня есть тип:

public class Human 
{ 
    public int Id { get; set; } 
    public string Address { get; set; } 
    public string Name { get; set; } 
    public List<ContactNumber> ContactNumbers { get; set; } 

    public Human(int id) 
    { 
     Id = id; 
    } 

    public Human(int id, string address, string name, 
       List<ContactNumber> contactNumbers) : 
     this(id) 
    { 
     Address = address; 
     Name = name; 
     ContactNumbers = contactNumbers; 
    }   
} 

Пожалуйста, наставит меня является среди лучших практик использования конструктора с для инициализации списка? Как инициализировать список с помощью конструктора?

Edit:

Пожалуйста, наставит меня является одним из наиболее эффективных методов использования конструктора с для инициализации списка? Как назначить значения для списка с помощью конструктора, поэтому, если значение, принятое по умолчанию, будет использоваться?

Благодаря

Answer 1

Использование коллекции инициализатору

От C# 3, вы можете использовать коллекцию инициализаторами для построения списка и заполнить его с помощью одного выражения. Следующий пример создает человек и его ContactNumbers:

var human = new Human(1, "Address", "Name") { 
    ContactNumbers = new List<ContactNumber>() { 
     new ContactNumber(1), 
     new ContactNumber(2), 
     new ContactNumber(3) 
    } 
} 

Специализируя Human конструктор

Вы можете изменить конструктор Human класса, чтобы обеспечить возможность заселить ContactNumbers свойства:

public class Human 
{ 
    public Human(int id, string address, string name, IEnumerable<ContactNumber> contactNumbers) : this(id, address, name) 
    { 
     ContactNumbers = new List<ContactNumber>(contactNumbers); 
    } 

    public Human(int id, string address, string name, params ContactNumber[] contactNumbers) : this(id, address, name) 
    { 
     ContactNumbers = new List<ContactNumber>(contactNumbers); 
    } 
} 

// Using the first constructor: 
List<ContactNumber> numbers = List<ContactNumber>() { 
    new ContactNumber(1), 
    new ContactNumber(2), 
    new ContactNumber(3) 
}; 

var human = new Human(1, "Address", "Name", numbers); 

// Using the second constructor: 
var human = new Human(1, "Address", "Name", 
    new ContactNumber(1), 
    new ContactNumber(2), 
    new ContactNumber(3) 
); 

Итог

Какая альтернатива является лучшей практикой? Или, по крайней мере, хорошая практика? Вы это судите! ИМО, лучшая практика заключается в том, чтобы написать программу как можно яснее всем, кто должен ее прочитать. Использование инициализатора коллекции является для меня победителем, в данном случае. С гораздо меньшим количеством кода он может делать почти то же самое, что и альтернативы - по крайней мере, альтернативы, которые я дал ...

Answer 2

ContactNumbers = new List<ContactNumber>(); 

Если вы хотите, чтобы передать в, просто взять

public Human(List<ContactNumber> numbers) 
{ 
ContactNumbers = numbers; 
} 

Answer 3

Вы можете инициализировать его так же, как и любой список:

public List<ContactNumber> ContactNumbers { get; set; } 

public Human(int id) 
{ 
    Id = id; 
    ContactNumbers = new List<ContactNumber>(); 
} 

public Human(int id, string address, string name) :this(id) 
{ 
    Address = address; 
    Name = name; 
    // no need to initialize the list here since you're 
    // already calling the single parameter constructor 
}  

Однако я даже сделают шаг дальше и сделают сеттер приватным, так как вам часто не нужно устанавливать список, но просто ac налог/изменить его содержимое:

public List<ContactNumber> ContactNumbers { get; private set; } 

Answer 4

В общем, не подвергать List<T> публично, и не обеспечивают сеттеры для свойств коллекции. Кроме того, вы можете скопировать элементы переданного списка (как показано ниже). В противном случае изменения в исходном списке будут влиять на экземпляр Human.

public class Human 
{ 
    public Human() 
    { 
    } 

    public Human(IEnumerable<ContactNumber> contactNumbers) 
    { 
     if (contactNumbers == null) 
     { 
      throw new ArgumentNullException("contactNumbers"); 
     } 

     _contactNumbers.AddRange(contactNumbers); 
    } 

    public IEnumerable<ContactNumber> ContactNumbers 
    { 
     get { return _contactNumbers; } 
    } 

    private readonly List<ContactNumber> _contactNumbers = new List<ContactNumber>(); 
} 

Другой вариант - использовать конструктор списка, который берет коллекцию и удаляет инициализатор поля.

Answer 5

Вы ищете это?

ContactNumbers = new List<ContactNumber>(){ new ContactNumber("555-5555"), 
              new ContactNumber("555-1234"), 
              new ContactNumber("555-5678") };