Replace multiline with single line in a string+Javascript

Question

I am trying to replace all multiplines with single line in a string in javascript but none is working . Below is my code :

var str=inputList.replace(/\n/gm,"\n");

input e.g.

abc,def <3 newlines>


xyz <1 newline>
opp

Expected output:

abc,def <1 newline>
xyz
opp

Actual output:

abc,def<3 newlines>


xyz<1 newline>
opp

Any help is appreciated.

You are replacing every occurrence of one \n with one \n … — C3roe
– C3roe, Commented Jan 4, 2016 at 17:37
Your goal is very unclear, and the fact you misuse the code snippet feature (it's supposed to help test your code) doesn't help — Denys Séguret
– Denys Séguret, Commented Jan 4, 2016 at 17:38
var str=inputList.replace(/\n+/gm,"\n"); ???? I am confused with the final output... So if there is only one, than you replace it with no line breaks? — epascarello
– epascarello, Commented Jan 4, 2016 at 17:39
Try var str=inputList.replace(/\n{2,}/g,"\n");. If that does not work, try also var str=inputList.replace(/(\r?\n|<br\s*\/?>){2,}/g,"\n");. — Wiktor Stribiżew
– Wiktor Stribiżew, Commented Jan 4, 2016 at 17:42

Jeremy Thille · Accepted Answer · 2016-01-04 17:51:39Z

3

(Edit : simplified version thanks to stribizhev)

If you are trying to replace two or more \n with one, try this :

var str = inputList.replace(/\n{2,}/gm,"\n");

{2,} means 2 or more

edited Jan 4, 2016 at 17:51

answered Jan 4, 2016 at 17:44

Jeremy Thille

26.4k12 gold badges48 silver badges64 bronze badges

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

2 Comments

Wiktor Stribiżew Over a year ago

There is no need escaping \n as \\n and the capturing group is redundant here. And /m modifier is also redundant.

Jeremy Thille Over a year ago

so what do you suggest? /\n{2,}/gm also seems to work, if that's what you mean

epascarello · Accepted Answer · 2016-01-04 17:44:04Z

0

You are matching just one and replacing it with one. I believe if you just add the + after the \n to match one or more. If you do not want to match just one use {2,} to match two or more.

var str=inputList.replace(/\n+/g,"\n");
                             ^

or

var str=inputList.replace(/\n{2,}/g,"\n");
                             ^^^^

answered Jan 4, 2016 at 17:44

epascarello

208k20 gold badges206 silver badges246 bronze badges

Comments

Community · Accepted Answer · 2016-01-04 20:29:04Z

0

You are replacing \n with \n in your code.

Instead do:

var str = inputList.replace(/\n/gm, "");

edited Jan 4, 2016 at 20:29

CommunityBot

11 silver badge

answered Jan 4, 2016 at 17:37

void

36.8k10 gold badges72 silver badges111 bronze badges

3 Comments

Jeremy Thille Over a year ago

I believe OP is trying to replace TWO \n or more with one \n.

Wiktor Stribiżew Over a year ago

Why use /m modifier?

Denys Séguret Over a year ago

@JeremyThille I believe we should wait to have a proper question before to guess the need

Collectives™ on Stack Overflow

Replace multiline with single line in a string+Javascript

3 Answers 3

2 Comments

Comments

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

Comments

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related