Node.js中Promise始终Pending，图片分数计算返回undefined问题求助

我来帮你排查下代码里的问题，一步步解决你的Promise pending、分数undefined和程序停滞的问题：

核心问题拆解 #

你的代码主要有几个关键的异步逻辑错误，导致了当前的问题：

1. Promise初始化时机错误：你在循环外提前创建了promiseImageScore，此时pathToImage还是空字符串，而且这个Promise只会执行一次——后面循环里修改pathToImage根本不会重新触发getImageScore，所有文件都复用了同一个初始的Promise（而这个Promise的resolve值是undefined）。
2. getImageScore没有正确封装异步流程：这个函数内部全是异步回调（img.toBuffer、img.size、str.on("parsed")），但它没有返回Promise，直接调用它会立刻返回undefined，导致你一开始的resolve(getImageScore())直接resolve了undefined。
3. 异步回调里的return无效：在str.on("parsed")回调里return分数完全没用，这个回调的返回值无法传递到外层的Promise里，必须手动调用Promise的resolve来传递结果。
4. 错误处理缺失：原来的代码里，异步回调的错误只是返回Error对象，没有触发Promise的reject，导致错误无法被捕获，甚至可能让程序卡住。

修复后的完整代码 #

我重构了你的代码，解决了所有上述问题，同时让异步逻辑更清晰：

const fs = require('fs');
const gm = require('gm').subClass({imageMagick: true});
const PNG = require("pngjs").PNG;

const pathToFolder = '/home/eugen/Pictures/wallpapers1';

// 重构为返回Promise的函数，把图片路径作为参数传入，避免全局变量副作用
function getImageScore(pathToImage) {
  return new Promise((resolve, reject) => {
    console.log(`开始处理图片: ${pathToImage}`);
    const img = gm(pathToImage);

    // 获取PNG buffer
    img.toBuffer("PNG", (err, buff) => {
      if (err) {
        console.error(`获取buffer失败: ${err}`);
        return reject(err);
      }
      console.log(`获取到图片buffer`);

      // 获取图片尺寸
      img.size((err, size) => {
        if (err) {
          console.error(`获取图片尺寸失败: ${err}`);
          return reject(err);
        }
        console.log(`图片尺寸: ${size.width}x${size.height}`);

        // 解析PNG文件
        const pngParser = new PNG();
        pngParser.on("parsed", (buffer) => {
          let totalScore = 0;
          // 遍历所有像素计算分数
          for (let y = 0; y < size.height; y++) {
            for (let x = 0; x < size.width; x++) {
              const pixelIndex = (size.width * y + x) << 2;
              const r = buffer[pixelIndex];
              const g = buffer[pixelIndex + 1];
              const b = buffer[pixelIndex + 2];
              totalScore += (r + g + b) / 765; // 归一化到0-1区间
            }
          }
          const averageScore = totalScore / (size.height * size.width);
          console.log(`图片处理完成，分数: ${averageScore}`);
          resolve(averageScore);
        });

        // 捕获PNG解析错误
        pngParser.on("error", (e) => {
          console.error(`PNG解析错误: ${e}`);
          reject(e);
        });

        // 写入buffer开始解析
        pngParser.end(buff);
      });
    });
  });
}

// 读取目录并处理每个文件，用async/await让异步流程更易读
fs.readdir(pathToFolder, async (err, files) => {
  if (err) {
    return console.error('无法扫描目录:', err);
  }
  console.log('目录中的文件:\n');

  // 遍历文件，逐个处理
  for (const file of files) {
    const pathToImage = `${pathToFolder}/${file}`;
    try {
      const imageScore = await getImageScore(pathToImage);
      console.log(`${file} 的分数是: ${imageScore}`);
    } catch (e) {
      console.error(`处理${file}失败:`, e);
    }
  }

  console.log('所有图片处理完成');
});

额外优化说明 #

• 把pathToImage作为参数传递给getImageScore，彻底避免了全局变量的副作用，代码更健壮。
• 使用async/await替代.then/.catch，让异步代码的逻辑顺序更符合直觉，可读性更高。
• 所有异步错误都通过reject传递，并且在调用时用try/catch捕获，避免单个图片处理失败导致整个程序崩溃。
• 如果处理的是超大分辨率图片，pngjs的parsed事件可能需要较长时间（因为要遍历所有像素），这是正常的计算耗时，不是程序停滞。如果想进一步优化，可以考虑分块处理像素，但对于一般壁纸图片，当前代码已经足够。

内容的提问来源于stack exchange，提问作者Alexandru-Eugen Toma