PHP检查图片是否存在并本地保存失败问题排查及优化

首先，你的代码无法保存图片的核心问题出在文件存在性检查的逻辑上，另外还有几个容易踩坑的细节需要处理，我逐个给你拆解：

一、当前代码的关键错误 #

1. 文件存在性检查逻辑完全失效 #

你用了foreach (glob($image) as $file)来遍历匹配的文件，但如果目标文件不存在，glob()会返回一个空数组，这个循环根本不会执行，自然永远不会调用save_image()函数。这也是为什么你看不到图片被下载的直接原因。

2. 目标目录可能不存在 #

如果folder/img这个目录还没创建，fopen($image_file, 'w+')会直接失败，因为PHP无法自动创建多级目录。

3. 缺少错误处理机制 #

CURL请求可能因为网络问题、URL无效等原因失败，但你没有检查curl_exec()的返回值，也没有捕获任何错误，出问题时完全不知道哪里出了问题。

4. 相对路径的不确定性 #

如果你的PHP脚本不在网站根目录，folder/img/这种相对路径可能会指向错误的位置，导致文件保存到意想不到的地方。

二、修复与优化后的代码 #

我把这些问题都修复了，同时优化了逻辑，让代码更健壮：

function save_image($image_url, $image_file) {
    // 先确保目标目录存在
    $dir = dirname($image_file);
    if (!is_dir($dir)) {
        mkdir($dir, 0755, true); // 递归创建多级目录
    }

    $fp = fopen($image_file, 'w+');
    if (!$fp) {
        error_log("无法打开文件句柄: $image_file");
        return false;
    }

    $ch = curl_init($image_url);
    curl_setopt($ch, CURLOPT_FILE, $fp);
    curl_setopt($ch, CURLOPT_FOLLOWLOCATION, true);
    curl_setopt($ch, CURLOPT_TIMEOUT, 1000);
    curl_setopt($ch, CURLOPT_USERAGENT, 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/114.0.0.0 Safari/537.36');
    // 如果需要忽略SSL证书，取消下面注释
    // curl_setopt($ch, CURLOPT_SSL_VERIFYPEER, false);
    // curl_setopt($ch, CURLOPT_SSL_VERIFYHOST, false);

    $success = curl_exec($ch);
    
    // 检查CURL错误
    if (!$success) {
        $error_msg = curl_error($ch);
        error_log("CURL请求失败: $error_msg，URL: $image_url");
        fclose($fp);
        unlink($image_file); // 删除空文件
        curl_close($ch);
        return false;
    }

    // 检查HTTP状态码是否为200（成功）
    $http_code = curl_getinfo($ch, CURLINFO_HTTP_CODE);
    if ($http_code !== 200) {
        error_log("HTTP请求失败，状态码: $http_code，URL: $image_url");
        fclose($fp);
        unlink($image_file);
        curl_close($ch);
        return false;
    }

    curl_close($ch);
    fclose($fp);
    return true;
}

function download_images() {
    global $db;
    $query = $db->get_results("SELECT `id`,`title`,`url_image` FROM table");
    
    if (!$query) {
        echo "No data found.";
        return;
    }

    // 建议使用绝对路径，避免相对路径的问题
    $base_dir = __DIR__ . '/folder/img/'; // __DIR__代表当前脚本所在目录
    foreach ($query as $data) {
        $image_filename = slugify($data->title) . "-img-" . $data->id . ".png";
        $image_path = $base_dir . $image_filename;

        // 直接检查文件是否存在，逻辑简单直接
        if (!file_exists($image_path)) {
            $save_result = save_image($data->url_image, $image_path);
            if ($save_result) {
                echo "成功下载图片: $image_filename<br>";
            } else {
                echo "下载失败: $data->url_image<br>";
            }
        } else {
            echo "图片已存在，跳过: $image_filename<br>";
        }
    }
}

三、关键优化点说明 #

1. 修复文件存在性检查：直接用file_exists($image_path)替代错误的glob()遍历逻辑，逻辑清晰且可靠。
2. 自动创建目录：在save_image()中先检查并递归创建目标目录，避免因目录不存在导致的保存失败。
3. 完善错误处理：
   • 检查fopen()是否成功打开文件句柄
   • 捕获CURL请求的错误信息
   • 检查HTTP响应状态码，确保图片请求成功
   • 失败时自动删除空文件，避免垃圾文件残留
4. 使用绝对路径：用__DIR__拼接绝对路径，彻底解决相对路径的不确定性问题。
5. 返回布尔值表示操作结果：让save_image()返回true/false，方便上层逻辑判断下载是否成功。

四、额外建议 #

• 可以给slugify()函数加个兜底逻辑，如果转换后的字符串为空，用id作为文件名的一部分，避免生成无效的文件名。
• 如果图片数量很多，建议加入批量处理的延迟（比如sleep(1)），避免对源服务器造成过大压力，同时防止被封禁。
• 可以在数据库中增加一个字段（比如local_image_path），保存本地图片的路径，下次就不用再拼接文件名，直接读取即可。

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