[Résolu] Convertir url en lien cliquable [php - regexp]

Voici comme promis le résultat de ma collecte :

$text = ereg_replace("[[:alpha:]]+://[^<>[:space:]]+[[:alnum:]/]", "<a href=\"\\0\">\\0</a>", $text);

/*** make sure there is an  http://  on all URLs ***/
$string = preg_replace("/([^\w\/])(www\.[a-z0-9\-]+\.[a-z0-9\-]+)/i", "$1http://$2",$string);
/*** make all URLs links ***/
$string = preg_replace("/([\w]+:\/\/[\w-?&;#~=\.\/\@]+[\w\/])/i","<a target=\"_blank\" href=\"$1\">$1</A>",$string);

$chaine = eregi_replace("([[:alnum:]]+)://([^[:space:]]*)([[:alnum:]#?/&=])",
"<A HREF=\"\\1://\\2\\3\" TARGET=\"_blank\">\\1://\\2\\3</A>",$chaine);

$str = ereg_replace("[[:alpha:]]+://[^<>[:space:]]+[[:alnum:]/]","<a href=\"\\0\">\\0</a>", $str);

$html = preg_replace('/\\b(https?|ftp|file):\/\/[-A-Z0-9+&@#\/%?=~_|!:,.;]*[-A-Z0-9+&@#\/%=~_|]/i', "<a href=\"#\" onclick=\"open_url('\\0')\";return false;>\\0</a>", $html);

Le motif suivant est prévu pour les adresses commençant par "www" mais l'explication du bonhomme (mattle) est tellement impressionnante de clarté qu'il fallait que je lui rendes hommage !
Voici ce qu'il propose :

$myRegex = "/(www\.[a-z\d-\.]*\.[a-z]{2,4}?\/?(?:[^\s]*[\/a-z\d]))/ig";
preg_replace($myRegex, "<a href='$1'>$1</a>", $text);  

Son message sur les forums de webhostingtalk : http://www.webhostingtalk.com/showpost.php?p=6504695&postcount=14

Je ne sais pas si ces fragments de codes vont pouvoir faire évoluer le schmilblic, mais j'avais dit que je le ferai, voilà qui est fait !
J'ai finalement modifié le motif comme suis :

(?:https?|ftp)://\S+?)(?=[[:punct:]]?(?:\s|\Z)|\Z)

Ce qui donne donc pour l'instant (peut-être que quelqu'un trouvera mieux ) :

function pxUrlToLink($text) {
	//Motifs permettant de retrouver les éléments à remplacer
	$in = array(
		'`((?:https?|ftp)://\S+?)(?=[[:punct:]]?(?:\s|\Z)|\Z)`',
		'`;\)`'
		);
	//Les chaînes qui vont remplacer les éléments repérés
	$out = array(  
		'<a href="$1" title="$1">$1</a>',
		'<img src="xmedia/emoticones/clin_d_oeil.gif" alt="Clin d\'oeil" />'
		); 
	// on éxécute l'expression régulière et on retourne le texte formaté 
	$text = preg_replace($in, $out, text);  
	return $text;
}

Je n'ai pas encore tout à fait compris l'utilité de certains des points d'interrogations dans ce motif, une petite explication pour le ?=[...]?
Modifié par jojaba (20 Apr 2010 - 21:49)

Non. Dans une classe de caractères, l'échappement est facultatif s'il n'y a pas d'ambiguïté. Une classe se termine nécessairement par un crochet fermant, donc tout ce qui se trouve entre en fait automatiquement partie. Ainsi, il n'y a pas besoin d'échapper ni le point, ni la parenthèse, ni les points d'exclamation et d'interrogation (ni un plus, ni un astérisque, ni un dollar non plus). Pas besoin d'échapper un tiret s'il est en premier ou en dernier (intervalle). Pas besoin d'échapper un circonflexe s'il n'est pas en premier (négation/complément). Cas spécial, une classe ne peut pas être vide donc pas besoin d'échapper un crochet fermant s'il est en premier. Sus aux backslash inutiles !

En ce qui concerne plus généralement cette seconde partie de motif, ?= délimite justement ce que j'ai expliqué dans mon post précédent : c'est un look ahead, et ça permet d'examiner la suite de la chaîne sans pour autant l'inclure dans le motif effectivement matché au final. Donc non, cette partie n'est pas capturée et il n'y a aucun traitement à faire. Tu pourras remarquer que $2 n'existe pas...

Au cas où le sujet t'intéresse, voici un lien vers la doc des regexp en php. Le look ahead est une assertion complexe, il est décrit dans la rubrique assertions.

Je suis un peu mazo, j'aime bien les regexp alors que tout le monde les a en horreur...
Modifié par QuentinC (21 Apr 2010 - 08:46)

a écrit :
) : est-il possible de faire réaliser ces 2 opérations (transformation des url en lien et raccourcissement éventuel si l'url est trop longue) par une seule expression régulière ? Si ce n'est pas possible, je pense qu'on pourra faire un traitement après le preg_replace (je ne sais pas trop si on peut et surtout comment récupérer les chaînes modifiées après le preg_replace juste avant l'affichage)...

IL faut alors utiliser preg_replace_callback. Au lieu de faire un remplacement simple, preg_replace_callback envoie chaque sous-chaîne trouvée à une fonction, et c'est cette fonction qui est chargée de retourner le texte de remplacement.
doc de la fonction preg_replace_callback

QuentinC a écrit :

IL faut alors utiliser preg_replace_callback.
doc de la fonction preg_replace_callback

Vraiment génial le php !
J'ai consulté sur la doc, mais ce n'était pas trop clair. J'ai trouvé une autre ressource qui proposait des exemples me permettant de mieux comprendre : Fonction preg_replace_callback() sur expreg.com. Seulement comme je tourne en php 5 (et non 5.3) j'ai été contraint d'utiliser la fonction create_function() à la place d'une fonction anonyme (comme ils proposent sur regexp.com). J'ai pris un peu de temps à comprendre comment définir cette fonction create-function() mais finalement, je pense avoir réussi. En tout cas, ça fonctionne chez moi comme ça.
Je rappelle donc ce que je voulais faire : avant de remplacer les URL par une URL "cliquable", je voulais les raccourcir pour éviter d'avoir de trop longues chaînes de caractères qui "cassent" le formatage de la page HTML. Dans le code ci-dessous, seules les URL sont traitées, les émoticônes je les traite à part en utilisant la fonction str_replace() amplement suffisante pour remplacer les combinaisons de 2 caractères et surtout évitant ainsi d'utiliser des expressions régulières qui, d'après ce que j'ai lu, sont relativement gourmandes en ressources serveur...
Le code donc :

//Motif permettant de retrouver les URL à remplacer
    	    	$pattern = '`((?:https?|ftp)://\S+?)(?=[[:punct:]]?(?:\s|\Z)|\Z)`'; 
	       	//Fonction permettant de raccourcir les longues URL lors du remplacement dans la fonction preg_replace_callback()
	    	$callback_function = create_function(
	    				'$matches',
	    				'$link_displayed = (strlen($matches[0]) > 25) ? substr( $matches[0], 0, 10).\'[…]\'.substr($matches[0], -10) : $matches[0];
	    				 return \'<a href="\'.$matches[0].\'" title="\'.$matches[0].\'" class="website">\'.$link_displayed.\'</a>\';'
	    				);
	    	$content = preg_replace_callback($pattern, $callback_function, $content);

Pour raccourcir le texte affiché, je ne me suis inspiré d'aucune ressource Web pour la simple raison que je n'ai même pas trouvé d'exemple pour faire ça. On trouve en revanche pas mal de services Web permettant de raccourcir des url... Cela m'a permis de me replonger un peu dans le traitement de chaînes en php et c'est très bien ainsi !
Voilà, je pense que la boucle est bouclée. Ceci dit, je suis toujours prêt à entendre d'autres conseils, astuces pour éventuellement améliorer ce script.
Je souhaite terminer en te remerciant chaleureusement Quentin. Tu as été un maître patient, pédagogique et disponible. A charge de revanche (si c'est dans mes moyens ) !
Modifié par jojaba (21 Apr 2010 - 21:19)

Petite remarque, le coup du create_function préconisé dans la doc n'est pas une obligation. Personnellement, je trouve que ça n'apporte rien par rapport à une fonction normale, c'est moche, pas forcément très clair, et ça complique inutilement. Les closures de php 5.3 sont par contre toutes indiquées. (Ceci dit, tout comme toi, je suis pour le moment bloqué à php 5.2.6 (debian oblige))

a écrit :
Je souhaite terminer en te remerciant chaleureusement Quentin. Tu as été un maître patient, pédagogique et disponible. A charge de revanche (si c'est dans
mes moyens

Merci à toi, tu es le bienvenu pour une prochaîne fois si je peux encore aider.

QuentinC a écrit :
Petite remarque, le coup du create_function préconisé dans la doc n'est pas une obligation. Personnellement, je trouve que ça n'apporte rien par rapport à une fonction normale, c'est moche, pas forcément très clair, et ça complique inutilement.

Cela veut-il dire qu'il y aune autre solution ? d'après ce que j'ai compris, je n'ai pas trop le choix, mais si tu pouvais confirmer...

QuentinC a écrit :
(Ceci dit, tout comme toi, je suis pour le moment bloqué à php 5.2.6 (debian oblige))

Moi c'est free qui me bloque : version 5.1.3RC4-dev. de php
Merci pour ta proposition d'aide future, je n'hésiterai pas !

a écrit :
Cela veut-il dire qu'il y aune autre solution ? d'après ce que j'ai compris, je n'ai pas trop le choix, mais si tu pouvais confirmer...

Tu peux très bien utiliser une fonction normale. Il suffit de passer son nom sous forme de chaîne. p.ex. :

function minuscules ($match) {
return strtolower($match[0]);
}
function balises_en_minuscules ($html) {
return preg_replace_callback('%</?\w+%', 'minuscules', $html);
}

ON peut aussi utiliser des méthodes, y compris privées. Pour plus d'infos, regarde la description du type callback dans la doc.

La doc préconise create_function parce que la fonction de callback n'est bien souvent utiliser qu'une seule fois, et il est inutile de charger l'espace global avec une fonction qui ne sert qu'à un endroit. Un cas tout indiqué pour utiliser une closure.

Par contre non seulement create_function est moche et pas pratique, mais en plus il est potentiellement dangereux et pas optimisé. Parce qu'en réalité, create_function, c'est rien d'autre qu'approximativement ça :

function create_function ($args, $code) {
$name = 'func'.rand(1000000, 9999999);
eval("function $name ($args) { $code }");
return $name;
}

Et eval, c'est le mal... bon dans ce cas je doute que des entrées utilisateur soient réutilisées, mais c'est quand même pas optimisé.
Modifié par QuentinC (22 Apr 2010 - 17:05)

QuentinC a écrit :

Tu peux très bien utiliser une fonction normale. Il suffit de passer son nom sous forme de chaîne. p.ex. :

function minuscules ($match) {
return strtolower($match[0]);
}
function balises_en_minuscules ($html) {
return preg_replace_callback('%</?\w+%', 'minuscules', $html);
}

Ah ben je croyais que c'était ça une fonction anonyme... Est-ce qu'on peut aussi faire comme ça ? :

function minuscules ($match) {
return strtolower($match[0]);
}
$html = preg_replace_callback('%</?\w+%', 'minuscules', $html);

Je suppose que non. Il faut donc placer preg_replace_callback dans une fonction, c'est ça ?

QuentinC a écrit :

Par contre non seulement create_function est moche et pas pratique, mais en plus il est potentiellement dangereux et pas optimisé.

Je vais donc modifier ça...
Modifié par jojaba (23 Apr 2010 - 07:20)

Ah ben petit souci.
Voici ce que j'ai fait :

//Pattern to retrieve the url in the comment
    	    	$pattern = '`((?:https?|ftp)://\S+?)(?=[[:punct:]]?(?:\s|\Z)|\Z)`'; 
	       	//Function to wrap long URLs into smaller one before replacement in the preg_replace_callback() function
	    	function wrap_url($match) {
	    		$link_displayed = (strlen($matches[0]) > 25) ? substr( $matches[0], 0, 10).'…'.substr($matches[0], -10) : $matches[0];
	    		return '<a href="'.$matches[0].'" title="'.$matches[0].'" class="website" style="font-style: normal; font-size: 0.7em">'.$link_displayed.'</a>';
	    	}
	    	//Replacement of the pattern
	    	function replace_pattern($content) {
	    		return preg_replace_callback($pattern, 'wrap_url', $content);
	    	}

Message d'erreur :

a écrit :

Fatal error: Cannot redeclare wrap_url() (previously declared in C:\wamp\www\jojaba2010\manager\tools\utils\functions.php:391) in C:\wamp\www\jojaba2010\manager\tools\utils\functions.php on line 391

Ligne 391 :

function wrap_url($match) {

Modifié par jojaba (23 Apr 2010 - 08:13)

a écrit :

Je suppose que non. Il faut donc placer preg_replace_callback dans une fonction, c'est ça ?

Pas du tout. Ce que tu proposes fonctione. Tant que la chaîne envoyée en 2ème paramètre de preg_replace_callback contient le nom d'une fonction existante, tout va bien.

Pour ta fatal error, c'est sûrement un problème d'inclusions multiples. Utilise require_once / include_once à la place de require / include.

QuentinC a écrit :

Pas du tout. Ce que tu proposes fonctionne. Tant que la chaîne envoyée en 2ème paramètre de preg_replace_callback contient le nom d'une fonction existante, tout va bien.

Je confirme, ça marche également sans utiliser de deuxième fonction.

QuentinC a écrit :

Pour ta fatal error, c'est sûrement un problème d'inclusions multiples. Utilise require_once / include_once à la place de require / include.

Non l'inclusion se fait bien par un require_once par contre, dans mon cas, la fonction de remplacement se trouve dans une autre fonction utilisée plusieurs fois dans la page Web d'où le problème. Il a suffit de ressortir la fonction wrap_url() de la fonction dont je parle pour que ça fonctionne...
Modifié par jojaba (23 Apr 2010 - 10:24)