尝试删除Node.js Express MongoDB Jade应用程序中的条目

Question

I'm trying to add a button to my app that will delete an entry (comprising of a date and link) but seem to be running into a wall. When I click the button I get this error: Error 404 not found I want to enter the date and link in a form on the view (that works already), view the entries (which works) and then delete an entry and return to the current view (that's where I run into issues).

My app.js file looks like this:

var express = require('express');
var path = require('path');
var favicon = require('serve-favicon');
var logger = require('morgan');
var cookieParser = require('cookie-parser');
var bodyParser = require('body-parser');

var routes = require('./routes/index');
var users = require('./routes/users');
var database = require('./routes/database');

var app = express();

// view engine setup
app.set('views', path.join(__dirname, 'views'));
app.set('view engine', 'jade');

// uncomment after placing your favicon in /public
//app.use(favicon(__dirname + '/public/favicon.ico'));
app.use(logger('dev'));
app.use(bodyParser.json());
app.use(bodyParser.urlencoded({ extended: false }));
app.use(cookieParser());
app.use('/public', express.static(__dirname + '/public'));
app.use(express.static(path.join(__dirname, 'public')));
app.use('public/js', express.static(path.join(__dirname + 'public/js')));
app.use('public/css', express.static(path.join(__dirname + 'public/css')));

app.use('/', routes);
app.use('/users', users);
app.use('/database', database)
app.use('/create', database)

// catch 404 and forward to error handler
app.use(function(req, res, next) {
  var err = new Error('Not Found');
  err.status = 404;
  next(err);
});

// error handlers

// development error handler
// will print stacktrace
if (app.get('env') === 'development') {
  app.use(function(err, req, res, next) {
    res.status(err.status || 500);
    res.render('error', {
      message: err.message,
      error: err
    });
  });
}

// production error handler
// no stacktraces leaked to user
app.use(function(err, req, res, next) {
  res.status(err.status || 500);
  res.render('error', {
    message: err.message,
    error: {}
  });
});


module.exports = app;

app.listen(8080);

console.log("The server has started");

My database.js file (which handles the GET, POST and DELETE actions) looks like this:

var router = express.Router();
var mongoose = require('mongoose');
var Schema = mongoose.Schema;

/* Database stuff */
//connect to the data store and the set up the database
var db = mongoose.connection;

//connect to the database
mongoose.connect('mongodb://localhost/Mandela_Diaries/data');

//Create a model which connects to the schema and entries collection in the Mandela_Diaries database
var Entry = mongoose.model("Entry", new Schema({date: 'date', link: 'string'}), "entries");

mongoose.connection.on("open", function() {
    console.log("mongodb is connected!");
});

//The route for getting data for the database - GET form
router.get("/", function(req, res) {
    //Send the current entries to the page
    Entry.find({}, function(err, entries) {
        console.log(entries);
        if(err) {
            res.status(404).json({"error": "not found", "err":err});
            return;
        } else {
            res.render('database', {title: 'database', entries: entries});
        }
    });

});

//The route for posting data to the database - POST
router.post('/', function(req, res) {
    var newEntry = new Entry(req.body);
    newEntry.save(function(err, entries){
        if (err !== null) {
            res.status(500).json(err);
        } else {
            res.redirect('database');
        };
    });
});

//The route for deleting data to the database - Delete/remove todo item by its id
router.delete('/', function (req, res) {
    Entry.findById(req.params.id)
        .exec(function(err, entries) {
            if (err || !doc) {
                res.statusCode = 404;
                res.send({});
            } else {
                entries.remove(function(err) {
                    if (err) {
                        res.statusCode = 403;
                        res.send(err);
                    } else {
                        res.send({});
                    }
                });
            }
        });
});

And the view code database.jade (where I want to route to) looks like this:

extends layout

block content
    .container
      .row
        .col-s-12
          h1 The Mandela Diaries Database
      .row
        .col-s-4
          h3 Add Entry
      .row
        .col-s-12
          form.form-inline(method='post', action='/create')
            .form-group
              label(for='date') Date: 
              input#datepicker.form-control.datepicker(type='text', name='date')
            .form-group
              label(for='link') Link: 
              input#link.form-control(type='string', name='link')
            button.btn.btn-default(type='submit') Submit
      .row
        p
              | Format options:
              br
              select#format
                option(value='mm/dd/yy') Default - mm/dd/yy
                option(value='yy-mm-dd') ISO 8601 - yy-mm-dd
                option(value='d M, y') Short - d M, y
                option(value='d MM, y') Medium - d MM, y
                option(value='DD, d MM, yy') Full - DD, d MM, yy
                option(value="'day' d 'of' MM 'in the year' yy") With text - 'day' d 'of' MM 'in the year' yy
      br
      ul#dbentries.col-s-12
        each entry in entries
          p= 'Date: ' + entry.date + ': Link: '
            span
              a(href="/delete/#{entries.id}") Delete

I have a feeling I'm not referencing the entries_id correctly but I could be wrong. Please help :-(

Answer 1

The problem seems to be that you haven't set a param in your route. Also you're going to have to change your html or jade in your case, you have a(href="/delete/#{entries.id}") Delete , this will send a GET request to /delete/{id} , you need to send a DELETE request to /database/{id} .

    // Changed path from `/` to `/:id`
    router.delete('/:id', function (req, res) {
        Entry.findById(req.params.id)
            .exec(function(err, entries) {
               // changed `if (err || !doc)` to `if (err || !entries)`
                if (err || !entries) {
                    res.statusCode = 404;
                    res.send({});
                } else {
                    entries.remove(function(err) {
                        if (err) {
                            res.statusCode = 403;
                            res.send(err);
                        } else {
                            res.send({});
                        }
                    });
                }
            });
    });

As for your jade, unless you do this request via Xhr your probably going to have to use a method override middleware . Here's an example of how a form in jade would look.

  ul#dbentries.col-s-12
    each entry in entries
      p= 'Date: ' + entry.date + ': Link: '
        span
          form(method='POST', action='/database/#{entry._id}?_method=DELETE')
            button(type='submit') Delete

Then in app.js ideally one of the first middlewares.

var methodOverride = require('method-override');
app.use(methodOverride('_method'));