❝ 5 commentaires ❞

1  Walane le

Je me suis permis de faire une version un peu plus "propre" à mon goût : https://gist.github.com/walane/5776828 ;)

 
2  Silvus le

Bon, ça ne marche pas fort cette histoire.
Sans trop creuser, parce que je n'ai pas réellement l'utilité de cette fonction en ce moment et qu'une expression régulière pour parsser du html... Voilà voilà...


J'ai finalement eu de meilleurs résultats avec ceci : http://pim.famnit.upr.si/blog/index.php?/archives/248-PHP-get-favicon-from-an-URL.html


Un coup de parssage via simplexml pour commencer, puis au cas où on regarde /favicon.ico et /favicon.png.
Le temps d'exécution est visiblement bien plus long mais sur la poignée de sites que j'ai testé, la différence du nombre de favicons trouvés est aussi très significative !

 
3  Bronco le

Je pense l'améliorer plus tard, mais elle marche sur les sites aux favicons 'normaux'...
Toutefois, dans l'appli pour laquelle j'ai fait cette fonction, le temps d'exécution était prioritaire sur le reste (la favicon est un plus mais n'est pas du tout indispensable.)
En outre, contrairement à la solution que tu proposes, cette fonction est courte, rapide et elle sauve une version locale de la favicon pour les prochains accès. De plus, elle renvoie toujours au moins une favicon par défaut...


(Je m'attendais, bien entendu, à la remarque des regexes pour parser du html voilàvoilà )

 
4  Silvus le

Oh oui c'est sûr que "ma" solution est incomplète et mériterait quelques lignes de plus pour traiter le favicon trouvé (ou non) comme il se doit, mais ce n'est pas le plus compliqué de l'histoire !


Du coup, si la rapidité d’exécution est un critère important et qu'on ne cible pas "l'exotique", on n'aurait pas un intérêt à au moins commencer par un test sur /favicon.ico et /favicon.png ?
'fin bon, si tu as prévu de l'améliorer, je vais attendre les évolutions...


Par contre, c'est moi ou il y a un problème d'échappement au niveau de l'expression régulière ? Étant une bille en regex, il est probable que j'ai raté un morceau mais je serais curieux de savoir quoi...

 
5  Bronco le

En effet, il y a bien un blème pour les guillemets J'ai corrigé, du coup: merci !
Pour ce qui est de l'exotique, il faut encore que je me penche sur les autres moyens de mettre une favicon: en effet, tant qu'il y a le link favicon, tout va bien, mais des fois, il n'y en a pas et le site affiche quand même une favicon...
Comme je n'ai jamais réfléchi à ça, il faudra que je regarde de plus près: ce bout de code en sera forcément retouché.


L'idée de chercher en premier simplement favicon.png et .ico n'est pas bête du tout... ça traiterait une grande partie des cas... du coup on ferait une première regex plus légère avant d'aller plus loin...
A creuser...

 

Fil RSS des commentaires de cet article

✍ Écrire un commentaire

les commentaires relevant du SPAM seront filtrés et dégagés direct...

Quelle est le septième caractère du mot xflu1rad ?