no-extra-boolean-cast

Запрещает ненужные преобразования к логическому типу

В таких контекстах, как условие оператора if, где результат выражения уже будет приведён к логическому типу, преобразование к логическому типу с помощью двойного отрицания (!!) или вызова Boolean не требуется. Например, следующие операторы if эквивалентны:

if (!!foo) {
    // ...
}

if (Boolean(foo)) {
    // ...
}

if (foo) {
    // ...
}

Подробности правила

Это правило запрещает ненужные преобразования к логическому типу.

Примеры неправильного кода для этого правила:

/*eslint no-extra-boolean-cast: "error"*/

const foo = !!!bar;

const foo1 = !!bar ? baz : bat;

const foo2 = Boolean(!!bar);

const foo3 = new Boolean(!!bar);

if (!!foo) {
    // ...
}

if (Boolean(foo)) {
    // ...
}

while (!!foo) {
    // ...
}

do {
    // ...
} while (Boolean(foo));

for (; !!foo; ) {
    // ...
}

Примеры правильного кода для этого правила:

/*eslint no-extra-boolean-cast: "error"*/

const foo = !!bar;
const foo1 = Boolean(bar);

function qux() {
    return !!bar;
}

foo = bar ? !!baz : !!bat;

Параметры

У этого правила есть параметр-объект:

• "enforceForInnerExpressions" при значении true, помимо проверки стандартных контекстов, проверяет наличие дополнительных преобразований к логическому типу в выражениях, результат которых используется в логическом контексте. См. примеры ниже. По умолчанию установлено значение false, то есть это правило по умолчанию не выдаёт предупреждения о дополнительных преобразованиях к логическому типу во внутренних выражениях.

Устарело: Свойство объекта enforceForLogicalOperands устарело (eslint#18222). Вместо него используйте enforceForInnerExpressions.

enforceForInnerExpressions

Примеры неправильного кода для этого правила, если параметр "enforceForInnerExpressions" имеет значение true:

/*eslint no-extra-boolean-cast: ["error", {"enforceForInnerExpressions": true}]*/

if (!!foo || bar) {
    //...
}

while (!!foo && bar) {
    //...
}

if ((!!foo || bar) && !!baz) {
    //...
}

const foo = new Boolean(!!bar || baz);

foo && Boolean(bar) ? baz : bat;

const ternaryBranches = Boolean(bar ? !!baz : bat);

const nullishCoalescingOperator = Boolean(bar ?? Boolean(baz));

const commaOperator = Boolean((bar, baz, !!bat));

// another comma operator example
for (let i = 0; console.log(i), Boolean(i < 10); i++) {
    // ...
}

Примеры правильного кода для этого правила, если параметр "enforceForInnerExpressions" имеет значение true:

/*eslint no-extra-boolean-cast: ["error", {"enforceForInnerExpressions": true}]*/

// Note that `||` and `&&` alone aren't a boolean context for either operand
// since the resultant value need not be a boolean without casting.
const foo = !!bar || baz;

if (foo || bar) {
    //...
}

while (foo && bar) {
    //...
}

if ((foo || bar) && baz) {
    //...
}

const foo1 = new Boolean(bar || baz);

foo && bar ? baz : bat;

const ternaryBranches = Boolean(bar ? baz : bat);

const nullishCoalescingOperator = Boolean(bar ?? baz);

const commaOperator = Boolean((bar, baz, bat));

// another comma operator example
for (let i = 0; console.log(i), i < 10; i++) {
    // ...
}

// comma operator in non-final position
Boolean((Boolean(bar), baz, bat));

Версия

Это правило появилось в ESLint v0.4.0.

Ресурсы

• Исходный код правила
• Исходный код тестов