Secondo questa guida al codice pulito, dovresti incapsulare condizionali:
function shouldShowSpinner() {
return fsm.state === 'fetching' && isEmpty(listNode);
}
if (shouldShowSpinner()) {
// ...
}
Perché non scrivere semplicemente:
const shouldShowSpinner = fsm.state === 'fetching' && isEmpty(listNode)
if (shouldShowSpinner) {
// ...
}
Perché
const shouldShowSpinner = fsm.state === 'fetching' && isEmpty(listNode)
viene valutato solo una volta. L'operatore di assegnazione = memorizza il risultato di una singola valutazione dell'espressione booleana.
Questo va bene se lo stato dell'espressione booleana non cambia mai durante l'esecuzione del codice nello scope pertinente, ma produrrà risultati errati se cambia, nel qual caso otterrai il risultato ora impreciso del calcolo originale.
Leggi altre domande sui tag coding-style javascript