Express/Mongoose 路由器：'Cast to ObjectId failed for value "undefined" at path "_id"'

Question

我在 Express 中有一个简单的 API，允许用户将 'post' 和 'delete' 一个 post 标题放入 MongoDB 数据库中。出于某种原因，当我添加一个 post 标题，然后添加 'delete' 它时，我在路径“_id”处得到 "Cast to ObjectId failed for value "undefined”。

创建post后调用'delete'时，似乎“_id”不存在。但是，当我刷新页面，然后单击 'delete'，它会完美地获取“_id”并删除该条目。

我在路由中做错了什么没有生成“_id”并且无法立即从 post 中提取吗？

module.exports = function(router) {

    var Post = require('../models/post.js');

    // middleware for the api requests
    router.use(function(req, res, next) {
        // do logging
        console.log('something is happening.');
        next(); // make sure we go to our next route and don't stop here
    });

    // test route to make sure everything is working (accessed at GET http://localhost:8080/api)

    router.get('/', function(req, res) {
        res.json({ message: 'hooray! welcome to our api!' });   
    });

    // all routes here

    // routes that end in /posts
    router.route('/posts')

        // create a Post (accessed at POST http://localhost:7777/api/posts)
        .post(function(req, res) {
            var post = new Post();
            post.postTitle = req.body.postTitle; // set the post name (comes from request) 
            console.log(post._id);

            // save post and check for errors
            post.save(function(err) {
                if (err)
                    return res.status(300).send(err);

                res.json({ message: 'post created!' });
            });
        })

        // get all Posts (accessed at GET http://localhost:7777/api/posts)
        .get(function(req, res) {
            Post.find(function(err, posts) {
                if (err)
                    return res.send(err);

                res.json(posts);
            });
        });

    // routes that end in /posts for specific id
    router.route('/posts/:post_id')

        // get the post with that id
        .get(function(req, res) {
            Post.findById(req.params.post_id, function(err, post) {
                if (err)
                    return res.send(err);

                res.json(post);
            });
        })

        // update the post with that id
        .put(function(req, res) {
            Post.findById(req.params.post_id, function(err, post) {
                if (err)
                    return res.send(err);

                post.postTitle = req.body.postTitle;

                // save the post
                post.save(function(err) {
                    if (err)
                        return res.send(err);

                    res.json({ message: 'post updated!' });
                });
            });
        })

        // deletes the post with that id
        .delete(function(req, res) {
            Post.findOne({
                _id:req.params.post_id
            }).remove(function(x){
                console.log("removed: ", x);
            });
        })

        .patch(function(req, res) {
            Post.findOne({
                _id: req.body._id
            }, function(err, doc) {
                for (var key in req.body) {
                    dock[key] = req.body[key];
                }
                doc.save();
                res.status(200).send();
            });
        });
}

/

function addPostItem(post){
        posts.push(post);
        triggerListeners();

        helper.post("/api/posts", post);
    }

function deletePost(post) {
        var index = posts.indexOf(post);
        console.log(post);
        posts.splice(index, 1);
        triggerListeners();

        helper.del('api/posts/' + post._id);
    }

/

var $ = require('jquery');

module.exports = {
    get: function(url) {
        return new Promise(function(success, error) {
            $.ajax({
                url: url,
                dataType: 'json',
                success: success,
                error: error
            });
        });
    },
    post: function(url, data) {
        return new Promise(function(success, error) {
            $.ajax({
                url: url,
                type: 'POST',
                data: data,
                success: success,
                error: error
            });
        });
    },
    patch: function(url, data) {
        return new Promise(function(success, error) {
            $.ajax({
                url: url,
                type: 'PATCH',
                data: data,
                success: success,
                error: error
            });
        });
    },
    del: function(url) {
        return new Promise(function(success, error) {
            $.ajax({
                url: url,
                type: 'DELETE',
                success: success,
                error: error
            });
        });
    }
};

Answer 1

在 addPostItem 中，您在调用 POST 之前将 post 添加到您的客户端模型列表中。您推送到 posts 数组的新创建的 post 将没有 _id，因为它是在服务器上生成的，而不是 returned。当您为这个新 post 向服务器传递一个未定义的 _id 时，任何针对它的 api 调用都会失败并出现相同的错误。

当您刷新页面并调用 get 函数时，您的所有 post 都有一个 _id，因此一切正常。

要遵循的典型模式是 return 在 post 创建时向客户端创建 ID（或整个 post），然后将其添加到您的项目数组（像这样的东西）：

function addPostItem(post){
    triggerListeners();

    helper.post("/api/posts", post, function(res){
        posts.push(res.data);
    });
}

您还需要将 POST 请求重写为 return 创建的实体：

post.save(function(err) {
            if (err)
                return res.status(300).send(err, o);

            res.json(o);
        });

一种变体是 return 只是 ._id。另一种方法是在客户端创建 id，但这样你就失去了原生 mongo ObjectIds 的一些优势。

Express/Mongoose 路由器：'Cast to ObjectId failed for value "undefined" at path "_id"'

Express/Mongoose Router: 'Cast to ObjectId failed for value "undefined" at path "_id"'

javascript

router

mongoose

node.js

express