reg ex

to add > only for incompleate tag in the last part of data i did something like this
$i_sales =~ s|\<\/(.*)$|\<\/$1\>|;
but it's not working it also add > for all other line.
any tips?

i don't wont to append > to this line .
<ul> <li>something<li>something <li>something something <li>something <li>something <li>something.
but i won't append > to correct the tag.
<ul> <li>something<li>something <li>something something <li>something <li>something <li>something </ul

Find more posts tagged with

Comments

madmax

sorry it was a mis typo , the last 2 line will be---
but i like to append > to correct the tag.
<ul> <li>something<li>something <li>something something <li>something <li>something <li>something </ul

jbonifaci

s|(</)[ \t]*(\w+)([ \t]*>)?|$1$2>|g
The above expression will turn this:
<tag1> blah blah blah </tag1
<tag2> blah blah blah </tag2 blah
<tag3> blah blah blah </ tag3 blah
<tag4> blah blah blah </ tag4 >
<tag5> blah blah blah </ tag5> blah blah
<tag6> blah blah blah </tag6 blah <tag7> blah blah </tag7 blah
Into this:
<tag1> blah blah blah </tag1>
<tag2> blah blah blah </tag2> blah
<tag3> blah blah blah </tag3> blah
<tag4> blah blah blah </tag4>
<tag5> blah blah blah </tag5> blah blah
<tag6> blah blah blah </tag6> blah <tag7> blah blah </tag7> blah
Is that what you're looking for?

nipper

Pretty good Jeff, took me a while to figure out what you are doing.

However will it work if I have an open tag like:

< tag blah blah </tag>

should it be:

s|<[/])...........

?

jbonifaci

If you wanted it to work for < blah as well, you would make the following change:

s|(</?)[ \t]*(\w+)([ \t]*>)?|$1$2>|g

That will turn this:

<tag1 blah blah blah </tag1
<tag2 blah blah blah </tag2 blah
<tag3 blah blah blah </ tag3 blah
<tag4 blah blah blah </ tag4 >
<tag5 blah blah blah </   tag5> blah blah
<tag6 blah blah blah </tag6 blah <tag7> blah blah </tag7 blah

Into This:

<tag1> blah blah blah </tag1>
<tag2> blah blah blah </tag2> blah
<tag3> blah blah blah </tag3> blah
<tag4> blah blah blah </tag4>
<tag5> blah blah blah </tag5> blah blah
<tag6> blah blah blah </tag6> blah <tag7> blah blah </tag7> blah

But that of course would break things like .

Adam Stoller

There's a problem here - the lines are not XHTML compliant - they have <li>'s without </li>'s - so if you're looking to close the last open tag at the end - you'll be looking for </li-with-missing-'>' not </ul-with-missing-'>'

(I misread the original code and thought it was trying to match closing tags to opening tags - thus the above statement. The code wasn't doing that, so the above statement is wrong)

Ignoring the cases you added for <-followed-by-white-space - I believe that: s|<(\S[^>\s]+)>?|<$1>|g;Will do the trick. If you want to deal with extraneous white-space before-and-after the various bits and pieces it becomes a bit more convoluted.

FYI - my test code:use strict;
while(<DATA>){
print;
s|<(/?)\s*(\S[^>\s]+)>?|<$1$2>|g;
print;
print "\n";
}
__DATA__
<ul> blah blah blah </ul
<li> blah blah blah </ul blah
<ol><ul> blah blah blah </ul blah blah blah </ol blah

--fish
Senior Consultant, Quotient Inc.
http://www.quotient-inc.com

Edited by ghoti on 03/24/05 06:18 PM (server time).

jbonifaci

Now, if you wanted it to take in account the attributes too, that is a little messier, but you could do something like this:

s|(</?)[ \t]*(\w+)(([ \t]*\w+[ \t]*=[ \t]*['"].*['"])*)([ \t]*>)?|$1$2$4>|g

Which would take something like this:

<tag1 attr1='1' blah blah blah </tag1
<tag2   attr1="1" attr2 = '1'blah blah blah </tag2 blah
<tag3  attr1  =  "1"blah blah blah </ tag3 blah
<tag4 attr1='' attr2=""blah blah blah </ tag4 >
<tag5   attr1=" " attr2="blah"blah blah blah </   tag5> blah blah
<tag6 blah blah blah </tag6 blah <tag7> blah blah </tag7 blah

And turn it into this:

<tag1 attr1='1'> blah blah blah </tag1>
<tag2   attr1="1" attr2 = '1'>blah blah blah </tag2> blah
<tag3  attr1  =  "1">blah blah blah </tag3> blah
<tag4 attr1='' attr2="">blah blah blah </tag4>
<tag5   attr1=" " attr2="blah">blah blah blah </tag5> blah blah
<tag6> blah blah blah </tag6> blah <tag7> blah blah </tag7> blah

Now, that leaves the spacing in the opening tag a little ugly, which could probably be tweaked, but I don't care to play with it that much right now, [smile].

madmax

thanks jeff. works great.

nipper

you guys need a llife

jbonifaci

Hey now, just because I've been at work until 10 or later every day for the last couple weeks and like to take a break from my work by doing other people's work for them on devnet doesn't mean I don't have a life. Errr... Nevermind, you're right...