Маршрутизатор возвращает 404 в Express.js

Question

Я сейчас учусь и новичок на Node.js.Я пытаюсь создать простой REST API и в настоящее время получаю error 404 при попытке отправить на определенный маршрут в почтальоне, чтобы проверить, были ли данные успешно отправлены на Mongo db.Я уверен, что мне не хватает.Я дважды проверил все свои маршруты, и они кажутся в порядке.Это работает, когда я делаю запрос на получение и проваливается, когда я делаю почтовый запрос.

Это мой app.js

var createError = require('http-errors');
var express = require('express');
var path = require('path');
var cookieParser = require('cookie-parser');
var bodyParser = require('body-parser');
var mongoose = require('mongoose');
var logger = require('morgan');
var indexRouter = require('./routes/index');
var usersRouter = require('./routes/users');
var Developer = require('./models/developers');
var app = express();

// view engine setup
app.set('views', path.join(__dirname, 'views'));
app.set('view engine', 'pug');
app.use(logger('dev'));
app.use(express.json());
app.use(express.urlencoded({ extended: false }));
app.use(cookieParser());
app.use(bodyParser.urlencoded({ extended: true}));
app.use(express.static(path.join(__dirname, 'public')));
app.use(bodyParser.json());
app.use('/', indexRouter);
app.use('/users', usersRouter);

// Connect to DB
mongoose.connect('mongodb://127.0.0.1:27017');

// API Routes
var router = express.Router();

// Routes will be prefixed with /api
app.use('/api', router);

// catch 404 and forward to error handler
app.use(function(req, res, next) {
  next(createError(404));
});

// error handler
app.use(function(err, req, res, next) {
  // set locals, only providing error in development
  res.locals.message = err.message;
  res.locals.error = req.app.get('env') === 'development' ? err : {};

  // render the error page
  res.status(err.status || 500);
  res.render('error');
});

module.exports = app;

// Test Route
router.get('/', function (req, res) {
  res.json({message: 'Welcome to my simple API!'});
});

router.route('/developers')
    .post(function (req, res) {
      var developer = new Developer(); // New instance of sa developer
      developer.firstName = req.body.firstName;
      developer.lastName = req.body.lastName;
      developer.jobTitle = req.body.jobTitle;

      developer.save(function (err) {
        if (err) {
          res.send(err);
        } else {
          res.json('Developer was successfully fetched');
        }
      });
    })

    .get(function (req, res) {
      Developer.find(function (err, developers) {
        if (err) {
          res.send(err);
        } else
          res.json(developers);
      });
    });

router.route('/developer/:developer_id')
    .get(function (req, res) {
      Developer.findById(res.params.developer_id, function (err, developer) {
        if (err) {
          res.send(err);
        }
        res.json(developer);
      });
    });

router.route('/developer/firstName/:firstName')
    .get(function (req, res) {
      Developer.find({firstName:res.params.firstName}, function (err, developer) {
        if (err) {
          res.send(err);
        }
        res.json(developer);
      });
    });

Моя модель - developer.js

var mongoose = require('mongoose');
var Schema = mongoose.Schema;

var DevelopersSchema = new Schema({
   firstName: String,
   lastName: String,
   jobTitle: String
});

module.exports = mongoose.model('Developers', DevelopersSchema);

Answer 1

если я прав, строка app.use('/api', router); ничего не делает.При поступлении запроса на конечную точку /api ответ должен быть отправлен в конце операции.Код передачи запроса на router.А маршрутизатор ничего не делает.

Тогда, если createError(404) вызывает методы res.send() или res.end(), вы должны переместить метод use() в конец вашей кодовой страницы:

// catch 404 and forward to error handler
app.use(function(req, res, next) {
  next(createError(404));
});

Потому что каждый раз он работает раньше других маршрутов.

ИМХО, использование стандартного стиля кодирования делает процесс чтения более читабельным.

Удачи ..

Answer 2

Для подключения к базе данных вместо

mongoose.connect('mongodb://127.0.0.1:27017');

Use,

mongoose.connect('mongodb://127.0.0.1:27017/yourDatabaseName');

Проверьте doc .

Требуется подключение к базе данных Mongooseимя базы данных в методе connect().

Ваш окончательный app.js должен быть таким,

var createError = require('http-errors');
var express = require('express');
var path = require('path');
var cookieParser = require('cookie-parser');
var bodyParser = require('body-parser');
var mongoose = require('mongoose');
var logger = require('morgan');
var indexRouter = require('./routes/index');
var usersRouter = require('./routes/users');
var Developer = require('./models/developers');
var app = express();

// view engine setup
app.set('views', path.join(__dirname, 'views'));
app.set('view engine', 'pug');
app.use(logger('dev'));
app.use(express.json());
app.use(express.urlencoded({ extended: false }));
app.use(cookieParser());
app.use(bodyParser.urlencoded({ extended: true}));
app.use(express.static(path.join(__dirname, 'public')));
app.use(bodyParser.json());
app.use('/', indexRouter);
app.use('/users', usersRouter);

// Connect to DB
mongoose.connect('mongodb://127.0.0.1:27017/my_unique_data_base_name');

// API Routes
var router = express.Router();

// Routes will be prefixed with /api
app.use('/api', router);

// catch 404 and forward to error handler
app.use(function(req, res, next) {
  next(createError(404));
});

// error handler
app.use(function(err, req, res, next) {
  // set locals, only providing error in development
  res.locals.message = err.message;
  res.locals.error = req.app.get('env') === 'development' ? err : {};

  // render the error page
  res.status(err.status || 500);
  res.render('error');
});

module.exports = app;

// Test Route
router.get('/', function (req, res) {
  res.json({message: 'Welcome to my simple API!'});
});

router.route('/developers')
    .post(function (req, res) {
      var developer = new Developer(); // New instance of sa developer
      developer.firstName = req.body.firstName;
      developer.lastName = req.body.lastName;
      developer.jobTitle = req.body.jobTitle;

      developer.save(function (err) {
        if (err) {
          res.send(err);
        } else {
          res.json('Developer was successfully fetched');
        }
      });
    })

    .get(function (req, res) {
      Developer.find(function (err, developers) {
        if (err) {
          res.send(err);
        } else
          res.json(developers);
      });
    });

router.route('/developer/:developer_id')
    .get(function (req, res) {
      Developer.findById(res.params.developer_id, function (err, developer) {
        if (err) {
          res.send(err);
        }
        res.json(developer);
      });
    });

router.route('/developer/firstName/:firstName')
    .get(function (req, res) {
      Developer.find({firstName:res.params.firstName}, function (err, developer) {
        if (err) {
          res.send(err);
        }
        res.json(developer);
      });
    });