Version complète: sur le forum Webmaster Hub : Tester si une image existe
Webmaster Hub > Création et exploitation de Sites Internet > Les langages du Net > PHP
v4np13
Bonjour,
je suis en train d'essayer de génerer mes bannières pour avoir un affichage aléatoire, pour le moment j'utilise ce code mais chaque fois que l'image n'est pas présente dans le répertoire, il m'affiche un warning.

CODE
<?
$background= "../galleriecda/bannieres/ff.jpg";
while (!file($background)){
 $section = "acc";
 $numimage = rand(1,20);
 $extension = "jpg";
 $background = "../galleriecda/bannieres/bancda".$section."_".$numimage.".".$extension;
}
?>

1/ Comment pourrais-je faire pour ne pas avoir les warning?

2/ Existe-il un équivalent à "repeat" de Delphi en PHP?

Merci d'avance smile.gif

<edit>
J'avais oublié de vous montrer quel warning il me donnait:
Warning: file(http://coeurdelardenne.info/galleriecda/bannieres/bancdaacc_2.jpg): failed to open stream: HTTP request failed! HTTP/1.1 404 Not Found in /home/www/999c0bdfb68d1d1b24a9076e2b196adb/web/include/header_.php on line 11
</edit>
captain_torche
Essaye de tester l'existence de l'image avec file_exists.
v4np13
Avec file_exists, il m'affiche un fatal error:
CITATION
Fatal error: Maximum execution time of 10 seconds exceeded in /home/www/999c0bdfb68d1d1b24a9076e2b196adb/web/include/header_.php on line 11


J'avais remplacé
while (!file($background)){

par
while (!file_exists($background)){
captain_torche
Là, ce n'est pas dû à la fonction, mais au fait qu'au bout de 10 secondes, le script ait enclenché une sécurité lui permettant de ne pas boucler infiniment.

Tu es sûr d'avoir des fichiers du type bancdaacc_X.jpg (où 1<X<20) ?

Sinon, personnellement, je te conseillerais une autre méthode :
- Chaque catégorie de bannières dans un sous-répertoire spécifique
- Avec readdir, tu listes tous les fichiers de ton dossier dans un array
- avec array_rand, tu choisis un de ces fichiers au hasard
- et tu affiches l'image retournée.
Ca t'évitera de faire des boucles 'à vide'.
v4np13
J'aurais préféré éviter de lister,... donc générer et tester
Dan
Si tu listes, tu le fais une fois pour chaque bannière, alors que si tu génères un nom au petit bonheur, tu devras tester jusqu'à ce que ce fichier existe réellement. Cela peut être beaucoup plus long, non ?

Dan
v4np13
Mon client supposait qu'il y ait 100bannières, si je les liste toutes avant de choisir, ça me prendrait + de temps que générer et tester, non?
captain_torche
J'ai pas de bench sous la main, mais je doute que lister 100 fichiers, soit si lent que ça.
Portekoi
Bonjour

Il vaut mieux lister : En asp, je traite 1500 fichiers en 3 secondes... Donc pour 100, sauf si tu es sur un pentium 75, ca devrait pas poser de problèmes smile.gif


Portekoi
v4np13
bancdaacc_1.jpg
bancdaacc_2.jpg
jusque 5.

Je viens d'appronfondir mes tests avec file_exists, j'ai réduis ma boucle pour qu'elle aille de 1 à 5 (les images existent), il m'affichait le fatal error.

Je suis revenu avec !file($background), cela fonctionne sauf que si je veux supprimer par exemple l'image banccdaacc_4.jpg il risque de m'afficher le warning:

Warning: file(http://coeurdelardenne.info/galleriecda/bannieres/bancdaacc_4.jpg): failed to open stream: HTTP request failed! HTTP/1.1 404 Not Found in /home/www/999c0bdfb68d1d1b24a9076e2b196adb/web/include/header_.php on line

Comment l'éviter?
Dan
Un petit essai en local: il me faut moins de 4/100èmes de seconde pour lister une centaine de fichiers sur mon vieux PC smile.gif
v4np13
Ok bah je vais creuser dans cette direction alors smile.gif

Merci
Dan
Un bout de code pour tester chez toi:
CODE
<?php
$time_start = microtime();

if ($handle = opendir('.')) {
   echo "Directory handle: $handle<br>";
   echo "Files:<br>";
   while (false !== ($file = readdir($handle))) {
       echo $file."<br>";
   }
   closedir($handle);
}

$time_end = microtime();
$time = $time_end - $time_start;
print "Temps en secondes: <b>".$time."</b><br>";
?>


Vérification faite précisément, je suis entre 0,000263 et 0,000282 seconde pour une centaine de fichiers. wink.gif
captain_torche
Dans le script final, bien sûr, tu n'oublieras pas de ne pas lister les entrées './' et '../'.

Edit : Dan, il y a une raison précise pour laquelle tu utilises print ou echo ?
v4np13
Arf laugh.gif j'en demandais pas tant Dan, je te croyais sur parole wink.gif Mais merci pour la petite démonstration biggrin.gif
Dan
CITATION
Edit : Dan, il y a une raison précise pour laquelle tu utilises print ou echo ?

Non, pas de raison particulière.
Je viens de tester, les deux se valent au niveau temps d'exécution.
Ceci est une version "bas débit" de notre forum. Pour voir la version complète avec plus d'information, la mise en page et les images, veuillez cliquer ici.