プロムスとモンゴースのベストプラクティス

Question

私はこの約束のコンセプトでかなり新しいです。私は確信していないが、これを見て、私はコールバックとして約束を使用していると私は約束地獄で終わると思う！

私はこの機能を利用してMongoUserデータベースからユーザオブジェクトを取得し、更新して再度保存します。ここに私のコードスニペットがあります：

var changePassword = function(data){ 
     return new Promise(function(fulfill, reject){ 
     MongoUser.findOne({username: data.username}).exec() 
      .then(function(mongoUser){ 
      //mongoUser = new MongoUser(); 
      //mongoUser.username = data.username; 
      mongoUser.password = data.password; 
      mongoUser.save().then(function(){ 
       fulfill(data); 
      }).catch(function(error){ 
       log.error("MongoDB Failed in updating data", {"error": error}); 
       reject(error); 
      }); 
      }) 
      .catch(function(error){ 
      log.error("MongoDB Failed in updating data", {"error": error}); 
      reject(error); 
      }); 
     }); 
}; 

任意のアイデア新しいものを作成せずに返された約束をどのように使用するのですか？

Answer 1

マングースすでにsupports promisesので、私はあなたがこれにあなたのコードを書き換えることができると思います。

var changePassword = function(data) { 
    return MongoUser.findOne({username: data.username}).then(function(mongoUser) { 
    mongoUser = new MongoUser(); 
    mongoUser.username = data.username; 
    mongoUser.password = data.password; 
    return mongoUser.save(); 
    }).catch(function(error){ 
    log.error("MongoDB Failed in updating data", {"error": error}); 
    throw error; 
    }); 
}; 

（私はなぜあなたが既に存在していることがあり、ユーザー作成しているかわからないが）を。

Answer 2

モンゴースはすでにpromisesをサポートしています。また、私はあなたが現在のユーザーのパスワードを変更する必要がある代わりに、新しい資格情報で新しいユーザーを作成する必要があるとは思わない。また、私は "changePassword"関数を呼び出している間にデータを返しました。

var changePassword = function(data){ 
    return MongoUser.findOne({username: data.username}).exec() 
     .then(function(mongoUser){ 
     mongoUser.username = data.username; 
     mongoUser.password = data.password; 
     return mongoUser.save(); 
     }) 
    .then(function(newSavedData) { 
       return newSavedData; // returns the new saved data 
     }) 
    .catch(function(error){ 
      log.error("MongoDB Failed in updating data", {"error": error}); 
      reject(error); 
     }); 
};