Discussion :

[Bruno.C]

Bonjour,

J'aimerai en fait appliquer un urldecode sur toutes les urls d'une page mais je n'y arrive pas, j'ai commencé par faire:

Code :

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

1
2
3
4
5
 
<?php
$chaine = '|href="(.*?)"|i';
$contenu = preg_replace($chaine,urldecode($1),$contenu);
?>

Vous vous en serez douté, ça ne fonctionne pas, il n'y a qu'à voir la coloration syntaxique du $1 pour s'en rendre compte.

Si vous étiez à ma place comment procéderiez vous ?

Merci.

[Eusebe]

Bonjour,

Tu pourrais utiliser preg_replace_callback. Tu trouveras des exemples qui devraient t'y aider dans la doc php (lein donné)

[Bruno.C]

Merci de votre réponse.
A vrai dire, j'ai visité la doc et je n'y ai rien compris. Même les exemples sont trop tordus.
Si ça te dérange de me donner la solution à mon problème pourrais tu stp me montrer un exemple de l'utilisation de cette fonction qui soit un peu moins difficile ?

Merci encore une fois.

EDIT

Voila un petit code qui explique ce que je veux faire, vous pouvez le tester chez vous et vous verrez que quand vous visualisez le code source de la page générée l'url n'a pas été décodée .

Code :

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

1
2
3
4
5
6
7
8
9
<?php
$url = 'ceci est un test <a href="http://www.actu-master.com/index.php%3Fpage%3Doutils" >test.html</a> encore un test';
 
	$chaine = '|href="(.*?)"|i';
	$contenu = preg_replace($chaine,'href="'.urldecode('$1').'"',$url);
 
 
echo $contenu;
?>

[Eusebe]

J'ai bien compris ce que tu voulais faire.

Mais ta solution n'est pas la bonne. Dans ton cas tu remplaces par 'href="'.urldecode('$1').'"' soit 'href="$1"' (puisque la chaîne '$1 n'est pas modifiée par urldecode).

La solution est donc de passer par une fonction de callback, comme proposé.

Un exemple après adaptation de la doc php (non testé) :

Code :

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

1
2
3
$url = preg_replace_callback('@(?<=href=").*(?=")@i',
    create_function('$matches', 'return urldecode($matches[0]);'),
    $url);

[s.n.a.f.u]

Bonjour,

Personnellement, je suis un adepte de la simplicité :

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
 
<?php
 
$buffer = 'ceci est un test <a href="http://www.actu-master.com/index.php%3Fpage%3Doutils" >test.html</a> encore un test';
$pattern = '|href="([^"]*)"|i';
if ( preg_match($pattern, $buffer, $matches) ) {
    $url = $matches[1];
    $buffer = str_replace($url, urldecode($url), $buffer);
}
 
echo $buffer;

En complément, je milite pour une utilisation minimale du .* dans les regex
C'est un gouffre à performances et une fuite de contrôle dans les regex.
Pour notre exemple,

Code :

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

$pattern = '/"(.*?)"/';

Sera avantageusement remplacé par

Code :

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

$pattern = '/"([^"]*)"/';

[Eusebe]

L'inconvénient de cette solution est qu'elle ne fonctionne que pour le premier lien. Il faut utiliser preg_match_all et modifier un peu le script pour remplacer "toutes les urls d'une page".

Et je ne trouve pas cette solution beaucoup plus simple que l'utilisation de preg_replace_callback... Mais ça c'est un avis personnel

Pour ce qui est de la regex, je suis d'accord pour le côté "fuite de contrôle" lorsqu'on utilise .* dans un regex. Pour ajouter ma pierre du côté gouffre de performance, l'utilisation des assertions me semble les améliorer, en évitant une capture.

[s.n.a.f.u]

L'inconvénient de cette solution est qu'elle ne fonctionne que pour le premier lien. Il faut utiliser preg_match_all et modifier un peu le script pour remplacer "toutes les urls d'une page".

Et je ne trouve pas cette solution beaucoup plus simple que l'utilisation de preg_replace_callback... Mais ça c'est un avis personnel

Pour ce qui est de la regex, je suis d'accord pour le côté "fuite de contrôle" lorsqu'on utilise .* dans un regex. Pour ajouter ma pierre du côté gouffre de performance, l'utilisation des assertions me semble les améliorer, en évitant une capture.

Je viens de me rappeler une alternative à la fonction de callback : l'option d'évaluation. Est-ce plus simple ? Je pense que c'est une affaire de goût...

Code php :

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

1
2
3
4
$buffer = 'ceci est un test <a href="http://www.actu-master.com/index.php%3Fpage%3D1" >test.html</a> encore un <a href="http://www.actu-master.com/index.php%3Fpage%3D2" >test</a>';
$pattern = '|href="([^"]*)"|ie';
$buffer = preg_replace($pattern, "'href=\"'.urldecode('\\1').'\"'", $buffer) ; 
echo $buffer;

Ou qui pourra être élégamment mixée avec les assertions pour avoir une fonction d'évaluation moins compliquée (pour qui maîtrise les assertions ):

Code php :

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

1
2
3
4
$buffer = 'ceci est un test <a href="http://www.actu-master.com/index.php%3Fpage%3D1" >test.html</a> encore un <a href="http://www.actu-master.com/index.php%3Fpage%3D2" >test</a>';
$pattern = '|(?<=href=")[^"]*(?=")|ie';
$buffer = preg_replace($pattern, "urldecode('\\0')", $buffer) ; 
echo $buffer;

[Eusebe]

Je viens de me rappeler une alternative à la fonction de callback : l'option d'évaluation.

Oui, ça je ne connaissais pas !
Mais je le note

[Bruno.C]

Merci à vous deux. Grace à vous j'ai résolu mon problème et j'ai appris deux trois choses .

Merci encore.