14

Mongoose と Express 3 を使用して登録フォームを作成しました

ユーザーがそのユーザー名で既に存在している可能性があります。その場合、err.code 11000 (重複キー) が返されます。既存のユーザーをどのように処理すればよいですか?

これは私が今やっていることです....しかし、エラーコードをチェックすることが最善の方法であるかどうかはわかりません:

  user.save(function(err){
    if ( err ) {
      console.log(err);
      console.log(err.code);

      //duplicate key
      if ( err.code == 11000 ) {
        req.flash('error', 'User already exists');
        res.redirect('/signup');
        return;
      }
    }

    res.locals.user = user;
    req.session.user = user;
    //res.locals.session = req.session;
    res.redirect('/');
  });

これを行うより良い方法はありますか？

4

4 に答える 4

22

これを試して：

user.save(function(err){
  if ( err && err.code !== 11000 ) {
    console.log(err);
    console.log(err.code);
    res.send('Another error showed up');
    return;
  }

  //duplicate key
  if ( err && err.code === 11000 ) {
    req.flash('error', 'User already exists');
    res.redirect('/signup');
    return;
  }

  res.locals.user = user;
  req.session.user = user;
  //res.locals.session = req.session;
  res.redirect('/');
});

この方法でエラー ログを埋めることはありません。

于 2012-09-28T16:52:30.473 に答える
5

私はまだこれを試していませんが、これはエラーの原因を避けるために私が考えていることです:

//look for existing user first
user.findOne({ username: req.body.username }, function(err, user) {
  if ( err ) throw err;

  //existing user found, stop registration
  if ( user ) {
      res.flash('error', "That user already exists");
      res.redirect('/signup');
      return;
  }

  //create new user
  var user = new User({ username: req.body.username });

  user.save(function(err){
   if ( err ) throw err;
      res.flash('info', "Your account has been created");
      res.redirect('/account');
  });
});
于 2012-09-29T20:54:26.720 に答える