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

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

新規登録して質問してみよう
ただいま回答率
85.50%
JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

Q&A

解決済

2回答

1046閲覧

Promise.allのエラー原因が分からないです

norio1435

総合スコア7

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

0グッド

1クリップ

投稿2022/12/13 01:17

前提

初めてPromiseを使います。
JavaScriptの関数の中でPromiseを返し、Promise.allで処理したいのですが、
reject()のところでエラーが出てしまいます。

原因が分からないので、教えていただけるとありがたいです。

発生している問題・エラーメッセージ

reject()のところで、以下のエラーが表示されます。

Uncaught (in promise) undefined

該当のソースコード

JavaScript

1function p1(){ 2 const promise1 = new Promise((resolve, reject) => { 3 setTimeout(() => { 4 console.log("promise1"); 5 reject(); 6 }, 2000) 7 }); 8 return promise1; 9} 10 11function p2(){ 12 const promise2 = new Promise((resolve, reject) => { 13 console.log("promise2"); 14 resolve(); 15 }); 16 return promise2; 17} 18 19p1(); 20p2(); 21 22Promise.all([p1, p2]).then(() => { 23 console.log("done"); 24}).catch(() => { 25 console.log("error"); 26});

試したこと

関数にしなければ正常に動作します。

const promise1 = new Promise((resolve, reject) => { setTimeout(() => { console.log("promise1"); reject(); }, 2000) }); const promise2 = new Promise((resolve, reject) => { console.log("promise2"); resolve(); }); Promise.all([promise1, promise2]).then(() => { console.log("done"); }).catch(() => { console.log("error"); });

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

rejectはエラー処理ですからpromiseのcatchでうけて後続にまわしてください

javascript

1function p1(){ 2 const promise1 = new Promise((resolve, reject) => { 3 setTimeout(() => { 4 console.log("promise1"); 5 reject('err'); 6 }, 2000) 7 }).catch(err=>{ 8 return err; 9 }); 10 return promise1; 11} 12 13function p2(){ 14 const promise2 = new Promise((resolve, reject) => { 15 setTimeout(() => { 16 console.log("promise2"); 17 resolve('success'); 18 }, 1000) 19 }); 20 return promise2; 21} 22Promise.all([p1(), p2()]).then((data) => { 23 console.log(data); 24});

投稿2022/12/13 01:30

yambejp

総合スコア114585

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

ベストアンサー

js

1p1(); 2p2(); 3 4Promise.all([p1, p2]).then(() => {

Promise.all() には関数の配列ではなく、Promise の配列を渡しましょう。

js

1Promise.all([p1(), p2()]).then(() => {

投稿2022/12/13 01:25

int32_t

総合スコア20672

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

norio1435

2022/12/13 02:32

正常に動作しました。 ありがとうございました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問