CHERCHER UNE IMAGE SUR FLICKR

cs_yoman64 Messages postés 592 Date d'inscription samedi 19 janvier 2002 Statut Membre Dernière intervention 4 décembre 2008 - 15 mars 2008 à 13:55
dvdstory Messages postés 192 Date d'inscription dimanche 23 janvier 2005 Statut Membre Dernière intervention 10 août 2009 - 19 mars 2008 à 14:07
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.

https://codes-sources.commentcamarche.net/source/46065-chercher-une-image-sur-flickr

dvdstory Messages postés 192 Date d'inscription dimanche 23 janvier 2005 Statut Membre Dernière intervention 10 août 2009
19 mars 2008 à 14:07
Pour ceux que ça interesse, l'on peut facilement chercher une image sur Flickr grace au framework ZEND qui intègre parfaitement le dialogue entre votre serveut et les API de Flickr
cs_BlAcKbUrRy Messages postés 6 Date d'inscription samedi 15 mars 2008 Statut Membre Dernière intervention 15 mars 2008
15 mars 2008 à 14:11
Salut,
Merci pour ton commentaire.

Cette source n'est pas encore terminer c'est pour ça.
cs_yoman64 Messages postés 592 Date d'inscription samedi 19 janvier 2002 Statut Membre Dernière intervention 4 décembre 2008
15 mars 2008 à 13:55
Salut,

Ton code est pas très propre, parfois tu utilise des simples quotes ' ' et parfois des doubles " " , tu devrais essayer d'être plus constant de ce coté la.... Des fois tu utilises echo avec des parenthèses et d'autres fois non.

Ensuite veux tu bien me dire à quoi ça sert ça:

# if (!isset($_GET['pge']))
# {
# define('page', 1);
# }
# else
# {
# if (is_numeric($_GET['pge']))
# {
# define('page', $_GET['pge']);
# }
# else
# {
# define('page', 1);
# }
# }

?

C'est tellement inutile de déclaré une constance ici :-/
Tout ça peut être remplacé avantageusement par

# if (!isset($_GET['pge']) || !is_numeric($_GET['pge'])) (int) $_GET['pge'] = 1;

Ici (bon cette remarque la est un peu casse couilles, c'est juste pour bien finir mes remarques ^^):

# for ($i = 1; $i != 8; $i++)
# {
# $e = page+$i;
# echo " $e";
# }

Il n'y a pas d'interêt à calculer à chaque fois $e = page+$i, aussi bien commencer à la bonne page
# for ($i $_GET['pge']+1 , $max $_GET['pge']+8; $i !== $max; ++$i)
# {
# echo " $i";
# }

Je crois aussi qu'on pourrait très bien se basé de phpflickr qui est probablement lourd pour rien ici mais bon cela n'engage que moi...

Sans parler de la qualité du code xhtml (les arguments devrait être entre double quotes, pas des simples. Les imgs doivent absolument avoir un texte alternatif, etc.

Bref beaucoup de ménages à faire :)

Bonne prog et @+
Rejoignez-nous