在nodejs混乱中返回next（）

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

为什么作者写return next()而不是next()？我知道next()是让流程跳转到下一个中​​间件或函数，但是为什么上面的next()需要return呢？

Answer 1

约定在前面加上return 以退出函数。另一种方法是使用if-else if-else 而不仅仅是if。在这种情况下，您只想退出函数并在中间件链上更进一步。

您会经常看到这种模式。例如，这很常见：

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

    console.log(result);
});

与此相比，它的嵌套更少，对大多数人来说更容易阅读：

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

第一个模式还可以防止您意外调用next() 两次甚至更多次，以防您的if-else-逻辑出现错误。 这正是您发布的next() 不应该发生的事情。它可以调用next() 并且在任何情况下仍然会导致重定向。