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

**jexl** · 15/06/2007, 13h45

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

**Korko Fain** · 15/06/2007, 14h17

Le probleme c'est que si tu as

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

la regex te prendra ça :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

[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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

**{Anthony}** · 15/06/2007, 14h18

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

**jexl** · 15/06/2007, 14h28

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

**Korko Fain** · 15/06/2007, 14h33

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. ^^

**{Anthony}** · 15/06/2007, 14h39

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

Enfin après tout dépend de l'objectif final de jexl

**Korko Fain** · 15/06/2007, 14h54

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.

**jexl** · 15/06/2007, 15h06

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.

@+

**jexl** · 19/06/2007, 23h22

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.

@+