web-dev-qa-db-ja.com

ESLint isNaNの早めぬ使用

Node.jsモジュールのarrow関数内でisNaNグローバル関数を使用しようとしていますが、このエラーが発生しています。

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

これは私のコードです：

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

module.exports = {
  isNumber,
};

私が間違っていることについて何か考えはありますか？

シモンズ：私はAirBnBスタイルガイドを使っています。

javascripteslint
94
Elias Garcia

ドキュメンテーションが示唆している として、 Number.isNaN を使用してください。

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

Airbnbのドキュメントを引用する：

どうして？グローバルなisNaNは非数を数に強制し、NaNを強制するものにはtrueを返します。この動作が望ましい場合は、明示的にしてください。

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

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

ちなみに、これはIEでは機能しません。ブラウザの互換性で こちら を確認してください。

10
thyforhtian

私の場合、5（整数）、5.4（10進）、 '5'、 '5.4'を数字として扱いたいが、他には何もしたくなかった。

同様の要件がある場合は、以下のほうが効果的です。

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))

負の数を含めるには：

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

これにより、isNaNのグローバルな使用に関する問題も解消されます。 isNum関数を通常のES5関数に変換すると、IEブラウザでも同様に機能します。

0
Noby Fujioka

@Andy Gaskell isNumber('1.2.3') return true、回答を編集し、Number()の代わりに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