Regular Expressions...

[ohrflieger]

Abend,

'/\[download=([^ ]+).*\](.*)\[\/download\]/'

Ein regulärer Ausdruck für eigenen "BBCode". Der Ausdruck soll eigentlich Download-Links umwandeln zu:

'<a class="downloadlink" href="download.php?file=$1">$2</a>'

Aber wenn man mehr als einen Downloadlink setzt, nimmt PHP leider den Inhalt von beiden als File. Beispiel: PHP macht aus dem hier

[download=file1.txt]File 1[/download][download=file2.txt]File 2[/download]

das hier

<a class="downloadlink" href="download.php?file=file1.txt]File 1[/download][download=file2.txt">File 2</a>

Wie kann man verhindern, dass beide Links zusammengefasst werden?

 

[Voodoo]

Das liegt am grundsätzlich "gierigen" Verhalten von Regulären Ausrücken. Das bedeutet ein * oder + belegt so viele Zeichen, wie möglich.
Der RegExp-Evaluator - RegExp-Tutorial - Modifier
Um dem entgegenzuwirken kannst du den Modifier U setzen oder ein Fragezeichen (siehe Codeschnipsel). Ich habe deinen Ausdruck mal ein wenig modifziert. Grundsätzlich ist es besser den Bereich der erlaubten Zeichen vorzugeben, als alle nicht erlaubten zu verbieten - dann hat man auch nicht so viele Probleme mit Greed. Ich habe außerdem mal eine Backreference (\1) gemacht. So kann man den Ausruck leicht erweitern.

#\[(download|url)=([A-z0-9\.-]+)\](.+?)\[/\1\]#is

Gruß
Voodoo

 

[ohrflieger]

Danke, funktioniert wunderbar! :)
Deinen Link hatte ich vorher schon besucht und es mit dem u-Modifier versucht, dann allerdings wurde mir gar nichts mehr ausgegeben (keine Ahnung warum).

 

[crash]

Wichtig: In PHP sind Regexe per Voreinstellung gierig. Das trifft seltener auf andere Sprachen zu.

 

[Gelöschtes Mitglied 3007]

#\[(download|url)=([A-z0-9\.-]+)\](.+?)\[/\1\]#is

dieser ausdruck ist zwar für den download tag zu gebrauchen für den url tag aber eher weniger da keine urls angegeben werden können

also nimm lieber den hier für downloads

/\[download=([A-z0-9\.-]+)\]([^\]]+)\[\/download\]/isU

dabei muss nur in der zweiten variable auf sicherheit geachtet werden, du könntest da aber auch sagen das da z.b. nur buchstaben und leerzeichen drins ein dürfen, das würden dann so aussehen

/\[download=([A-z0-9\.-]+)\]([A-z ]+)\[\/download\]/isU

für urls kannst du diesen nehmen da musst du aber bei beiden parametern auf sicherheit achten

/\[url=([^\]]+)\]([^\]]+)\[\/url\]/isU