Mongoose Versprechen brechen dann Kette während der Behandlung von Fehlern

Question

Bearbeiten 2018-01-18: Verwenden Sie async/erwarten statt Versprechen. Es wird alle Ihre Probleme lösen.

Ich habe diesen Code für Mungo in 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); 
     }) 

Ich wünschte, die Hinrichtung zu stoppen, wenn es keine Nutzer gefunden, und nur senden „Es wurden keine Nutzer.“ ohne etwas anderes auszuführen.

Ich bin mir bewusst, dass ich stattdessen throw res.send('No users found.'); verwenden könnte.

aber dann würde das es mir unmöglich machen, echte Fehler - die zum Beispiel beim Speichern oder Aktualisieren passieren könnten - zu erfassen und zu verwalten.

Wie soll ich das angehen? Sollte ich eine andere Codestruktur verwenden? Ich mag, wie einfach und wartungsfreundlich diese Struktur ist, abgesehen von diesem einen Nachteil.

Answer 1

Sie können Fehlerklassen in 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' 
    } 
} 

Ihre Fehlerbehandlungscode erweitern dann bei mehr Metadaten in den Fehler suchen kann die Entscheidung darüber, wie zur Antwort zu machen.

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

Sie wollen wahrscheinlich einen allgemeinen express API Request/Response-Handler erstellen, so dass Sie nicht die Antwort wiederholen und Fehler für jeden Handler Handhabung. Dann muss der Handler-Code nur Daten zurückgeben oder Fehler werfen.

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