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));
}
}
}