giloum
Messages postés17Date d'inscriptiondimanche 30 septembre 2007StatutMembreDernière intervention26 février 2009 26 févr. 2009 à 15:17
Bonjour LaurentKOogar, je viens de voir ton message ... De ce que je viens de voir sur google curl() est une fonction permettant de vérifier si une url est en ligne, c'est ça ???
Mais je viens de découvrir que mon script a des limites (de très grandes limites :-( ), pour le moment il ne prend pas en compte les url composées (comme par exemple /blabla.php?lg=fr&...)
En plus je n'ai pas encore réussi a faire la liste des liens contenue sur une autre page que la page d'accueil ...
Mais ça va venir, enfin j'espère.
Avez vous une idée de toutes les extension d'une page (pouvant être lu par un navigateur) possible ???
Merci encore de votre aide.
Giloum.
LaurentKOogar
Messages postés369Date d'inscriptionsamedi 4 septembre 2004StatutMembreDernière intervention20 octobre 2013 26 févr. 2009 à 10:52
faudrais commencer par verifier si la page est en ligne, curl est ton ami :)
neigedhiver
Messages postés2480Date d'inscriptionjeudi 30 novembre 2006StatutMembreDernière intervention14 janvier 201119 25 févr. 2009 à 21:05
Salut,
Moi j'ai pour habitude de ne pas noter...
Mais j'ai quelque chose à dire ^^
Ligne 22 tu utilises ereg() : c'est dommage, parce que preg_match() est beaucoup plus performant à motif équivalent. C'est d'autant plus 'important' que tu l'utilises dans une boucle et que tu ne sais pas combien de fois elle va être exécutée.
Et même, pour aller encore plus vite, comme tu veux juste vérifier la présence d'une simple chaîne de caractère dans une autre, l'utilisation de strpos() serait encore plus judicieuse.
Autre petit truc : tu utilises une variable $url_presente, qui n'est pas définie si le lien n'a pas été trouvé. PHP déclenchera une erreur de type notice, qui s'affichera si error_reporting() n'est pas à 0. Et au lieu d'utiliser une chaîne dans la variable, autant utiliser un booléen (true/false), dont c'est justement l'utilité ici (un peu plus performant, surtout si le test est effectué avec === ).
Bon ce ne sont que des détails qui ne remettent pas en question la méthode ;)
26 févr. 2009 à 15:17
Mais je viens de découvrir que mon script a des limites (de très grandes limites :-( ), pour le moment il ne prend pas en compte les url composées (comme par exemple /blabla.php?lg=fr&...)
En plus je n'ai pas encore réussi a faire la liste des liens contenue sur une autre page que la page d'accueil ...
Mais ça va venir, enfin j'espère.
Avez vous une idée de toutes les extension d'une page (pouvant être lu par un navigateur) possible ???
Merci encore de votre aide.
Giloum.
26 févr. 2009 à 10:52
25 févr. 2009 à 21:05
Moi j'ai pour habitude de ne pas noter...
Mais j'ai quelque chose à dire ^^
Ligne 22 tu utilises ereg() : c'est dommage, parce que preg_match() est beaucoup plus performant à motif équivalent. C'est d'autant plus 'important' que tu l'utilises dans une boucle et que tu ne sais pas combien de fois elle va être exécutée.
Et même, pour aller encore plus vite, comme tu veux juste vérifier la présence d'une simple chaîne de caractère dans une autre, l'utilisation de strpos() serait encore plus judicieuse.
Autre petit truc : tu utilises une variable $url_presente, qui n'est pas définie si le lien n'a pas été trouvé. PHP déclenchera une erreur de type notice, qui s'affichera si error_reporting() n'est pas à 0. Et au lieu d'utiliser une chaîne dans la variable, autant utiliser un booléen (true/false), dont c'est justement l'utilité ici (un peu plus performant, surtout si le test est effectué avec === ).
Bon ce ne sont que des détails qui ne remettent pas en question la méthode ;)
25 févr. 2009 à 16:40
Puisque tu est nouveau je te souhaite la bienvenue sur PHPCS :)
Ta source est très utile et je t'encourage à continuer sur la voie du partage.
Bonne prog à toi,
akh
PS : J'ai noté 10 pour t'encourager et aussi car y'à rien à dire (critique) sur ta source. Bonne continuation