Laravel7实现多图片打包为zip文件下载报错如何解决

• ZipArchive::addFile() 仅支持本地文件系统路径，你代码中直接传入远程HTTP/HTTPS链接作为文件路径，是导致报错的核心原因。
• 生成的ZIP文件未指定合法的可写存储路径，直接在代码运行目录创建，大概率存在权限不足的问题。
• 未做异常兜底逻辑：如果ZIP创建失败，代码依然尝试返回下载响应，会触发二次报错。
• 可优化点：原逻辑会在服务器残留生成的ZIP文件，占用磁盘空间，建议下载后自动删除。

public function download($id) // 修正原方法名拼写错误
{
    $inbox = Inbox::findOrFail($id); // 改用findOrFail避免id不存在时触发后续报错
    $folderName = $inbox->id . '-' . str_replace(' ', '-', $inbox->order_name);
    $fileName = $folderName . '.zip';
    // 指定存储到storage临时目录，确保目录可写
    $zipPath = storage_path('app/temp/' . $fileName);
    // 自动创建临时目录避免目录不存在报错
    if (!file_exists(storage_path('app/temp'))) {
        mkdir(storage_path('app/temp'), 0755, true);
    }

    $zip = new \ZipArchive;
    if ($zip->open($zipPath, \ZipArchive::CREATE | \ZipArchive::OVERWRITE) === TRUE) {
        $urlPrefix = config('yourstitchart.file_url');
        $inboxFiles = json_decode($inbox->file); // 复用已查询的inbox对象，无需重复查库

        foreach ($inboxFiles as $filePath) {
            // 先读取远程文件内容
            $fileContent = file_get_contents($urlPrefix . $filePath);
            $relativeName = basename($filePath);
            // 用addFromString添加文件内容，无需提前落地原文件到本地
            $zip->addFromString($relativeName, $fileContent);
        }
        $zip->close();
    } else {
        abort(500, 'ZIP文件创建失败，请检查目录权限');
    }

    // 下载完成后自动删除临时ZIP文件，避免占用磁盘空间
    return response()->download($zipPath)->deleteFileAfterSend(true);
}

• 确保storage/app/temp目录权限为755，运行PHP的进程用户（如www-data）拥有该目录的读写权限
• 如果远程文件体积较大，建议改用流式读取逻辑避免内存溢出，也可以先将远程文件下载到本地临时目录后再添加到ZIP
• 可按需添加远程文件存在性校验，避免文件读取失败导致ZIP内容缺失

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