Event emitter を返す関数を作りたいのですが、望ましい書き方はありますか？

Question

以下のような、連想配列的なオブジェクトを返すが、メンバとして持っているメソッドが完了したときにイベントをemitする、EventEmitter の機能を追加したいとき、単純に EventEmitter を作って、それに持たせたいプロパティを Object.assign() を使って合成するという作り方で問題ないでしょうか。
もう少し望ましい、効率的な書き方がある、この書き方は良くない、間違っているなどの注意点があれば教えていただけないでしょうか。

function createEmitter(arg){
  let em = new require('events').EventEmitter();
  const handler1 = ()=>{
    em.foo += em.arg;
    em.emit('done1', em.foo);
  }
  const handler2 = ()=>{
    //...some processing
    em.foo -= em.arg;
    em.emit('done2', em.foo);
  }
  Object.assign(em, {
    foo: 5,
    arg,
    handler1,
    handler2,
  });
  return em
}

const em1 = createEmitter(4);
em1.on('done1', result=>{
  console.log(result);
})
em1.handler1();

Comments

[dameo]

まず、require('events').EventEmitterは()で括った方がいいです。
他に良し悪しは分からないので、回答は出来ないのですが、主観に基づき意見だけ少し書いておきます。
「EventEmitterの継承や委譲の方が一般的だと思います」
以下Hoge1(class使わない版),Hoge2(class使う版)が継承、Hoge3が委譲の例です。

const EventEmitter = require('events').EventEmitter;

function Hoge1(arg) {
EventEmitter.call(this);
this.foo = 5;
this.delta = arg;
}
Hoge1.prototype = Object.create(EventEmitter.prototype);
Hoge1.prototype.method1 = function () {
this.foo += this.delta;
this.emit('done1', this.foo);
};
Hoge1.prototype.method2 = function () {
this.foo -= this.delta;
this.emit('done2', this.foo);
};

class Hoge2 extends EventEmitter {
constructor(arg) {
super();
this.foo = 5;
this.delta = arg;
}
method1() {
this.foo += this.delta;
this.emit('done1', this.foo);
}
method2() {
this.foo -= this.delta;
this.emit('done2', this.foo);
}
}

class Hoge3 {
constructor(arg) {
this.em = new EventEmitter();
this.foo = 5;
this.delta = arg;
}
method1() {
this.foo += this.delta;
this.em.emit('done1', this.foo);
}
method2() {
this.foo -= this.delta;
this.em.emit('done2', this.foo);
}
}

let hoge = new Hoge1(4);
hoge.on('done1', result=>{
console.log(result);
})
hoge.method1();

hoge = new Hoge2(4);
hoge.on('done1', result=>{
console.log(result);
})
hoge.method1();

hoge = new Hoge3(4);
hoge.em.on('done1', result=>{
console.log(result);
})
hoge.method1();

Answer 1

この書き方は良くない、間違っている

イベントハンドラを固定しようとすることが間違いです。

Node.jsコアAPIの多くは、特定の種類のオブジェクト（「エミッタ」と呼ばれる）がFunctionオブジェクト（「リスナー」）を呼び出す名前付きイベントを発する慣用的な非同期イベント駆動型アーキテクチャを中心に構築されています。(NodeJS Document:EventsよりGoogle翻訳抜粋)

イベント駆動型アーキテクチャですので、イベントを発火する側、データを受ける側を明確にするべきです。
（データを受ける側が自由にその後の処理としてイベントハンドラを設定できるから意味があります）

よって、質問コメント内で dameo さんが示した継承、移譲という手法のコードが「回答」です。

NodeJS の標準モジュール util には継承用の util.inherits() が用意されています。

var EventEmitter = require("events").EventEmitter;
var util = require("util");

var SampleClass = function () {
};
util.inherits(SampleClass, EventEmitter);

console.log( utils.inherits.toString() ) とすると Hoge1 と同等の継承を行うユーティリティ関数と確認できます。

---

「リクエストリプライ方式」「イベントドリブン方式」があり、
Webメディア（ITmedia エンタープライズ）の古い記事 ですが、参考になるかもしれません。