regular expression to remove comment javascript

Question

I am using below regular expression to remove comments from string

<\!{1}\-{2}(.*?)\-{2}\s*>

This is working fine except for mult-iline string

var search = '<\!{1}\-{2}(.*?)\-{2}\s*>';

  var re = new RegExp(search, "gm");

  var subject = <multi-line string>;
  result = subject.replace(re, '');

what should I do to get it working with multiline strings

A bad idea. Consider: var s = ""; — Bart Kiers
– Bart Kiers, Commented Jul 9, 2010 at 13:18

Bart Kiers · Accepted Answer · 2010-07-09 13:29:39Z

2

. does not allow linebreaks.

This one should work:

^(<\!\-{2})((.|\s)*?)\-{2}>$

Fix:

<!--[\S\s]*?-->

I removed the \s at the beginning and the end of the expression and added it in the middle so multiline-comments are allowed.

But you shoud have a look at BartKs comment ;)

regards

edited Jul 9, 2010 at 13:29

Bart Kiers

171k38 gold badges307 silver badges297 bronze badges

answered Jul 9, 2010 at 13:19

Philipp Grathwohl

2,8763 gold badges28 silver badges38 bronze badges

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

3 Comments

Bart Kiers Over a year ago

IMO, -- is more to the point than -{2}, and (.|\s) could be replaced by [\s\S]. Also, ! and - don't need escaping.

MikeD Over a year ago

For the meat of the expression, wouldn't we just want ([^\-]*), question mark. (I would have felt bad ending this comment with a '?' when it could easily be construed as part of the regex, which is not the case.) Edit: It would fail on , I haven't had my morning coffee.

Bart Kiers Over a year ago

I made a slight edit: the class [\S|\s] does not need to contain the | (OR): inside a character class, | just matches the literal "|". Also, The regex doesn't need to be anchored with ^ and $ and [\s\S] does not need to be grouped by parenthesis. But @Sourabh should be aware that replacing comments like that could result in unexpected behavior as I commented under his question.

Collectives™ on Stack Overflow

regular expression to remove comment javascript

1 Answer 1

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related