PHP for array only loops once

Question

This for array only loops once, instead of 3 (numbers of files i want to insert into the database), i've been looking at it for 2 hours now, and cant seem to locate the error. Please help.. (It's the insert for loop that only runs once, the 2 foreach works fine)

$upload = $_FILES['upload'];

$i=0;
foreach($upload['tmp_name'] as $key=>$value){
$tmp_name[$i]=$value;
$i++;
}

$i=0;
foreach($upload['name'] as $key=>$value){
$name[$i]=$value;
$i++;
}

for($i=0; $i < count($name); $i++){
    $insert_image = "INSERT INTO ".$image_table." (";

    $lastImage = end($image_rows);
    for ($i=0; $i < count($image_rows); $i++){
    $insert_image .= "".$image_rows[$i]."";
    if($image_rows[$i] != $lastImage){
        $insert_image .= ", ";
    }
    }

    $insert_image .= ") VALUE ('".$upload['name'][$i]."', '".$latest_id."')";
    mysql_query($insert_image) or die(mysql_error ());
}

HTML: <input type='file' name='upload[]'>

why three? why not for 4? i dont understand from where you are getting three. — robert
– robert, Commented Sep 29, 2013 at 14:04
I have 3 file uploads, therefore the $name array only has 3 file names. — Sultenhest
– Sultenhest, Commented Sep 29, 2013 at 14:07
Try var_dump(count($name)); after your line for($i=0; $i < count($name); $i++). What will it output? — nni6
– nni6, Commented Sep 29, 2013 at 14:14

Kyle · Accepted Answer · 2013-09-29 14:06:16Z

5

Probably because you are reassigning the value of $i:

for ($i=0; $i < count($image_rows); $i++){

answered Sep 29, 2013 at 14:06

Kyle

4,0342 gold badges33 silver badges46 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Sultenhest Over a year ago

Still only the last file gets insert in the database

Sultenhest Over a year ago

Changed everything in the for loop to: $insert_image = "INSERT INTO ".$image_table." (".$image_rows.") VALUE ('".$upload['name'][$i]."', '".$latest_id."')";. Works now! Thanks

Collectives™ on Stack Overflow

PHP for array only loops once

1 Answer 1

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related