質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.92%

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

解決済

回答 3

投稿

  • 評価
  • クリップ 0
  • VIEW 2,424

d415uk35470

score 45

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); //一応できた
  }
}

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

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 3

+1

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

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

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

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

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

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/05/02 05:58

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

    キャンセル

checkベストアンサー

0

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

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/05/02 05:57

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

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

    キャンセル

0

例示されたコードで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 さん

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/05/02 06:00

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.92%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る