Não dá para generalizar e definir que qualquer parâmetro boolean
é um problema. Se é um problema ou não, vai variar do objetivo, contexto ou mesmo tamanho do método/classe que está recebendo este parâmetro booleano. E para resolvê-lo, também irá depender se estamos falando de uma classe ou método recebendo este parâmetro.
Casos mais simples: poucos parâmetros
Por exemplo, se eu estiver querendo popular um objeto Saldo
:
new Saldo("1.12", false);
O primeiro parâmetro é o valor do crédito e o segundo é um booleano indicando se o saldo entrando está bloqueado ou não.
Para um caso assim, simples e direto, este booleano não vai lhe gerar dores de cabeça ou confundi-lo no uso da classe.
Agora vamos ao método. Usando seu exemplo, teríamos uma chamada deste método assim:
teste(objeto, false);
Assim como ocorre com a classe, são poucos parâmetros e geralmente não criará confusões. Claro que quando vemos simplesmente um parâmetro false
, não temos qualquer indicação do que aquilo significa fora da classe e método. Nestes casos simples, uma simples extração de variável pode ajudar:
new Saldo("1.12", isSaldoBloqueado);
teste(objeto, isEspecial);
Algumas linguagens também contam com named parameters, o que ajuda muito no entendimento do código e substitui esta técnica de extração de variável. Outras IDEs, como o Intellij IDEA do Java, por padrão mostram o nome das variáveis booleanas internas do método na sua chamada (simulando um named parameter), facilitando muito o entendimento do código.
Em casos simples assim, acabo sendo mais prático ao invés de tentar quebrar em vários métodos ou classes o código apenas para tentar sumir com o boolean. Não estou dizendo que deixar assim é uma boa prática, mas não quero ser extremista a ponto de falar que isto não deve existir em hipótese alguma no código. Mas, sem dúvida, são códigos que merecem atenção.
Deixemos isto para os casos mais complexos, explicados a seguir.
Casos mais complexos: muitos parâmetros
Quando temos muitos parâmetros, a situação muda de figura. Imagine a seguinte instanciação de classe e chamada de métodos:
new Saldo("1,23", true, false, false);
teste(objeto, true, dataLimite, true, false);
Embora possa extrair algumas variáveis e melhorar a situação, nem sempre o resultado é satisfatório. Por ter mais variáveis booleanas, as chances de se confundir são maiores e, principalmente no caso do método, temos um inicio de bad smell: provavelmente este método está fazendo coisas demais ou o código internamente está confuso.
Como resolver nos objetos?
Nos objetos, normalmente é mais simples resolver. Você tem duas opções:
- Quebrar em mais classes, distribuindo melhor as responsabilidades
- Usar um pattern como Builder.
Ou seja, para um objeto inicialmente assim: new Saldo("1,23", true, false, false);
, poderíamos refatorar para ter:
new Saldo("1,23", new Bloqueado(true, dataLimite), new Composto(false, false));
Podendo usar, também, a própria presença do objeto ou não para compor o conteúdo do próprio objeto e usando diferentes construtores para isto. No exemplo acima, poderíamos ter dois construtores para Saldo, um deles aceitando o Bloqueado
e o outro não.
Outra alternativa:
Saldo.builder
.valor("1,23")
.bloqueado(true)
.dataLimite(dataLimite)
.saldoComposto(false)
.saldoCompostoBloqueavel(true)
.build();
Particularmente não gosto do Builder
se existir a possibilidade de refatorar para separar em mais objetos as informações. Com o Builder
, você pode facilmente esquecer de definir uma das informações acima e quebrar seu código sem perceber.
Agora, no caso de métodos, as coisas podem complicar um pouco.
Como resolver em métodos?
Com os métodos, temos mais opções, mas nem todas parecem boas. Vamos começar com um exemplo.
public String enviar (String mensagem, boolean erro) {
if (!erro)
service.enviarSucesso(mensagem);
else
service.enviarErro(mensagem);
}
Este método tem mais de uma responsabilidade. A chamada do mesmo seria:
enviar("mensagem", true);
Só quer você não quer ter este boolean
. Para resolver este caso, você pode passar a responsabilidade para quem chama o método decidir:
enviarErro("mensagem");
enviarSucesso("mensagem");
Desta maneira, os métodos farão apenas uma única coisa (única responsabilidade). Contudo, isto pode resultar apenas em transferência da condição (mudar o if
de classe), dependendo do código.
Outra alternativa, mais OOP, poderia ser:
Mensagem mensagem = new Sucesso("mensagem");
Mensagem mensagem = new Erro("mensagem");
Inclusive, no caso de uso de classes orientadas à objetos, técnicas como Decorator podem eliminar vários parâmetros booleanos, como demonstrado anteriormente no exemplo do Saldo
.