Merge pull request #101 from skatcat31/validateNumber

Fix: validateNumber: check cooercian
2017-12-14 10:20:33 +02:00
parent aac891ad96 f37272cd01
commit 562cbdb26b
1 changed files with 2 additions and 1 deletions
--- a/snippets/validate-number.md
+++ b/snippets/validate-number.md
@ -2,8 +2,9 @@

 Use `!isNaN` in combination with `parseFloat()` to check if the argument is a number.
 Use `isFinite()` to check if the number is finite.
+Use `Number()` to check if the coercion holds.

 ```js
-const validateNumber = n => !isNaN(parseFloat(n)) && isFinite(n);
+const validateNumber = n => !isNaN(parseFloat(n)) && isFinite(n) && Number(n) == n;
 // validateNumber('10') -> true
 ```