Stripping commas and non-numeric characters from string in javascript

Question

should be simple but RegExs never seem to be :). Can anyone help on how to strip both a comma and any non-numeric characters from a string? Thanks. It's in the var result block. Apparently when you put an operator in the number it bombs out.num1 and num2. I also need to strip out any dashes.

 function calcTotalRetailVal() {
    var num1 = $oneTimeCostField.val();
    var num2 = $recurringTotalCostField.val();
   //In the replace method
    var result = parseFloat(num1.replace(/,/g, '')) + parseFloat(num2.replace(/,/g, ''));
    if (!isNaN(result)) {
        $totalRetailAmountField.text('$' + result.toFixed(2));
    }   
}

Dupocas · Accepted Answer · 2019-04-10 20:40:12Z

4

const clearText = plainText.replace(/\D/g,'')

\D will match all non-digits, but if you need to preserve dots and dashes:

replace(/[^\d.-]/g, '')

edited Apr 10, 2019 at 20:40

answered Apr 10, 2019 at 20:32

Dupocas

21.5k7 gold badges44 silver badges59 bronze badges

Sign up to request clarification or add additional context in comments.

5 Comments

Barmar Over a year ago

This will also remove a minute sign, which is needed for negative numbers, and the decimal point.

erics15 Over a year ago

Thank you, i also need to strip the dash b/c of the case of someone putting in a negative amount.

Dupocas Over a year ago

replace(/[^\d.]/g, '') then

erics15 Over a year ago

I also need to remove commas (sorry to keep bothering)

Dupocas Over a year ago

replace(/[^\d,-]/g, '') this Will strip dashes an commas

Hamed Taheri · Accepted Answer · 2019-04-10 20:56:10Z

1

You should use this regex /(,|[^\d.-]+)+/g to detect comma and any non-numeric value such as characters, operators, spaces in the groups and faster than the individual detection. a negative number (ex -1) and . will be included.

I rewrite your code.

function calcTotalRetailVal() {
    var num1 = $oneTimeCostField.val();
    var num2 = $recurringTotalCostField.val();
   //In the replace method
    var result = parseFloat(num1.replace(/(,|[^\d.-]+)+/g, '')) + parseFloat(num2.replace(/(,|[^\d.-]+)+/g, ''));
    if (!isNaN(result)) {
        $totalRetailAmountField.text('$' + result.toFixed(2));
    }   
}

edited Apr 10, 2019 at 20:56

answered Apr 10, 2019 at 20:48

Hamed Taheri

1602 silver badges11 bronze badges

Comments

mbojko · Accepted Answer · 2019-04-10 20:35:50Z

1

With a regexp. num1.replace(/[^0-9.]/, '')

answered Apr 10, 2019 at 20:35

mbojko

14.8k2 gold badges21 silver badges28 bronze badges

1 Comment

Barmar Over a year ago

You should also allow a minute sign for negative numbers.

Collectives™ on Stack Overflow

Stripping commas and non-numeric characters from string in javascript

3 Answers 3

5 Comments

Comments

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

5 Comments

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related