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, Aug 07 '12 at 13:12

hvgotcodes · Accepted Answer · 2012-08-07T15:15:50.613

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 07 '12 at 15:15

answered Aug 07 '12 at 13:06

hvgotcodes

118,147
33
203
236

3

+1 (but with single quotes you wouldn't need to escape the double quotes...) – Rob I Aug 07 '12 at 13:07
@RobI thanx, added that to my answer – hvgotcodes Aug 07 '12 at 13:09
1

Also, be suge to specify the `/g` option to match all occurences. – Austin Aug 07 '12 at 13:15
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. – Jorge Antonio Díaz-Benito Aug 07 '12 at 16:28
@user1540432, I tested it in the console; are you sure? – hvgotcodes Aug 07 '12 at 16:32
Sure, give me some time and I'll retest. – Jorge Antonio Díaz-Benito Aug 07 '12 at 16:45
Yeah you're right it worked now, I messed myself up with that extra 'comma' not properly highlighted and tried to pass the flag as an argument instead adding it to the first one. – Jorge Antonio Díaz-Benito Aug 07 '12 at 18:01

Austin · Answer 2 · 2012-08-07T13:13:14.473

4

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

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

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

edited Aug 07 '12 at 13:13

answered Aug 07 '12 at 13:06

Austin

6,026
2
24
24

score 2 · Answer 3 · answered Aug 07 '12 at 13:07

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 07 '12 at 13:07

score 0 · Answer 4 · answered Aug 07 '12 at 13:12

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

Writing double quotes in javascript string

4 Answers4