ritorno next() in confusione nodejs

Question

https://github.com/hwz/chirp/blob/master/module-5/completed/routes/api.js

function isAuthenticated (req, res, next) { 
    // if user is authenticated in the session, call the next() to call the next request handler 
    // Passport adds this method to request object. A middleware is allowed to add properties to 
    // request and response objects 

    //allow all get request methods 
    if(req.method === "GET"){ 
     return next(); 
    } 
    if (req.isAuthenticated()){ 
     return next(); 
    } 

    // if the user is not authenticated then redirect him to the login page 
    return res.redirect('/#login'); 
}; 

Perché l'autore fanno return next() invece di next()? So che next() è quello di consentire al flusso di passare al middleware o alla funzione successiva, ma perché è necessario un codice return per next() precedente?

Answer 1

È una convenzione per anteporre un valore a return per uscire dalla funzione. L'alternativa sarebbe utilizzare if-else if-else anziché solo if. In questo caso, devi solo uscire dalla funzione e andare oltre sulla catena del middleware.

Vedrai questo schema abbastanza spesso. Ad esempio, questo è abbastanza comune:

someFunction(function(err, result) { 
    if (err) { 
     return console.error(err); 
    } 

    console.log(result); 
}); 

E 'meno di nidificazione e legge più facile alla maggior parte dei poeple rispetto a questo:

someFunction(function(err, result) { 
    if (err) { 
     console.error(err); 
    } else { 
     console.log(result); 
    } 
}); 

Il primo modello che mantiene anche di chiamare accidentalmente next() due volte o anche più volte in caso di errori nel tuo if-else -logic. E questo è esattamente ciò che non dovrebbe accadere con next() in quel caso che hai postato. Potrebbe chiamare next() e causare comunque il reindirizzamento in ogni caso.