Каков стандартный формат выражений присваивания?

Question

У меня есть следующая функция в JavaScript и при запуске через JSLint она кричала на меня, как я ее называю.

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 
    }; 
} 

В частности, выражение inline присваивания в цикле while. Я полагал, что двойные круглые скобки были стандартным способом сказать: «Я ожидаю, что возвращаемое значение из выражения присваивания будет занесено в тип Boolean для условного выражения». JSLint, похоже, не согласен, даже когда я включаю выражения присваивания. Затем я попытался добавить !! спереди, а JSLint жалуется, что это «запутанное использование». Итак, мой вопрос в том, как правильно форматировать это?

EDIT: Под «этим» я имел в виду конкретно выражение для встроенного присваивания. Цель моего вопроса состояла в том, чтобы прояснить, какой приемлемый стандарт для этой конкретной линии был, если кто-то действительно хотел использовать это, и хотя я согласен с тем, что ответ на этот вопрос является самым правильным способом записи функции, это не ответ на вопрос, который я задавал.

Answer 1

Согласно this article, для JSHint, ESHint и JSLint (от до июля 2013 года) вы можете заставить его в условный, как это:

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

Для более поздних версий JSLint вы не повезло и вам нужно отделить свое задание, если вы оцениваете все зеленые.

Answer 2

Лучше не иметь выражений присваивания в условных выражениях. Но, я считаю, что while подходит для этого случая лучше, чем do...while. Вы можете преобразовать код в это

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 
    }; 
} 

Теперь у условного выражения нет заданий, и я считаю, что это выглядит чистым.