• 0

Regex c# remove white space but not inside <pre> and </pre>


Question

Hi all

 

I am trying to write some regex to remove all white space from html.

 

but the regex i am currently using doesn't factor in that in the pre tags there maybe opening "<" and closing ">" tags

 

This matches everything inside the pre tags

 

(<)\s*?(pre\b[^>]*?)(>)([\s\S]*?)(<)\s*(/\s*?pre\s*?)(>)

 

e.g.

(009)156 (010) (0)<pre> <test> edehofo<w<dieoj >  ></pre>     yuui u    ji 

will match 

<pre> <test> edehofo<w<dieoj >  ></pre>

 

and 

(?<=\s)\s+(?![^<>]*</pre>)

 

eg will almost work but does not work if there is an "<" or ">" in the mark up.

 

space[     ]spaces <pre>[          ]spaces</pre>space[      ]spaces 

 

will result in 

space[ ]spaces <pre>[ ]spaces</pre>space[ ]spaces 

 

but if there is a "<" or ">" in the pre tags then it will not work.

 

Could anyone help me

 

 

 

 

 

 

 

 

 

 

10 answers to this question

Recommended Posts

  • 0

A warning: http://stackoverflow.com/questions/1732348/regex-match-open-tags-except-xhtml-self-contained-tags

 

A solution: http://stackoverflow.com/questions/8762993/remove-white-space-from-entire-html-but-inside-pre-with-regular-expressions (same as your regex)

 

Just be careful with regex and HTML, parsing it to a C# object and then outputting it without whitespace is probably a better way to do this (likely to be slower though).

 

You shouldn't include < or > in <pre>, they should be escaped to &lt; and &gt;

  • 0

Thanks, but I've already been to those sites and a lot of others trying to find an solution.

 

From that its highlighted that I maybe able to do it in to step then...

 

1. encode all html with in pre tags

2. then use (?<=\s)\s+(?![^<>]*</pre>) to remove white space.

 

I am finding it hard to now find regex which will encode html with in a pre tag.

I love cats and dogs but dogs > than cats

<spaces>       <spaces>
<pre>
    1 > 0  = ?

    <code>       dedw</code>
</pre>

should be 

I love cats and dogs but > than cats <spaces> <spaces>
<pre>
    1  &gt; 0 = ?
    &glt;code&gt;       dedw &gt;/code &gt;
</pre>

It would be easier if i had some regex which basically ignored everything between a given tag.

 

I am doing my own research and trying to find the answer... but thought i would clarify what I am trying to do.

 

Thanks if you can help me find a solution

  • 0

Hi Eric

 

I'm trying to do it for c#

 

Basically I can get what i want if I can strip out white-space but not between two sets of chars

 

e.g.  <pre> anything </pre> or <code> anything </code>

 

seems more challenging that i thought it would be.

 

 (?<=\s)\s+(?![^<>]*</pre>)  is awesome but if i could get it to basically ignore everything inside of a given tag

 

My understanding of how to get the right syntax for regex is limited.

 

Match everything but not between tag(pre or code) and remove white space.

 

any help with getting this to work would be so awesome.

  • 0

How about the following: http://snipd.net/parsing-xhtml-into-a-dom-tree-in-c

You avoid using regex as you simply parse then print the HTML instead.

 

 

You might also want to re-read my first link, which points out that you cannot parse arbitrary (X)HTML using a regex. It is not possible as regex is not powerful enough to parse (X)HTML. (Parsing a limited subset is possible)

  • 0

to anyone who is good with regex please help

 

(?<=\s)\s+(?![^[<p]+?[e>])

 

cant not get this to work but must be close?

 

to: Lant

 

Sorry but i don't need to parse the HTML... i am not trying to validate it or read it really... 

 

I am wanting to remove white space but only when outside of a given tag like say <pre> or <code>

 

It has been working fine with (?<=\s)\s+(?![^<>]*</pre>)  until i hit a new requirement which.... is that the <pre> tag can contain < or >

 

which this regex does not support. One suggestion was to replace < and > with their HTML versions &lt; and &gt;

 

but again i would need to have something which only applied this on html with in tag the <pre> tag and I would need to do it in c#

 

seems easier to just get the regex to ignore things between tags some how...

 

Thanks

  • 0

If you don't mind doing the replace in C#, you can use capture groups with this pattern.

(?:\<pre\>)(.*)(?:\<\/pre\>)

It should capture everything between the pre tags and the value of the capture can then be string.Replaced on. I am working on a Regex replace version so you can use RegexOptions.Compiled. This regex was tested with CaseInsensitive and SingleLine (despite it being multi-line, using that to adjust how it handles .*)

 

1cKDh7e.png

 

 

so for example:

 

private static Regex preBlockMatch = new Regex(@"(?:\<pre\>)(.*)(?:\<\/pre\>)", RegexOptions.Compiled | RegexOptions.Singleline | RegexOptions.IgnoreCase);

 

(code block)

string newValue = preBlockMatch.Captures[0].Value.Replace("<", "&lt;");

(code block)

 

obviously room for some enhancement but it gets you started

  • 0

Thought I would share what I got working......

 

Also thanks!!! to Squirrelington for the regex for the between stuff

 public class MinifiedStream : MemoryStream
        {
            private readonly Stream _output;
            public MinifiedStream(Stream stream)
            {
                _output = stream;
            }

            private static readonly Regex Whitespace = new Regex(@"(?<=\s)\s+(?![^<pre>]*</pre>)",RegexOptions.Compiled);

            private static readonly Regex PreBlockMatch = new Regex(@"(?:\<pre\>)(.*)(?:\<\/pre\>)", RegexOptions.Compiled | RegexOptions.Singleline | RegexOptions.IgnoreCase);

            Dictionary<string, string> result = new Dictionary<string, string>();

            public override void Write(byte[] buffer, int offset, int count)
            {
                var html = Encoding.UTF8.GetString(buffer);

                //------------------------------------
                var matches = PreBlockMatch.Matches(html);
                int loopcount = 1;
                foreach (Match match in matches)
                {
                    var token = Guid.NewGuid().ToString() + "_"+ loopcount;
                    html = html.Replace(match.Value, token);
                    result.Add(token, match.Value);
                    loopcount ++;
                }
                //------------------------------------



                html = Whitespace.Replace(html, string.Empty);
                html = html.Trim();

                //-----------------------------------
                foreach (var match in result)
                {
                    html = html.Replace(match.Key, match.Value);
                }

                //-----------------------------------

                _output.Write(Encoding.UTF8.GetBytes(html), offset, Encoding.UTF8.GetByteCount(html));
            }
        }

But still feels as if you should be able to create a straight regex version.

 

if anyone can help me improve that would be awesome

  • 0

I agree that there should be a way to get it exclusively in regex but I am only really good at matching things, I don't have a lot of experience in replacement syntax. I know you can do $1 for the first group, $2 for 2nd or ${name} for named groups but how to modify the contents of those groups in the replacement is beyond me atm. Maybe someone with more experience will be able to chime in. :) Glad it is working thus far though. \o/

 

btw the regex I gave you, I probably went a little too crazy with the non-capture groups. I think at the time I was trying some funky magic to try and capture parts of it for replacement purposes but ended up with that.

<pre>(.*)</pre>

would probably be fine as well. lol

This topic is now closed to further replies.
  • Recently Browsing   0 members

    • No registered users viewing this page.
  • Posts

    • Honestly, I think the long-term play here is for Microsoft to ditch the idea of a traditional console entirely and just turn Xbox into a full-on operating system. They (or anyone!) could release hardware like a Mac Mini or a typical console with built-in GPU and RAM, but instead of being locked into a console ecosystem, they run the Xbox OS. It makes total sense because it pushes Game Pass, Windows, and all their other software. The handheld angle is really interesting too. You’d basically have a portable PC. Hook up a keyboard and mouse, and suddenly you can edit videos or get some real work done while on the go. Something like a prebuilt Xbox PC would be more than just a gaming box. It could be a decent little PC that people might actually upgrade with each generation. And since it’s running a PC-based OS, you’d get all the usual perks like mod support, cheaper games across different stores, and no extra charges just to plug in a webcam or other standard accessories. Plus, if they let you install the OS on your own rig, then you’ve got full upgradability too. Best bit also being Microsoft wont even need to build them anyone can and when they do Microsoft just wins. The competition is kind of stuffed. Those maybe relying on SteamOS might be ok but will still have software support issues and no Game Pass. Sony and Nintendo can’t really offer the same kind of flexibility at all not sure how they will live on. They’d struggle to match something that works as a desktop OS, console, handheld, streaming box, and media hub, all with your game library ready to go, never needing to rebuy games. Console exclusives are the only way they can live I think, but if they ever get blocked by antitrust rulings down the line, it’s game over. I'm waiting to see the bloat that's still left on these and if they let you install on a regular old PC. Fingers crossed this could be the lightweight OS we have been asking for.
    • I take my Apple Watch off at night and put it on the charger when I go to bed. Then I wake up the next morning I put it back on.
    • I don’t get what an ai device like a pendant does for me. As a man, I’m not wearing pendant. Then there is battery life. Battery life on my Apple Watch is horribly embarrassing. Having to charge every 36 hours is horribly inconvenient. Fitbits that hold a charge for 5-7 days are much better, so the pendant would need that level of charge. Voice is actually jarring for me. I find it inconvenient when people talk into their phones and I overhear their request. Speaker phones are fine in a closed office but not when others are around. If the device is screen less, it must either respond vocally or respond via the screen on my phone. if this another health device, does the marketplace need another?
  • Recent Achievements

    • Reacting Well
      NeoToad777 earned a badge
      Reacting Well
    • Reacting Well
      eric79XXL earned a badge
      Reacting Well
    • First Post
      brynmot earned a badge
      First Post
    • Reacting Well
      brynmot earned a badge
      Reacting Well
    • Week One Done
      Al_ earned a badge
      Week One Done
  • Popular Contributors

    1. 1
      +primortal
      479
    2. 2
      +FloatingFatMan
      277
    3. 3
      ATLien_0
      243
    4. 4
      snowy owl
      209
    5. 5
      Edouard
      187
  • Tell a friend

    Love Neowin? Tell a friend!