TheSin
Messages postés331Date d'inscriptionmardi 12 novembre 2002StatutMembreDernière intervention10 février 2009
-
13 mai 2006 à 09:21
Teclis01
Messages postés1423Date d'inscriptionmardi 14 décembre 2004StatutMembreDernière intervention29 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.
Teclis01
Messages postés1423Date d'inscriptionmardi 14 décembre 2004StatutMembreDernière intervention29 décembre 20124 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és4Date d'inscriptionvendredi 24 janvier 2003StatutMembreDernière intervention13 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és2268Date d'inscriptionmercredi 27 novembre 2002StatutMembreDernière intervention13 septembre 20134 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és331Date d'inscriptionmardi 12 novembre 2002StatutMembreDernière intervention10 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és331Date d'inscriptionmardi 12 novembre 2002StatutMembreDernière intervention10 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 ...
14 mai 2006 à 01:58
J attend de voir les améliorations ...
13 mai 2006 à 17:56
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.
13 mai 2006 à 10:39
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
13 mai 2006 à 09:26
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);
13 mai 2006 à 09:21
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 ...