node.js - mongooseコールバックでスローされたcatchエラー

Question

私のコントローラーでは、結果を得るか、呼び出された関数からエラーをキャッチすることを期待して、別の関数を呼び出すエンドポイント関数があります。呼び出された関数generateInvitesで

exports.createInvites = (req, res) => { 
    // code... 

    try { 

    generateInvitations(requirements, data => { 

     res.status(200).json({data}); 
    }); 

    } catch (err) { 
    console.log(`caught the error: ${err}`); 

    return res.status(500).json(err); 
    } 
} 

、私はマングースを使ってMongoDBのために生成されたオブジェクトを保存し、そして1がある場合は、エラーをスローします。上記のコード

function generateInvitations(requirements, cb) { 

    const { expirationDate, credits, numOfUse, numOfInvites } = requirements; 
    let invitationCodes = []; 
    for (let i = 0; i < numOfInvites; i++) { 

    const code = randomString(CODE_LENGTH); 
    invitationCodes.push(code); 

    const invitation = new Invitation({ 
     // code, 
     numOfUse, 
     expirationDate, 
     credits 
    }); 

    invitation.save((err, obj) => { 

     if (err) { 
     throw err; // it should throw a ValidationError here 
     } 
    }); 
    } 
    cb(invitationCodes); 

} 

generateInvitesでスローされたエラーを処理する私の試みです。もちろん、私はどんなエラーも捕まえることができませんでした。私は何を間違えたのですか？代わりに私は何をすべきですか？

Answer 1

コールバックは、自分自身のコールバックにあるため、エラーがスローされるよりもずっと前に発生するという事実に対処する必要があります。私はPromise.allが約束を返すすべての保存操作を待つことを提案します。

function generateInvitations(requirements) { 
    const { expirationDate, credits, numOfUse, numOfInvites } = requirements; 
    let promises = []; 

    for (let i = 0; i < numOfInvites; i++) { 
    const code = randomString(CODE_LENGTH); 

    const invitation = new Invitation({ 
     // code, 
     numOfUse, 
     expirationDate, 
     credits 
    }); 

    promises.push(invitation.save()); 
    } 
    return Promise.all(promises): 
} 

そして：

generateInvitations(requirements) 
    .then(data => res.status(200).json({data})) 
    .catch(err => { 
    console.log(`caught the error: ${err}`); 
    return res.status(500).json(err); 
    });