Considera una funzione senza parametri ( modifica: non necessariamente) che esegue una singola riga di codice e viene chiamata una sola volta nel programma (anche se non è impossibile che sia necessario di nuovo in futuro).
Potrebbe eseguire una query, controllare alcuni valori, fare qualcosa che coinvolge regex ... qualcosa di oscuro o "hacky".
La logica alla base di ciò sarebbe quella di evitare valutazioni difficilmente leggibili:
if (getCondition()) {
// do stuff
}
dov'è getCondition()
la funzione di una riga.
La mia domanda è semplicemente: è una buona pratica? Mi sembra a posto ma non so a lungo termine ...
getCondition
? Se è piccolo e usato di rado come dici, dargli un nome non significa ottenere nulla.