there is a script:

app.get('/:userId', function(req, res) { var userId = +req.params.userId; users.forEach(function (user) { if (user.user_id === userId) { console.log(1); res.send('<h1>' + user.username + '</h1>' + '<h2> Id: ' + user.user_id + '</h2>'); res.end(); } }); console.log(2); res.send('user with id: ' + userId + ' not found'); res.end(); });

when the request is executed, then 1 and 2 are output to the console, why, because res.end should end the request and work as return; or not?

If the user is found, he will return and 1 will be displayed in the console, and if not, 2 will be displayed.

    1 answer 1

    Because res.end(); completes the response to the request, but does not exit the function.

    You can do this:

     app.get('/:userId', function(req, res) { var userId = +req.params.userId; for (user of users) { if (user.user_id === userId) { console.log(1); res.send('<h1>' + user.username + '</h1>' + '<h2> Id: ' + user.user_id + '</h2>'); res.end(); return; } } console.log(2); res.send('user with id: ' + userId + ' not found'); res.end(); });

    And another option:

     app.get('/:userId', function(req, res) { var userId = +req.params.userId; var found = users.some(function (user) { if (user.user_id === userId) { console.log(1); res.send('<h1>' + user.username + '</h1>' + '<h2> Id: ' + user.user_id + '</h2>'); return true; } }); if (!found) { console.log(2); res.send('user with id: ' + userId + ' not found'); } res.end(); });
    • That is, you have to write return after res.end every time? and no way around it? jsfiddle.net/sju8werg - Vitali
    • @Vitaly, and what prevents you from just using the cycle? Especially recently added for-of. - Qwertiy ♦
    • @Vitaly, and one more option added. - Qwertiy ♦