Para emissão de relatórios da minha aplicação eu criei várias classes, e cada classe é responsável por um relatório, entretanto, existe repetição de muito código por parte dessas classes que eu gostaria de evitar, reescrevendo o código de uma forma que aproveite mais os recursos de orientação a objeto da linguagem C#.
Vou apresentar duas classes para ilustrar a situação, e todas as outras terão a estrutura parecida com poucos atributos que varriam.
Classe RelAlunoAtivoOuInativo
:
using fyiReporting.RDL;
using fyiReporting.RdlViewer;
using System.Data;
using System.Data.SQLite;
using DAL;
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
namespace BLL
{
public class RelAlunoAtivoOuInativo
{
private RdlViewer reportViewer;
private ViewerToolstrip reportStrip;
private String nomeArquivo;
private StringBuilder query;
private List<string> erros;
private DALConexao conexao;
private Boolean ativoOuInativo;
public RelAlunoAtivoOuInativo(DALConexao conexao, bool ativoOuInativo)
{
this.conexao = conexao;
this.erros = new List<string>();
this.reportViewer = new RdlViewer();
this.reportStrip = new ViewerToolstrip(this.reportViewer);
this.reportStrip.Viewer = this.reportViewer;
this.query = new StringBuilder();
this.query.Append("SELECT id_aluno, nome, telefone, informatica, participa_encontro FROM Alunos WHERE ativo = ?");
this.nomeArquivo = "relatorioGeralAlunoAtivoInativo.rdl";
this.ativoOuInativo = ativoOuInativo;
emitir();
}
public void emitir()
{
PathRelatorio caminhoArquivo = new PathRelatorio(this.nomeArquivo);
if (caminhoArquivo.existeArquivo())
{
try
{
this.reportViewer.SourceFile = new Uri(caminhoArquivo.Path);
this.reportViewer.Parameters += string.Format("conexao={0}", this.conexao.Conexao.ConnectionString);
this.reportViewer.Report.DataSets["Data"].SetData(obterTabela());
this.reportViewer.Rebuild();
this.reportViewer.HideRunButton();
this.reportViewer.Dock = System.Windows.Forms.DockStyle.Fill;
}
catch (Exception ex)
{
this.erros.Add(string.Format("Erro: {0}", ex.Message));
}
}
}
public RdlViewer ReportViewer
{
get
{
return this.reportViewer;
}
}
public ViewerToolstrip ReportStrip
{
get
{
return this.reportStrip;
}
}
public List<string> Erros
{
get
{
return this.erros;
}
}
private DataTable obterTabela()
{
DataTable tabela = new DataTable();
try
{
using (SQLiteConnection conexao = this.conexao.Conexao)
{
using (SQLiteCommand command = conexao.CreateCommand())
{
command.CommandType = CommandType.Text;
command.CommandText = this.query.ToString();
command.Parameters.Add("ativo", DbType.Int32).Value = this.ativoOuInativo ? 1 : 0;
conexao.Open();
using (SQLiteDataAdapter dataAdapter = new SQLiteDataAdapter(command))
{
dataAdapter.Fill(tabela);
conexao.Close();
}
}
}
}
catch (Exception ex)
{
this.erros.Add("Erro:" + ex.Message);
return null;
}
return tabela;
}
}
}
Classe RelGeralAluno
:
using fyiReporting.RDL;
using fyiReporting.RdlViewer;
using System.Data;
using System.Data.SQLite;
using DAL;
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
namespace BLL
{
public class RelGeralAluno
{
private RdlViewer reportViewer;
private ViewerToolstrip reportStrip;
private String nomeArquivo;
private StringBuilder query;
private List<string> erros;
private DALConexao conexao;
public RelGeralAluno(DALConexao conexao)
{
this.conexao = conexao;
this.erros = new List<string>();
this.reportViewer = new RdlViewer();
this.reportStrip = new ViewerToolstrip(this.reportViewer);
this.reportStrip.Viewer = this.reportViewer;
this.query = new StringBuilder();
this.query.Append("SELECT nome, data_cadastro, telefone, celular, endereco, email, idade FROM Alunos ");
this.nomeArquivo = "relatorioGeralAlunos.rdl";
emitir();
}
public void emitir()
{
PathRelatorio caminhoArquivo = new PathRelatorio(this.nomeArquivo);
if (caminhoArquivo.existeArquivo())
{
try
{
this.reportViewer.SourceFile = new Uri(caminhoArquivo.Path);
this.reportViewer.Parameters += string.Format("conexao={0}", this.conexao.Conexao.ConnectionString);
this.reportViewer.Report.DataSets["Data"].SetData(obterTabela());
this.reportViewer.Rebuild();
this.reportViewer.HideRunButton();
this.reportViewer.Dock = System.Windows.Forms.DockStyle.Fill;
}
catch (Exception ex)
{
this.erros.Add(string.Format("Erro: {0}", ex.Message));
}
}
}
public RdlViewer ReportViewer
{
get
{
return this.reportViewer;
}
}
public ViewerToolstrip ReportStrip
{
get
{
return this.reportStrip;
}
}
public List<string> Erros
{
get
{
return this.erros;
}
}
private DataTable obterTabela()
{
DataTable tabela = new DataTable();
try
{
using (SQLiteConnection conexao = this.conexao.Conexao)
{
using (SQLiteCommand command = conexao.CreateCommand())
{
command.CommandType = CommandType.Text;
command.CommandText = this.query.ToString();
conexao.Open();
using (SQLiteDataAdapter dataAdapter = new SQLiteDataAdapter(command))
{
dataAdapter.Fill(tabela);
conexao.Close();
}
}
}
}
catch (Exception ex)
{
this.erros.Add("Erro:" + ex.Message);
return null;
}
return tabela;
}
}
}
Implementação das classes:
private void FormRunRelatorio_Load(object sender, EventArgs e)
{
var erros = new List<string>();
switch (this.relatorioSelecionado)
{
case "GeralAluno":
RelGeralAluno relGeralAluno = new RelGeralAluno(new DALConexao(new DadosConexao().String_Conexao));
this.Controls.Add(relGeralAluno.ReportStrip);
this.Controls.Add(relGeralAluno.ReportViewer);
erros = relGeralAluno.Erros;
break;
case "GeralAlunoInativoOuAtivo":
RelAlunoAtivoOuInativo relAlunoAtivoInativo = new RelAlunoAtivoOuInativo(new DALConexao(new DadosConexao().String_Conexao), false);
this.Controls.Add(relAlunoAtivoInativo.ReportStrip);
this.Controls.Add(relAlunoAtivoInativo.ReportViewer);
erros = relAlunoAtivoInativo.Erros;
break;
case mais condicao...
mais codigo...
}
if (erros.Count > 0)
{
foreach (var erro in erros)
MessageBox.Show(erro, "Erro", MessageBoxButtons.OK, MessageBoxIcon.Error);
this.Close();
}
}
Repetição entre métodos e atributos
Os seguintes atributos e métodos se repetem muito entre as classes, são eles:
Atributos
- Atributo reportViewer:
private RdlViewer reportViewer;
- Atributo reportStrip:
private ViewerToolstrip reportStrip;
- Atributo nomeArquivo:
private String nomeArquivo;
- Atributo query:
private StringBuilder query;
- Atributo erros:
private List<string> erros;
- Atributo conexao:
private DALConexao conexao;
Métodos
- Método emitir():
public void emitir()
- Método obterTabela():
private DataTable obterTabela()
A maneira como esta definida as classes não me parecer a melhor forma, sei que existe uma forma, porem meu conhecimento ainda é limitado.
Pergunta
Como eu posso evitar toda esta repetição de código, se é possível utilizar herança ou interface ou alguma outra característica da orientação a objeto, dentro das boas praticas de programação? Ou até mesmo se existe algum padrão ou ferramenta dentro da engenharia de software para evitar a situação acima?
Obs: Utilizo o Microsoft Visual C# 2012 e a versão do C# é 5.0 junto com o banco de dados SQLite, e a ferramenta de emissão de relatório é o My-FyiReporting.