When to use next() and return next() in Node.js

As @Laurent Perrin's answer:

If you don't do it, you risk triggering the callback a second time later, which usually has devastating results

I give an example here if you write middleware like this:

app.use((req, res, next) => {  console.log('This is a middleware')  next()  console.log('This is first-half middleware')})app.use((req, res, next) => {  console.log('This is second middleware')  next()})app.use((req, res, next) => {  console.log('This is third middleware')  next()})

You will find out that the output in console is:

This is a middlewareThis is second middlewareThis is third middlewareThis is first-half middleware

That is, it runs the code below next() after all middleware function finished.

However, if you use return next(), it will jump out the callback immediately and the code below return next() in the callback will be unreachable.

Some people always write return next() is to ensure that the execution stops after triggering the callback.

If you don't do it, you risk triggering the callback a second time later, which usually has devastating results. Your code is fine as it is, but I would rewrite it as:

app.get('/users/:id?', function(req, res, next){    var id = req.params.id;    if(!id)        return next();    // do something});

It saves me an indentation level, and when I read the code again later, I'm sure there is no way next is called twice.

next() is part of connect middleware. Callbacks for router flow doesn't care if you return anything from your functions, so return next() and next(); return; is basically the same.

In case you want to stop the flow of functions you can use next(err) like the following

app.get('/user/:id?',     function(req, res, next) {         console.log('function one');        if ( !req.params.id )             next('No ID'); // This will return error        else               next(); // This will continue to function 2    },    function(req, res) {         console.log('function two');     });

Pretty much next() is used for extending the middleware of your requests.