Promiseの中でawaitを使ってよいのか

Question

  Test() {
    return new Promise( async function(resolve, reject) {
      await sleep(10000);
      return resolve();
    });
  },

Promiseを返す関数の中でawaitを使用したい場合は、上記のように記述して問題ないのでしょうか？

Answer 1

構わないですが、戻りがpromiseなので
呼び出し側がasyncで処理してawaitしないと思った値はとれないかもしれません

Answer 2

動くことは動きますが、new Promiseの引数にした関数について、返り値は参照されません（おまけに、async関数をawaitせずに呼んだ場合、「Promiseを返す関数」として動作します）。

return resolve()としても無意味なので、単にresolve();とだけ書いておきましょう。

Comments

[maisumakun]

わざわざnew Promiseせずに、「return sleep(10000).then(() => {/* 略 */})」のように書くほうがスッキリしていていいかもしれません。

[Anon_]

なるほど。await/asyncにばかり気を取られておりましたが、promise内ではthenで繋いでもよさそうですね。

[Anon_]

https://qiita.com/okashoi/items/b786f94f534372efc705
returnの件ですが、上記の記事にresolveで処理が中断しないと書かれているのでreturnを書いておいたのですが、記事が間違っているという事でしょうか？

[maisumakun]

> promise内ではthenで繋いでもよさそうですね。

いえ、そうではなく、Test()の返り値自体を「return sleep(10000).then」のように書いていく、という話です。

[maisumakun]

> 記事が間違っているという事でしょうか？

記事は正しいですが、今回はresolve()でnew Promiseの引数となっている関数が終わりなので、もともと「中断せず」に実行されるコード自体がありません。

[Anon_]

＞いえ、そうではなく、Test()の返り値自体を「return sleep(10000).then」のように書いていく、という話です。

すみません、認識を間違っておりました。
実際のコードではPromise内で同期処理をさせたく、promise内でawaitを使用しても問題ないか質問させていただいた次第です。

>記事は正しいですが、今回はresolve()でnew Promiseの引数となっている関数が終わりなので、もともと「中断せず」に実行されるコード自体がありません。

ありがとうございます。
そういう事でしたか。

[maisumakun]

> 実際のコードではPromise内で同期処理をさせたく

状況によっては、「他の方法で作ったPromiseにthenでつないで新たなPromiseを作る」ということも可能です。promise.thenの返り値も別なpromiseです。

[Anon_]

なるほどです。ありがとうございました。