JavaScriptで、複数のイベントから受け渡されるデータが条件を満たしているときに特定の処理を実行させたい

Question

JavaScriptで、複数のイベントから受け渡されるデータが条件を満たしているときに特定の処理を実行させたい

JavaScriptのイベント処理(非同期)で悩んでいます。
通信のためのライブラリで、複数のon('...')イベントに対してそれぞれ関数を設定しています。
関数では受け取ったデータが一定の条件をみたしているかをチェックしています。

// バイナリデータを受け取ったときに発火
dataConnection.on('data', (data)=>{
  // 特定のdataを受け取るという条件A
});

// メディアストリームを受け取ったときに発火
mediaConnection.on('stream', (stream)=>{
  // streamを受け取るという条件B
});

このような場合で、AとBの条件を両方満たした場合に、特定の処理をさせたいのですが、どうすれば実現できるでしょうか。
(Primiseやasyncみたいな仕組みで対応できるのでしょうか...?)

Comments

[Lhankor_Mhy]

.on() はどういったオブジェクトのメソッドですか？　jQuery でしょうか。

[kttl]

通信のために使っているライブラリ(SkyWay)ですね。EventEmitterを使って実装されているようです。

[Lhankor_Mhy]

では、そのライブラリのドキュメントがないと回答しようがないと思うのですが、いかがでしょうか。

[kttl]

一般的なイベントハンドラの特性のことだと思い省略していました。ドキュメントはこちらです。 https://webrtc.ecl.ntt.com/skyway-js-sdk-doc/ja/dataconnection/

[Lhankor_Mhy]

すみません、ドキュメント見つけられました。ソースコード読んだ感じ、普通に EventEmitter ですね......

[Lhankor_Mhy]

普通に、ラップしてpromise返すか、emitメソッドを上書きしてpromise返すか、ぐらいしかなさそうな気がしました。他の回答者を待ちたい。

[karamarimo]

data と stream は別々のタイミングで来るわけですが、「AとBを両方満たすとき」というのはどういうタイミングなのでしょうか。

[kttl]

質問中の擬似コードのコメント部分にif(A)のような条件文があり、AとBの条件文がどちらもtrueとなり、実行された状態です。

Answer 1

Primiseやasyncみたいな仕組みで対応できるのでしょうか...?

可能ですが、依存元のライブラリがPromiseインスタンスを投げ返してくれる前提です。

一応解説しておくと、Promise.allというメソッドが用意されており、
Promiseインスタンスの配列を投げ込むと
全て実行完了になった時に.thenが発火する仕組みになっています。

const func1 = () => new Promise((resolve, reject) => resolve(123))
const func2 = () => new Promise((resolve, reject) => resolve(234))

Promise.all([func1(), func2()])
  .then(results => {
    console.log(results) // [123, 234]
  })

---

該当の処理がPromiseを返さない場合は無理です。
まぁ、Promiseで包んでやれば動くんじゃないですかね？

const getData = () => new Promise((resolve, reject) => {
  // バイナリデータを受け取ったときに発火
  dataConnection.on('data', (data)=>{
    // 特定のdataを受け取るという条件A
    resolve(data);
  });
});

const getMedia = () => new Promise((resolve, reject) => {
  // メディアストリームを受け取ったときに発火
  mediaConnection.on('stream', (stream)=>{
    // streamを受け取るという条件B
    resolve(stream);
  });
})

Promise.all([getData(), getMedia()])
  .then(([data, media]) => {
    console.log(data);
    console.log(media);
  });

---

ES2017のasync/await構文はPromiseのthenがダサいということで糖衣構文で表現しているだけなので、
Promiseを返さないライブラリ相手では使えません。

やってる事は似たようなもんです。

const getData = () => new Promise((resolve, reject) => {
  // バイナリデータを受け取ったときに発火
  dataConnection.on('data', (data)=>{
    // 特定のdataを受け取るという条件A
    resolve(data);
  });
});

const getMedia = () => new Promise((resolve, reject) => {
  // メディアストリームを受け取ったときに発火
  mediaConnection.on('stream', (stream)=>{
    // streamを受け取るという条件B
    resolve(stream);
  });
})

// await構文を使うには、async関数内である条件があるので、即時実行関数で発火
(async functon () {
  const promises = {
    data: getData(),
    media: getMedia(),
  };
  const data = await promises.data;
  const stream = await promises.media;
  console.log(data);
  console.log(stream);
})();

---

asyncライブラリ(ES2017のasync/await構文とは別) を利用する手もあります。
リンクを張っておいたので使い方は見て下さい。

ただし、基本的に非同期関数はtry〜catchで握れない為、
第一引数がerrであるという前提で考える慣習があります。

その辺が許容出来るならasyncライブラリは中々良い選択肢だと思います。

Comments

[kttl]

丁寧な回答ありがとうございます。
イベントハンドラの登録ごとPromiseで包んでしまうという方法ですね。

Promise.allでまとめて登録&条件監視する場合はできそうですが、
イベントハンドラの登録のタイミングも別々、という場合にはどうすればよいでしょうか。

[miyabi-sun]

Promiseのインスタンスを作った後、中の処理はこっそり実行中です。
Promiseの状態はインスタンス作成直後はPendingで、resovle・rejectメソッドの実行と共に成功・失敗に状態が切り替わります。

Promise.allが引数として要求する配列は、状態が完了のPromiseが混ざっていても構いません。
作れるPromiseは順次生成して何処かの変数に保存しましょう。

Promise.allは全てのイベントハンドラが出揃ってから、
引数に退避しておいたPromise達で配列を作って投げ込めば大丈夫です。

[kttl]

丁寧にありがとうございました

[kttl]

確かに全体の構造としてはそうするのがよいですね。
ご教授いただきありがとうございます。

Answer 2

単にイベントごとに条件が合致するかどうかのフラグを持たせて、最終的なイベントが起こった時にそのフラグをチェックして起動させる、というのでいいかと

Comments

[kttl]

回答ありがとうございます。

イベントの発生順番が不定なので、すべてのイベントにその他のイベントが発生したかのチェックとイベント発生したことを登録するという方法で実装していました。
しかし管理が大変なので、よりスマートな方法で実装できないか質問させていただいた次第です。
やはりシンプルにそう実装するのがよいのかもしれません。ありがとうございました。