Qual è il formato standard per le espressioni di assegnazione?

Question

Ho la seguente funzione in JavaScript e durante l'esecuzione tramite JSLint, mi ha urlato contro, come la chiamo io.

function getPos(event, element) { 
    var x = event.clientX, 
     y = event.clientY, 
     currentElement = element; 

    do { 
     x -= currentElement.offsetLeft - currentElement.scrollLeft; 
     y -= currentElement.offsetTop - currentElement.scrollTop; 
    } while ((currentElement = currentElement.offsetParent)); 

    return { 
     x: x, 
     y: y 
    }; 
} 

In particolare sull'espressione dell'assegnazione inline nel ciclo while. Ho immaginato che la doppia parentesi fosse il modo standard per dire: "Mi aspetto che il valore restituito dall'espressione di assegnazione venga digitato in un codice Boolean per il condizionale". JSLint sembra essere in disaccordo, anche quando abilito le "espressioni di assegnazione". Poi ho provato ad aggiungere uno !! davanti e JSLint si lamenta che si tratta di "uso confuso". Quindi la mia domanda è, qual è il modo giusto per formattare questo?

MODIFICA: Con "questo", intendevo specificamente l'espressione dell'assegnazione in linea. L'intento della mia domanda era di chiarire quale fosse uno standard accettabile per quella particolare linea, se si voleva davvero usarlo, e anche se sono d'accordo sul fatto che la risposta di courtèye è il modo più corretto per scrivere la funzione, non è la risposta a la domanda che stavo chiedendo.

Answer 1

Secondo this article, per JSHint, ESHint e JSLint (da prima del luglio 2013) è possibile forzare in una condizione come questa:

do { 
    // ... 
} while ((currentElement = currentElement.offsetParent) !== null); 

Per le versioni più recenti di JSLint sei fuori di fortuna e devi separare il tuo incarico se apprezzi tutto il verde.

Answer 2

È meglio non avere espressioni di assegnazione nelle espressioni condizionali. Ma, credo che while si addicesse a questo caso meglio di do...while. Si potrebbe trasformare il codice a questo

function getPos(event, element) { 
    var x = event.clientX, 
     y = event.clientY, 
     currentElement = element; 

    while (currentElement) { 
     x -= currentElement.offsetLeft - currentElement.scrollLeft; 
     y -= currentElement.offsetTop - currentElement.scrollTop; 
     currentElement = element.offsetParent; 
    } 

    return { 
     x: x, 
     y: y 
    }; 
} 

Ora, non v'è alcuna assegnazione coinvolti nell'espressione condizionale e credo che questo sembra pulito.