How to replace all \" in a JS string?

Question

How to replace all \" to " in a string?

I tried, but it doesn't works: var foobar = ("foo\\\"bar\\\"foo").replace(/"\\\""/,'"');

The result is foo\"bar\"foo , but it should be foo"bar"foo

Madara's Ghost · Accepted Answer · 2012-04-23 15:01:05Z

5

You don't need to use quotes inside of a RegEx pattern, the // delimiters act as ones.

var foobar = "foo\\\"bar\\\"foo".replace(/\\"/g,'"');

Works for me.

answered Apr 23, 2012 at 15:01

Madara's Ghost

176k52 gold badges274 silver badges314 bronze badges

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

Comments

Niet the Dark Absol · Accepted Answer · 2012-04-23 15:00:50Z

5

Try .replace(/\\"/g,'"'); - regexes don't need quotes around them, I'm surprised you get any result at all.

answered Apr 23, 2012 at 15:00

Niet the Dark Absol

326k86 gold badges480 silver badges604 bronze badges

2 Comments

Oleg V. Volkov Over a year ago

Well, why not? He is trying to replace something that doesn't exists in original string, so he just gets it back unmodified.

Niet the Dark Absol Over a year ago

That's what I meant, but for some reason I didn't type that >_>

Madara's Ghost · Accepted Answer · 2012-04-23 15:06:32Z

5

You need to fix your regex, you need to do

replace(/\\\"/g, "\"")

edited Apr 23, 2012 at 15:06

Madara's Ghost

176k52 gold badges274 silver badges314 bronze badges

answered Apr 23, 2012 at 15:04

slowpoison

6165 silver badges21 bronze badges

2 Comments

Madara's Ghost Over a year ago

Please format your code properly. A code block is inserted by indenting 4 spaces before any line of code. I've formatted the code for you this time, but please format it properly next time. For further help, see the Editing FAQ

Madara's Ghost Over a year ago

My bad, confused about your multiple slashed there for a minute (Note that you don't need to escape quotes inside of a regex pattern)

Felix Kling · Accepted Answer · 2012-04-23 15:01:51Z

3

Your quoting is wrong and you're not using g - global flag. It should be:

var foobar = ("foo\\\"bar\\\"foo").replace(/\\"/g,'"');

edited Apr 23, 2012 at 15:01

Felix Kling

820k181 gold badges1.1k silver badges1.2k bronze badges

answered Apr 23, 2012 at 15:01

Oleg V. Volkov

22.6k4 gold badges49 silver badges68 bronze badges

Comments

MilkyWayJoe · Accepted Answer · 2012-04-23 15:02:07Z

3

Try defining it like this

var foobar = ("foo\\\"bar\\\"foo").replace(/"\\\""/g,'"');

note that the .replace has a /g which makes it global

answered Apr 23, 2012 at 15:02

MilkyWayJoe

9,1122 gold badges41 silver badges53 bronze badges

1 Comment

Madara's Ghost Over a year ago

This will fail. No quotes around regex patterns.

Ryan Fiorini · Accepted Answer · 2012-04-23 15:05:11Z

1

jsfiddle

// initial string
var str = "AAAbbbAAAccc";

// replace here
str = str.replace(/A/g, "Z");
alert(str);

answered Apr 23, 2012 at 15:05

Ryan Fiorini

7985 silver badges9 bronze badges

3 Comments

MilkyWayJoe Over a year ago

Why the down vote on this one? It replaces the "A"s for "Z"s and shows how to use the replace...

Madara's Ghost Over a year ago

@MilkyWayJoe: Hover over the downvote icon, "This answer is not useful" yeah, pretty much sums it up for me. While it does show the correct use of str.replace() it doesn't, in fact, help in any way to the OP.

Ryan Fiorini Over a year ago

I must admit. This is a really bad answer to the question. I must have been half sleeping when I wrote this. :)

Collectives™ on Stack Overflow

How to replace all \" in a JS string?

6 Answers 6

Comments

2 Comments

2 Comments

Comments

1 Comment

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

6 Answers 6

Comments

2 Comments

2 Comments

Comments

1 Comment

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related