ESLint一貫性のあるリターンとネストされたコールバック

Question

私は、ESLintが提供するconsistent-returnルールにいくつか問題があります。以下のコードでは、User.findOneコールバック関数の警告がconsistent-returnに送信されます。私が知る限り、findErrがなく、existingUserがある場合にのみ、.removeアクションを実行できます。

これらのネストされたコールバックを回避するためのベストプラクティスは何ですか？私はconsistent-return警告を渡したいと思いますが、実際には解決策をオンラインで見たことはありません。これらのネストされたコールバックを回避するためのベストプラクティスでしょう何

function remove(req, res) { 
    User.findOne({ username: req.params.username }, (findErr, existingUser) => { 
    if (findErr) return res.status(500).send(errorHandler.getErrorMessage(findErr)); 
    if (!existingUser) return res.status(404).send({ message: 'User not found' }); 

    existingUser.remove((removeErr) => { 
     if (removeErr) return res.status(500).send(errorHandler.getErrorMessage(removeErr)); 

     return res.json({ message: `${existingUser.username} successfully deleted` }); 
    }); 
    }); 
} 

Answer 1

？

約束

私は、あなたが最良の早期リターンを使用しないと思います

警告一貫したリターンを渡したいと思います。代わりに、別の方法として

function remove(req, res) { 
    User.findOne({ username: req.params.username }, (findErr, existingUser) => { 
    if (findErr) res.status(500).send(errorHandler.getErrorMessage(findErr)); 
    else if (!existingUser) res.status(404).send({ message: 'User not found' }); 

    else existingUser.remove((removeErr) => { 
     if (removeErr) res.status(500).send(errorHandler.getErrorMessage(removeErr)); 

     else res.json({ message: `${existingUser.username} successfully deleted` }); 
    }); 
    }); 
} 

を書き、あなたはまた、{ "treatUndefinedAsUnspecified": true }オプションを有効にしてreturn void …を使用することができます

function remove(req, res) { 
    User.findOne({ username: req.params.username }, (findErr, existingUser) => { 
    if (findErr) { 
     res.status(500).send(errorHandler.getErrorMessage(findErr)); 
     return; 
    } 
    if (!existingUser) { 
     res.status(404).send({ message: 'User not found' }); 
     return; 
    } 
    existingUser.remove((removeErr) => { 
     if (removeErr) { 
     res.status(500).send(errorHandler.getErrorMessage(removeErr)); 
     return; 
     } 
     res.json({ message: `${existingUser.username} successfully deleted` }); 
    }); 
    }); 
} 

行うことができる必要があります。