6

Tenho um projeto que estou desenvolvendo em camadas, até agora tenho as seguintes: Acesso a dados, apresentação, negócios e objetos de transferência. A dúvida é a seguinte:

Exemplo: Tenho um form para cadastrar uma pessoa jurídica no banco de dados. Para salvar os dados que o usuário inseriu no form seria correto neste form que o usuário inseriu os dados eu criar um objeto pessoaJuridica e passar este objeto para uma classe na camada de negócios com o nome de SalvaPessoaJuridica.cs por exemplo, esta por sua vez acessa a camada de acesso a dados e efetua a gravação no banco?

Para cada form eu teria que criar uma classe para manter os dados ou eu já usaria a camada de acesso a dados diretamente no form e já manteria os dados sem criar uma classe específica para cada tipo de operação?

Exemplo:

    private void button1_Click(object sender, EventArgs e)
    {
        if (txtNome.Text == string.Empty || txtIdade.Text == string.Empty || txtEmail.Text == string.Empty)
        {
            MessageBox.Show("Informe os valores para nome, idade e email do aluno.");
            return;
        }
        else
        {
            try
            {
                string sqlInsert = "Insert into Alunos (Nome,Idade,Email) values (@Nome,@Idade,@Email)";
                string[] campos = { "@Nome", "@Idade", "@Email" };
                string[] valores = { txtNome.Text, txtIdade.Text, txtEmail.Text };
                accDB.Salvar(campos, valores, sqlInsert);
                gdvDados.DataSource = accDB.getRegistro(sqlSelect);
            }
            catch (Exception ex)
            {
                MessageBox.Show("Erro " + ex.Message);
            }
        }
     }

Ou:

private void button1_Click(object sender, EventArgs e)
    {
             try
            {
                Alunos aluno = new Alunos();
                aluno.Nome = "Teste";
                aluno.Idade = 22;
                aluno.Email = "[email protected]" 

                SalvaAluno salva = new SalvaAluno();
                salva.Manter(aluno);
            }
            catch (Exception ex)
            {
                MessageBox.Show("Erro " + ex.Message);
            }
        }
     }
2
  • A segunda opção facilita a manutenção do sistema. Qualquer novo integrante de sua equipe conseguirá entender facilmente o que é feito neste código. Observação: Cuidado ao exibir mensagens de exceção para o usuário final e muito cuidado principalmente ao concatenar strings para montar uma query. Você pode abrir uma brecha de segurança em seu sistema. 26/10/2016 às 14:35
  • Realcionado. Esta pergunta e as perguntas vinculadas podem dar uma visao melhor do que fazer 28/10/2016 às 18:24

1 Resposta 1

4
+50

O seu segundo exemplo é o que chegou mais próximo das melhores práticas, pois ele está aplicando o conceito de separação de responsabilidades.

Esse segundo código é algo bem parecido com que os conceitos MVC ou MVVM fariam por exemplo.

No primeiro exemplo a camada de apresentação está sendo responsável por lógica de inserção no banco dados, como criar a instrução SQL e salvar. A responsabilidade do form é coletar e exibir dados.

Essa sua classe SalvaAluno seria uma espécie de repositório se estivesse usando o Repository Pattern, pois ela apenas adiciona entidades do tipo aluno.

Para seu código atual ficar melhor ainda em vez de utilizar a classe SalvaAluno diretamente, você poderia utilizar uma interface ISalvaAlunopara aplicar conceitos de injeção de dependência e inversão de controle. Dessa forma você não se preocuparia com o tipo de implementação da sua camada DAL, essa implementação poderia ser tanto com Entity Framework, Dapper etc

Para tudo isso citado há muito conteúdo disponível nesse site e na web.

Respondendo sua pergunta:

Para cada form eu teria que criar uma classe para manter os dados ou eu já usaria a camada de acesso a dados diretamente no form e já manteria os dados sem criar uma classe específica para cada tipo de operação?

Você teria que criar uma classe para persistir dados para cada form(ou melhor para cada entidade) para que haja separação de responsabilidades.

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .