PHP文件上传-多个文件

Question

我正在尝试使用以下 PHP 上传多个图像。上传的图像数量可以变化。

我似乎遇到的问题是图像编号 1 没有上传，但它的文件路径正在打印到屏幕上。

代码：-

if ($_FILES['pac_img_1']['name']>""){
    echo("You have uploaded the following images:-<ul>");
    for ($i=1; $i<=$imagesCount; $i++){ 
        $target_path = "files/" . $companyName . "/images/";
        $target_path = $target_path . basename( $_FILES['pac_img_' . $i]['name']); 
        if(move_uploaded_file($_FILES['pac_img_' . $i]['tmp_name'], $target_path)) {
            echo "<li><a href='". $target_path . "'>".  basename( $_FILES['pac_img_' . $i]['name']). "</a></li>";
        } else{
            echo "There was an error uploading an image";
        }
};
    echo("</ul>");
}else{
    echo("None uploaded");
};

我根据以前使用过的一些代码改编了它，所以我怀疑我在这里犯了“小学生”错误。

如有帮助，将不胜感激。

编辑以添加 $imagesCount 通过 $_POST 请求从表单元素获取其值。当仅上传一张图像时，该值 = 0。

Answer 1

根本不是 php-dude，我会尝试更改

for ($i=1; $i<=$imagesCount; $i++){

为

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

- 或者可能

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

取决于 $imagesCount 的设置方式。

Answer 2

您的 for 循环需要修改。数组索引从 0 开始。最后一个元素应该是数组长度 - 1；
您的 for 循环需要修改为以下代码示例。

实际上，它循环了几个 $_POST 项。他的 HTML 可能有类似这样的内容：

<input type="file" name="pac_img_1">
<input type="file" name="pac_img_2">
<input type="file" name="pac_img_3">

他正在尝试获取这些图像。

我会采取不同的做法。

HTML：（

<input type="file" name="pac_img[]" />
<input type="file" name="pac_img[]" />
<input type="file" name="pac_img[]" />

请注意，您可以动态添加文件输入，而不必担心名称）

PHP：

if (count($_FILES['pac_img']) > 0){
    echo("You have uploaded the following images:-<ul>");

    foreach($_FILES['pac_img'] as $key => $file){
        $target_path = "files/" . $companyName . "/images/";
        $target_path = $target_path . basename( $file['name']); 
        if(move_uploaded_file($file['tmp_name'], $target_path)) {
            echo "<li><a href='". $target_path . "'>".  basename( $file['name'] ). "</a></li>";
        } else{
            echo "There was an error uploading an image";
        }
    }
    echo("</ul>");
}else{
    echo("None uploaded");
}

最后但并非最不重要的一点：始终检查上传的文件是否符合其预期。 （http://www.acunetix.com/websitesecurity/upload-forms-threat。嗯)

Answer 3

您的 for 循环需要修改。数组索引从 0 开始。最后一个元素应该是数组长度 - 1；

您的 for 循环需要修改为以下代码示例。

if ($_FILES['pac_img_1']['name']>""){
    echo("You have uploaded the following images:-<ul>");
    for ($i=0; $i<$imagesCount; $i++){ 
        $target_path = "files/" . $companyName . "/images/";
        $target_path = $target_path . basename( $_FILES['pac_img_' . $i]['name']); 
        if(move_uploaded_file($_FILES['pac_img_' . $i]['tmp_name'], $target_path)) {
            echo "<li><a href='". $target_path . "'>".  basename( $_FILES['pac_img_' . $i]['name']). "</a></li>";
        } else{
            echo "There was an error uploading an image";
        }
};
    echo("</ul>");
}else{
    echo("None uploaded");
};