Comecei a usar OO a pouco tempo e em todos projetos que eu vejo na net, eu vejo a seguinte maneira de declaração.
public class Trabalhador
{
public int Id { get; set; }
public string Nome { get; set; }
public virtual ICollection Dependente { get; set; }
public virtual ICollection Dependente { get; set; }
public virtual ICollection LicencaMedica{ get; set; }
}
public class Dependente
{
public int ID { get; set; }
public string Nome { get; set; }
public virtual Trabalhador Trabalhador { get; set; }
}
public class Ferias
{
public int ID { get; set; }
public string Nome { get; set; }
public virtual Trabalhador Trabalhador { get; set; }
}
public class LicencaMedica
{
public int ID { get; set; }
public string Nome { get; set; }
public virtual Trabalhador Trabalhador { get; set; }
}
Entretanto estou trabalhando em um projeto em conjunto com outros desenvolvedores, e um deles defende com unhas e dentes que as minhas classes não devem possuir o "virtual" como atributo de navegação, defende ele que as classes Dependente, Ferias e Licença pelo fato de ja terem uma ICollection em Trabalhador, nao devem possuir nenhuma chave, pois serão todas acessiveis a partir da classe trabalhador, está correto isso do ponto de vista da OO? Expus minha opiniao e falei que isso estava errado, pois estaria meio que criando uma classe Root e tudo passaria por ela sem necessidade, meio que sobrecarregando uma classe com muita responsabilidade, a abordagem dele seria a seguinte:
public class Trabalhador
{
public int Id { get; set; }
public string Nome { get; set; }
public virtual ICollection Dependente { get; set; }
public virtual ICollection Dependente { get; set; }
public virtual ICollection LicencaMedica{ get; set; }
}
public class Dependente
{
public int? ID { get; set; }
public string Nome { get; set; }
}
public class Ferias
{
public int? ID { get; set; }
public string Nome { get; set; }
}
public class LicencaMedica
{
public int? ID { get; set; }
public string Nome { get; set; }
}
Está correto isso do ponto de vista da OO? Pra mim ter acesso a um objeto da classe filha eu devo obrigatoriamente passar pela classe pai? Segundo ele o Ninject que acusa erro no sistema, pois o fato de instanciar a classe pai e depois instanciar a classe filha segundo ele daria um loop no sistema, por isso ele arranca todas as propriedade de navegação das classes filhas.
(Nao estamos usando Entity, estamos usando ADO com SP no Banco). Tentei ser o mais especifico possivel, qual o problema de cada uma das abordagens?
sealed
por default. stackoverflow.com/a/6389669/221800. herança deve ser usada em última caso, quando realmente você tem uma relação é-um, quando isto traz benefícios. Portantovirtual
não faz sentido se não precisar de polimorfismo. Acostume-se criar suas classes comselead
e só o retire se achar um bom motivo para isto.