[php] Mein BBCode verlangsamt meine Seite erheblich

[Yannik Hampe]

Matthias Schlich wrote:
> Hallo liebe PHP-Gemeinde,
Hallo Matthias,

> ich hoffe ihr k�nnt mir bei meinem Problem helfen.
> Ich betreibe auf meinem root-sever eine kleine Community.
> Dabei setze ich ein CMS ein (Ilch - www.ilch.de), das keinen richtigen 
> bbcode nutzt,
> daher hab ich ein "fremdes" Modul installiert, dass diese L�cke schlie�t.
> Nun zu meinem Problem: Wenn ich �ber den uns allbekannten code 
> [img]bild.jpg[/img]
> ein Bild einf�gen m�chte, so pr�ft der bbcode zuerst die pixel-gr��e des 
> Bildes um es
> evt zu verkleinern wenn es eine gewisse h�he und breite �berschreitet.
> Diese Funktion arbeitet nat�rlich mit 'getimagesize' und verlangsamt 
> damit meine Seite
> ERHEBLICH wenn server-externe Bilder geladen werden. Liegen die Bilder 
> jedoch auf meinem Server, arbeitet das System in einer sehr guten 
> geschwindigkeit.
> Ich will jetzt nat�rlich den bbcode erhalten und meine Seite schneller 
> machen.
> Meinen Usern zu sagen sie sollen alle Bilder die sie im Forum posten auf 
> meinen Server zu laden,
> ist nat�rlich auch nicht wirklich eine Option.
> Dazu habe ich einmal den betreffenden Code hochgeladen unter:
> http://www.matthias-schlich.de/bbcode_img.txt
> Es handelt sich hier nur um die img-funktion. Falls gew�nscht, kann ich 
> den Rest des BBCodes auch hochladen.
> Au�erdem ist mir folgendes aufgefallen: Das CMS l�d alles von oben nach 
> unten der Reihenfolge nach. Also wenn man im Forum einen Artikel mit 
> Bildern aufruft, l�d das Script den Text nur bis zum ersten Bild aus der 
> Datenbank, dann k�mmert es sich um das Bild und erst wenn das Bild 
> vollst�ndig geladen hat, geht es mit dem Text weiter. Eine erhebliche 
> Erleichterung w�re es, wenn zu erst der gesammte Text laden w�rde und 
> sich das Script erst dann um das Bild k�mmern k�nnte, so wie in jedem 
> anderen bulletinboard auch.

Es ist grunds�tzlich schonmal eine schlechte Idee den BBCode JEDESMAL zu
parsen, wenn er angezeigt werden soll.
Das ist n�mlich entweder viel zu langsam (regex,...) oder unsicher
(str_replace()).
Du solltest beim editieren eines Eintrags den Eintrag "vorparsen".
phpBB macht das Beispielsweise so:

Text des users:
$text ='bla [b]blubb[/b] foobar';
$randomid ='zufall'; //bei phpBB ist das ein zuf�lliger Hexwert.
$text
=preg_replace('/\\[b\\](.*?)\\[/b\\]/',"[b:$randomid]\\1[/b:$randomid]",$text);

In $text steht jetzt:
bla [b:zufall]blubb[/b:zufall] foobar

$text und $randomid speicherst du dann in der Datenbank.
Und beim Anzeigen musst du nurnoch:
str_replace(Array("[b:$randomid]","[/b:$randomid]"),Array('<b>','</b>'),$text);
machen.
Das ist deutlich schneller als das regex JEDES MAL beim anzeigen wieder
anzuwenden. Trotzdem kannst du aus dieser Zwischenform auch wieder den
bbcode erstellen, den der User am Anfang erstellt hat.

Was soll die $randomid??
Stell dir vor du hast
$text ='[b]';
Dann w�rde str_replace dir da den �ffnenden Tag <b> hinschreiben, aber
der Tag w�rde nie wieder geschlossen werden. So kannst du dir das
aussehen der ganzen Seite kaputt machen, oder schlimmeres. Die randomid
verhindert das. Denn da diese bei jedem editieren neu generiert wird,
ist sie unbekannt f�r den Verfasser und der muss es erst mal schaffen
die randomid zu treffen. Dies ist keine 100%-L�sung, aber je nach l�nge
der $randomid eine 99,999999999999%-ige. Und der Performancegewinn ist
es wert.

Bei Bildern ist das nat�rlich schon etwas schwieriger, denn du musst die
Gr�sse deines Bildes irgendwie speichern. Das ist der Punkt, wo phpBB
aufgibt und doch wieder preg_replace nimmt. Aber unm�glich w�re es
nicht. Du kannst zum Beispiel etwas in der Form generieren:
[img:$randomid]src="source" width="breite" height="h�he"[/img:$randomid]
mit
str_replace(Array("[img:$randomid]","[/img:$randomid]"),Array('<img
','>'),$text);
Den html-Text generieren...
preg_replace("/\\[img:$randomid]src=".*?"\\[/img:$randomid\\]/",'[img]\\1[/img]');
kannst du dann wieder den bbcode generieren.
(Den Teil habe ich mir gerade so im Kopf zusammen gebastelt. Also keine
Garantie! Absolut ungetestet!).
(Oder du kannst bei dem Schutz darauf verzichten, dass der User das Bild
nach dem erstellen des Beitrags durch ein riesen Teil austauscht. Oder
einfach vorschreiben, dass das Bild auf deinem Sever liegen muss, damit
der user das Bild nicht einfach austauschen kann.)

Zu guter letzt: Wenn ich mir den qt ansehe, den du uns zur Verf�gung
stellst, stelle ich fest, dass der Autor scheinbar grunds�tzlich kein
Performanceprofi war... Also schreib' es besser selber neu, oder nimm
ein anderes Plugin...

Yannik