ALBUM PHOTO (ANOTHER ONE)

Signaler
Messages postés
331
Date d'inscription
mardi 12 novembre 2002
Statut
Membre
Dernière intervention
10 février 2009
-
Messages postés
1423
Date d'inscription
mardi 14 décembre 2004
Statut
Membre
Dernière intervention
29 décembre 2012
-
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

Messages postés
1423
Date d'inscription
mardi 14 décembre 2004
Statut
Membre
Dernière intervention
29 décembre 2012
4
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 ...
Messages postés
4
Date d'inscription
vendredi 24 janvier 2003
Statut
Membre
Dernière intervention
13 mai 2006

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.
Messages postés
2268
Date d'inscription
mercredi 27 novembre 2002
Statut
Membre
Dernière intervention
13 septembre 2013
3
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
Messages postés
331
Date d'inscription
mardi 12 novembre 2002
Statut
Membre
Dernière intervention
10 février 2009

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);
Messages postés
331
Date d'inscription
mardi 12 novembre 2002
Statut
Membre
Dernière intervention
10 février 2009

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 ...