Javascript regex - get string between two matches, multiple times

Question

I'm trying to remove all style rules like this <style>blablabla</style> including <style> tag. I'm using the following script to accomplish this

  var text = '<   style  type= "text/css"  > hello1  <  /  sTyle  > 
               some text
              <   style  type= "text/css"  > hello2  <  /  sTyle  >';

  var regX = /< *style .*>([\s\S]*?)< *\/ *style *>/ig;
  text.replace(regX, "");

but it removes the whole text including "some text" between two style definitions. I want to keep "some text" in this example.

Paul Draper · Accepted Answer · 2014-04-12 16:26:30Z

1

The problem is here

< *style .*>

The .* matches everything.

Change your regex to

/< *style .*?>([\s\S]*?)< *\/ *style *>/ig

or

/< *style [^>]*>([\s\S]*?)< *\/ *style *>/ig

Note, however that currently your regex misses cases, for example

<style></style>

You may consider using an actual HTML parser.

edited Apr 12, 2014 at 16:26

answered Apr 12, 2014 at 16:17

Paul Draper

84.2k53 gold badges216 silver badges303 bronze badges

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

Comments

tewathia · Accepted Answer · 2014-04-12 16:51:11Z

1

It's best to avoid this .* greedy fellow in a RegEx. Try something like:

text.replace(/< *style[\w"= \/]*>[\w ]*<[ \/]*style *>/ig, '')

DEMO

edited Apr 12, 2014 at 16:51

answered Apr 12, 2014 at 16:32

tewathia

7,3283 gold badges25 silver badges28 bronze badges

Collectives™ on Stack Overflow

Javascript regex - get string between two matches, multiple times

2 Answers 2

Comments

Comments

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Related