| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268 |
- /**
- * @fileoverview Rule to flag comparisons to the value NaN
- * @author James Allardice
- */
- "use strict";
- //------------------------------------------------------------------------------
- // Requirements
- //------------------------------------------------------------------------------
- const astUtils = require("./utils/ast-utils");
- //------------------------------------------------------------------------------
- // Helpers
- //------------------------------------------------------------------------------
- /**
- * Determines if the given node is a NaN `Identifier` node.
- * @param {ASTNode|null} node The node to check.
- * @returns {boolean} `true` if the node is 'NaN' identifier.
- */
- function isNaNIdentifier(node) {
- if (!node) {
- return false;
- }
- const nodeToCheck =
- node.type === "SequenceExpression" ? node.expressions.at(-1) : node;
- return (
- astUtils.isSpecificId(nodeToCheck, "NaN") ||
- astUtils.isSpecificMemberAccess(nodeToCheck, "Number", "NaN")
- );
- }
- //------------------------------------------------------------------------------
- // Rule Definition
- //------------------------------------------------------------------------------
- /** @type {import('../types').Rule.RuleModule} */
- module.exports = {
- meta: {
- hasSuggestions: true,
- type: "problem",
- docs: {
- description: "Require calls to `isNaN()` when checking for `NaN`",
- recommended: true,
- url: "https://eslint.org/docs/latest/rules/use-isnan",
- },
- schema: [
- {
- type: "object",
- properties: {
- enforceForSwitchCase: {
- type: "boolean",
- },
- enforceForIndexOf: {
- type: "boolean",
- },
- },
- additionalProperties: false,
- },
- ],
- defaultOptions: [
- {
- enforceForIndexOf: false,
- enforceForSwitchCase: true,
- },
- ],
- messages: {
- comparisonWithNaN: "Use the isNaN function to compare with NaN.",
- switchNaN:
- "'switch(NaN)' can never match a case clause. Use Number.isNaN instead of the switch.",
- caseNaN:
- "'case NaN' can never match. Use Number.isNaN before the switch.",
- indexOfNaN:
- "Array prototype method '{{ methodName }}' cannot find NaN.",
- replaceWithIsNaN: "Replace with Number.isNaN.",
- replaceWithCastingAndIsNaN:
- "Replace with Number.isNaN and cast to a Number.",
- replaceWithFindIndex:
- "Replace with Array.prototype.{{ methodName }}.",
- },
- },
- create(context) {
- const [{ enforceForIndexOf, enforceForSwitchCase }] = context.options;
- const sourceCode = context.sourceCode;
- const fixableOperators = new Set(["==", "===", "!=", "!=="]);
- const castableOperators = new Set(["==", "!="]);
- /**
- * Get a fixer for a binary expression that compares to NaN.
- * @param {ASTNode} node The node to fix.
- * @param {function(string): string} wrapValue A function that wraps the compared value with a fix.
- * @returns {function(Fixer): Fix} The fixer function.
- */
- function getBinaryExpressionFixer(node, wrapValue) {
- return fixer => {
- const comparedValue = isNaNIdentifier(node.left)
- ? node.right
- : node.left;
- const shouldWrap = comparedValue.type === "SequenceExpression";
- const shouldNegate = node.operator[0] === "!";
- const negation = shouldNegate ? "!" : "";
- let comparedValueText = sourceCode.getText(comparedValue);
- if (shouldWrap) {
- comparedValueText = `(${comparedValueText})`;
- }
- const fixedValue = wrapValue(comparedValueText);
- return fixer.replaceText(node, `${negation}${fixedValue}`);
- };
- }
- /**
- * Checks the given `BinaryExpression` node for `foo === NaN` and other comparisons.
- * @param {ASTNode} node The node to check.
- * @returns {void}
- */
- function checkBinaryExpression(node) {
- if (
- /^(?:[<>]|[!=]=)=?$/u.test(node.operator) &&
- (isNaNIdentifier(node.left) || isNaNIdentifier(node.right))
- ) {
- const suggestedFixes = [];
- const NaNNode = isNaNIdentifier(node.left)
- ? node.left
- : node.right;
- const isSequenceExpression =
- NaNNode.type === "SequenceExpression";
- const isSuggestable =
- fixableOperators.has(node.operator) &&
- !isSequenceExpression;
- const isCastable = castableOperators.has(node.operator);
- if (isSuggestable) {
- suggestedFixes.push({
- messageId: "replaceWithIsNaN",
- fix: getBinaryExpressionFixer(
- node,
- value => `Number.isNaN(${value})`,
- ),
- });
- if (isCastable) {
- suggestedFixes.push({
- messageId: "replaceWithCastingAndIsNaN",
- fix: getBinaryExpressionFixer(
- node,
- value => `Number.isNaN(Number(${value}))`,
- ),
- });
- }
- }
- context.report({
- node,
- messageId: "comparisonWithNaN",
- suggest: suggestedFixes,
- });
- }
- }
- /**
- * Checks the discriminant and all case clauses of the given `SwitchStatement` node for `switch(NaN)` and `case NaN:`
- * @param {ASTNode} node The node to check.
- * @returns {void}
- */
- function checkSwitchStatement(node) {
- if (isNaNIdentifier(node.discriminant)) {
- context.report({ node, messageId: "switchNaN" });
- }
- for (const switchCase of node.cases) {
- if (isNaNIdentifier(switchCase.test)) {
- context.report({ node: switchCase, messageId: "caseNaN" });
- }
- }
- }
- /**
- * Checks the given `CallExpression` node for `.indexOf(NaN)` and `.lastIndexOf(NaN)`.
- * @param {ASTNode} node The node to check.
- * @returns {void}
- */
- function checkCallExpression(node) {
- const callee = astUtils.skipChainExpression(node.callee);
- if (callee.type === "MemberExpression") {
- const methodName = astUtils.getStaticPropertyName(callee);
- if (
- (methodName === "indexOf" ||
- methodName === "lastIndexOf") &&
- node.arguments.length <= 2 &&
- isNaNIdentifier(node.arguments[0])
- ) {
- /*
- * To retain side effects, it's essential to address `NaN` beforehand, which
- * is not possible with fixes like `arr.findIndex(Number.isNaN)`.
- */
- const isSuggestable =
- node.arguments[0].type !== "SequenceExpression" &&
- !node.arguments[1];
- const suggestedFixes = [];
- if (isSuggestable) {
- const shouldWrap = callee.computed;
- const findIndexMethod =
- methodName === "indexOf"
- ? "findIndex"
- : "findLastIndex";
- const propertyName = shouldWrap
- ? `"${findIndexMethod}"`
- : findIndexMethod;
- suggestedFixes.push({
- messageId: "replaceWithFindIndex",
- data: { methodName: findIndexMethod },
- fix: fixer => [
- fixer.replaceText(
- callee.property,
- propertyName,
- ),
- fixer.replaceText(
- node.arguments[0],
- "Number.isNaN",
- ),
- ],
- });
- }
- context.report({
- node,
- messageId: "indexOfNaN",
- data: { methodName },
- suggest: suggestedFixes,
- });
- }
- }
- }
- const listeners = {
- BinaryExpression: checkBinaryExpression,
- };
- if (enforceForSwitchCase) {
- listeners.SwitchStatement = checkSwitchStatement;
- }
- if (enforceForIndexOf) {
- listeners.CallExpression = checkCallExpression;
- }
- return listeners;
- },
- };
|