[Erledigt] Eigener BBCode mit mehreren Optionen - Fehler

Arego · 06.03.2009, 17:30

Ich hab hier ein problem mit einem Parser zu einem sehr schönen eigenen BBcode.

Erstmal worums geht:
im BBcode kann man verschiedene Modifikatoren angeben.
zb:
BBcode (Standard):
[test]*text*[/test]

BBcode mit zusatz1:
[test]*text*[/test rev]

BBcode mit zusatz2:
[test]*text*[/test max=*param*]

BBcode mit zusatz1 und zusatz2:
[test]*text*[/test max=*param* rev]

BBcode mit zusatz3 :
[test=*parameter*]*text*[/test]

BBcode mit zusatz3 und zusatz1 :
[test=*parameter*]*text*[/test rev]

BBcode mit zusatz3 und zusatz2 :
[test=*parameter*]*text*[/test max=*parameter*]

BBcode mit zusatz3 und zusatz2 und zusatz1 :
[test=*parameter*]*text*[/test max=*parameter* rev]

Das sind alle möglichkeiten wie der BBcode aussehen kann...

Der Parser:

PHP-Code:

      $s = preg_replace("/\[test\]((\s|.)+?)\[\/test\]/i", 'Parser1', $s); 
    $s = preg_replace("/\[test\]((\s|.)+?)\[\/test ([r][e][v])\]/i", 'Parser2', $s);
    $s = preg_replace("/\[test\]((\s|.)+?)\[\/test max=([0-9]+?)\]/i", 'Parser3', $s);
    $s = preg_replace("/\[test\]((\s|.)+?)\[\/test max=([0-9]+?) ([r][e][v])\]/i", 'Parser4', $s);
    $s = preg_replace("/\[test=([a-f0-9]+?)\]((\s|.)+?)\[\/test\]/i", 'Parser5', $s);
    $s = preg_replace("/\[test=([a-f0-9]+?)\]((\s|.)+?)\[\/test ([r][e][v])\]/i", 'Parser6', $s);
    $s = preg_replace("/\[test=([a-f0-9]+?)\]((\s|.)+?)\[\/test max=([0-9]+?)\]/i", 'Parser7', $s);
    $s = preg_replace("/\[test=([a-f0-9]+?)\]((\s|.)+?)\[\/test max=([0-9]+?) ([r][e][v])\]/i", 'Parser8', $s);

Die Beispiele von oben sind hier jetzt mal mit dem Parser geparst und als screenshot, wie das dann aussieht wenn es funktioniert; das Obere im Roten Rahmen ist der verwendete BBcode (vor dem parsing das oben angegeben ist) und das darunter im Grünen Rahmen hinter dem zensierten Namen ist das Ergebnis gemäss parser. Zur verdeutlichung habe ich die zugehörigen auch nich mit einer gelben linie verbunden.

Das sind jetzt die Ergebnisse aus den Beispielen die ich oben angeführt habe.

Nun nehme ich anstatt *text* einen richtigen text der Lautet:

Zitat:

Das ist der Text // Option: reversiv // Bildung:[ test ] deintext [/test rev ]

Dieser ist jetzt im nächsten Test auch immer gleich, nur der BBcode wird variiert:

Wie hier schon auffällt gehen alle BBcodes mit Modifikatoren/Parametern nicht mehr und sind LEER. Wie ich herausgefunden habe, ist das $s im parser durch den diese teile laufen bereits nach der ersten parsingzeile von [test] leer, sprich nach:

PHP-Code:

     $s = preg_replace("/\[test\]((\s|.)+?)\[\/test\]/i", 'Parser1', $s);

und das AUCH wenn dieser gar nicht verwendet, bzw angesprochern wird (&$count liefert 0 zurück, dh keine ersetzung). Unmittelbar vor diesem ersten
parser ist der BBcode noch da.

Zur weiteren Info
vor dem parsing wird $s mit htmlspecialchars gefiltert
Weiterhin habe ich zum Debug hier den test-text

Zitat:

Das ist der Text // Option: reversiv // Bildung:[ test ] deintext [/test rev ]

nochmal in 2 teile geteilt und damit wiederum alle Möglichkeiten durchgetestet um fehler im Satz auszuschließen (zb. weil vll die sonderzeichen, sprich Eckigen Klammern nicht gehen?)
Screen mit Testreihe Teil1 vom Satz:

Screen mit Testreihe Teil2 vom Satz:

Ich hoff, ich hab das Problem ausführllich genug dokumentiert und ihr versteht mein (meiner meinung nach) etwas komplexes und komisches
Problem.

Ps.: Die tatsächlichen Ersetzungen habe ich rausgenommen weil diese eh keine rolle spielen und auch zu lang sind, auch wenn die ersetzung leer ist oder irgendwas drin ist was auf jeden fall gehen muss (zb. abc), geht es nicht, daher wende ich mich dem pattern als Fehlerursache zu.

Ps2.: Die angeführten Beispiele funktionieren mit dem angegebenen Parser, dh auch zum selbertesten wo die ursache sein könnte, ich habe sie bid jetzt nicht herausgefunden und forsche seit ca 12 h nach dem fehler.

Vermute dass es vll auch an den Leerzeichen zwischen den argumenten liegt, wüsste aber nicht warum und wieso und wie man es besser machen könnte

nikosch · 06.03.2009, 18:44

[MOD: verschoben]

Arego · 07.03.2009, 20:05

Ich habe nun einen neuen kompletten Parser geschrieben.

Damit kann das Thema als erledigt angesehen werden.
Hilfe habe ich aber trotzdem gehabt, leider nicht von euch.

nikosch · 07.03.2009, 20:07

Ich fand die Frage ehrlich gesagt auch relativ unverständlich. Nächstes mal aufs Problem reduzieren und unbedingt Crosspostings als solche markieren!

php.de - Ankündigungen im Forum : PHP Einsteiger

Arego · 07.03.2009, 22:38

Naja also ich hatte 20 h keine EINZIGE antwort bekommen... und das andere forum hat auch nicht wirklich gefunden...

Ich habe mit einem meiner kleinen Kontakte telefoniert der mir geraten hat einen komplett neuen Parser zu schreiben.

Ich weiß schon was Crossposting ist, aber wenn nach gut einem Tag keine einzige Antwort erfolgt ist, nicht mal ein Tipp, kann mans echt vergessen.

Für Unannehmlichkeiten entschuldige ich mich.

nikosch · 07.03.2009, 22:40

Ich meine auch nur, Du sollst bitte das andere Forum zukünftig verlinken, damit wir fremde Lösungsstände berücksichtigen können. Ansonsten ist ein Thread auch schon mal deswegen dicht.

Zitat:

Ich habe mit einem meiner kleinen Kontakte telefoniert der mir geraten hat einen komplett neuen Parser zu schreiben.

Leider ist das oft die beste Lösung.

nikosch · 07.03.2009, 22:42

PS: Ich finde die Lösung mit den Optionen im Endtag irgendwie unsinnig. Über lange Markierungen sucht man sich imho dämlich nach Problemen und schlecht zu parsen ist das auch noch. Hast Du das jetzt wieder so drin?

Arego · 09.03.2009, 08:21

Nein....

aus den 8 Einzelnen sind jetzt nur noch zwei geworden,
einer ohne optionen und einer mit Optionen, die jeweils nur vorne angegeben werden und im komplettpaket an eine neue Funktion übergeben werden die die Optionen Trennt und richtig zugeordnet.
Praktisch ein Parser für die Optionen extra im parser für den BBcode.

Also so:
$text = preg_replace("/\[test=((\s|.)+)\]((\s|.)+)\[test\]/ie", "preg_parse_test('\\2','\\1')");

in dem ersten
((\s|.)+)
stehen alle Optionen egal welche und wieviele.

nikosch · 09.03.2009, 18:00

Leere Tags werden bei Dir nicht geparst?

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Thema bewerten
Thema bewerten:

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
Fehlermeldung - aber kein fehler...	Tschuu	HTML, Usability und Barrierefreiheit	16	14.03.2005 15:56
[Erledigt] mysql fehler		PHP Tipps 2004	2	03.11.2004 00:32
array_push nur in begrenzter Anzahl ausführen ?		PHP Tipps 2004	2	07.09.2004 09:05


Mister Ad PHP Code Flüsterer Registriert seit: 21.08.2005 Beiträge: 4682 PHP-Kenntnisse: Fortgeschritten

06.03.2009, 18:44
nikosch moderatives Dielektrikum Registriert seit: 21.05.2008 Beiträge: 35.989 PHP-Kenntnisse: Fortgeschritten	[MOD: verschoben] __________________ -- One pixel is still too big. Please make it smaller. ASAP. Initiative Mittelstand. Die wichtigste Gestaltungsregel im Screendesign ist Pi mal Daumen des Arbeitgebers. --

07.03.2009, 20:05
Arego Benutzer Registriert seit: 06.03.2009 Beiträge: 30	Ich habe nun einen neuen kompletten Parser geschrieben. Damit kann das Thema als erledigt angesehen werden. Hilfe habe ich aber trotzdem gehabt, leider nicht von euch.

07.03.2009, 20:07
nikosch moderatives Dielektrikum Registriert seit: 21.05.2008 Beiträge: 35.989 PHP-Kenntnisse: Fortgeschritten	Ich fand die Frage ehrlich gesagt auch relativ unverständlich. Nächstes mal aufs Problem reduzieren und unbedingt Crosspostings als solche markieren! php.de - Ankündigungen im Forum : PHP Einsteiger __________________ -- One pixel is still too big. Please make it smaller. ASAP. Initiative Mittelstand. Die wichtigste Gestaltungsregel im Screendesign ist Pi mal Daumen des Arbeitgebers. --

07.03.2009, 22:38
Arego Benutzer Registriert seit: 06.03.2009 Beiträge: 30	Naja also ich hatte 20 h keine EINZIGE antwort bekommen... und das andere forum hat auch nicht wirklich gefunden... Ich habe mit einem meiner kleinen Kontakte telefoniert der mir geraten hat einen komplett neuen Parser zu schreiben. Ich weiß schon was Crossposting ist, aber wenn nach gut einem Tag keine einzige Antwort erfolgt ist, nicht mal ein Tipp, kann mans echt vergessen. Für Unannehmlichkeiten entschuldige ich mich.

07.03.2009, 22:42
nikosch moderatives Dielektrikum Registriert seit: 21.05.2008 Beiträge: 35.989 PHP-Kenntnisse: Fortgeschritten	PS: Ich finde die Lösung mit den Optionen im Endtag irgendwie unsinnig. Über lange Markierungen sucht man sich imho dämlich nach Problemen und schlecht zu parsen ist das auch noch. Hast Du das jetzt wieder so drin? __________________ -- One pixel is still too big. Please make it smaller. ASAP. Initiative Mittelstand. Die wichtigste Gestaltungsregel im Screendesign ist Pi mal Daumen des Arbeitgebers. --

09.03.2009, 08:21
Arego Benutzer Registriert seit: 06.03.2009 Beiträge: 30	Nein.... aus den 8 Einzelnen sind jetzt nur noch zwei geworden, einer ohne optionen und einer mit Optionen, die jeweils nur vorne angegeben werden und im komplettpaket an eine neue Funktion übergeben werden die die Optionen Trennt und richtig zugeordnet. Praktisch ein Parser für die Optionen extra im parser für den BBcode. Also so: $text = preg_replace("/\[test=((\s\|.)+)\]((\s\|.)+)\[test\]/ie", "preg_parse_test('\\2','\\1')"); in dem ersten ((\s\|.)+) stehen alle Optionen egal welche und wieviele.

09.03.2009, 18:00
nikosch moderatives Dielektrikum Registriert seit: 21.05.2008 Beiträge: 35.989 PHP-Kenntnisse: Fortgeschritten	Leere Tags werden bei Dir nicht geparst? __________________ -- One pixel is still too big. Please make it smaller. ASAP. Initiative Mittelstand. Die wichtigste Gestaltungsregel im Screendesign ist Pi mal Daumen des Arbeitgebers. --