Problema script multi upload

Ciao a tutti, ho fatto questo script per il caricamento di più di un'immagine. Tutte le immagini caricate si chiamano image1, image2 ecc.. Ecco lo script:

[...]

$idDiario = $row['mid'] + 1;
mkdir("immagini/diari/".$idDiario, 0777);

//UPLOAD
for($i = 0; $i; $i++){
            
    if(!isset($_FILES['image'.$i])) break;
                
    $path = "immagini/diari/";
    $max_size = 20000000;

    if(is_uploaded_file($_FILES['image'.$i]['tmp_name'])){
        if($_FILES['image'.$i]['size'] > $max_size){ echo "Il file &egrave; troppo grande e non potr&agrave; essere caricato!<br>\n"; exit; }
        $res = move_uploaded_file($_FILES['image'.$i]['tmp_name'], $path.$idDiario."/".$i.".png");
        if(!$res){ echo "Upload fallito!<br>\n"; exit; }
    }
} 

Però non carica nessuna immagine. Cosa c'è che non va?

[alemoppo]

Esistono le cartelle "immagini", "diari" e "id_del_diario"?

Poi:

if(is_uploaded_file($_FILES['image']['tmp_name'])){ 

non dovrebbe essere:

if(is_uploaded_file($_FILES['image'.$i]['tmp_name'])){ 

?

Ciao!

Si, i percorsi sono giusti, tanto che la cartella la crea con il mkdir! Già provato anche quello, ma l'upload non va comunque..

[alemoppo]

Ah ecco... prima non c'era la mkdir.

Comunque, hai corretto l'errore nella is_uploaded_file() ?

Lo script, restituisce qualche errore?

Ciao!

Corretto, si. Inoltre lo script non restituisce errori..


EDIT: Ho fatto stampare dei messaggi all'interno del ciclo for, ma non li stampa affatto. Significa che il ciclo non parte proprio. Ma perchè?

[alemoppo]

Dove lo hai messo l'echo? Prima o dopo della riga:

if(!isset($_FILES['image'.$i])) break; 

?

Perché, se l'hai messo dopo, FORSE può essere che in realtà non esistono le varie $_FILES['image'.$i]...

Ciao!

E' questo il bello. L'ho messo anche prima..

[alemoppo]

ma LOL

Scusami, ma il for non l'avevo nemmeno letto!

for($i = 0; $i; $i++){ 

Ovviamente non può andare, perché all'inizio $i è 0, quindi false...

Dovresti fare una cosa come:

for($i = 0; $i<$k; $i++){ 

Dove $k è il numero di file caricati...

Oppure, (soluzione brutta), nel tuo caso visto che hai messo quel break, puoi sostituirlo anche con un:

$i=0;
while(1) {
///codice
++$i;
} 

oppure

for($i = 0; ; $i++){ 

... ed esce dal ciclo tramite il break che hai messo...

Ciao!

Grazie mille! Adesso va :D