Best Practices für NodeJS: Fehler für die Ablaufsteuerung?

Question

In Node.js, sollte ich Fehler für die Ablaufsteuerung verwenden, oder sollte ich sie eher als Ausnahmen verwenden?

Ich schreibe einen Authentifizierungs-Controller und einige Komponententests in Sails.js, und derzeit überprüft meine Registrierungsmethode, ob ein Benutzer mit demselben Benutzernamen existiert. Wenn ein Benutzer bereits mit dem Benutzernamen vorhanden ist, ruft mein Modell Methode seine Rückruf Argumente mit einem neuen Error-Objekt, etwa so:

Modell:

exists: function (options, cb) { 
    User.findOne({ 
     where: { username: typeof options === 'Object' && options.username ? options.username : options }, 
    }).exec(function (err, user) { 
     if (err) return cb(err); 
     if (user) return cb(new Error("A user with that username already exists.")); 
     cb(null, !!user); 
    }); 
}, 

Controller:

User.exists(req.body.user.username, function (err, exists) { 
    if (err) { 
    console.log("error: ", err); 
    return res.status(409).json({ 
     message: err 
    });  
    } 

    User.create(req.user).then(function (data) { 
    res.status(201).json({ 
     user: data 
    }); 
    }); 
}); 

Ist das beste trainieren? Ich bin mir nicht sicher, ob Knotenkonventionen Fehler für Ausnahmefälle oder für die Ablaufsteuerung bevorzugen. Ich denke, ich sollte das umschreiben, aber ich möchte Konventionen kennen, bevor ich das tue. Ich glaube, ich habe einige Beispiele in Sails gesehen. Vielen Dank!

Answer 1

Die obige Antwort ist gut für Express, aber in Sails-Controller sollten Sie nicht anrufen next; Best Practice ist immer eine Antwort zurückzugeben. In den meisten Beispiel-Sails-Code sehen Sie nicht einmal next als Argument für eine Controller-Action-Funktion. Beachten Sie auch, dass Sails mit einigen default response methods direkt in das res Objekt, wie res.serverError und res.badRequest, sowie res.negotiate eingebaut ist, die versuchen werden, den Fehler basierend auf dem Statuscode an den entsprechenden Handler für Sie weiterzuleiten. So könnte Ihr Beispiel werden gezwickt als:

Modell:

exists: function (options, cb) { 
    User.findOne({ 
     where: { username: typeof options === 'Object' && options.username ? options.username : options }, 
    }).exec(function (err, user) { 
     // res.negotiate will default to a 500 server error 
     if (err) return cb(err); 
     // res.negotiate will just output the status code and error object 
     // as JSON for codes between 400 and 500, unless you 
     // provide a custom view as api/responses/badRequest.ejs 
     if (user) return cb({ 
      status: 409, 
      message: "A user with that username already exists." 
     }); 
     cb(null, !!user); 
    }); 
}, 

Controller:

User.exists(req.body.user.username, function (err, exists) { 
    // Let Sails handle those errors for you 
    if (err) {return res.negotiate(err);} 

    User.create(req.user).then(function (data) { 
    res.status(201).json({ 
     user: data 
    }); 
    }); 
}); 

Answer 2

Node (oder Javascript wirklich) können Fehler auf Ausnahmen auslösen, indem das Schlüsselwort throw:

if (something_went_wrong) { 
    throw new Error('Doh!'); 
} 

Sie können auch zusätzliche Parameter an das Objekt Standardfehler hinzufügen mehr semantische Bedeutung zu Ihren Fehlern in Ihrem Programm zu geben, . Wenn Sie das gesagt haben, möchten Sie keinen Fehler in Ihrem Routen-Handler ausgeben, da dies den Prozess und Ihren Server zum Absturz bringen würde.

Wenn Sie Routen-Handler in Sails verwenden (oder wirklich ausdrücken), sollten Sie die Art des Fehlers unbedingt prüfen und entsprechend auf den Client reagieren.

// -- Route handler 
app.get('/something', function (req, res, next) { 

    DB.create({ username: 'user' }, function (err, docs) { 

    if (err) { 

     // This checks to see if we have a unique error from MongoDB 
     // and send the appropriate response to the client 
     if (err.code === 11000 || error.code === 11001) { 
     return res.send(409); // or return res.json(409, {message: "User exists"}); 
     } 

     // Any other error type is assumed to be an internal error so we pass it 
     // down the chain to the error handler middleware 
     return next(err); 

    } 

    // This is a client error, not an internal error so we respond to the client 
    // with the appropriate client error type 
    if (docs.length === 0) return res.send(404); 

    if (user.notAuthorized) return res.send(403); 

    res.send('All Good'); 

    }); 

}); 

Beachten Sie, dass in dem Fall, dass die DB mit einem internen Fehler reagiert, wir die next() Funktion übergeben, die durch die Fehlerbehandlung Middleware in der Kette aufgenommen wird. Jede Middleware mit einer Arrigkeit von 4 ist als Fehlerbehandlungs-Middleware definiert. Sails hat wahrscheinlich einen Standard-Fehlerhandler, aber Sie können ihn wahrscheinlich auch überschreiben - Sie müssen die entsprechenden Dokumente für diese Informationen überprüfen, da ich das Steuerelement bevorzuge, das ich mit Express allein erhalte.