Ho appena visto uno snippet di codice che utilizzava un'assegnazione pop array in condizioni while; mi chiedevo se questo è accettabile / buone pratiche?
var arr = [0,1,2,3,4,5];
var current;
while (current = arr.pop())
{
// do stuff to each element stored in current
}
È non necessariamente sbagliato se il codice fa ciò che deve fare, tuttavia, decisioni come questa possono essere una questione di gusto o opinione in una scrittura personale codice. Viceversa, evitare questa particolare disposizione della logica può essere un requisito di un datore di lavoro.
Ecco le cose da considerare.
Per essere più specifici, ecco alcuni pro e contro usando il codice nel tuo esempio.
undefined , null , 0 , false , NaN , "" o ha "buchi", in questo modo avrà esito negativo. "Fori" indica quando la matrice ha una lunghezza maggiore di 0 e ad alcuni (o tutti) gli elementi dell'array non è stato assegnato un valore.
"Verità" significa valori che non valutano in falso (vedere i contro). Non dimenticare NaN o "" , l'ho quasi fatto.
Il codice nel tuo esempio contiene già un bug. Il primo elemento dell'array sarà perso. Perché? Il primo elemento è zero, che quando usato in un'istruzione di assegnazione restituisce il valore zero, che viene trattato come falso dal ciclo while, vale a dire che il ciclo while si arresta prima di elaborare l'ultimo elemento sullo "stack", o piuttosto il primo elemento nell'array.
Se l'intenzione è di elaborare l'array come una coda o una pila, in cui gli elementi vengono rimossi mentre vengono elaborati, sì. Questo è un approccio abbastanza standard.
In effetti, a volte vedi questo tipo di loop per attraversamenti di alberi in cui lo stack overflow, o la necessità di mettere in pausa e riprendere, sono un potenziale problema nella soluzione ricorsiva.
Leggi altre domande sui tag loops programming-practices conditions javascript array