Etiqueta: brackets

$high_consuming_resources = true;
($high_consuming_resources)
  ? set_time_limit(0)
  : set_time_limit(30);

$high_consuming_resources = false;
($high_consuming_resources)
  ? init_set('memory_limit', '2048M')
  : null;

Depois que a gente começa a trabalhar com Clean Code umas das tendências é tentar ao máximo eliminar os famigerados if / else com brackets.

Então estes dias eu estava adicionando novas features em uma classe e conforme o escopo eu poderia ou não executar um processo de acordo com os parâmetros recebidos, mas não queria dar o braço a torcer… então decidi experimentar algo diferente.

Usando um operador ternário eu simplesmente desprezei a variável para armazenar o resultado da operação. Como as minhas operações eram VOID elas não produziam nenhum retorno mesmo.

Depois foi simples, na parte TRUE / FALSE do ternário coloquei as instruções para executar as operações desejadas. Tem que ser uma operação única, ou então uma operação em cadeia tipo um objeto que retorna self e permite encadear as operações.

Caso você tenha duas ou mais operações você pode encapsulá-las em um novo método e então no ternário teria uma única operação a ser invocada.

Também surgiu a situação onde eu tinha uma operação somente para a parte TRUE do ternário. Então a solução mais lógica foi apenas adicionar um NULL na parte FALSE fechando o ciclo do operador.

O mesmo código acima poderia ser escrito desta forma com o tradicional IF / ELSE…

$high_consuming_resources = true;
if ($high_consuming_resources) {
    set_time_limit(0);
} else {
    set_time_limit(30);
}

$high_consuming_resources = false;
if ($high_consuming_resources) {
    init_set('memory_limit', '2048M');
}

E aí? Qual destas formas você acha mais legível ou mais agradável para debugar mentalmente?

Confesso que acabei me acostumando com o ternário, na hora de ler o código parece que flui melhor o raciocínio.

Sei lá, pode ser só preferência pessoal mesmo!