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.

Ресурсы

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