JavaScriptのPromiseのresolveを引数の関数外から呼ぶ方法を知りたいです

Question

JavaScriptのPromiseのresolveを引数の関数外から呼ぶ方法を知りたいです。一応動作はさせられたんですが明らかにイレギュラーな使い方なので、もっと良い方法があれば知りたいです。

まず通常の例です。foo.bar()を呼び出すと、Promiseを返し、1000ms後に完了したいとします。
この例は動作します。

class Foo{
  bar(){
    return new Promise((resolve)=>{ 
      setTimeout(()=>{ resolve() }, 1000 ); //1000msかかる処理があるとする
    });
  }
}

次に、動作しない例です。こういうことがしたい、というイメージです。
つまりnew Promise()に渡す無名関数に何でもつっこむのが嫌になった時に、後で別の関数からresolveを呼び出したいです。
なぜそう思ったかというと、その無名関数が巨大になったり、外から中断したい可能性のあるいくつかの非同期実行される関数を経て、最終的にresolveを呼びたい（または中断された場合には呼ばない）ということがしたかった、というのが理由です。

class Foo{
  bar(){
    this.promise = new Promise((resolve)=>{}); //とりあえずpromiseを保持しておく
    setTimeout(this.onTimeout.bind(this),1000); //1000msかかる処理があるとする
    return this.promise;
  }
  onTimeout(){
    //処理完了
    this.promise.resolve(); //そのようなメソッドは存在しないのでエラーです
  }
}

きれいではない方法ですが、動作をさせることができたサンプルです。

class Foo{
  bar(){
    this.promise = new Promise((resolve)=>{
      this.resolveFunction = resolve; //resolveも保持しておく
    }); //とりあえずpromiseを保持しておく
    setTimeout(this.onTimeout.bind(this),1000); //1000msかかる処理があるとする
    return this.promise;
  }
  onTimeout(){
    //処理完了
    this.resoveFunction.call(promise); //一応できた
  }
}

どう見てもイレギュラーな方法なので、良い書き方があれば知りたいです。

Answer 1

、外から中断したい可能性のあるいくつかの非同期実行される関数を経て、最終的にresolveを呼びたい（または中断された場合には呼ばない）ということがしたかった

上記のしたいことがPromiseの役割です。
したがって、そもそも意味が無いことをしているのかもしれません。
もっと具体的に詰まっているコードを示していただいたほうが回答しやすいかもしれません

その無名関数が巨大になったり

無名関数のコードが巨大になってしまうならきちんと名前をつけて関数分割してあげて下さい。
無名関数だらけだと後々コードを読むのが困難になります。

以下のPromiseに関するドキュメントを一通り読むことをオススメします
http://azu.github.io/promises-book/

Comments

[d415uk35470]

なるほど、その状況自体がすでに良くない、ということですね。良い指摘ありがとうございます。設計思想を再勉強してみます。ありがとうございます。

Answer 2

例示されたコードでPromiseを使う意図が分かりませんでした。
もう少し、意味のおるコードを出した方が的確なアドバイスが付くと思います。

関数外から関数を呼ぶだけなら、コールバック関数を外部スコープに置く方法が第一に考えつきます。

class Foo{
  bar(){
    return new Promise(this.onTimeout).then(this.resolve);
  }
  onTimeout (resolve) {
    setTimeout(resolve, 1000);
  }
  resolve () {
    sonsole.log('1 second later');
  }
}

Re: terutaka-kondo さん

Comments

[d415uk35470]

確かに、例示したコードが良くなかったように思います。ご指摘ありがとうございます。
該当のメソッドを呼び出して完了待ちをしている間に、何かしら外から任意のタイミングで中断したい、そして中断した場合にはresolveのfunctionではなくrejectのfunctionを呼びたい、という感じでした。

Answer 3

きれいではない方法ですが、動作をさせることができたサンプルです。
どう見てもイレギュラーな方法なので、良い書き方があれば知りたいです。

Promiseを拡張したDeferredというものが該当します。
考え方はterutaka-kondoさんの考えられた方法と余り変わりません。

https://developer.mozilla.org/en-US/docs/Mozilla/JavaScript_code_modules/Promise.jsm/Deferred
詳しくは、この記事を読んでください。
polyfillを見ると考え方が一緒なのが分ります。

Comments

[d415uk35470]

なるほど！jQuery.Deferredは以前から使っており、インスタンスを保持して後から使う、ということをするケースがあったので、同様のことをしたい場合はどうしたら、となって質問を投稿しました。

拡張したDeferredを考えた人がいるということは、そうしたいというシーンが生まれること自体はあながち完全に間違った状態である、ということではないということかなと理解しました。（とはいえ他の方が書いていただいているように、理想的な状態ではないのかもしれません。）