powrócić next() w nodejs zamieszania

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

Dlaczego autor zrobić return next() zamiast next()? Wiem, że next() to pozwolić przepływowi przejść do następnego oprogramowania pośredniego lub funkcji, ale dlaczego potrzebuje on return dla next() powyżej?

Answer 1

Konwencja ma na celu dodanie return do wyjścia z funkcji. Alternatywą byłoby użycie if-else if-else zamiast tylko if. W tym przypadku wystarczy wyjść z funkcji i krok dalej w łańcuchu oprogramowania pośredniego.

Ten wzór będzie dostępny dość często. Na przykład, jest to dość powszechne:

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

    console.log(result); 
}); 

To mniej gniazdowania i czyta łatwiej większości ludzie ciężko w porównaniu do tego:

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

Pierwszy wzór trzyma cię również przed przypadkowym wywołaniem next() dwukrotnie lub nawet więcej razy na wypadek, gdyby pojawił się błąd w twojej metodzie if-else. I to jest dokładnie to, co nie powinno się zdarzyć z next() w tym przypadku, który wysłałeś. Może wywołać next() i nadal powodować przekierowanie w każdym przypadku.