Mongoose Promiseはエラー処理中にチェーンを切断します

Question

編集2018-01-18：約束の代わりにasync/awaitを使用してください。すべての問題を解決します。

は、私は私が見つけたユーザーが一人もいない場合、実行を停止し、ちょうど送りたいNodeJS

User.find({ name: 'John' }) 
     .then((users) => { 
     if (!users.length) return res.send('No users found.'); 
     return Request.find({ cost: 100 }) 
     }) 
     .then((requests) => { 
     console.log('should not get here'); 
     return res.json(requests); 
     }) 
     .catch((err) => { 
     res.status(500).json(err); 
     }) 

にマングースのために、このコードを持っている「いいえユーザーは見つかりませんでした。」他の何も実行せずに。

代わりにthrow res.send('No users found.');を使用することができます。

しかし、それは私が本物のエラー（例えば保存や更新中に起こる可能性がある）をキャッチして管理することを不可能にします。

私はこれにどのようにアプローチすればよいですか？私は別のコード構造を使うべきですか？この構造の欠点を除いて、この構造がいかにシンプルで維持可能であるかは好きです。

Answer 1

あなたは、その後どのように応答する程度それの決定を行うために、エラーに多くのメタデータを見ることができますES2015/6

class ExtendedError { 
    constructor(message){ 
    super(message) 
    this.name = this.constructor.name 
    this.message = message 
    if (typeof Error.captureStackTrace === 'function'){ 
     Error.captureStackTrace(this, this.constructor) 
    } else { 
     this.stack = (new Error(message)).stack 
    } 
    } 
} 

class NotFoundError extends ExtendedError { 
    constructor(message, options){ 
    super(message) 
    this.status = 404 
    this.code = 'NF0001' 
    } 
} 

あなたのエラー処理のコードでエラークラスを拡張することができます。

User.find({ name: 'John' }) 
    .then((users) => { 
    if (!users.length) throw new NotFoundError('No users found.')   
    return Request.find({ cost: 100 }) 
    }) 
    .then((requests) => { 
    console.log('should not get here') 
    return res.json(requests); 
    }) 
    .catch((err) => { 
    let status = err.status || 500 
    res.status(500).json({ error: err }) 
    }) 

あなたはおそらく、あなたがすべてのハンドラのハンドリングレスポンスやエラーを繰り返していませんので、一般的な明示のAPIリクエスト/レスポンスハンドラを作成します。ハンドラコードはデータを返すか、エラーを投げるだけでよい。

User.find({ name: 'John' }) 
    .then((users) => { 
    if (!users.length) throw new NotFoundError('No users found.')   
    return Request.find({ cost: 100 }) 
    })