RACCOURCISSEUR D'URL

Commentaire de X_Cli le 07/03/2010 10:21:09

Je n'ai pas lu les sources en entier, mais rien que le concept est à mon avis pas très bon : pourquoi utiliser une base de données pour qqch d'aussi simple... il est à mon sens bcp plus performant et fiable d'utiliser une arborescence filesystem. Ensuite, la table est très mal concu/optimisée : les champs text flinguent les performances, il n'y a pas d'index etc. Stocker une IP en text, notamment, est ridicule : tu peux la stocker en char(15) si tu veux, ou mieux en unsigned integer et convertir REMOTE_ADDR avec des fonctions comme ip2long.
Je vois également des utilisations de fonction ereg* qui sont considérées comme deprecated.

Je trouve bien en revanche d'avoir pensé à la déduplication des URLs.

Commentaire de LeFauve42 le 08/03/2010 11:46:26

Bonjour,

C'est sympa comme point de depart, mais j'ai quelques petites remarques :
- Tes requetes SQL utilisent beaucoup de conversion de collation. Pourquoi ne pas creer directement tes tables en UTF8 plutot que de te trainer le latin1_swedish_ci ?
- Tu n'as pas besoin de stoquer le code dans ta base, puisque tu peux l'obtenir simplement a partir de l'id, et inversement. Comme ca tu n'utilises plus que le champs id pour retrouver tes donnees, qui est deja indexe (les indexes sont importants comme le faisait remarquer X_Cli).
- Le type "LongText" me parait un peu "overkill" pour stoquer une URL. Un VARCHAR(100) devrait suffire, non ?
- Tu n'as aucune protection contre les SQL injection. Regarde un peu les fonctions mysql_escape_string() et mysql_real_escape_string() pour corriger ca.
- Ta fonction CheckURL est plutot maladroite. Tu devrais pouvoir la remplacer par une seule expression reguliere, et comme le disait aussi X_Cli, utiliser les expressions regulieres PERL au lieux des POSIX qui sont obsoletes depuis PHP 5.3. Ca devrait donner quelque chose comme ca :
function CheckURL($url)
{
  return preg_match('@^https?://.+$@i',urldecode(trim($url)),$matches)?$matches[0]:'error';
}
(a tester, mais ca devrait etre bon).
- Ta fonction CheckURL() appelait deux fois urldecode(). Ca peut poser des problemes avec les url contenant des '+' ou des '%' par exemple.
- Faire deux requetes par redirection est quand meme couteux. Si tu veux vraiment garder un compteur, tu peux peut-etre faire renvoyer la redirection avant de faire la seconde requete. La charge sera sensiblement la meme sur tou serveur, mais le client attendra un peu moins longtemps.

Voila, il doit y avoir d'autres ameliorations, mais c'est deja un bon depart :o)

Bon courrage,
Eric

Commentaire de hairai le 08/03/2010 20:37:51

Bonjour,

tout d'abord merci pour vos commentaires constructifs.

L'intérêt de la BDD est de prévoir de futures améliorations, comme un panel d'administration, etc
Effectivement la table était assez crade, j'ai modifié la structure pour cette version.
Pour ce qui est de la sanitization, je l'ai également ajoutée.
Pour le compteur étant donné que le site de base n'était pas trés visitée ce n'était pas dérangeant,
maintenant, il est configurable, et il y a la possibilité de le desactiver, de l'utiliser en mode simple ou de l'utiliser avec APC,
le principe est simple, les informations sont stockées dans un tableau en cache jusqu'à atteindre une limite configurable,
puis tout est ajouté dans la base. Cela permet de gagner en performances.
La table est également maintenant en UTF-8.
Merci LEFAUVE42 pour ta fonction de verification des URL, je l'ai ajoutée quasiment telle quelle, elle marche bien :)

Voila voila, encore une fois merci et n'hésitez pas à me donner vos impressions sur ce nouveau code :)