no-useless-call

Запрещает ненужные вызовы .call() и .apply()

Вызов функции можно записать с помощью Function.prototype.call() и Function.prototype.apply(). Но Function.prototype.call() и Function.prototype.apply() работают медленнее, чем обычный вызов функции.

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

Это правило предназначено для обнаружения использования Function.prototype.call() и Function.prototype.apply(), которые можно заменить обычным вызовом функции.

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

/*eslint no-useless-call: "error"*/

// These are same as `foo(1, 2, 3);`
foo.call(undefined, 1, 2, 3);
foo.apply(undefined, [1, 2, 3]);
foo.call(null, 1, 2, 3);
foo.apply(null, [1, 2, 3]);

// These are same as `obj.foo(1, 2, 3);`
obj.foo.call(obj, 1, 2, 3);
obj.foo.apply(obj, [1, 2, 3]);

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

/*eslint no-useless-call: "error"*/

// The `this` binding is different.
foo.call(obj, 1, 2, 3);
foo.apply(obj, [1, 2, 3]);
obj.foo.call(null, 1, 2, 3);
obj.foo.apply(null, [1, 2, 3]);
obj.foo.call(otherObj, 1, 2, 3);
obj.foo.apply(otherObj, [1, 2, 3]);

// The argument list is variadic.
// Those are warned by the `prefer-spread` rule.
foo.apply(undefined, args);
foo.apply(null, args);
obj.foo.apply(obj, args);

Параметры

У этого правила нет параметров.

Известные ограничения

Это правило статически сравнивает код, чтобы определить, изменяется ли thisArg. Поэтому если код, связанный с thisArg, содержит динамическое выражение, правило не сможет сделать правильный вывод.

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

/*eslint no-useless-call: "error"*/

a[i++].foo.call(a[i++], 1, 2, 3);

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

/*eslint no-useless-call: "error"*/

a[++i].foo.call(a[i], 1, 2, 3);

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

Если вы не хотите получать уведомления о ненужных .call() и .apply(), это правило можно отключить.

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

• prefer-spread

Версия

Это правило появилось в ESLint v1.0.0-rc-1.

Ресурсы

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