Tava pensando em separar isso em um método mesmo, o que acha? O resto das
linhas está  com chamadas a métodos, algo como:
getNossoNumeroParaCarteiras17e18.. O que acha? Só uma sugestão.


On Mon, Apr 14, 2014 at 9:31 PM, Rodrigo Turini <notifications@github.com>wrote:

> In
> stella-boleto/src/main/java/br/com/caelum/stella/boleto/bancos/BancoDoBrasil.java:
>
> > @@ -39,7 +39,7 @@ public String geraCodigoDeBarrasPara(Boleto boleto) {
> >  		} else if (emissor.getCarteira().equals("17") || emissor.getCarteira().equals("18")) {
> >  			campoLivre.append("000000");
> >  			campoLivre.append(getNumeroConvenioDoEmissorFormatado(emissor));
> > -			campoLivre.append(getNossoNumeroDoEmissorFormatado(emissor).substring(7));
> > +			campoLivre.append(emissor.getCarteira().equals("17") ? getNossoNumeroDoEmissorFormatado(emissor).substring(1) : getNossoNumeroDoEmissorFormatado(emissor).substring(7));
>
> é minor, mas se quiser deixar essa linha um pouco mais legivel, usa o
> ternário só no indice:
>
> int indice = emissor.getCarteira().equals("17") ? 1 : 7;campoLivre.append(getNossoNumeroDoEmissorFormatado(emissor).substring(indice));
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/caelum/caelum-stella/pull/111/files#r11615439>
> .
>