Problema al poner imágenes aleatorias

[volaveruntmissit]

Encontré éste código navegando, tiene un buen funcionamiento, pero un error.

Es para poner imágenes de forma aleatoria que estén dentro de una carpeta, puedo tener 10 o 20 imágenes, no hace falta agregar una por una en el código, las encuentra en automático tan sólo con estar dentro de la carpeta, eso es lo que necesito:

Código PHP:

  <?php 
 function imagen_aleatoria($path) {  
 $F = 0;  
 $dir = opendir($path);  
 while ($file = readdir($dir)) { if ($file != "." && $file != "..") { $archivos[$F] = $file; $F++; } }    
 $muestra = rand(0,$F);    
return "<img src=$path/$archivos[$muestra]>"; 
 }  
 ?>   

Y para publicar la imagen en el cuerpo del contenido:

Código PHP:

  <?  echo imagen_aleatoria("imagenes"); ?>   

(Donde "imagenes" es la carpeta donde quiero que búsque)

...
...

EL PROBLEMA:

Si dentro de esa carpeta tengo por ejemplo las imagenes: 1.jpg, 2.jpg y 3.jpg, el error es que a veces de forma aleatoria aparece unicamente la carpeta raiz:

<img src=www.mi-web.com/imagenes/3.jpg>
<img src=www.mi-web.com/imagenes/>
<img src=www.mi-web.com/imagenes/1.jpg>
<img src=www.mi-web.com/imagenes/2.jpg>
<img src=www.mi-web.com/imagenes/>


No se como solucionar ese problema para que únicamente aparezcan las imágenes, y solucionar el error de que a veces sólo aparece la dirección de la carpeta. Agradezco su ayuda!
.

[carlos_belisario]

realiza una validación de que no este vacío

Código PHP:

if(!empty($archivos[$muestra])) {
   return "<img src=\"$path/$archivos[$muestra]\">";
}
 

además también pudieras validar que es una imagen, saludos

[Ekel]

la mejor manera de listar archivos dentro de un directorio es con la funcion glob


algo asi

Código PHP:

  <?php 
$files = glob("directorio/*.jpg"); 
 
foreach ($files as $nombre_archivo) { 
    echo "Tamaño de $nombre_archivo " . filesize($nombre_archivo) . "\n"; 
} 
?>   

y si quieres hacerlo aleatoriamente podrías:

Código PHP:

  $files = glob('firectorio/*.jpg'); 
$rand = $files[mt_rand(0,(count($files)-1))];  
   

esto te devolveria una imagen aleatoria almacenada en directorio...


http://php.net/manual/es/function.glob.php

en el manual encuentras más información.

un saludo ;D

[volaveruntmissit]

Iniciado por carlos_belisario

realiza una validación de que no este vacío

Código PHP:

if(!empty($archivos[$muestra])) {
   return "<img src=\"$path/$archivos[$muestra]\">";
}
 

además también pudieras validar que es una imagen, saludos

Gracias por ayudar! Pero, bueno, tu código funciona como un "parche", es decir si pongo 5 imagenes de forma aleatoria de esta forma:

Código PHP:

  
 
<?  echo imagen_aleatoria("imagenes"); ?> 
<?  echo imagen_aleatoria("imagenes"); ?> 
<?  echo imagen_aleatoria("imagenes"); ?> 
<?  echo imagen_aleatoria("imagenes"); ?> 
<?  echo imagen_aleatoria("imagenes"); ?>   

Y cuando encuentra el error en la tercera linea de leer solo la carpeta como es el problema que menciono, hace esto, se imprimen unicamente 2 lineas aunque haya puesto 5.

<img src="www.mi-web.com/imagenes/1.jpg">
<img src="www.mi-web.com/imagenes/3.jpg">

..

[volaveruntmissit]

Iniciado por Ekel

la mejor manera de listar archivos dentro de un directorio es con la funcion glob


algo asi

Código PHP:

  <?php 
$files = glob("directorio/*.jpg"); 
 
foreach ($files as $nombre_archivo) { 
    echo "Tamaño de $nombre_archivo " . filesize($nombre_archivo) . "\n"; 
} 
?>   

y si quieres hacerlo aleatoriamente podrías:

Código PHP:

  $files = glob('firectorio/*.jpg'); 
$rand = $files[mt_rand(0,(count($files)-1))];  
   

esto te devolveria una imagen aleatoria almacenada en directorio...


[url]http://php.net/manual/es/function.glob.php[/url]

en el manual encuentras más información.

un saludo ;D

GRACIAS!!!

Problema solucionado, tu código funciona correcto. Saludos y gracias!


.