web-dev-qa-db-fra.com

ESLint Utilisation inattendue d'isNaN

J'essaie d'utiliser la fonction globale isNaN à l'intérieur d'une fonction de flèche dans un module Node.js mais j'obtiens cette erreur:

[eslint] Unexpected use of 'isNaN'. (no-restricted-globals)

Ceci est mon code:

const isNumber = value => !isNaN(parseFloat(value));

module.exports = {
  isNumber,
};

Une idée sur ce que je fais mal?

PS: J'utilise le guide de style AirBnB.

javascripteslint
74
Elias Garcia

Comme le propose documentation , utilisez Number.isNaN .

const isNumber = value => !Number.isNaN(parseFloat(value));

Citant la documentation d'Airbnb:

Pourquoi? Le global isNaN contraint les non-nombres en nombres, renvoyant true pour tout ce qui oblige à NaN. Si ce comportement est souhaité, rendez-le explicite.

// bad
isNaN('1.2'); // false
isNaN('1.2.3'); // true

// good
Number.isNaN('1.2.3'); // false
Number.isNaN(Number('1.2.3')); // true
138
Andy Gaskell

Pour votre information, cela ne fonctionnera pas pour IE . Vérifiez ici avec la compatibilité du navigateur.

9
thyforhtian

Dans mon cas, je voulais traiter 5 (entier), 5,4 (décimal), «5», «5,4» comme des nombres, mais rien d’autre par exemple.

Si vous avez les mêmes exigences, vous trouverez ci-dessous un meilleur fonctionnement:

const isNum = num => /^\d+$/.test(num) || /^\d+\.\d+$/.test(num);

//Check your variable if it is a number.
let myNum = 5;
console.log(isNum(myNum))

Pour inclure des nombres négatifs:

const isNum = num => /^-?\d+$/.test(num) || /^-?\d+\.\d+$/.test(num);

Cela supprimera également votre problème d'utilisation globale d'isNaN . Si vous convertissez la fonction isNum en une fonction ES5 normale, cela fonctionnera également sur le navigateur IE.

0
Noby Fujioka

@Andy Gaskell isNumber('1.2.3') return true, vous pouvez modifier votre réponse et utiliser Number() à la place de parseFloat().

    const isEmpty = value => typeof value === 'undefined' || value === null || value === false;
    const isNumeric = value => !isEmpty(value) && !Number.isNaN(Number(value));

  console.log(isNumeric('5')); // true
  console.log(isNumeric('-5')); // true
  console.log(isNumeric('5.5')); // true
  console.log(isNumeric('5.5.5')); // false
  console.log(isNumeric(null)); // false
  console.log(isNumeric(undefined)); // false
0
Vincent Baronnet