Laravel多文件上传控制器失效：foreach循环无法正常执行

你的代码里存在几个明显的逻辑错误，同时还要注意前端表单的配置，我来逐个帮你梳理：

核心代码错误分析 #

• 循环内误用变量：你在foreach($file as $doc)里把单个文件赋值给了$doc，但后续调用getClientOriginalName()和move()时，却用了外层的$file（这是所有上传文件的集合），这不仅会导致逻辑混乱，甚至会直接抛出错误，让循环无法正常执行。
• 数组赋值错误：你初始化了$doc_url = []数组来存储所有文件路径，但赋值时写成了$doc_url = $filePath，这会把数组直接覆盖成单个字符串，而不是逐个追加元素。
• 变量命名混淆：外层的$file其实是多个文件的集合，建议改成$files，这样语义更清晰，避免和循环内的单个文件变量混淆。

修正后的完整代码 #

if($request->hasfile('document')){ 
    $doc_name = []; 
    $doc_url = []; 
    // 这里把变量名改成$files，更符合多文件集合的语义
    $files = $request->file('document'); 
    foreach($files as $doc){ 
        // 使用当前循环的$doc对象获取原文件名
        $fileName = "emp". $emp_id ."-". time() . $doc->getClientOriginalName(); 
        $doc_name[] = $fileName; 
        // 调用当前$doc对象的move方法，而不是request里的document
        $doc->move(public_path("/storage/expense"), $fileName); 
        $filePath = '/storage/expense/'. $fileName; 
        // 以数组追加的方式存储每个文件路径
        $doc_url[] = $filePath; 
    }
    // 这里可以添加把$doc_name和$doc_url存入数据库等后续逻辑
}

前端表单必须检查的配置 #

如果后端代码修正后还是不行，那大概率是前端表单的问题：

• 表单必须设置enctype="multipart/form-data"，否则服务器无法解析上传的文件
• 文件输入框的name属性必须写成document[]（带方括号），并且加上multiple属性，这样才能支持多选文件

示例表单代码：

<form method="POST" action="/your-upload-route" enctype="multipart/form-data">
    @csrf
    <!-- 其他员工ID等表单字段 -->
    <input type="file" name="document[]" multiple>
    <button type="submit">上传文件</button>
</form>

内容的提问来源于stack exchange，提问作者 Developer