Skip to main content
tags editadas
Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil
Tweeted twitter.com/#!/StackOverflowPT/status/576909415925878784
Post restaurado por SomeDeveloper
Post removido por SomeDeveloper
Fonte Link
SomeDeveloper
  • 18,8mil
  • 9
  • 77
  • 135

Essa é uma prática comum em orientação a objetos?

Em orientação a objetos uma forma de garantir o encapsulamento é manter os atributos das classes privados e modificar seu estado via métodos. Além disso, para garantir a integridade dos objetos, é comum exigirmos que dados essenciais para a existência do objeto sejam passados no construtor.

Em classes com poucos atributos isso funciona bem. Acontece que em classes com um número muito grande de atributos, começa a ficar meio estranho. Um exemplo disso é essa classe PessoaJuridica (que eu reconheço não estar muito boa):

public class PessoaJuridica
{
    public string NomeFantasia { get; set; }

    public string RazaoSocial { get; set; }

    public string Endereco { get; set; }

    public string Bairro { get; set; }

    public string Cidade { get; set; }

    public string Estado { get; set; }

    public int CEP { get; set; }

    public int Telefone1 { get; set; }

    public int Telefone2 { get; set; }

    public int Fax { get; set; }

    public int CNPJ { get; set; }

    public int InscricaoEstadual { get; set; }

    public int InscricaoMunicipal { get; set; }

    public DateTime DataConstituicao { get; set; }

    public DateTime DataCadastro { get; set; }
    
    public string Observacoes { get; set; }
}

A classe tem diversos atributos. Passar tudo isso no construtor é inviavel, e mesmo que escolhesse um subconjunto para passar ainda ficaria ruim. Além disso construir um método para cada atributo não teria muita vantagem em relação ao que já está ai. Por isso pensei na seguinte melhoria:

Olhei na classe algumas coisas que fazem sentido agrupadas. Por exemplo:

  • Endereço, Bairro, Cidade, Estado, CEP formam coletivamente o Endereço da empresa
  • Telefone1, Telefone2 e Fax são tipos de telefones
  • CNPJ, InscricaoEstadual, InscricaoMunicipal e DataConstituicao formam uma documentação da empresa

Por isso codifiquei uma struct Endereco com propriedades "Rua, Numero, Bairro, Cidade, Estado, CEP", uma struct Telefone com propriedades "DDD, Numero, TipoTelefone" sendo "TipoTelefone" um enum que pode ser "Fixo" ou "Fax". Finalmente codifiquei uma struct DocumentacaoEmpresa com atributos "RazaoSocial, CNPJ, InscricaoEstadual, InscricaoMunicipal e DataConstituicao". Com isso fiquei com a classe

public class PessoaJuridica
{
    public string NomeFantasia { get; private set; }

    public Endereco Endereco { get; private set; }

    public IList<Telefone> Telefones { get; private set; }

    public DocumentacaoEmpresa Documentacao { get; private set; }
    
    public string Observacoes { get; set; }
}

E aí fica mais fácil usar métodos e o construtor para lidar com essa classe. O que eu fiz foi pegar uma classe com vários atributos, agrupar esses atributos em conceitos que fazem sentido e codificar esses conceitos. Além disso, agora tem uma lista de telefones para obedecer o Open/Closed principle.

O que eu gostaria de saber é: essa é uma prática comum na orientação a objetos? A construção dessas structs não foi feita por causa de serem conceitos do domínio, mas sim para melhorar a escrita de uma classe. Existe algum design pattern, ou algo do gênero sobre isso?

Eu sei que não é necessário um design pattern para algo ser válido, mas queria saber se isso é comum e se existe mais informação sobre isso para saber se em geral é uma boa ideia fazer esse tipo de coisa.