no-constant-condition

Запрещает константные выражения в условиях

Константное выражение (например, литерал) в качестве условия проверки может быть опечаткой или триггером разработки для определённого поведения. Например, следующий код выглядит так, будто он не готов к использованию в рабочей среде.

if (false) {
    doSomethingUnfinished();
}

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

Это правило запрещает константные выражения в условии проверки для следующих конструкций:

• if, for, while или do...while оператор
• тернарное выражение ?:

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

/*eslint no-constant-condition: "error"*/

if (false) {
    doSomethingUnfinished();
}

if (void x) {
    doSomethingUnfinished();
}

if (x &&= false) {
    doSomethingNever();
}

if (class {}) {
    doSomethingAlways();
}

if (new Boolean(x)) {
    doSomethingAlways();
}

if (Boolean(1)) {
    doSomethingAlways();
}

if (undefined) {
    doSomethingUnfinished();
}

if (x ||= true) {
    doSomethingAlways();
}

for (;-2;) {
    doSomethingForever();
}

while (typeof x) {
    doSomethingForever();
}

do {
    doSomethingForever();
} while (x = -1);

const result = 0 ? a : b;

if(input === "hello" || "bye"){
  output(input);
}

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

/*eslint no-constant-condition: "error"*/

if (x === 0) {
    doSomething();
}

for (;;) {
    doSomethingForever();
}

while (typeof x === "undefined") {
    doSomething();
}

do {
    doSomething();
} while (x);

const result = x !== 0 ? a : b;

if(input === "hello" || input === "bye"){
  output(input);
}

Параметры

checkLoops

Эта строковая настройка принимает следующие значения:

• "all" — запрещает константные выражения во всех циклах.
• "allExceptWhileTrue" (по умолчанию) — запрещает константные выражения во всех циклах, кроме циклов while с выражением true.
• "none" — разрешает константные выражения в циклах.

Также вместо этого можно задать для значения checkLoops логические значения: true эквивалентно "all", а false эквивалентно "none".

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

/*eslint no-constant-condition: ["error", { "checkLoops": "all" }]*/

while (true) {
    doSomething();
};

for (;true;) {
    doSomething();
};

/*eslint no-constant-condition: ["error", { "checkLoops": true }]*/

while (true) {
    doSomething();
};

do {
    doSomething();
} while (true)

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

/*eslint no-constant-condition: ["error", { "checkLoops": "all" }]*/

while (a === b) {
    doSomething();
};

/*eslint no-constant-condition: ["error", { "checkLoops": true }]*/

for (let x = 0; x <= 10; x++) {
    doSomething();
};

Пример правильного кода, если checkLoops имеет значение "allExceptWhileTrue":

/*eslint no-constant-condition: "error"*/

while (true) {
    doSomething();
};

Примеры правильного кода, если checkLoops имеет значение "none" или false:

/*eslint no-constant-condition: ["error", { "checkLoops": "none" }]*/

while (true) {
    doSomething();
    if (condition()) {
        break;
    }
};

do {
    doSomething();
    if (condition()) {
        break;
    }
} while (true)

/*eslint no-constant-condition: ["error", { "checkLoops": false }]*/

while (true) {
    doSomething();
    if (condition()) {
        break;
    }
};

for (;true;) {
    doSomething();
    if (condition()) {
        break;
    }
};

Связанные правила

• no-constant-binary-expression

Версия

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

Ресурсы

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