Remove redundant number characters check

Remove unnecessary multiple array making with `.split()`
Reducing lines and making it more terse and readable
This commit is contained in:
iamsoorena
2017-12-22 15:00:46 +03:30
parent 9a8566568e
commit 7d47cf05db

View File

@ -6,12 +6,8 @@ Use `toString()` to convert the float `num` to a string, then use regex to separ
```js ```js
const toDecimalMark = (num) => { const toDecimalMark = (num) => {
let cleanNum = num.toString().split('').filter(n => '0123456789.'.includes(n)).join('') let numberParts = num.toString().split('.')
let wholeNum = cleanNum.split('.')[0].replace(/\B(?=(\d{3})+(?!\d))/g, ",") return `${numberParts[0].replace(/\B(?=(\d{3})+(?!\d))/g, ',')}.${numberParts[1]}`
let decNum = `.${cleanNum.split('.')[1]}`
return wholeNum + decNum;
} }
// toDecimalMark(12305030388.9087) //-> '12,305,030,388.9087' // toDecimalMark(12305030388.9087) //-> '12,305,030,388.9087'
// toDecimalMark(123.889087e2) //-> '12,388.9087'
// toDecimalMark('12305abc030388.9087') // -> '12,305,030,388.9087'
``` ```