6

我正在尝试创建一个可以通过 slug 或 id 查询的 mongoose 查询,但是问题在于我不知道要处理哪一个,因为我将其作为 Express 路由:

app.get('/animals/dogs/:id', function (req, res, next) {
     Dogs.find({$or: [{slug: id}, {_id: id}]}, function (err, docs) {
         if (err) {
             return next(err);
         }
         // ....
     });
});

我希望能够通过其中任何一个进行搜索,但是上面的这种方法会引发Invalid ObjectId错误。

另一种方法是嵌套查询,但这感觉有点麻烦:

app.get('/animals/dogs/:id', function (req, res, next) {
     Dogs.find({slug: id}, function (err, docs) {
         if (err) {
             return next(err);
         }
         if (!docs) {
             Dogs.findById(id, function (err, docs) {
             // ...
             });
         }
         // ....
     });
});

还有其他我没有考虑过的方法吗?我知道我可以将我的 slug 变成 ObjectId,但如果可能的话,我宁愿避免这种情况。

4

2 回答 2

6

测试是否id是有效的 ObjectId,然后仅在查询中包含该术语(如果有效):

app.get('/animals/dogs/:id', function (req, res, next) {
    var query = {$or: [{slug: id}]};
    if (id.match(/^[0-9a-fA-F]{24}$/)) {
        query.$or.push({_id: id});
    }
    Dogs.find(query, function (err, docs) {
        if (err) {
            return next(err);
        }
        // ....
    });
});
于 2012-12-11T18:10:00.993 回答
6

我绝对更喜欢以下语法(使用 Mongoose 验证器ObjectId):

const mongoose   = require('mongoose');
const isObjectId = mongoose.ObjectId.isValid;

app.get('/animals/dogs/:id', function (req, res, next) {
  const id = req.params.id;
  const promise = isObjectId(id) ? Dogs.find(id) : Dogs.findOne({ slug: id });

  promise.then(dog => res.send(dog)).catch(next)
});
于 2017-01-04T09:19:02.023 回答