Ao desenvolver uma tela para uma determinada parte de um sistema, me deparo com uma situação clássica de possuir alguns if
s para determinar qual ação deverá ser executada. Logo associei esse caso a exemplos de violação do OCP.
Contudo, vi que a tela é bem coesa e com um escopo definido, onde não vejo, e nem a equipe vê em algum momento próximo, uma possível mudança nas ações. Ou seja, não serão, teoricamente, adicionados novas condições.
Em casos como esse, deve-se seguir o OCP fielmente, criar a classe base e suas extensões, ou manter os if
s? levando em consideração que não devem aparecer novas demandas?
Esse é o código atual que teoricamente viola o OCP.
Controlador:
[HttpPost]
public ActionResult InserirOrcamento(OrcamentoViewModel orcamento, string acao)
{
if (acao == "consultaCPF")
{
var retornoConsultaCPF = OrcamentoService.consultaCPF(orcamento);
return Json(retornoConsultaCPF, JsonRequestBehavior.AllowGet);
}
if (acao == "listaTipoProduto")
{
var retornoTipoProduto = orcamento.lstTipoProduto = OrcamentoService.listaTipoProduto(orcamento);
return Json(retornoTipoProduto, JsonRequestBehavior.AllowGet);
}
}
Service
public class OrcamentoService
{
public static OrcamentoViewModel consultaCPF(OrcamentoViewModel orcamento)
{
using(ToqueEntities db = new ToqueEntities()){
var retorno = db.PESSOA_FISICA
.Where(x => x.PESF_CPF == orcamento.cpfPessoa)
// .Join(db.CONTATOS, x => x.PES_CDID, c => c.PES_CDID, (x,c) => new { x, c })
.Select(x => new OrcamentoViewModel
{
cpfPessoa = x.PESF_CPF,
nomeClienteOrcamento = x.PESF_NOME,
sobrenomeClienteOrcamento = x.PESF_SOBRENOME,
PES_CDID = x.PES_CDID
}).FirstOrDefault();
if (!retorno.Equals(null))
{
var lstContatos = db.CONTATOS.Where(x => x.PES_CDID == retorno.PES_CDID && x.CON_TIPOCONTATO != "EMAIL")
.Select(x => x.CON_CONTEUDO)
.ToList();
if (!lstContatos.Count().Equals(0))
{
foreach (var item in lstContatos)
{
retorno.Contatos += item + ";";
}
}
else
{
retorno.Contatos = "";
}
}
orcamento.nomeClienteOrcamento = retorno.nomeClienteOrcamento;
orcamento.sobrenomeClienteOrcamento = retorno.sobrenomeClienteOrcamento;
orcamento.cpfPessoa = retorno.cpfPessoa;
orcamento.Contatos = retorno.Contatos;
return orcamento;
}
}