function

[Rossella_75]

ciao a tutti,

utilizzo questa funzione per caricare un'immagine a caso dentro una cartella, ma non capisco perchè a volte non viene fuori nulla anche se nella cartella almeno un file di immagine c'è!

function leggi_cartella($path){

$immagine=array();

if (is_dir("$path"))
{
$handle=opendir($path);
$i = 0;
while (false!==($file = readdir($handle)))
{
if ($file != "." && $file != ".." && $file != "Thumbs.db" && $file != "big")
{
$immagine[$i] = "$file";
$i++;
}
}
closedir($handle);
}
return $immagine[rand(0, $i)];
} 

grazie!

[kodode]

perchè questa funzione restituisce un valore casuale... $immagine[rand(0, $i)];
e alcune vuolte restituiusce NULL

[Rossella_75]

grazie mille, come posso evitare questo inconveniente?

[kodode]

dipende da quello che vuoi fare... ad esempio:

<?php
function leggi_cartella($path){ 
    
    $immagine=array(); 
    
    if (is_dir("$path")) { 
        $handle=opendir($path); 
        $i = 0; 
            while (false!==($file = readdir($handle))) { 
                if ($file != "." && $file != ".." && $file != "Thumbs.db" && $file != "big") { 
                $immagine[$i] = "$file"; 
                $i++; 
            } 
        } 
        closedir($handle); 
    } 
    
    return $immagine; 
}

print_r( leggi_cartella("img"));

cosi restituisce l'array intero... ( con print_r vedi il risultato)

[Rossella_75]

io voglio un'immagine a caso... pescata dalla cartella... :master:

[cubalibre810]

<?php
function random_image($path)
{
    $allowed = array('jpg', 'gif', 'png');
    $files = array();
    
    if(!is_dir($path)) die("Cartella non valida.");
    
    foreach(glob($path . "/*") AS $file)
    {
        if(in_array(end(explode('.', $file)), $allowed)) 
            $files[] = $file;
    }
    
    return $files[rand(0, count($files)-1)];
}

echo random_image('immagini'); // Path senza slash finale
?>

perchè questa funzione restituisce un valore casuale... $immagine[rand(0, $i)];
e alcune vuolte restituiusce NULL

Che idea fantasiosa
La tua funzione non va perchè incrementi $i dopo aver riempito l'array, quindi se ad esempio nella cartella hai 4 file ($files[3] come indice massimo) avrai il secondo parametro della funzione rand() settato a 4 e se esce 4 come numero casuale non lo trova nell'array

[kodode]

le due frasi "restituuisce NULL" e " come numero casuale non lo trova nell'array"

alla fine della giostra non sono cosi diverse... o no?!?

[cubalibre810]

Il filtro per estensione si può fare direttamente con glob()

<?php
function random_image($path)
{
    $files = array();
    
    if(!is_dir($path)) die("Cartella non valida.");
    
    foreach(glob($path . "/*.{jpg,gif,png}", GLOB_BRACE) AS $file)   $files[] = $file;
    
    return $files[rand(0, count($files)-1)];
}

echo random_image('immagini'); // Path senza slash finale
?>

[Rossella_75]

grazie mille, cosi è perfetto, solo un piccolo problema

io poi la richiamo cosi

$immagine = random_image("img/immibili/$id"); 

e stampo in questo modo

<tr><td align="center" style="padding-top:11px; padding-bottom:11px"><a href="'.$immagine.'" rel="clearbox[gal1]">[img]'.$immagine.'[/img]</td></tr>

come vedi

<a href="'.$immagine.'" rel="clearbox[gal1]">

l'hafer serve per aprire l'ingrandimento ma dovrebbe puntare ad una sotto cartella (big) per pescare l'immagine ingrandita...

ma cosi non riesco a dargli il percorso...

[cubalibre810]

Originariamente inviato da kodode
le due frasi "restituuisce NULL" e " come numero casuale non lo trova nell'array"

alla fine della giostra non sono cosi diverse... o no?!?

Insomma, NULL è diverso da 4 (nel caso di 3 file), la funzione è giusta, è il contatore che è di troppo, sarebbe stato sufficiente un $files[].
Non restituisce mai NULL comunque