Regex - avoid match if pattern prefixed with // - comment outs

Question

I have the following regex (JS.match) pattern /@import ['"](.*)['"]/g allowing for the external files inclusion so e.g.

@import "/my/path" returns /my/path

all works as expected but I would like to enable commenting out with // and /* */

so if string contains //@import "/my/path" or /* @import "/my/path" */ then there should be NO match.

any ideas?

possible duplicate of Javascript: negative lookbehind equivalent? — Barmar
– Barmar, Commented Dec 24, 2013 at 16:07
Something like this: /(/*|//) @import ['"](.*)['"](*/)/g, but need to escape slashes. — Parkash Kumar
– Parkash Kumar, Commented Dec 24, 2013 at 16:10
@Barmar it is not a duplicate since looking behind is one thing but it also needs to look forward due to closing */ — Marcin
– Marcin, Commented Dec 24, 2013 at 16:12
@ParkashKumar your solution is wrong as it should NOT match when // or /* — Marcin
– Marcin, Commented Dec 24, 2013 at 16:13

revo · Accepted Answer · 2014-01-18 07:19:08Z

1

(?![/*])[^/* ]@import ['"](.*?)['"]

Live demo

Update

(?![/*])[^/* ]@import ['"](.*?)['"](?![^/]*?\*\/)

Live demo

Update #2

Use below regex with m flag:

(?:(?![/*]])[^/* ]|^ *)@import ['"](.*?)['"](?![^*]*?\*\/)

Live demo

edited Jan 18, 2014 at 7:19

answered Jan 14, 2014 at 17:58

revo

49k15 gold badges84 silver badges123 bronze badges

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

2 Comments

Marcin Over a year ago

Thanks @Revo, its almost there, the only thing it doesn't account for is commenting out multi lines with /*\n\n*/ e.g. (can't add break line in comments but I hope you know what I mean) /* @import "/my/path14" \n@import "/my/path14" */

Marcin Over a year ago

@revo, almost there, just one more issue, when /* is followed by break line and then multiple imports, each on its own line, it seems to include all of them except the last one, here is an example - regex101.com/r/zP4cJ7

Collectives™ on Stack Overflow

Regex - avoid match if pattern prefixed with // - comment outs

1 Answer 1

2 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related