[PHP] BBCode probleem

Quia Ego Sic Dico.

De parser van phpBB is AFAIK helemaal niet stackbased?

In phpBB gebruiken ze intern [quote:23495] tags, die bij het editen weer teruggezet worden naar [quote]. Op die manier maken ze nesting mogelijk met regexps. Dus tenzij hun systeem veranderd is in een van de nieuwere versies gebruiken ze in phpBB ook "gewoon" regexps.

Overigens is een stackbased parser hier inderdaad wel een goede oplossing. Een andere manier is om met assertions te werken, maar daar worden regexps ook niet leesbaarder op en uiteindelijk krijg je een product dat functioneel ondergeschikt is aan zo'n parser.

'E's fighting in there!' he stuttered, grabbing the captain's arm.
'All by himself?' said the captain.
'No, with everyone!' shouted Nobby, hopping from one foot to the other.

vrijdag 5 mei 2006 22:09

Acties:

Topicstarter

Oke, en nu effe in n00btaal alstublieft?

offtopic:
Stackbased...?

vrijdag 5 mei 2006 22:10

Acties:

Quia Ego Sic Dico.

Je kàn natuurlijk ook even Google gebruiken als je een term tegenkomt die je niet kent.

[google=php stackbased parser]
[google=php stack based parser]

'E's fighting in there!' he stuttered, grabbing the captain's arm.
'All by himself?' said the captain.
'No, with everyone!' shouted Nobby, hopping from one foot to the other.

zondag 7 mei 2006 13:00

Acties:

vrijdag 26 mei 2006 11:49

Devver

Pixelated

Een andere manier is om vantevoren de matches op code-tags op te slaan en later weer terug te zetten:

PHP:

preg_match_all('/\\[code\](.*?)\\[\/code\]/is', $content, $codematches);

// andere replaces

if (!empty($codematches))
{
    $i = 0;
    $content = preg_replace('/\\[code\](.*?)\\[\/code\]/ise', '\'<pre>\'.htmlspecialchars(\$codematches[1][\$i++]).\'</pre>\'', $content);
}

het is een workaround maar werkt...

Intentionally left blank

Acties:

vrijdag 26 mei 2006 11:56

Topicstarter

zodra ik de $codematches door een functie haal waar html bij wordt gemaakt (bijv. een tabel met regelnummering) gaat het mis... door een functie halen zónder html gaat wel goed. Hoe maak ik 't dan dat het ook mét html werkt? $_/-\o_$

Acties:

vrijdag 26 mei 2006 11:59

Devver

Pixelated

Wat heb je al en wat gaat er precies fout?

Intentionally left blank

Acties:

vrijdag 26 mei 2006 12:26

Topicstarter

Hij geeft helemaal geen post meer weer waar een [code ] in zit.

code:

<?
    function code($f){

        $regels = count(explode("<br>", nl2br($f)));
        $regels2 = count(explode("<br \>", nl2br($f)));
        $regels3 = count(explode("\n",  nl2br($f)));
        $f = str_replace("&quot;", "\"", $f);
        $f = str_replace("\"", "\\\"", $f);
        $f = str_replace("&acute;", "`", $f);
        $return = "<div style=\"width: 550px; overflow: auto; table-style: fixed;\">Code:<br><table width=\"550\" cellspacing=\"0\" style=\"border-right: black solid 1px;border-top: black solid 1px;border-left: black solid 1px;\"><tr><td width=\"10\" bgcolor=\"#DCD9D6\"><strong>";
        for ($i = 1; $i < ($regels + $regels2 + $regels3); $i++){
            $return .=  $i . "<br>";
        }
        $return .=  "</strong></td><td valign=\"top\" nowrap>" .  (stripslashes($f)) . "</td></tr></table></div>";
        return $return;

    }

function quote($f){
    return "<div class=\"mainquote\">Quote:<div class=\"quote\">" . $f. "</div></div>";
}
function ubb($f){

    $f = nl2br($f); 
preg_match_all('/\\[code\](.*?)\\[\/code\]/is', $f, $codematches); 
    $query = mysql_query("SELECT * FROM `" . $GLOBALS['prefix'] . "smilies`");
    while ($row = mysql_fetch_array($query)){
        $f = str_replace($row['shortcut'], "[img]'"[/img]", $f);
    }
    



    $f = preg_replace("/\[quote](.+?)\\[\/quote]/ise", "quote('\\1')", ($f));
    $f = preg_replace("/\[message=(.*?)\](.+?)\\[\/message]/isD", '<a href="singlepost.php?id=\\1#\\1">\\2</a>', ($f));
    $f = preg_replace("/\[img]http:\/\/(.+?)\\[\/img]/isD", '[img]"http://\\1"[/img]', ($f));
    $f = preg_replace('#(^|[\n ])(http://[^ \n\r\t<]*)#is', '\\1<a href="\\2" target="_BLANK">\\2</a>', $f);
    $f = preg_replace('#(^|[\n ])(www.[^ \n\r\t<]*)#is', '\\1<a href="http://\\2" target="_BLANK">\\2</a>', $f);


    $f = preg_replace('/\[b](.*?)\\[\/b]/si', '<strong>\\1</strong>', $f);
    $f = preg_replace('/\[u](.*?)\\[\/u]/si', '<u>\\1</u>', $f);
    $f = preg_replace('/\[i](.*?)\\[\/i]/si', '<em>\\1</em>', $f);
if (!empty($codematches)) 
{ 
    $j = 0; 
    $f = preg_replace('/\\[code\](.*?)\\[\/code\]/ise', code('$codematches[1][$j++]'), $f); 
} 

    return $f;
}

Acties:

vrijdag 26 mei 2006 12:30

Quia Ego Sic Dico.

preg_replace_callback().

'E's fighting in there!' he stuttered, grabbing the captain's arm.
'All by himself?' said the captain.
'No, with everyone!' shouted Nobby, hopping from one foot to the other.

Acties:

vrijdag 26 mei 2006 14:04

Topicstarter

En dan parsed-ie weer gewoon de smilies en opmaak d'rbij

hij parsed gewoon alles, behalve de [code ]'s

dus krijg gewoon smilies d'rin en géén code-tabelletje d'romheen

[ Voor 64% gewijzigd door lennartkocken op 26-05-2006 12:38 ]

Acties:

vrijdag 26 mei 2006 14:07

Topicstarter

code:

<?
    function code($f){
        //$f = str_replace("&amp;", "&", $f);
        $f = str_replace("&quot;", "\"", $f);
        //$f = str_replace("\"", "\\\"", $f);   
        $regels = count(explode("<br>", nl2br($f)));
        $regels2 = count(explode("<br \>", nl2br($f)));
        $regels3 = count(explode("\n",  nl2br($f)));
        $f = str_replace("<br \>", "", $f);
        $f = str_replace("\\", "\\\\", $f);

        $return = "<div style=\"width: 550px; overflow: auto; table-style: fixed;\">Code:<br><table width=\"550\" cellspacing=\"0\" style=\"border-right: black solid 1px;border-top: black solid 1px;border-left: black solid 1px;\"><tr><td width=\"10\" bgcolor=\"#DCD9D6\" valign=\"top\"><strong>";
        for ($i = 1; $i < ($regels + $regels2 + $regels3); $i++){
            $return .=  $i . "<br>";
        }
        $return .=  "</strong></td><td valign=\"top\" nowrap>" .  (($f)) . "</td></tr></table></div>";
        return $return;

    }

function quote($f){
    return "<div class=\"mainquote\">Quote:<div class=\"quote\">" . $f. "</div></div>";
}
function ubb($f){

    $f = nl2br($f); 
    preg_match_all('/\\[code\](.*?)\\[\/code\]/is', $f, $codematches); 
    $query = mysql_query("SELECT * FROM `" . $GLOBALS['prefix'] . "smilies`");
    while ($row = mysql_fetch_array($query)){
        $f = str_replace($row['shortcut'], "[img]'"[/img]", $f);
    }
    



    $f = preg_replace("/\[quote](.+?)\\[\/quote]/ise", "quote('\\1')", ($f));
    $f = preg_replace("/\[message=(.*?)\](.+?)\\[\/message]/isD", '<a href="singlepost.php?id=\\1#\\1">\\2</a>', ($f));
    $f = preg_replace("/\[img]http:\/\/(.+?)\\[\/img]/isD", '[img]"http://\\1"[/img]', ($f));
    $f = preg_replace('#(^|[\n ])(http://[^ \n\r\t<]*)#is', '\\1<a href="\\2" target="_BLANK">\\2</a>', $f);
    $f = preg_replace('#(^|[\n ])(www.[^ \n\r\t<]*)#is', '\\1<a href="http://\\2" target="_BLANK">\\2</a>', $f);


    $f = preg_replace('/\[b](.*?)\\[\/b]/si', '<strong>\\1</strong>', $f);
    $f = preg_replace('/\[u](.*?)\\[\/u]/si', '<u>\\1</u>', $f);
    $f = preg_replace('/\[i](.*?)\\[\/i]/si', '<em>\\1</em>', $f);
if (!empty($codematches)) 
{ 
    $j = 0; 
    $f = preg_replace('/\\[code\](.*?)\\[\/code\]/is', code($codematches[1][$j++]), $f); 
} 

    return $f;
}

Hij werkt $_/-\o_$ $_/-\o_$ Thnx Crisp en -Nme

Acties:

vrijdag 26 mei 2006 14:17

Devver

Pixelated

volgens mij werkt 'ie nu alleen voor de 1e code-tag

Intentionally left blank

Acties:

vrijdag 26 mei 2006 14:18

Topicstarter

shit

Acties:

vrijdag 26 mei 2006 14:23

Devver

Pixelated

Ik denk dat je beter eerst door je codematches array heen kan lopen en door code() gooien en daarna pas de replace doen

Intentionally left blank

Acties:

vrijdag 26 mei 2006 15:51

Topicstarter

code:

if (!empty($codematches)) 
{ 
    for ($j = 0; $j < count($codematches); $j++){
  
    $f = preg_replace('/\\[code\](.*?)\\[\/code\]/is', code($codematches[1][$j++]), $f); 
    }
}

zoals dit? Dit werkt namelijk niet

Acties:

Scott

Ik ben, dus ik tweak

Nee, moet je het zo doen:

PHP:

if (!empty($codematches)) 
{ 
    for ($j = 1; $j < count($codematches); $j++)  
        $codematches[1][$j-1] = code($codematches[1][$j-1]);

    for ($i = 1; $i < count($codematches); $i++)
        $f = preg_replace('/\\[code\](.*?)\\[\/code\]/is', $codematches[1][$i-1], $f);
}

Werkt dit wel ?

vrijdag 26 mei 2006 15:53

Acties:

vrijdag 26 mei 2006 16:13

Topicstarter

Werkt niet...

krijg nog steeds 2x dezelfde code waar het 2x verschillend moet zijn...

[ Voor 75% gewijzigd door lennartkocken op 26-05-2006 16:00 ]

Acties:

vrijdag 26 mei 2006 16:16

Devver

Pixelated

Of zo:

PHP:

for ($j = 0; $j < $codematches[1]; $j++)
    $codematches[1][$j] = code($codematches[1][$j]);

$j = 0;
$f = preg_replace('/\\[code\](.*?)\\[\/code\]/ise', '\$codematches[1][\$j++]', $f);

of zo:

PHP:

1 2	$j = 0; $f = preg_replace('/\\[code\](.*?)\\[\/code\]/ise', 'code(\$codematches[1][\$j++])', $f);

Intentionally left blank

Acties:

zaterdag 27 mei 2006 12:54

Topicstarter

Die laatste lijkt te werken :D:D:D $_/-\o_$

maar nou wordt er geen preg_replace() meer uitgevoerd die ná de code's komen

[ Voor 51% gewijzigd door lennartkocken op 26-05-2006 16:21 ]

Acties:

zaterdag 27 mei 2006 12:59

Topicstarter

*kick*

Acties:

Borizz

lennartkocken schreef op zaterdag 27 mei 2006 @ 12:54:
*kick*

Iets meer zelf onderzoeken mag ook wel hoor? Er zijn zat open source producten waar je even in de code kan kijken hoe zij het opgelost hebben. Daar leer je meer van dan dat wij het voor je op gaan lossen.

If I can't fix it, it ain't broken.

zaterdag 27 mei 2006 13:09

Acties:

zaterdag 27 mei 2006 23:35

Quia Ego Sic Dico.

Sowieso is kicken hier niet gewenst voordat je topic uit de topiclisting is verdwenen, en dat is pas na 24 uur. Mensen zien je topic vanzelf wel staan zolang het in de listing staat.
Trouwens:

crisp schreef op zondag 07 mei 2006 @ 13:00:
het is een workaround maar werkt...

Waarom hang je zo vast aan regexps als al vanaf het begin gezegd wordt dat het eigenlijk niet echt een oplossing is? Met een stackbased parser heb je geen van de problemen die je hier probeert aan te kaarten en zò moeilijk is het ook weer niet te implementeren.

'E's fighting in there!' he stuttered, grabbing the captain's arm.
'All by himself?' said the captain.
'No, with everyone!' shouted Nobby, hopping from one foot to the other.

Acties:

zaterdag 27 mei 2006 23:44

Devver

Pixelated

-NMe- schreef op zaterdag 27 mei 2006 @ 13:09:
[...]
Met een stackbased parser heb je geen van de problemen die je hier probeert aan te kaarten en zò moeilijk is het ook weer niet te implementeren.

Ik ga hier toch wat tegengas geven: een stackbased parser is inderdaad niet moeilijk te implementeren, maar om een goede te schrijven is andere koek

er zijn wel goede opzets te vinden though

[ Voor 8% gewijzigd door crisp op 27-05-2006 23:35 ]

Intentionally left blank

Acties:

zaterdag 27 mei 2006 23:54

Quia Ego Sic Dico.

In de tijd die hij nu bezig is met knutselen aan regexps die niet of maar half werken had ie al een heel aardig opzetje af kunnen hebben.

'E's fighting in there!' he stuttered, grabbing the captain's arm.
'All by himself?' said the captain.
'No, with everyone!' shouted Nobby, hopping from one foot to the other.

Acties:

orf

Maar met regexen kan het toch ook prima?
Ik heb dat een tijd geleden ingebouwd in een forum

zaterdag 27 mei 2006 23:58

Acties:

Devver

Pixelated

orf schreef op zaterdag 27 mei 2006 @ 23:54:
Maar met regexen kan het toch ook prima?
Ik heb dat een tijd geleden ingebouwd in een forum

Je loopt vroeg of laat toch tegen problemen op die met reguliere expressies niet of nauwelijks meer te behappen zijn, zeker als je je tag-ondersteuning gaat uitbreiden.
Klassieke voorbeelden zijn problemen met nestings van dezelfde tags, het afvangen van verkeerde nestings en afhandeling van context-afhankelijke tags (tags die bijvoorbeeld enkel binnen een bepaalde andere tag mogen worden gebruikt of juist niet).

Intentionally left blank

zondag 28 mei 2006 00:12

Acties:

orf

Ik gebruikte voor tags die niet in andere tags gebruikt mochten worden placeholder tags in combinatie met preg_replace_callback(). Werkt op zich prima, binnen de code tags heb ik verschillende high-light parsers.

zondag 28 mei 2006 00:25

Acties:

maandag 29 mei 2006 16:07

Devver

Pixelated

orf schreef op zondag 28 mei 2006 @ 00:12:
Ik gebruikte voor tags die niet in andere tags gebruikt mochten worden placeholder tags in combinatie met preg_replace_callback(). Werkt op zich prima, binnen de code tags heb ik verschillende high-light parsers.

Ik zeg ook niet dat het onmogelijk is, enkel dat het vaak erg complex en on-onderhoudbaar wordt (en vaak ook traag).
Op dit moment worstelen we eigenlijk een beetje met hetzelfde probleem op de frontpage waar nog steeds een regexp-based parser wordt gebruikt die tesamen met het feit dat reakties enkel geparsed worden opgeslagen (en dus voor editting teruggeparsed moeten worden) voor steeds meer problemen zorgt... Mettertijd willen we daarvoor ook een stackbased parser gaan inzetten aangezien het gewoonweg niet meer te onderhouden valt.

Intentionally left blank

zondag 28 mei 2006 01:02

Acties:

orf

Ik sla ook geparsed op. Bij een beetje volume site vind ik dat naast dubbel opslaan de enige goede methode. Terugparsen is ook niet zo'n probleem, totdat je wijzigingen in je parser aanbrengt. Gewoon één keer bouwen en nooit meer naar kijken.

Acties: