web-dev-qa-db-ja.com

Express Router CRUD API。削除できません

まあ、私はMEAN Machineの本を読んでいて、その例に従っています。私は自分のコードの何が問題なのかを理解しようとしているため、DELETE要求は行われません。 GET、PUTおよびPOSTは正常に機能します。

私のserver.jsにこのコードがあります:

app.all('*', function(req, res, next) {
  res.header('Access-Control-Allow-Origin', '*');
  res.header('Access-Control-Allow-Methods', 'PUT, GET, POST, DELETE, OPTIONS');
  res.header('Access-Control-Allow-Headers', 'X-Requested-With, content-type, Authorization');
  next();
});

var apiRouter = express.Router();

apiRouter.route('/users/:user_id')
.get( function (req, res) {
    User.findById( req.params.user_id, function (err, user) {
        if (err) res.send (err);
        res.json(user);
    });
})

.put( function (req, res) {
    User.findById(req.params.user_id, function (err, user) {
        if (err) res.send(err);

        if (req.body.name) user.name = req.body.name;
        if (req.body.username) user.username = req.body.username;
        if (req.body.password) user.password = req.body.password;

        user.save( function (err){
            if (err) send (err);
            res.json({message: 'User updated'});
    });
})

.delete( function (req, res) {
    User.remove({
        _id: req.params.user_id
    }, function (err, user) {
        if (err) return res.send(err);
        res.json({ message: 'Deleted' });
    });
});

});

私はModulus MongoDBデータベースのセットのユーザーを持っていますが、localhost:8080/api/users/5610e5576d827dc41fb8e6eでPOSTMANを使用しようとすると、POSTMANは言います

Cannot DELETE /api/users/5610e5576d827dc41fb8e6e 

私のNode Morganを備えたサーバーが言う間

DELETE /api/users/5610e5576d827dc41fb8e6e 404

なぜ404が表示されるのですか?何が悪いのでしょうか?

完全なコード

11
Rodmentou

put()の閉じ括弧を間違った場所に配置しました。つまり、put()ルーター内にdeleteルーターを定義しています。

これは適切なインデント後のコードです:

.put( function (req, res) {
    User.findById(req.params.user_id, function (err, user) {
        if (err) res.send(err);

        if (req.body.name) user.name = req.body.name;
        if (req.body.username) user.username = req.body.username;
        if (req.body.password) user.password = req.body.password;

        user.save( function (err){
            if (err) send (err);
            res.json({message: 'User updated'});
        });
    })

    .delete( function (req, res) {     // <===== defined inside 'put',
        User.remove({
            _id: req.params.user_id
        }, function (err, user) {
            if (err) return res.send(err);
            res.json({ message: 'Deleted' });
        });
    });
})

したがって、deleteルーターコールバックの外にputルーターを移動するだけです

25
hassansin