Having trouble with javascript Replace()

Question

This code will replace the comma's no problem, but will leave the $ for some reason... Is it set up wrong? Trying to replace the $ also.

function doValidate() { 
    var valid = true;  
    document.likeItemSearchForm.sup.value = document.likeItemSearchForm.sup.value.replace(/\$|,/g, "")  
    return valid;   
}

"$,,$,$".replace(/\$|,/g, "") === "" for me.

pimvdb
– pimvdb

2011-09-26 14:57:11 +00:00
Commented Sep 26, 2011 at 14:57 — pimvdb
– pimvdb, Commented Sep 26, 2011 at 14:57
what is going on with mine????? lol

Doc Holiday
– Doc Holiday

2011-09-26 15:01:20 +00:00
Commented Sep 26, 2011 at 15:01 — Doc Holiday
– Doc Holiday, Commented Sep 26, 2011 at 15:01

user657496 · Accepted Answer · 2011-09-26 14:59:07Z

2

Try this:

'asd$asd,asd,asd$,asd'.replace(/[\$,]/g,'');

JSFIDDLE -edit- fixed link

answered Sep 26, 2011 at 14:59

user657496

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

2 Comments

pimvdb Over a year ago

You don't need \ inside a character class.

user657496 Over a year ago

Thanks, didn't check the link (woops)

Ya Zhuang · Accepted Answer · 2011-09-26 14:59:17Z

1

try this:

"$12,121.30".replace(/[\$,]/g, "");

answered Sep 26, 2011 at 14:59

Ya Zhuang

4,66033 silver badges40 bronze badges

Collectives™ on Stack Overflow

Having trouble with javascript Replace()

2 Answers 2

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related