如何在JavaScript的exec中声明全局数组并解决PID数组未更新问题

我正在编写一个JS脚本，用来在服务器上定位所有.pid文件、提取PID，再对每个PID执行ps -fp {pid}并提取内存占用。目前的代码如下：

var exec = require('child_process').exec;
exec('(cd ~/; find . -type f -name "*.pid" )', (error, stdout, stderr) => { 
  if (error) { return 1 } 
  var fileps = stdout.toString().split('\n'); 
  fileps.pop() 
  var pidsarr = new Array; 
  function getpid(path) { 
    var command = `(cd ~/; cat ${path} )`; 
    exec(command, (error, stdout, stderr) => { 
      if (error) { return 1 } 
      pid = parseInt(stdout.toString()); 
      pidsarr.push(pid); 
    }); 
  } 
  fileps.forEach(getpid); 
  console.log(pidsarr); 
});

问题在于pidsarr数组始终是空的，没有被正确更新。我怀疑是因为exec的异步特性或者变量作用域的问题，请问怎么才能让pidsarr在每次exec调用完成后都能追加元素？另外也想了解在Node.js的exec调用里怎么正确处理全局数组或者共享变量？

这个问题的核心其实是Node.js中exec的异步特性，而非变量作用域问题——你声明的pidsarr在getpid函数里是可以访问到的，只是你在所有异步的exec调用完成之前就打印了数组，所以看到的是空数组。下面给你几个可行的解决思路：

1. 使用Promise和async/await（推荐，代码更清晰） #

把每个exec调用包装成Promise，然后用Promise.all等待所有调用完成后再处理数组：

const { exec } = require('child_process');

// 封装exec为Promise函数
function execPromise(command) {
  return new Promise((resolve, reject) => {
    exec(command, (error, stdout, stderr) => {
      if (error) {
        reject(error);
        return;
      }
      resolve(stdout.trim()); // 去除多余的换行和空格
    });
  });
}

async function getPidsAndMemory() {
  try {
    // 第一步：找到所有.pid文件
    const findOutput = await execPromise('cd ~/ && find . -type f -name "*.pid"');
    const pidFiles = findOutput.split('\n').filter(file => file); // 过滤空行，替代pop()
    
    // 第二步：逐个读取PID
    const pidsarr = await Promise.all(
      pidFiles.map(async (filePath) => {
        const pidStr = await execPromise(`cd ~/ && cat ${filePath}`);
        return parseInt(pidStr, 10); // 明确指定进制，避免意外
      })
    );
    
    // 第三步：处理每个PID的内存占用
    const memoryInfo = await Promise.all(
      pidsarr.map(async (pid) => {
        const psOutput = await execPromise(`ps -fp ${pid}`);
        // 这里可以根据ps输出格式提取内存占用，比如截取对应列
        return { pid, memoryDetails: psOutput };
      })
    );
    
    console.log('提取到的PID数组:', pidsarr);
    console.log('各PID内存占用信息:', memoryInfo);
  } catch (error) {
    console.error('执行出错:', error);
    process.exit(1);
  }
}

getPidsAndMemory();

2. 使用回调计数（传统异步处理方式） #

如果不想用Promise，可以通过计数来跟踪所有异步调用是否完成：

var exec = require('child_process').exec;
exec('(cd ~/; find . -type f -name "*.pid" )', (error, stdout, stderr) => { 
  if (error) { 
    console.error('查找文件出错:', error);
    return process.exit(1); 
  } 
  var fileps = stdout.toString().split('\n'); 
  var pidFiles = fileps.filter(file => file); // 过滤空行
  var pidsarr = [];
  let completedTasks = 0; // 计数器，记录完成的异步调用数

  function getpid(path) { 
    var command = `(cd ~/; cat ${path} )`; 
    exec(command, (error, stdout, stderr) => { 
      completedTasks++; // 每次完成调用就加1
      if (error) { 
        console.error(`读取文件${path}出错:`, error);
      } else {
        const pid = parseInt(stdout.toString(), 10);
        if (!isNaN(pid)) { // 确保是有效的数字PID
          pidsarr.push(pid);
        }
      }
      
      // 当所有调用都完成时，再处理数组
      if (completedTasks === pidFiles.length) {
        console.log('提取到的PID数组:', pidsarr);
        // 继续执行ps命令获取内存信息
        pidsarr.forEach(pid => {
          exec(`ps -fp ${pid}`, (err, out) => {
            if (!err) {
              console.log(`PID ${pid}的内存信息:\n`, out);
            }
          });
        });
      }
    }); 
  } 

  pidFiles.forEach(getpid); 
});

关于全局数组的说明 #

其实你完全不需要把pidsarr声明成全局变量——在你的原代码里，getpid是在exec的回调函数内部定义的，它可以通过JavaScript的闭包特性访问外层作用域的pidsarr。问题出在你在forEach调用getpid后立刻打印数组，而此时所有的exec异步调用都还没完成，所以数组是空的。

如果一定要声明全局数组，只需要把var pidsarr = [];放在所有函数的外面（比如文件的最顶部），这样整个脚本里的函数都能访问到它，但这种方式不推荐，因为全局变量容易引发意外的副作用，尤其是在异步代码里。

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