TDD E Encapsulamento

1 README.md

TDD e Encapsulamento

Testes que lidam demais com outros objetos ao invés de lidar com o objeto sob teste podem estar avisando o desenvolvedor em relação a problemas de encapsulamento. A própria não utilização da Lei de Demeter, tanto nos testes quanto no código de produção, também pode avisar sobre os mesmos problemas.

2 TestaFatura.java

public class TestaFatura {
  
  @Test
  public void deveMarcarFaturaComoPagaCasoBoletoUnicoPagueTudo() {
      ProcessadorDeBoletos processador = new ProcessadorDeBoletos();
      Fatura fatura = new Fatura("Cliente", 150.0);
      Boleto b1 = new Boleto(150.0);

      List<Boleto> boletos = Arrays.asList(b1);
      processador.processa(boletos, fatura);
      // Onde a fatura acaba sabendo que foi paga?
      assertTrue(fatura.isPago());
  }
}

3 ProcessaBoleto.java

public class ProcessaBoleto {
  
  public void processa(List<Boleto> boletos, Fatura fatura) {
    double valorTotal = 0;
    for(Boleto boleto : boletos) {
      Pagamento pagamento = new Pagamento(boleto.getValor(), MeioDePagamento.BOLETO);
      fatura.getPagamentos().add(pagamento);

      valorTotal += boleto.getValor();
    }

    if(valorTotal >= fatura.getValor()) {
      fatura.setPago(true);
    }
  }

}

4 refatorando.md

Rafatorando

Nós acabamos de perceber que o processador de boletos sabe demais sobre a classe de Fatura. Quando mandamos processar o boleto a nós checamos se o valor acumulado é maior ou igual ao da fatura e a marcamos como paga.

E se nós encapsularmos esse comportamento dentro da propria fatura? Evitariamos que o gerador de boleto soubesse demais sobre a fatura e o seu código se tornaria mais enxuto.

Primeiro vamos adicionar o método addPagamento(Pagamento p) a fatura e posteriormente remover o if de dentro do gerenciador de boleto.

5 Fatura.java

public class Fatura {
  private List<Pagamento> pagamentos;
  private boolean paga;
  private double valor;
  
  public Fatura(double valor) {
    this.pagamentos = new ArrayList<Pagamento>();
    this.paga = false;
    this.valor = valor;
  }
  
  /*
  * Removemos a lógica do gerenciador de boletos para a classe interessada
  */
  public addPagamento(Pagamento p) {
    this.pagamentos.add(p);
    
    double valorTotla = 0;
    
    for (Pagamento pagamento : this.pagamentos) {
      valorTotal += pagamento.getValor();
    }
    
    if (valorTotal >= this.valor) {
      this.paga = true;
    }
  }

}

6 ProcessaBoleto.java

public class ProcessaBoleto {
  
  public void processa(List<Boleto> boletos, Fatura fatura) {

    for(Boleto boleto : boletos) {
      Pagamento pagamento = new Pagamento(boleto.getValor(), MeioDePagamento.BOLETO);
      fatura.addPagamento(pagamento);
    }

  }

}