ALBUM PHOTO (ANOTHER ONE)

TheSin Messages postés 331 Date d'inscription mardi 12 novembre 2002 Statut Membre Dernière intervention 10 février 2009 - 13 mai 2006 à 09:21
Teclis01 Messages postés 1423 Date d'inscription mardi 14 décembre 2004 Statut Membre Dernière intervention 29 décembre 2012 - 14 mai 2006 à 01:58
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/37567-album-photo-another-one

Teclis01 Messages postés 1423 Date d'inscription mardi 14 décembre 2004 Statut Membre Dernière intervention 29 décembre 2012 4
14 mai 2006 à 01:58
désolé mais même si ma gallerie ne sait qu'utiliser le jpg faute de temps j'ai eu honte de la poster et là ça me fait mal de voir qqun qui a lister 2 dossiers et qui fait des echo (pardon printf).
J attend de voir les améliorations ...
LeVicking Messages postés 4 Date d'inscription vendredi 24 janvier 2003 Statut Membre Dernière intervention 13 mai 2006
13 mai 2006 à 17:56
Salut,
pour ce qui est du HTML, j'ai utilisé un wysiwyg car je ne suis pas orienté web dans mes codes, alors, il se peut en effet que je en soit pas à la page. Ensuite je me doutais que ce code était optimisable et qu'on pouvait faire les miniatures par le code, mais j'ai voulu faire au plus simple et surtout au plus rapide. Mais je prends en compte les remarques et j'optimiserai le code dès que je pourrai.
coockiesch Messages postés 2268 Date d'inscription mercredi 27 novembre 2002 Statut Membre Dernière intervention 13 septembre 2013 4
13 mai 2006 à 10:39
Salut!
Petites remarques supplémentaires:
- le HTML est pas très beau ( center, font, ... on les utilise plus )
- je préférerai que le script liste les fichiers contenus dans le dossier pour les grandes images et qu'ils crée les miniatures qui n'existent pas

@++

R@f
TheSin Messages postés 331 Date d'inscription mardi 12 novembre 2002 Statut Membre Dernière intervention 10 février 2009
13 mai 2006 à 09:26
par ailleurs, t'as vraiment besoin du $i ? je remplacerais :
while($image = readdir($dossier))
{
// Mémoriser dans un tableau le nom des images
$images[] = $image;
$i++;
}
par
while($image = readdir($dossier))
$images[] = $image; # Là je ferais même un array_push, mais à chacun sa méthode ^^


$i = count($images);
TheSin Messages postés 331 Date d'inscription mardi 12 novembre 2002 Statut Membre Dernière intervention 10 février 2009
13 mai 2006 à 09:21
arf, on est obligé d'avoir 2x l'image ....
en fait c'est un simple listage de dossier que tu nous fait là ^^
c'est quand tu veux toi aussi pour changer les ptintf() en "echo", et les guillemets (") en apostrophes (') ;-)

n'empeche que utiliser du printf, dans ton cas c'est pas un peu exagérer ?
(dans le sens où on peut faire un "echo 'text'.$var1.'suite'.$var2;" et que ça ne sert qu'à ça).

Bref ça serait bien optimisable ce petit bout de code ...
Rejoignez-nous