JavaScript replace all comma in a string

Question

I want to convert all comma in below string to space or say blank, i tried below code which is only taking care of first comma, I tried global indicator /g as well but that do nothing.

What I am doing wrong?

var str="D'Or, Megan#LastName Jr., FirstName#BMW, somename#What, new";
str=str.replace(',','');
alert(str)

Output

D'Or Megan#LastName Jr., FirstName#BMW, somename#What, new

expected

D'Or Megan#LastName Jr. FirstName#BMW somename#What new

Can also do str.replace(',', '', 'g'). From the docs: "To perform a global search and replace, either include the g switch in the regular expression or if the first parameter is a string, include g in the flags parameter." — Anthony Grist
– Anthony Grist, Commented Mar 3, 2014 at 11:41
i tried this global option as mentioned not working for me :( — Dev G
– Dev G, Commented Mar 3, 2014 at 11:42
Thanks Andolasoft....my bad i was using single coma for regex str=str.replace('/,/g',''); — Dev G
– Dev G, Commented Mar 3, 2014 at 11:44

lin · Accepted Answer · 2015-06-02 14:28:21Z

4

To replace any given string u need to use regular expressions. You need to use a RegExp Object to ensure crossbrowser compatibility.

The use of the flags parameter in the String.replace method is non-standard. For cross-browser compatibility, use a RegExp object with corresponding flags.

//Init
var str = "D'Or, Megan#LastName Jr., FirstName#BMW, somename#What, new";
var regex = new RegExp(',', 'g');

//replace via regex
str = str.replace(regex, '');

//output check
console.log(str); // D'Or Megan#LastName Jr. FirstName#BMW somename#What new

See that fiddle: http://jsfiddle.net/m1yhwL3n/1/ example. Thats how it will work fine for all browsers.

edited Jun 2, 2015 at 14:28

answered Mar 3, 2014 at 11:42

lin

18.5k4 gold badges66 silver badges88 bronze badges

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

Comments

athms · Accepted Answer · 2014-03-03 11:41:41Z

3

You need to use the global option in the following way:

str=str.replace(/,/g,'');

answered Mar 3, 2014 at 11:41

athms

9585 silver badges8 bronze badges

3 Comments

athms Over a year ago

The global flag used in that way will work fine in Firefox, but not across other browsers as it is non-standard. Please see the following answer: stackoverflow.com/questions/4981501/…

Anthony Grist Over a year ago

That's what I get for skimming documentation. Good to know. Going to delete my previous comment since it's now redundant.

athms Over a year ago

I can see why they introduced it, but it doesn't seem to have caught on. No worries and thanks for the upvote :)

Vinod VT · Accepted Answer · 2014-03-03 11:44:41Z

2

Try this code,

var str="D'Or, Megan#LastName Jr., FirstName#BMW, somename#What, new";
val=str.replace(/,/g, '');
alert(val);

answered Mar 3, 2014 at 11:44

Vinod VT

7,18912 gold badges54 silver badges78 bronze badges

Comments

Amit · Accepted Answer · 2014-03-03 11:41:59Z

1

Try this

var str="D'Or, Megan#LastName Jr., FirstName#BMW, somename#What, new";
str=str.replace(/,/g ,'');
alert(str)

DEMO

answered Mar 3, 2014 at 11:41

Amit

15.4k8 gold badges49 silver badges69 bronze badges

Comments

Dev G · Accepted Answer · 2014-03-03 11:46:31Z

-1

My Bad i was using comma for regex

wrong str=str.replace('/,/g','');

good str=str.replace(/,/g,'');

Thanks all for correct this..I will add points for all of you. :)

answered Mar 3, 2014 at 11:46

Dev G

1,4074 gold badges28 silver badges45 bronze badges

Collectives™ on Stack Overflow

JavaScript replace all comma in a string

Output

expected

5 Answers 5

Comments

3 Comments

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

Output

expected

5 Answers 5

Comments

3 Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related