Discussion :

[Empty_body]

Salut,

J'ai une chaîne de caractère qui ressemble à ceci :

Code :

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

<span style="font-style: italic;">Mon texte...</span>

Et j'aimerais la transformer en ceci :

Code :

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

<i>Mon texte...</i>

Quelqu'un aurait une astuce à me donner?
J'ai essayé les expression régulière mais je coince...

D'avance merci.

[dervish]

salut,
un truc du style:

Code :

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

1
2
 
echo preg_replace("/<[^>]*>([^<>])*.*/i", "<li>\\1</li>", '<span style="font-style: italic;">Mon texte...</span>');

[sielfried]

Expressions régulières à éviter quand on peut...

http://simplehtmldom.sourceforge.net/

Quelque chose comme ça devrait fonctionner :

Code :

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

1
2
3
4
5
6
7
 
$dom = new simple_html_dom();
$dom->load($tontexte);
foreach ($dom->find('span[style*=font-style: italic]') as $italic_span) {
  $italic_span->outertext = '<i>' . $italic_span->innertext . '</i>';
}
$tontexte = $dom->save();

[Empty_body]

Sympathique idée, sielfried, je ne connaissais pas... Malheureusement, mon hébergeur est limité à php4...

[sielfried]

Tu as ça, en PHP 4 :
http://fr.php.net/manual/en/ref.domxml.php

Le code sera certainement un peu plus complexe que ce que j'ai posté plus haut, mais ça aura le mérite de rester propre et adapté.

Une regexp marchera pour un cas simple, mais deviendra très vite peu fiable dès que tu veux faire des choses un peu plus avancées. Mais là dans ton cas précis, rien que pour gérer des <span> imbriqués, tu vas devoir faire du récursif si tu veux que ça fonctionne.

[FCYPBA]

Mais là dans ton cas précis, rien que pour gérer des <span> imbriqués, tu vas devoir faire du récursif si tu veux que ça fonctionne.

Pareil avec ton code, car le span imbriqué se retrouvera dans le $italic_span->innertext

[FCYPBA]

Expressions régulières à éviter quand on peut...

Raison ?

L'expression régulière réponds parfaitement à son besoin et ne requiert pas de dépendances supplémentaires.

Le code semble moins lisible mais j'imagine qu'il va effectuer un retraitement de fichiers par lot. ( un gros rechercher/remplacer )

Au niveau de la rapidité, je pense qu'il n'y a pas photo entre les deux, l'expression régulière va exploser le dom.

Mis à part cela , cette classe est toutefois très intéressante.

[sielfried]

Raison ?

L'expression régulière réponds parfaitement à son besoin et ne requiert pas de dépendances supplémentaires.

Je considère que lorsqu'on peut éviter d'utiliser les expressions régulières, c'est un bien. Elles sont par définition crades, illisibles et/ou difficiles à maintenir (sauf pour les trucs simples). C'est un outil formidable et d'une puissance exceptionelle, mais quand il y a des solutions différentes et efficaces, je trouve franchement pas l'option judicieuse.
edit: attention, je parle bien des choses 'lourdes', cf ma parenthèse. Parce que pour les trucs simples (tout ce qui est vérification de chaîne notamment), c'est souvent bien plus concis qu'un équivalent "non-regexp".

Mais surtout, dès qu'on rentre dans le domaine du parsing, rien ne vaut un... parseur... Pour le HTML/XML, il y a DOM qui est un outil parfaitement adapté à la situation, donc je ne vois pas pourquoi s'en priver.

C'est comme tenter de parser du BBcode à coups de regexp (le BBcode a aussi ses parseurs, notamment une extension PHP qui fait ça très bien). On peut à peu près y arriver (après pas mal d'efforts), mais pour gérer tous les cas d'imbrications possibles ça devient très, très vite un gigantesque bordel que personne ne peut aller impacter sauf celui qui l'a codé (et encore, vaut mieux pas qu'il y revienne quelques mois plus tard).

Au niveau de la rapidité, je pense qu'il n'y a pas photo entre les deux, l'expression régulière va exploser le dom.

C'est possible (j'en sais rien du tout, à vrai dire), mais pas certain à partir du moment où il faut de la récursion. Va falloir faire du preg_replace_callback dans tous les sens, et ça dépend toujours énormément des masques. Faudrait faire des benchs, mais je suis pas convaincu.