压缩多个承诺

Question

我有以下 JavaScript 承诺链。它按预期工作。

signUp (data) {
  return oneFunction(username).then((usernameExist) => {
    return firebaseAuth.createUserWithEmailAndPassword(data.email, data.password).then((user) => {
      firebaseDb.ref('users/' + user.uid + '/public/').set(userData).then()
      utils.updateUsernameMapping(data.username, user.uid).then()
      return user.updateProfile({
        displayName: data.displayName
      }).then(function () {
        return user
      }, error => {
         throw error
      })
    })
  }).catch(error => {
    throw error
  })
}

但是，我相信 signUp 函数由于嵌套级别而难以破译。我尝试将其更改为以下方法：

userPromise
.then()
.then()
.then();

但我无法让它工作，因为用户变量需要向下传递。理想情况下，为了提高可读性，我想尽量减少这段代码，并使用一个 catch() 来提高效率。任何想法表示赞赏。

更新：根据 Bergi 的反馈，以下是我的更新代码：

signUp (email, password, displayName, username) {
  const userData = { username: username, lastLogin: Firebase.database.ServerValue.TIMESTAMP }
  return utils.checkIfUserExists(username).then(usernameExist => {
    return firebaseAuth.createUserWithEmailAndPassword(email, password)
  }).then(user => {
    return Promise.all([
      firebaseDb.ref('users/' + user.uid + '/public/').set(userData),
      utils.updateUsernameMapping(username, user.uid),
      user.updateProfile({displayName})
    ]).then(() => user)
  })
},

Answer 1

对于多个承诺使用

p1 = new Promise(); p2 = new Promise(); p3 = new Promise(); Promise.all([p1, p2, p3])

Promise.all documentation

Answer 2

只重新抛出错误的错误处理程序是没有意义的，省略它们。

您可以在最外层使用usernameExist 变量unnest，这是您在其他任何地方都不需要的：

signUp (data) {
  return oneFunction(username).then(usernameExist => {
    return firebaseAuth.createUserWithEmailAndPassword(email, password);
  }).then(user => {
    return Promise.all([
      firebaseDb.ref('users/' + user.uid + '/public/').set(userData),
      utils.updateUsernameMapping(username, user.uid),
      user.updateProfile({displayName})
    ]).then(() => user);
  });
}

嵌套的then 没有任何问题，可确保最终返回user。有a few approaches可以解决这个问题，嵌套闭包就好了。