Node.js/Express - выдавать ошибку, если страница не найдена

У меня есть следующее определение контроллера/маршрута в Node.js(с помощью Express и Mongoose). Каким будет наиболее подходящий способ обработки Error, когда пользователь запрашивает страницу, которая не существует?

  app.get('/page/:pagetitle', function(req, res) {
      Page.findOne({ title: req.params.pagetitle}, function(error, page) {
          res.render('pages/page_show.ejs',
            { locals: {
                title: 'ClrTouch | ' + page.title,
                page:page
            }
          });
      });
  });

В настоящее время это прерывает мое приложение. Я верю, потому что я ничего не делаю с ошибкой, я просто передаю ее мнению, как успех?

TypeError: Cannot read property 'title' of null

Большое спасибо.

Ответ 1

Просмотрите пример экспресс error-pages. Принцип заключается в том, чтобы сначала зарегистрировать маршруты приложений, а затем зарегистрировать улов всех 404-обработчиков для всех других запросов, которые не отображаются на маршрут. Наконец, регистрируется 500-х обработчик следующим образом:

// "app.router" positions our routes 
// specifically above the middleware
// assigned below

app.use(app.router);

// Since this is the last non-error-handling
// middleware use()d, we assume 404, as nothing else
// responded.

app.use(function(req, res, next){
  // the status option, or res.statusCode = 404
  // are equivalent, however with the option we
  // get the "status" local available as well
  res.render('404', { status: 404, url: req.url });
});

// error-handling middleware, take the same form
// as regular middleware, however they require an
// arity of 4, aka the signature (err, req, res, next).
// when connect has an error, it will invoke ONLY error-handling
// middleware.

// If we were to next() here any remaining non-error-handling
// middleware would then be executed, or if we next(err) to
// continue passing the error, only error-handling middleware
// would remain being executed, however here
// we simply respond with an error page.


app.use(function(err, req, res, next){
  // we may use properties of the error object
  // here and next(err) appropriately, or if
  // we possibly recovered from the error, simply next().
  res.render('500', {
      status: err.status || 500
    , error: err
  });
});

Ответ 2

Одной из основных проблем с Node.JS является то, что нет чистой ошибки. Обычный способ, как правило, для каждой функции обратного вызова, первый аргумент - это не null, если есть ошибка, например:

function( error, page ){
   if( error != null ){
       showErrorPage( error, req, res );
       return;
   }
   ...Page exists...
}

Через некоторое время ситуация может стать уродливой со слишком большим количеством обратных вызовов, и я рекомендую использовать что-то вроде async, так что если есть один error, он переходит непосредственно к обратному сообщению об ошибке.

EDIT: вы также можете использовать экспресс-обработку ошибок.