路由器在Express.js中返回404

时间:2018-12-12 06:42:15

标签: javascript node.js mongodb express mongoose

我正在学习,并且是Node.js的新手。我正在尝试构建一个简单的REST API,并在尝试邮递到邮递员中的特定路由以测试数据是否已成功发送到error 404时得到了Mongo db。我确定我所缺少的。我已经仔细检查了所有路线,它们看起来还不错。当我发出get请求时有效,而当我发出post请求时失败。

这是我的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);

2 个答案:

答案 0 :(得分:0)

连接数据库,而不是

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

使用

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

检查doc

猫鼬数据库连接需要使用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);
      });
    });

答案 1 :(得分:0)

如果我是正确的,则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));
});

因为它每次都能在其他路线之前工作。

恕我直言,使用标准的编码样式可使程序流更具可读性。

祝你好运。