Skip to main content
adicionou 842 caracteres ao conteúdo
Fonte Link

Com poucos minutos foi possível abstrair praticamente toda a duplicidade de seu código. A partir daqui, sugiro que leia as sugestões passadas acima e dê continuidade a refatoração. Vale lembrar apenas sobre um pequeno detalhe: over engeneering. É fácil "se perder" nas refatorações e querer abraçar o mundo utilizando todas as boas praticas, design patterns e princípios de programação possíveis (e vejo isso acontecer dia após dia). Pondere apenas tais refatorações/abstrações com o custo a ser implementado e também complexidade final do código. Afinal, de nada adianta um código perfeito se o produto atrasou tanto a ser desenvolvido que não atende mais, ou mesmo, um código tão cheio de patterns que outros desenvolvedores menos experientes (que são os que irão manter o código), demoram 1 dia apenas para entender o que acontece...

Com poucos minutos foi possível abstrair praticamente toda a duplicidade de seu código. A partir daqui, sugiro que leia as sugestões passadas acima e dê continuidade a refatoração. Vale lembrar apenas sobre um pequeno detalhe: over engeneering. É fácil "se perder" nas refatorações e querer abraçar o mundo utilizando todas as boas praticas, design patterns e princípios de programação possíveis (e vejo isso acontecer dia após dia). Pondere apenas tais refatorações/abstrações com o custo a ser implementado e também complexidade final do código. Afinal, de nada adianta um código perfeito se o produto atrasou tanto a ser desenvolvido que não atende mais, ou mesmo, um código tão cheio de patterns que outros desenvolvedores menos experientes (que são os que irão manter o código), demoram 1 dia apenas para entender o que acontece...

Fonte Link

Sim, existem diversas formas de evitar a repetição de código, e você está começando bem em criticar tal repetição. Um princípio básico em programação é o DRY (don't repeat yourself) e prega justamente isso. Categoricamente, se você estiver repetindo 2 vezes o mesmo código, SEMPRE haverá uma forma de abstrair tal lógica e reaproveitar. Repetição de código é realmente um dos piores problemas quando se fala em manutenção de código. No entanto, apenas pondere entre adicionar complexidade extrema a um código apenas para evitar repetição de duas ou três linhas de código (mais que isso, provavelmente é melhor abstrair).

Em seu código existem diversas refatorações a serem feitas, entre as básicas estão:

  • Código bilingue (evite usar portugues e inglês, seja coerente).
  • C# Coding standards: Métodos são pascal case, membros (de classe) começam com undescore, etc
  • "Implicitly Typed Local Variables": a utilização do "var" força implicitamente a boa nomenclatura de membros, que dirige para um código mais limpo e legível (inclusive sem repetição, veja em seu código, várias vezes a variável possui o mesmo nome da classe, que fica desnecessariamente repetitivo na mesma linha).

Outras refatorações são um pouco mais complexas, e demandam um conhecimento maior sobre orientação a objeto, boas práticas e padrões, sugiro que dê uma olhada em cada uma delas:

  • SRP - Single responsability principle: Uma classe deve possuir apenas uma responsabilidade
  • ORM: Em .Net o framework para ORM de lonoge mais utilizado é o Entity Framework e com sua utilização você evita de ter queries hardcoded em suas classes, utilizar recursos como o migrations e também deixar seu código testável.
  • Injeção de dependência: A utilização de DI (dependency injection) permite a criação de testes unitários (através de Mocks) e desacoplam as camadas de seu projeto.

Por fim, outra boa prática é o desenvolvimento através de "baby-steps", ou seja, pequenos passos, evitando grandes refatorações o que podem causar problemas na identificação de bugs introduzidos pela mesma. A combinação da utilização de testes com esses "baby-steps" é recomendada (crie um teste, refatore e depois veja se o teste ainda passa).

Como já identificou, o primeiro passo é realmente a remoção da duplicidade de todo esse código. Vale lembrar mais uma vez que, se precisou repetir, mesmo que apenas uma vez, é possível abstrair e reaproveitar.

O exemplo a seguir é apenas uma sugestão inicial para refatoração, você pode, com o tempo, usar principalmente um ORM e injeção de dependência para conseguir testar unitariamente seu código, e ter o mesmo livre de códigos SQL "hardcoded". Em seu caso, utilizando herança, uma refatoração básica seria a seguinte (código não testado, apenas para demonstração de como fazer as refatorações):

public abstract class BaseReport
{
    private RdlViewer _reportViewer;
    private ViewerToolstrip _reportStrip;
    private String _nomeArquivo;
    private String _query;
    private List<string> _erros;
    private DALConexao _conexao;

    public BaseReport(DALConexao conexao, string query, string nomeArquivo) 
    {            
        _conexao = conexao;
        _erros = new List<string>();
        _reportViewer = new RdlViewer();
        _reportStrip = new ViewerToolstrip(_reportViewer);
        _reportStrip.Viewer = _reportViewer;
        _query = query;
        _nomeArquivo = nomeArquivo;
        Emitir();
    }
    
    protected abstract AdicionarParametros( SQLiteCommand cmd );

    private void Emitir() 
    {
        PathRelatorio caminhoArquivo = new PathRelatorio(_nomeArquivo);

        if (caminhoArquivo.existeArquivo())
        {
            try
            {
                _reportViewer.SourceFile = new Uri(caminhoArquivo.Path);
                _reportViewer.Parameters += string.Format("conexao={0}", _conexao.Conexao.ConnectionString);
                _reportViewer.Report.DataSets["Data"].SetData(ObterTabela());
                _reportViewer.Rebuild();
                _reportViewer.HideRunButton();
                _reportViewer.Dock = System.Windows.Forms.DockStyle.Fill;
            }
            catch (Exception ex)
            {
                _erros.Add(string.Format("Erro: {0}", ex.Message));
            }
        }
    }

    public RdlViewer ReportViewer
    {
        get
        {
            return _reportViewer;
        }
    }

    public ViewerToolstrip ReportStrip
    {
        get
        {
            return _reportStrip;
        }
    }

    public List<string> Erros
    {
        get
        {
            return _erros;
        }
    }


    

    private DataTable ObterTabela()
    {
        DataTable tabela = new DataTable();

        try
        {
            using (SQLiteConnection conexao = _conexao.Conexao)
            {
                using (SQLiteCommand command = conexao.CreateCommand())
                {
                    command.CommandType = CommandType.Text;
                    command.CommandText = _query.ToString();

                    adicionarParametros(command);

                    conexao.Open();

                    using (SQLiteDataAdapter dataAdapter = new SQLiteDataAdapter(command))
                    {
                        dataAdapter.Fill(tabela);

                        conexao.Close();
                    }
                }
            }
        }
        catch (Exception ex)
        {
            _erros.Add("Erro:" + ex.Message);

            return null;
        }

        return tabela;
    }
}

Com pouquíssimas alterações, foi extraída uma "classe base" com todas as funcionalidades e código que antes eram duplicados.

Ambas as classes ficariam assim:

public class RelAlunoAtivoOuInativo : BaseReport
{
    public RelAlunoAtivoOuInativo(DALConexao conexao) : base( conexao,  "SELECT nome, data_cadastro, telefone, celular, endereco, email, idade FROM Alunos ", "relatorioGeralAlunoAtivoInativo.rdl" ) 
    { }
    
    protected AdicionarParametros( SQLiteCommand cmd ){
        cmd.Parameters.Add("ativo", DbType.Int32).Value = this.ativoOuInativo ? 1 : 0;
    }
}

public class RelGeralAluno: BaseReport
{
    public RelGeralAluno(DALConexao conexao) : base( conexao,  "SELECT nome, data_cadastro, telefone, celular, endereco, email, idade FROM Alunos , "relatorioGeralAlunos.rdl" ) 
    { }
    
    protected AdicionarParametros( SQLiteCommand cmd ){ }
}

Uma sugestão para uma primeira refatoração no método é a extração da instanciação de suas classes de reports (utilização de uma factory simples).

private void FormRunRelatorio_Load(object sender, EventArgs e)
{
    var erros = new List<string>();
    var reportFactory = new SelectedReportFactory();
    var reportInstance = reportFactory.CreateReportFor(this.relatorioSelecionado);

    Controls.Add(reportInstance.ReportStrip);
    Controls.Add(reportInstance.ReportViewer);

    if (reportInstance.Erros.Count > 0)
    {
        foreach (var erro in reportInstance.Erros)
            MessageBox.Show(erro, "Erro", MessageBoxButtons.OK, MessageBoxIcon.Error);

        this.Close();
    }
}

public class SelectedReportFactory{
    public BaseReport CreateReportFor(string name){
        switch (name) 
        {
            case "GeralAluno":
                return new RelGeralAluno(new DALConexao(new DadosConexao().String_Conexao));

            case "GeralAlunoInativoOuAtivo":
                return new RelAlunoAtivoOuInativo(new DALConexao(new DadosConexao().String_Conexao), false);

            default: throw new Exception(string.Format("Invalid report name [{0}]", name)); 
        }
    }
}