Estou revisando algumas abordagens de programação e cai em um dilema com relação a onde devemos instanciar nossas propriedades de navegação do EF
ICollection
.
Muitos casos vejo que o pessoal instancia elas no construtor, isso seria uma boa pratica?
Em poucos casos vejo exemplos que instanciam somente quando realmente necessário, o que ao meu ver seria uma abordagem mais correta.
Então, tendo o exemplo a entidade Pais
que tem uma lista de Estados
Entidade Pais
[Table("Paises")]
[DisplayColumn("Nome")]
public class Pais : Entidade
{
public Pais()
{
Estados = new HashSet<Estado>();
}
[Key]
public Guid PaisId { get; set; }
[Required]
[StringLength(200)]
public string Nome { get; set; }
[Required]
[StringLength(3)]
public string Sigla { get; set; }
public ICollection<Estado> Estados { get; set; }
}
Entidade estado
[Table("Estados")]
[DisplayColumn("Nome")]
public class Estado : Entidade
{
[Key]
public Guid EstadoId { get; set; }
public Guid PaisId { get; set; }
[Required]
[StringLength(200)]
public String Nome { get; set; }
[Required]
[StringLength(2)]
public String Sigla { get; set; }
public virtual Pais Pais { get; set; }
public ICollection<Cidade> Cidades { get; set; }
}
Na primeira abordagem vejo no construtor de pais a criação de um HashSet
public Pais()
{
Estados = new HashSet<Estado>();
}
Em outra abordagem vejo somente na hora do uso, como mostra o exemplo a seguir
pais = new Pais();
if (pais.Estados == null)
pais.Estados = new List<Estado>();
pais.Estados.Add(new Estado() { Sigla = "RS", Nome = "Rio Grande do Sul" });
Sendo que nesse segundo caso, se a criação do ICollection<Estado>
estivesse na entidade Pais
, não seria necessário o if
para verificar se o objecto Estados
é nulo, bastando somente adicionar mais itens na coleção.