Лучшая практика использования обещаний с Mongoose

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); 
      }); 
     }); 
}; 

Любая идея, как использовать возвращенное обещание от Mongoose без создания нового?

Answer 1

Mongoose 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

Правильно Сказано: Mongoose поддерживает 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); 
     }); 
};