リターン

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()のために必要ですか？

Answer 1

この機能を終了するには、returnの前に付けるのが一般的です。代わりに、ifの代わりにif-else if-elseを使用することもできます。この場合、関数を終了してミドルウェアチェーンをさらに進めたいだけです。

このパターンはかなり頻繁に表示されます。たとえば、これはかなり一般的です：

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

    console.log(result); 
}); 

それは以下のネストだとこれに比べて最もpoepleに簡単に読み取ります

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

最初のパターンはまた、誤って二回あるいは複数回next()を呼び出すからあなたを保ちますあなたのif-else -logicに何らかのエラーがある場合に備えてください。 あなたが掲示したその場合、それはnext()で起こるはずのことではありません。next()を呼び出しても、いずれの場合でもリダイレクトが発生する可能性があります。