Considere uma função sem parâmetros ( editar: não necessariamente) que executa uma única linha de código e é chamada apenas uma vez no programa (embora não seja impossível que seja necessário novamente no futuro).
Pode realizar uma consulta, verificar alguns valores, fazer algo que envolva regex ... qualquer coisa obscura ou "hacky".
A lógica por trás disso seria evitar avaliações dificilmente legíveis:
if (getCondition()) {
// do stuff
}
Onde getCondition()
é a função de uma linha.
Minha pergunta é simples: essa é uma boa prática? Parece bom para mim, mas não sei a longo prazo ...
getCondition
? Se é tão pequeno e pouco usado como você diz, então dar um nome a ela não está conseguindo nada.