Pas mal, il manque juste selon moi un test pour éviter d’avoir un séparateur - en fin d’url, c’est disgracieux ^^

(exemple, un titre avec un signe de ponctuation style “On m’a volé mon ipod !” donnerait avec cette fonction “on-m-a-vole-mon-ipod-”, le tiret final n’est pas utile.)

Génial !! A part ce petit soucis qui n’en est pas un, cette astuce est vraiment géniale ! Merci beaucoup pour cette info, ce blog est vraiment une super idée ! Continuez !!

Il serait plus judicieux de retirer également les mots de moins de 3 lettres dans les urls car ils sont peu utile et si on les laisse, on peut arriver à des urls assez longues…

Jai une parse erreur à la ligne :
$texte_pre_pre = eregi_replace(‘[^a-zA-Z0-9_]’, ‘-’, $texte_pre_pre_pre);

Quand je remplace les ‘ par des ‘ j’en ai plus mais il me retourne aucun résultat.

@Damien, c’est mis à jour.

@Tommy: c’est une question de point de vue. Moi je préfère “roi-de-la-glisse” que “roi-glisse”

@saian: en fait, il faut mieux éviter le copier-coller, visiblement, wordpress a modifié les cotes à sa façon.

Pour éviter de faire trop souvent appel à la fonction, on peut aussi enregistrer le résultats dans la base à la manière des identifiants de Wordpress.
Dans ce cas on utilise la fonction uniquement lors de l’enregistrement de l’article. Ca permet de gagner un peu de temps lorsque l’on a beaucoup d’url à générer (style annuaire), on peut aussi avoir la main sur l’url rewritée et eventuellement faire des modifications.

@Antoine, tout à fait, mais cette remarque vaut pour toute les fonctions lourdes.

Mais ici, vous remarquerez que vous pouvez mettre à jour quasiment n’importe quel site avec cette technique, sans perte de liens puisque la page répondra toujours à son ancien URL.

@Chris, dans ce cas une petite redirection 301 en plus pour éviter le duplicata.

Salut,

Pour le tiret de la fin comme ça c’est quand même plus simple ..

$texte_final = preg_replace(’/\-$/’,”,$texte_final);

PS1 : L’effet + les couleurs des commentaires piquent les yeux IMHO

PS2 : Tu devrais mettre sur ton blog un doctype transitional, la plupart des erreurs de validation disparaitraient (enfin on passerait de 16 à 6 erreurs pour cette page)

@boris: Bien vu le regex, je prend note.
“PS1″ chez moi, les couleurs sont agréable, j’y songerai pour la V2
“PS2″ Non, je corrigerai plutôt les erreurs, lorsque j’aurai le temps, mais j’ai réalisé ce thème sans connection internet.

Quelques remarques sur le code de la fonction :

#3 : $texte_pre_pre_pre… Pourquoi créer autant de variables temporaires ? Le fait de ne garder qu’une variable ($texte par exemple) permet d’en économiser 5

#4 : eregi_replace est plus lent que preg_replace…

#6 substr() admet des valeurs numériques, donc sans apostrophes autour

#7-#8 : la création d’une variable $end est un peu inutile à mon sens, il suffit simplement de mettre la fonction substr() directement dans la condition : même résultat mais plus propre et plus rapide)

Sinon, 128 caractères de long, peut-être est-ce un peu trop pour un identifiant d’URL, non ? Google ne pénalise t-il pas les URLs trop longues ?

Voilà, j’espère que vous tiendrez compte de ces petites suggestions

Bonne continuation,
Thomas

@Thomas. bien sur qu’on prend en compte les remarques, c’est tout l’intérêt d’avoir des commentaires ouvert.

En fait j’ai trouvé cette fonction il y a longtemps, je ne l’ai pas codé, ni optimisée et avec tes remarques, je vois effectivement qu’elle pourrait être optimisée.

Avec une petite optimisation on pourrait obtenir ceci:

function pre_url($texte) {
$texte = html_entity_decode($texte);
$tofind = utf8_decode( ‘ÀÁÂÃÄÅàáâãäåÒÓÔÕÖØòóôõöøÈÉÊËéèêëÇçÌÍÎÏìíîïÙÚÛÜùúûüÿÑñ’ );

$replac = utf8_decode( ‘AAAAAAaaaaaaOOOOOOooooooEEEEeeeeCcIIIIiiiiUUUUuuuuyNn’ );

$texte = trim(strtolower(strtr($texte,$tofind,$replac)));

$texte = eregi_replace(’[^a-zA-Z0-9_]’, ‘-’, $texte);

$texte = eregi_replace(’-+’, ‘-’, $texte);

$texte = substr($texte, 0, 128);

$texte = preg_replace(’/\-$/’,”,$texte);

return $texte;

}

Le problème c’est que le doctype strict est très limitant, par exemple tu ne peux pas mettre d’attribut href sur les anchors, ce qui peut être pénalisant, surtout si tu ne maîtrise pas tout le contenu (commentaires, etc …)

Mais c’est toi qui voit

Euh, il fallait lire “d’attribut target”