Writing double quotes in javascript string

Question

I'm using a method to iteratively perform a replace in a string.

function replaceAll(srcString, target, newContent){
  while (srcString.indexOf(target) != -1)
  srcString = srcString.replace(target,newContent);
  return srcString;
}

But it doesn't work for the target text that I want, mainly because I can't think of how to properly write that text: What I want to remove is, literally, "\n", (included the comma and the quotes), so what to pass as second param in order to make it work properly?

Thanks in advance.

This is horribly inefficient and not necessary because you can use a regex with the global flag. — Esailija
– Esailija, Commented Aug 7, 2012 at 13:12

hvgotcodes · Accepted Answer · 2012-08-07 15:15:50Z

7

You need to escape the quotes, if you use double quotes for the first argument to replace

'some text "\n", more text'.replace("\"\n\",", 'new content');

or you can do

'some text "\n", more text'.replace('"\n",', 'new content');

Note in the second example, the first argument to replace uses single quotes to denote the string, so you don't need to escape the double quotes.

Finally, one more option is to use a regex in the replaceinvocation

'some text "\n", more text "\n",'.replace(/"\n",/g, 'new content');

the "g" on the end makes the replace a replace-all (global).

edited Aug 7, 2012 at 15:15

answered Aug 7, 2012 at 13:06

hvgotcodes

121k33 gold badges209 silver badges237 bronze badges

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

7 Comments

Rob I Over a year ago

+1 (but with single quotes you wouldn't need to escape the double quotes...)

hvgotcodes Over a year ago

@RobI thanx, added that to my answer

Austin Over a year ago

Also, be suge to specify the /g option to match all occurences.

Jorge Antonio Diaz-Benito Over a year ago

The '/g' flag solution doesn't work. However, that information about the single quotes to denote the string has allowed me to solve the problem using my custom although unefficient replaceAll method.

hvgotcodes Over a year ago

@user1540432, I tested it in the console; are you sure?

|

Austin · Accepted Answer · 2012-08-07 13:13:14Z

4

To remove "\n", simply use String.replace:

srcString.replace(/"\n"[,]/g, "")

You can replace using the Regular Expression /"\n"[,]/g

edited Aug 7, 2012 at 13:13

answered Aug 7, 2012 at 13:06

Austin

6,0542 gold badges26 silver badges24 bronze badges

Comments

user657496 · Accepted Answer · 2012-08-07 13:07:06Z

2

There is no need for such a function. The replace function has an extra parameter g, which replaces ALL occurrences instead of the first one:

'sometext\nanothertext'.replace(/\n/g,'');

answered Aug 7, 2012 at 13:07

user657496

Comments

jackwanders · Accepted Answer · 2012-08-07 13:12:50Z

0

Regardless of whether the quotes within the string are escaped or not:

 var str = 'This string has a "\n", quoted newline.';

or

var str = "This string has a \"\n\", escaped quoted newline.";

The solution is the same (change '!!!' to what you want to replace "\n", with:

 str.replace(/"\n",/g,'!!!');

jsFiddle Demo

answered Aug 7, 2012 at 13:12

jackwanders

16.1k3 gold badges42 silver badges40 bronze badges

Collectives™ on Stack Overflow

Writing double quotes in javascript string

4 Answers 4

7 Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

7 Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related