Question

我在Express中有一个简单的API，允许用户将帖子标题“发布”和“删除”到MongoDB数据库中。出于某种原因，当我添加一个帖子标题，然后“删除”它，我得到“Cast to ObjectId failed for value”undefined“at path”_id“。

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

我是否在路由中做错了没有生成“_id”并且能够立即从帖子中提取？

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调用之前，将帖子添加到客户端模型列表中。您推送到posts数组的新创建的帖子将没有_id，因为它是在服务器上生成的而不是返回的。当您将服务器的未定义_id传递给服务器以获取此新帖子时，任何针对它的api调用都将失败并出现相同的错误。

当您刷新页面并调用get函数时，所有帖子都有一个_id，所以一切正常。

要遵循的典型模式是在创建帖子时将创建的id（或整个帖子）返回给客户端，然后将其添加到items数组中（如下所示）：

function addPostItem(post){
    triggerListeners();

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

您还需要重写POST请求以返回创建的实体：

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

            res.json(o);
        });

变体只是返回._id。另一个是在客户端创建id，但是你失去了原生mongo ObjectIds的一些优点。

Express / Mongoose路由器：'转换为ObjectId失败，因为值“未定义”在路径“_id”'

1 个答案: