[Système] bbcode imbriqué qui ne marche pas

Version imprimable

Bonjour @ tous,

Je cherche à faire un système de BBcode, Ca marche à peu près comme il faut sauf quand des balises BBcode sont imbriqués. Avez-vous une idée pour que cela passe.

Voici ci dessous ma manière de faire.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
$sHtml = '[citation=bob]njvkldjvfopsvjdsopvndfp[/citation]';
$sCitationEtendu = '<span class="forumTexteCitation">$1 à écrit :</span><div class="forumDivCitation">$2</div>';
$sHtml = preg_replace('#\[citation=([^\]]*)\](.*)\[/citation\]#Usi', $sCitationEtendu, $sHtml);
 
echo $sHtml;
 
echo'<br />';
echo'<br />';
$sHtml2 = '<span class="forumTexteCitation">bob à écrit :</span><div class="forumDivCitation">njvkldjvfopsvjdsopvndfp</div>';
$sCitationEtendu2 = '[citation=$1]$2[/citation]';
$sHtml2 = preg_replace('#\<span class="forumTexteCitation">([^\]]*) à écrit :</span><div class="forumDivCitation">(.*)</div>#Usi', $sCitationEtendu2, $sHtml2);
 
echo $sHtml2;

D'avance merci

Le probleme c'est que si tu as

Code:

[citation=machin]trk[citation],fdk[/citation][/citation]

le mode U (non gourmand ou Ungreedy en anglais) te prendra ça :

Code:

[citation=machin]trk[citation],fdk[/citation][/citation]

Et sans le mode U, dans un cas simple comme celui la :

Code:

[citation]trklterk[/citation][citation]fdklmf[/citation]

la regex te prendra ça :

Code:

[citation]trklterk[/citation][citation]fdklmf[/citation]

Et ceci sera concidéré dans les 2 cas comme 1 balise. Tout comme le probleme ici : http://www.developpez.net/forums/sho...d.php?t=359034

Il te faut simuler un langage à pile (c'est ce qu'on utilise pour le principe des parentheses). Je te propose un truc comme ça :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
<?php
 
$xml = '[citation=machin]trk[citation],fdk[/citation][/citation][citation]trklterk[/citation][citation]fdklmf[/citation]';
 
function langage_pile($xml)
{
	$pile = array();
	$last = -1;
	$offset = 0;
 
	preg_match_all('#[/?citation(?: *=(.+?) *)?>#', $xml, $matches, PREG_OFFSET_CAPTURE + PREG_SET_ORDER);
 
	foreach($matches as $match)
	{
		/**
		* Format de $match :
		* [0] =>
		* 	[0] => Balise complete (ex : [/citation] ou [citation=toto])
		*	[1] => Indice du caractère [
		*
		* [1] =>
		*	[0] => Nom de la personne citée
		*	[1] => Indice du début du nom de la personne (inutil)
		*/
 
		// On test si c'est une balise ouvrante ou fermante
		if( strpos($match[0][0], '/') === FALSE )
		{
			// Balise Ouvrante
			$pile[++$last] = $match;
		}
		else
		{
			// Le début de la chaine à remplacer est la position du premier [ de la balise ouvrante (ici $pile)
			// à laquelle on ajoute une variable car comme on change la chaine, les indices changent
			$debut_avec_balises = $pile[$last][0][1]+$offset;
 
			// La longueur c'est la position du premier [ de la balise fermante ($match ici)
			// à laquelle on ajoute la longueur de la balise et à laquelle on enleve la position du début
			// on y ajoute aussi l'offset car la longueur n'est pas modifiée (+offset -offset)
			$longueur_avec_balises = $match[0][1]+strlen($match[0][0])+$offset - $debut_avec_balises;
 
			// Le début sans balise (la citation en elle même)
			// c'est la position du premier [ de la balise ouvrante à laquelle on ajoute sa longueur
			$debut_sans_balises = $pile[$last][0][1] + strlen($pile[$last][0][0]);
 
			// La longueur sans balise c'est la position du [ de la balise fermante
			// moins le début sans balises et plus l'offset (encore une fois +offset -offset)
			$longueur_sans_balises = $match[0][1]+$offset-$debut_sans_balises;
 
			// On récupère la citation
			$citation = substr($xml, $debut_sans_balises, $longueur_sans_balises);
 
			// Le code de remplacement
			$remplacement = '<span class="forumTexteCitation">'.$match[1][0].' à écrit :</span><div class="forumDivCitation">'.$citation.'</div>';
 
			// On remplace dans la chaine
			$xml = substr_replace($xml, $remplacement, $debut_avec_balises, $longueur_avec_balises);
 
			// On supprime la derniere entrée de $pile
			unset($pile[$last--]);
		}
	}
 
	return $xml;
}	
 
var_export( langage_pile($xml) );
 
?>

J'ai essayé de le commenté au maximum

Bonjour,
C'est peut-être un peu bête mais tu as essayé quelque chose dans ce style ?
Code:

1 2 3 4 5 6 7 8 $pattern = '/\[citation[\s]*=[\s]*(.*?)\](.*?)\[\/citation\]/s'; $count = 1; while ($count) { $file = preg_replace($pattern, $html, $file, -1, $count); }
Anthony

15/06/2007, 15h28
jexl
Merci à vous deux pour vos propositions, je suis entrain de les tester et de tenter de les comprendre completement.

Autrement, j'avais fais quelque chose qui semblait marcher, mais je n'ai pas vu encore les limites c'est ca :
Code:

1 2 3 $sCitationEtendu2 = '[citation=$1]'; $sHtml2 = preg_replace('#\<span class="forumTexteCitation">([^\]]*) à écrit :</span><div class="forumDivCitation">#Usi', $sCitationEtendu2, $sHtml2);
Pour remplacer la premiere balise puis le même principe pour la balise fermante.

Je vous tiens au courant.

Et en tout cas, merci beaucoup pour le temps passé, et les différents commentaires très appreciables pour bien comprendre.
15/06/2007, 15h33
Korko Fain

Oui ta méthode fonctionne aussi. Elle est meme tres surement plus rapide que la mienne. ^^ Disons que la mienne différencie les différents niveaux alors que la tienne n'en tiens pas compte. ^^
15/06/2007, 15h39
{Anthony}

Oui clairement, je suis allez au plus simple/rapide, ça n'a rien à voir avec ta solution bien plus complète (et complexe :p) mais fort bien commenté :)

Mais parfois rien ne sert de compliquer inutilement les choses :lol:
Enfin après tout dépend de l'objectif final de jexl ;)
15/06/2007, 15h54
Korko Fain

Non Anthony, ta méthode ne fonctionne pas dans tous les cas. Regarde le début de mon post, ta regex est dans le premier. (le U etant chez toi remplacé par les ? apres les * ou les +).

Je parlai de la méthode de jexl.
15/06/2007, 16h06
jexl

Mon but très simplement est de finaliser le forum que je suis entrain de créer.
J'en avais marre d'adapter phpBB et autres IPB à mes sites, alors j'en fais un qui réponds à 100% de mes besoins.
Je souhaitais donc l'agrémenter de quelques fonctionnalités bien utiles comme ces regex.
C'est vrai que je préfère miser sur l'efficacité du code, et sur sa rapidité d'execution, en revanche l'histoire des niveaux peut m'être utile... alors je vais tester ca dans tous les sens.

C'est toujours difficile de bien voir les limites de telle ou telle chose quand on ne maitrise pas tout.
Mon domaine à moi c'est plus le PHP et le SQL, mais pour être bien sure de mon code, je suis obligé de me mettre aux expressions régulière.

Je vous remercie encore pour vos conseils avisé et votre temps offert, et vous tiens au courant dès que j'aurai bien recetté l'execution de ce petit script dans tous les cas de figure.

@+
20/06/2007, 00h22
jexl

Pour info, j'ai fais au plus simple et ca marche très bien, en gros je gère [/citation] en str_replace et [citation=$1] avec preg_replace, ca marche nickel...

En tout cas, merci encore pour tout vos eclaircissements. J'ai même pu fais plusieurs expressions régulières qui ont marché du premier coup !!! C'est que ca rentre.:mouarf:

@+
20/06/2007, 10h39
Korko Fain

parfait ;)
Un ptit "Résolu" ? ^^
20/06/2007, 12h01
jexl

Bah Korko Fain, je l'ai mis le résolu ;)
22/06/2007, 13h21
Korko Fain

dsl jlavais pas vu il est pas dans le chemin de navigation :p