[JavaScript] mijn RegExp werkt niet

[wizzkizz]

In een editable divje (IE6) heb ik een soortement tekstverwerker gemaakt. Alles werkt perfect, behalve het importeren van teksten vanuit MS Word (XP). Hierbij staan veel te veel opmaak instructies, want ik wil dat alles aangepast wordt aan de huisstijl van de website (verdana 10pt). Hierbij moeten bestaande marges, fonts etc plaatsmaken voor de huisstijl opmaak. Fontgrootte, fontkleur, tekstopmaak (vet, cursief etc) en achtergrondkleuren mogen wel blijven staan.

Nu heb ik het volgende probleem. Om lege tags te zoeken, gebruik ik de volgende RegExp-expressie:

<("[^"]*"|'[^']*'|[^'">])*>[&nbsp;|\s]*<\/[^>]>

(Deze is gedeeltelijk overgenomen uit "Mastering Regular Expressions 2nd Edition" van O'Reilly.)

Maar ik krijg niet de gewenste output. (Evenals bij andere, theoretisch goede expressies). Ik heb een klein logje gemaakt mbv JS:

JavaScript:

Hiero de uitkomsten: 

input: 
<DIV class=article id=article_body>
    U kunt nu het artikel maken door het gewoon hier in te typen
</DIV>
<P></P>
<P>&nbsp;</P>
<P>  </P>

index: 113 
LastIndex: 126 
0: <P>&nbsp;</P>
1: P 

Gegevens van de Regular Expression: 

source: <("[^"]*"|'[^']*'|[^'">])*>[&nbsp;|\s]*<\/[^>]> 
multiline: false 
global: true 
ignoreCase: true

Het is de bedoeling dat ALLE <p>...</p> tags worden gepakt, zodat die later kunnen worden vervangen (tekst is alleen maar om te testen).
Zoals je kunt zien krijg ik echter alleen maar de 2e tag, terwijl (volgens mij) theoretisch gezien alle tags gematched zouden moeten worden.

Weet iemand een oplossing(kje) voor dit probleem?

BVD (nee, geen Margarita-gebeuren )

[ Voor 21% gewijzigd door wizzkizz op 09-08-2003 18:27 ]

[crisp]

code:

[ |\s]

hier zet je een tekenreeks in een character class; vervang dat eens door:

code:

( |\s)*

persoonlijk zou ik gebruik maken van de in javascript 1.5 geintroduceerde non-greedy modifier, en een backreference gebruiken, zoiets dus (niet getest):

code:

/<(\w+).*?>(&nbsp;|\s)*<\/\1>/g

[ Voor 14% gewijzigd door crisp op 09-08-2003 18:36 . Reden: f*cking html-rechten... ]

[wizzkizz]

Dat van die back-reference is een goed idee. Dat [ |\s] is een foutje, zie de topic zoals die er nu staat.

[crisp]

wizzkizz schreef op 09 augustus 2003 @ 18:36:
Dat van die back-reference is een goed idee. Dat [ |\s] is een foutje, zie de topic zoals die er nu staat.

bij mij ook; na een edit wordt het een gewone spatie...

[wizzkizz]

Helaas, maar jouw oplossing, Crisp, werkt ook niet. Ik krijg precies hetzelfde effect als bij die van mij.

Dat [ of ( maakte qua uitkomst ook niet uit.

[ Voor 6% gewijzigd door wizzkizz op 09-08-2003 19:53 ]

[crisp]

wizzkizz schreef op 09 augustus 2003 @ 19:20:
Helaas, maar jouw oplossing, Crisp, werkt ook niet. Ik krijg precies hetzelfde effect als bij die van mij.

Dat [ of ( maakte qua uitkomst ook niet uit.

werkt bij mij toch prima anders...
testscript

[ Voor 11% gewijzigd door crisp op 09-08-2003 20:53 ]

[wizzkizz]

crisp schreef op 09 August 2003 @ 19:51:
[...]

werkt bij mij toch prima anders...
testscript

nog niet geupload?

[wizzkizz]

Sorry, er zat (ook) nog een foutje in mijn gebruiken van de RegExp, hoewel de expressie zelf dus wel goed was. In elk geval bedankt, crisp!

[crisp]

wizzkizz schreef op 09 August 2003 @ 19:58:
[...]


nog niet geupload?

nee, typo (is mijn eigen server thuis)