Как лучше разделять строку в операторе «if»?

Question

[Ventus]

Имеется условный оператор if, который проверяет несколько условий:

if (typeof(mandatory) === 'string'
        && typeof(mandatory) !== null
        && typeof(expenseAmount) !== null
        && mandatory !== ''
        && expenseAmount !== '') {
        console.log('done');
    }

Правильно ли я сделал перевод на следующую строку? Как это чаще делается на практике?

Comments

[Тигран Абрамян]

https://github.com/airbnb/javascript

[Ventus]

TAbrahamyan, спасибо, нашел в пункте 17.1.

Answer 1

[Ventus]

Ссылку в комментарии к этому вопросу дал TAbrahamyan: https://github.com/airbnb/javascript

Объяснение нашел в пункте. 17.1

Answer 2

[Алексей Ярков]

В данном конкретном случае я бы первое условие и закрывающую скобку перенёс тоже.

if (
        typeof(mandatory) === 'string'
        && typeof(mandatory) !== null
        && typeof(expenseAmount) !== null
        && mandatory !== ''
        && expenseAmount !== ''
) {
        console.log('done');
}

В таком случае, если я захочу добавить или изменить условие, то diff будет понятнее. Ну это лично моё мнение ))

Comments

[Ventus]

Да, спасибо, ваше решение совпадает с рекомендацией Airbnb JavaScript Style.

Только табуляция, как я понял, лишняя перед списком проверяемых условий.

[Алексей Ярков]

Ventus,

[Ventus]

Алексей Ярков, забыл, что на qna табуляция в 2 раза длиннее))

[Алексей Ярков]

Ventus,
Так а где тут табуляция?