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

Question

前提

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

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

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

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

Uncaught (in promise) undefined

該当のソースコード

function p1(){
    const promise1 = new Promise((resolve, reject) => {
        setTimeout(() => {
            console.log("promise1");
            reject();
        }, 2000)
    });
    return promise1;
}

function p2(){
    const promise2 = new Promise((resolve, reject) => {
        console.log("promise2");
        resolve();
    });
    return promise2;
}

p1();
p2();

Promise.all([p1, p2]).then(() => {
    console.log("done");
}).catch(() => {
    console.log("error");
});

試したこと

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

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");
});

Answer 1

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

function p1(){
  const promise1 = new Promise((resolve, reject) => {
    setTimeout(() => {
      console.log("promise1");
      reject('err');
    }, 2000)
    }).catch(err=>{
      return err;
    });
    return promise1;
}

function p2(){
  const promise2 = new Promise((resolve, reject) => {
    setTimeout(() => {
      console.log("promise2");
      resolve('success');
    }, 1000)
    });
  return promise2;
}
Promise.all([p1(), p2()]).then((data) => {
  console.log(data);
});

Answer 2

p1();
p2();

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

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

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

Comments

[norio1435]

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