NodeJS中遍历图片调用getEXIF函数返回undefined问题排查

核心问题 #

你的getEXIF函数没有返回值，且内部依赖异步回调逻辑：

• 调用getEXIF(element)时，函数仅启动了ExifImage的异步任务，但本身没有任何return语句，所以必然返回undefined。
• 单独调用时看到的输出是函数内部console.log(exifData.image)的结果，并非函数返回值，这造成了“函数有返回值”的错觉。

修复方案 #

需要将getEXIF重构为返回Promise的异步函数，同时用支持异步等待的循环替代forEach（forEach无法正确处理异步函数的等待逻辑）。

修改后的完整代码

const fs = require('fs');
const mime = require('mime');
const ExifImage = require('exif').ExifImage;

function getFiles(dir, files = []) {
  const fileList = fs.readdirSync(dir);
  for (const file of fileList) {
    const name = `${dir}/${file}`;
    if (fs.statSync(name).isDirectory()) {
      getFiles(name, files);
    } else {
      files.push(name);
    }
  }
  return files;
}

// 重构为返回Promise的异步函数
function getEXIF(filepath) {
  return new Promise((resolve, reject) => {
    try {
      new ExifImage({ image: filepath }, function (error, exifData) {
        if (error) {
          console.log('Error: ' + error.message);
          reject(error);
        } else {
          resolve(exifData.image);
        }
      });
    } catch (error) {
      console.log('Error: ' + error.message);
      reject(error);
    }
  });
}

// 使用async/await + for...of处理异步遍历
async function processImages() {
  const filearr = getFiles('D://Photoes', []);
  for (const element of filearr) {
    const ftype = mime.getType(element);
    if (ftype === 'image/jpeg') {
      try {
        const exifResult = await getEXIF(element);
        console.log(exifResult); // 此处可获取真实的EXIF结果
      } catch (err) {
        // 可在此处理单个文件的读取错误
      }
    }
  }
}

// 启动图片处理流程
processImages();

关键改动说明

1. getEXIF函数重构：

   • 用Promise包裹异步逻辑，成功时通过resolve返回EXIF数据，失败时通过reject抛出错误。
   • 移除函数内部的console.log，将结果交付给调用方处理，保证函数职责单一。

2. 遍历方式替换：

   • 改用for...of循环配合async/await，确保每张图片的EXIF读取完成后再处理下一张，避免异步任务执行顺序混乱。
   • 用try/catch捕获单个文件的处理错误，不会因某一张图片读取失败导致整个流程中断。

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