结合 Promise 和 EventEmitter

Question

我创建了这个“简单模式”，适用于组合 Promise 和 EventEmitter（与 nodejs）。

但是：我想知道是否有更好的进球方式？

const { EventEmitter } = require('events');
const fs = require('fs');

function doSomething(parameters) {
  const emitter = new EventEmitter();

  const promise = new Promise((resolve, reject) => {
    // DO DIRTY JOB
    fs.readdir(parameters.directory, (err, files) => {
      if (err) {
        reject(err);
        return;
      }
      files.forEach(file => emitter.emit('update-event', file));
      resolve(`I'm done: ${parameters.param} world`);
    });
  });
  return { promise, emitter };
}

const work = doSomething({ param: 'hello', directory: './' });
work.emitter.on('update-event', data => console.log(`Update ${data}`));
work.promise.then(console.log).catch(console.error);

我在想：

doSomething(...).on(...).then(...)

但我不知道该怎么做。

Answer 1

Node.js 为此构建了一个函数：require('events').once 函数！这里是PR。

已随 Node [v11.13] (https://nodejs.org/en/blog/release/v11.13.0/) 发布

一个示例用法（来自文档）：

const { once, EventEmitter } = require('events');
async function run() {
  const ee = new EventEmitter();
  process.nextTick(() => {
    ee.emit('myevent', 42);
  });
  const [value] = await once(ee, 'myevent');
  console.log(value); // 42

  const err = new Error('kaboom');
  process.nextTick(() => {
    ee.emit('error', err);
  });

  try {
    await once(ee, 'myevent');
  } catch (err) {
    console.log('error happened', err);
  }
}

run();

Answer 2

就我个人而言，我不确定接受的答案与 OP 的问题有何关系，无论如何我认为我确实找到了一种相当简单（但可能不是很好）的方法来完成 OP 提出的特定 doSomething(...).on(...).then(...) 事情.以 OP 的示例代码为例，我们可以执行以下操作：

const { EventEmitter } = require('events');
const fs = require('fs');

function doSomething(parameters) {
  var resolves;
  var rejects;
  const emitter = new EventEmitter();
  const promise = new Promise((resolve, reject) => {
    resolves = resolve;
    rejects = reject;
  });
  promise.on = emitter.on;
  promise.emit = emitter.emit;

  // DO DIRTY JOB
  fs.readdir(parameters.directory, (err, files) => {
    if (err) {
      rejects(err);
      return;
    }
    files.forEach(file => promise.emit('update-event', file));
    resolves(`I'm done: ${parameters.param} world`);
  });

  return promise;
}

const work = doSomething({ param: 'hello', directory: './' });
work.on('update-event', data => console.log(`Update ${data}`))
  .then(console.log)
  .catch(console.error);

到目前为止，它适用于我的有限情况，据我所知，事件和承诺都可以毫无问题地链接起来。对于我还没有遇到过的更复杂的用例，可能会有问题，但它确实起到了像 OP 所要求的那样链接 doSomething(...).on(...).then(...) 的目的。

Answer 3

不，您不应该将事件发射器和承诺组合在一个对象中。像您一样单独退回它们是可以的。

当然，在您的特定示例中，根本没有理由使用事件发射器。无论如何，它都会在承诺实现时触发。简单得多：

const fs = require('fs');
function doSomething(parameters) {
  return new Promise((resolve, reject) => {
    // DO DIRTY JOB
    fs.readdir(parameters.directory, (err, files) => {
      if (err) reject(err);
      else resolve(Object.assign(files, parameters));
    });
  });
}

doSomething({ param: 'hello', directory: './' }).then(files => {
  for (const data of files) {
    console.log(`Update ${data}`)
  }
  return `I'm done: ${files.param} world`;
}).then(console.log, console.error);

Answer 4

我建议这样做：

import EventPromised from "event-promised";

function doSomething(parameters) {
    return new EventPromised((resolve, reject, emit) => {
        fs.readdir(parameters.directory, (err, files) => {
            if (err) {
                reject(err);
                return;
            }
            files.forEach(file => emit('update-event', file));
            resolve(`I'm done: ${parameters.param} world`);
        });
    });
}
doSomething({ param: 'hello', directory: './' })
    .on('update-event', data => console.log(`Update ${data}`))
    .then(console.log)
    .catch(console.error);