no-sequences

Запрет оператора запятой

Оператор запятой включает в себя несколько выражений, где ожидается только одно. Он вычисляет каждый операнд слева направо и возвращает значение последнего операнда. Однако это часто маскирует побочные эффекты, и его использование часто является случайностью. Вот некоторые примеры последовательностей:

var a = (3, 5); // a = 5

a = b += 5, a + b;

while (a = next(), a && a.length);

(0, eval)("doSomething();");

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

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

• В инициализирующей или обновляющей части for оператора.
• По умолчанию, если последовательность выражений явно заключена в скобки. Это исключение можно удалить с помощью опции allowInParentheses.

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

/*eslint no-sequences: "error"*/

foo = doSomething(), val;

0, eval("doSomething();");

do {} while (doSomething(), !!test);

for (; doSomething(), !!test; );

if (doSomething(), !!test);

switch (val = foo(), val) {}

while (val = foo(), val < 42);

with (doSomething(), val) {}

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

/*eslint no-sequences: "error"*/

foo = (doSomething(), val);

(0, eval)("doSomething();");

do {} while ((doSomething(), !!test));

for (i = 0, j = 10; i < j; i++, j--);

if ((doSomething(), !!test));

switch ((val = foo(), val)) {}

while ((val = foo(), val < 42));

with ((doSomething(), val)) {}

Примечание о телах стрелочных функций

Если тело стрелочной функции представляет собой оператор, а не блок, и этот оператор содержит последовательность, необходимо использовать двойные скобки вокруг оператора, чтобы указать, что последовательность является намеренной.

Примеры неправильного кода для стрелочных функций:

/*eslint no-sequences: "error"*/
const foo = (val) => (console.log('bar'), val);

const baz = () => ((bar = 123), 10);

const qux = () => { return (bar = 123), 10 }

Примеры правильного кода для стрелочных функций:

/*eslint no-sequences: "error"*/
const foo = (val) => ((console.log('bar'), val));

const baz = () => (((bar = 123), 10));

const qux = () => { return ((bar = 123), 10) }

Параметры

Это правило принимает один параметр — объект — со следующими свойствами:

• "allowInParentheses": Если установлено значение true (по умолчанию), это правило разрешает последовательности выражений, которые явно заключены в скобки.

allowInParentheses

Примеры неправильного кода для этого правила с параметром { "allowInParentheses": false }:

/*eslint no-sequences: ["error", { "allowInParentheses": false }]*/

foo = (doSomething(), val);

(0, eval)("doSomething();");

do {} while ((doSomething(), !!test));

for (; (doSomething(), !!test); );

if ((doSomething(), !!test));

switch ((val = foo(), val)) {}

while ((val = foo(), val < 42));

with ((doSomething(), val)) {}

const foo = (val) => ((console.log('bar'), val));

Примеры правильного кода для этого правила с параметром { "allowInParentheses": false }:

/*eslint no-sequences: ["error", { "allowInParentheses": false }]*/

for (i = 0, j = 10; i < j; i++, j--);

Когда не следует использовать это правило

Отключите это правило, если последовательности выражений с оператором запятой допустимы. Еще один случай — если вы хотите сообщать обо всех использованиях оператора запятой, даже в цикле for. Вы можете добиться этого, используя правило no-restricted-syntax:

{
    "rules": {
        "no-restricted-syntax": ["error", "SequenceExpression"]
    }
}

Версия

Это правило было введено в ESLint v0.5.1.

Ресурсы

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