37

編集したデータをマングースに保存する前に検証を行う方法は?

たとえばsample.name、データベースに既に存在する場合、ユーザーは何らかのエラーを受け取ります。そのようなものです。以下に私のコードを示します。

//Post: /sample/edit
app.post(uri + '/edit', function (req, res, next) {
  Sample.findById(req.param('sid'), function (err, sample) {

    if (err) {
      return next(new Error(err));
    }

    if (!sample) {
      return next(new Error('Invalid reference to sample information'));
    }

    // basic info
    sample.name = req.body.supplier.name;
    sample.tin = req.body.supplier.tin;

    // contact info
    sample.contact.email = req.body.supplier.contact.email;
    sample.contact.mobile = req.body.supplier.contact.mobile;
    sample.contact.landline = req.body.supplier.contact.landline;
    sample.contact.fax = req.body.supplier.contact.fax;

    // address info
    sample.address.street = req.body.supplier.address.street;
    sample.address.city = req.body.supplier.address.city;
    sample.address.state = req.body.supplier.address.state;
    sample.address.country = req.body.supplier.address.country;
    sample.address.zip = req.body.supplier.address.zip;

    sample.save(function (err) {
      if (err) {
        return next(new Error(err));
      }

      res.redirect(uri + '/view/' + sample._id);
    });

  });
});
4

8 に答える 8

69

通常、mongoose 検証を使用できますが、非同期の結果 (既存の名前の db クエリ) が必要であり、バリデーターは約束をサポートしていないため (私が知る限り)、独自の関数を作成してコールバックを渡す必要があります。次に例を示します。

var mongoose = require('mongoose'),
    Schema = mongoose.Schema,
    ObjectId = Schema.ObjectId;

mongoose.connect('mongodb://localhost/testDB');

var UserSchema = new Schema({
    name: {type:String}
});

var UserModel = mongoose.model('UserModel',UserSchema);

function updateUser(user,cb){
    UserModel.find({name : user.name}, function (err, docs) {
        if (docs.length){
            cb('Name exists already',null);
        }else{
            user.save(function(err){
                cb(err,user);
            });
        }
    });
}

UserModel.findById(req.param('sid'),function(err,existingUser){
   if (!err && existingUser){
       existingUser.name = 'Kevin';
       updateUser(existingUser,function(err2,user){
           if (err2 || !user){
               console.log('error updated user: ',err2);
           }else{
               console.log('user updated: ',user);
           }

       });
   } 
});

更新: より良い方法

pre フックは、保存を停止するためのより自然な場所のようです:

UserSchema.pre('save', function (next) {
    var self = this;
    UserModel.find({name : self.name}, function (err, docs) {
        if (!docs.length){
            next();
        }else{                
            console.log('user exists: ',self.name);
            next(new Error("User exists!"));
        }
    });
}) ;

更新 2: 非同期カスタム バリデーター

マングースは非同期カスタムバリデーターをサポートしているように見えるので、おそらくそれが自然な解決策になるでしょう:

    var userSchema = new Schema({
      name: {
        type: String,
        validate: {
          validator: function(v, cb) {
            User.find({name: v}, function(err,docs){
               cb(docs.length == 0);
            });
          },
          message: 'User already exists!'
        }
      }
    });
于 2013-06-02T14:14:11.067 に答える
10

@nfreeze を使用した例を続行する別の方法は、次の検証方法です。

UserModel.schema.path('name').validate(function (value, res) {
    UserModel.findOne({name: value}, 'id', function(err, user) {
        if (err) return res(err);
        if (user) return res(false);
        res(true);
    });
}, 'already exists');
于 2015-11-29T19:22:26.107 に答える
1

一意のインデックスで検索する場合は、int だけを返すのではなく、ドキュメント全体を返す (つまり、読み取りを行う) ため、UserModel.count を使用する方が UserModel.findOne よりも実際に適している場合があります。

于 2016-07-27T05:33:45.277 に答える